-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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.
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
aws_availability_zones started proposing changes on each apply #14405
Comments
Hi @vladimirtiukhtin 👋 Thank you for submitting this and sorry you ran into trouble here. Can you please provide the relevant data source configuration? |
Thanks for the prompt response @bflad. My configuration is incredibly simple
|
…omputed instead of Optional Reference: #14405 Output from acceptance testing: ``` --- PASS: TestAccAWSAvailabilityZones_Filter (7.67s) --- PASS: TestAccAWSAvailabilityZones_basic (7.83s) --- PASS: TestAccAWSAvailabilityZones_AllAvailabilityZones (7.88s) --- PASS: TestAccAWSAvailabilityZones_stateFilter (7.88s) --- PASS: TestAccAWSAvailabilityZones_ExcludeNames (9.35s) --- PASS: TestAccAWSAvailabilityZones_ExcludeZoneIds (9.41s) ```
This should fix it: #14412 I'm guessing there is going to be other instances of this unexpected behavior with other data sources in Terraform 0.13 as well, but its going to be a week or two before we can easily gain more visibility into this type of issue. |
…omputed instead of Optional (#14412) Reference: #14405 Output from acceptance testing: ``` --- PASS: TestAccAWSAvailabilityZones_Filter (7.67s) --- PASS: TestAccAWSAvailabilityZones_basic (7.83s) --- PASS: TestAccAWSAvailabilityZones_AllAvailabilityZones (7.88s) --- PASS: TestAccAWSAvailabilityZones_stateFilter (7.88s) --- PASS: TestAccAWSAvailabilityZones_ExcludeNames (9.35s) --- PASS: TestAccAWSAvailabilityZones_ExcludeZoneIds (9.41s) ```
The fix for this has been merged and will release with version 3.1.0 of the Terraform AWS Provider, next week. 👍 |
This has been released in version 3.1.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks! |
@bflad I'm afraid the issue still exists in 3.1.0. It looks like the root cause is in ID which became dependant on current data/time |
Should we open a new issue or re-open this one? |
This still seems to be broken:
and:
it also seems to be that many data providers are suffering from this; I also see it from:
and its dumping my secrets to console as plain text!!!!! also: |
Hi folks 👋 There was some unexpected changes with data source handling that occurred in the Terraform CLI version 0.13.0 release that are causing this additional data source output in addition to the unstable
Please upvote and subscribe to those for further updates. 👍 I'm going to lock this issue to encourage any discussion in those. |
Terraform CLI and Terraform AWS Provider Version
Affected Resource(s)
Debug Output
Expected Behavior
No changes proposed
The text was updated successfully, but these errors were encountered: