Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug]: 24029c2 broke aws_ami_ids datasource sorting #40528

Open
remilapeyre opened this issue Dec 11, 2024 · 2 comments · May be fixed by #40529
Open

[Bug]: 24029c2 broke aws_ami_ids datasource sorting #40528

remilapeyre opened this issue Dec 11, 2024 · 2 comments · May be fixed by #40529
Labels
bug Addresses a defect in current functionality. prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. regression Pertains to a degraded workflow resulting from an upstream patch or internal enhancement. service/ec2 Issues and PRs that pertain to the ec2 service.

Comments

@remilapeyre
Copy link

remilapeyre commented Dec 11, 2024 •
edited
Loading

Terraform Core Version

any

AWS Provider Version

5.74 and up

Affected Resource(s)

data.aws_ami_ids

Expected Behavior

The sort is now reversed.

Actual Behavior

The sorting should be correct and made according to sort_ascending.

Relevant Error/Panic Output Snippet

No response

Terraform Configuration Files

terraform {
  required_providers {
    aws = {
      source = "hashicorp/aws"
      version = "5.73.0"
    }
  }
}

data "aws_ami_ids" "almalinux" {
  owners = ["764336703387"]
}

output "ids" {
  value = data.aws_ami_ids.almalinux.ids
}

Steps to Reproduce

Run the Terraform code and check the order in the list, compare with the 5.74.0.

Debug Output

No response

Panic Output

No response

Important Factoids

No response

References

No response

Would you like to implement a fix?

No
@remilapeyre remilapeyre added the bug Addresses a defect in current functionality. label Dec 11, 2024
@github-actions GitHub Actions
Copy link

Community Note

Voting for Prioritization

  • Please vote on this issue by adding a 👍 reaction to the original post to help the community and maintainers prioritize this request.
  • Please see our prioritization guide for information on how we prioritize.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

Volunteering to Work on This Issue

  • If you are interested in working on this issue, please leave a comment.
  • If this would be your first contribution, please review the contribution guide.

@github-actions github-actions bot added service/ec2 Issues and PRs that pertain to the ec2 service. needs-triage Waiting for first response or review from a maintainer. labels Dec 11, 2024
@remilapeyre remilapeyre linked a pull request Dec 11, 2024 that will close this issue
Open
@ewbankkit ewbankkit removed the bug Addresses a defect in current functionality. label Dec 11, 2024
@ewbankkit
Copy link
Contributor

Relates #39899.

@ewbankkit ewbankkit added regression Pertains to a degraded workflow resulting from an upstream patch or internal enhancement. bug Addresses a defect in current functionality. and removed needs-triage Waiting for first response or review from a maintainer. labels Dec 11, 2024
@terraform-aws-provider terraform-aws-provider bot added the prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. label Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Addresses a defect in current functionality. prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. regression Pertains to a degraded workflow resulting from an upstream patch or internal enhancement. service/ec2 Issues and PRs that pertain to the ec2 service.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix sorting in data.aws_ami_ids remilapeyre/terraform-provider-aws
2 participants
@ewbankkit @remilapeyre