-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix inconsistent final plan when labels are added to resources #7923
Fix inconsistent final plan when labels are added to resources #7923
Conversation
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
Hello! I am a robot who works on Magic Modules PRs. I've detected that you're a community contributor. @slevenick, a repository maintainer, has been assigned to assist you and help review your changes. ❓ First time contributing? Click here for more detailsYour assigned reviewer will help review your code by:
You can help make sure that review is quick by running local tests and ensuring they're passing in between each push you make to your PR's branch. Also, try to leave a comment with each push you make, as pushes generally don't generate emails. If your reviewer doesn't get back to you within a week after your most recent change, please feel free to leave a comment on the issue asking them to take a look! In the absence of a dedicated review dashboard most maintainers manage their pending reviews through email, and those will sometimes get lost in their inbox. |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 2 files changed, 2 insertions(+)) |
Tests analyticsTotal tests: Action takenFound 8 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccStorageBucket_labels|TestAccDataSourceGoogleFirebaseAndroidAppConfig|TestAccComputeFirewallPolicyRule_multipleRules|TestAccApigeeKeystoresAliasesPkcs12_ApigeeKeystoresAliasesPkcs12Example|TestAccAlloydbBackup_missingLocation|TestAccAlloydbCluster_missingLocation|TestAccApigeeKeystoresAliasesKeyCertFile_apigeeKeystoresAliasesKeyCertFileTestExample|TestAccDataSourceAlloydbLocations_basic |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
This looks fine to me, I guess I'm confused how our tests were passing if this was causing such a problem. Is there an alternative way to create buckets with labels that doesn't hit the inconsistent plan? |
Sadly i'm missing the required Terraform-internal knowledge to give you a helpful answer here, but this bug is kinda strange. This works with the latest provider (4.64.0):
As soon as you add a label with a derived value it fails since 4.40.0 on the first
|
Users reported an issue creating buckets with labels via
google_storage_bucket
since #6518 was merged. Creation failed with the following error message:I am by far no Terraform expert but i looked at the other resources which had a
DiffSuppressFunc
set on theirlabels
property and all of them hadcomputed: true
EXCEPTgoogle_dataflow_job
. So i ran a small test withand it seems that this resource has the same problem as
google_storage_bucket
:Setting both properties to
computed: true
in alignment with the others fixed the problem on both resources. But please don't ask me why 😁@prauc FYI
Fixes hashicorp/terraform-provider-google#12804
If this PR is for Terraform, I acknowledge that I have:
make test
andmake lint
in the generated providers to ensure it passes unit and linter tests.Release Note Template for Downstream PRs (will be copied)