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.

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

Use set instead of to avoid infinite changes #1828

Merged
merged 4 commits into from
Aug 11, 2022

Conversation

vavsab
Copy link
Contributor

@vavsab vavsab commented Aug 9, 2022

Closes #1802

@vavsab vavsab requested a review from jacobbednarz as a code owner August 9, 2022 12:15
@github-actions
Copy link
Contributor

github-actions bot commented Aug 9, 2022

changelog detected ✅

@jacobbednarz
Copy link
Member

thanks for the PR! you'll need to update the acceptance tests as well since right now they are based on TypeList - https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/internal/provider/resource_cloudflare_fallback_domain_test.go

@vavsab
Copy link
Contributor Author

vavsab commented Aug 10, 2022

thanks for the PR! you'll need to update the acceptance tests as well since right now they are based on TypeList - https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/internal/provider/resource_cloudflare_fallback_domain_test.go

The test is green now.
image

@jacobbednarz
Copy link
Member

this is odd that the test is passing with the following assertion

resource.TestCheckResourceAttr(name, "domains.0.description", "example domain"),

for TypeSet, the index is a computed hash of the inner elements (such as 12345678). i'll see if i can find an explanation as to why this is still working after it should have broke.

@jacobbednarz jacobbednarz merged commit 77438f8 into cloudflare:master Aug 11, 2022
@github-actions github-actions bot added this to the v3.22.0 milestone Aug 11, 2022
github-actions bot pushed a commit that referenced this pull request Aug 11, 2022
@jacobbednarz
Copy link
Member

i had a look into this and it seems like it somehow works with nested schemas. i can't find specifically where the difference in nested vs simple schemas is (that shows the behaviour difference) but it works 🤷

@github-actions
Copy link
Contributor

This functionality has been released in v3.22.0 of the Terraform Cloudflare 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. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cloudflare_fallback_domain shows perpetual changes to apply
2 participants