-
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
fix apigw cache waiting logic when cache size provided but not cache enabled #23091
fix apigw cache waiting logic when cache size provided but not cache enabled #23091
Conversation
…bled and setting cache size
523cf85
to
f76a539
Compare
Relates: #18561. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀.
% make testacc TESTS=TestAccAPIGatewayStage_ PKG=apigateway ACCTEST_PARALLELISM=2
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/apigateway/... -v -count 1 -parallel 2 -run='TestAccAPIGatewayStage_' -timeout 180m
=== RUN TestAccAPIGatewayStage_basic
=== PAUSE TestAccAPIGatewayStage_basic
=== RUN TestAccAPIGatewayStage_cache
=== PAUSE TestAccAPIGatewayStage_cache
=== RUN TestAccAPIGatewayStage_cache_size_cache_disabled
=== PAUSE TestAccAPIGatewayStage_cache_size_cache_disabled
=== RUN TestAccAPIGatewayStage_Disappears_referencingDeployment
=== PAUSE TestAccAPIGatewayStage_Disappears_referencingDeployment
=== RUN TestAccAPIGatewayStage_tags
=== PAUSE TestAccAPIGatewayStage_tags
=== RUN TestAccAPIGatewayStage_disappears
=== PAUSE TestAccAPIGatewayStage_disappears
=== RUN TestAccAPIGatewayStage_disappears_restApi
=== PAUSE TestAccAPIGatewayStage_disappears_restApi
=== RUN TestAccAPIGatewayStage_accessLogSettings
=== PAUSE TestAccAPIGatewayStage_accessLogSettings
=== RUN TestAccAPIGatewayStage_AccessLogSettings_kinesis
=== PAUSE TestAccAPIGatewayStage_AccessLogSettings_kinesis
=== RUN TestAccAPIGatewayStage_waf
=== PAUSE TestAccAPIGatewayStage_waf
=== CONT TestAccAPIGatewayStage_basic
=== CONT TestAccAPIGatewayStage_disappears
--- PASS: TestAccAPIGatewayStage_disappears (44.03s)
=== CONT TestAccAPIGatewayStage_AccessLogSettings_kinesis
--- PASS: TestAccAPIGatewayStage_basic (81.00s)
=== CONT TestAccAPIGatewayStage_waf
--- PASS: TestAccAPIGatewayStage_waf (76.80s)
=== CONT TestAccAPIGatewayStage_Disappears_referencingDeployment
--- PASS: TestAccAPIGatewayStage_Disappears_referencingDeployment (31.59s)
=== CONT TestAccAPIGatewayStage_tags
--- PASS: TestAccAPIGatewayStage_AccessLogSettings_kinesis (213.45s)
=== CONT TestAccAPIGatewayStage_accessLogSettings
--- PASS: TestAccAPIGatewayStage_tags (68.41s)
=== CONT TestAccAPIGatewayStage_disappears_restApi
--- PASS: TestAccAPIGatewayStage_disappears_restApi (32.89s)
=== CONT TestAccAPIGatewayStage_cache_size_cache_disabled
--- PASS: TestAccAPIGatewayStage_accessLogSettings (103.58s)
=== CONT TestAccAPIGatewayStage_cache
--- PASS: TestAccAPIGatewayStage_cache_size_cache_disabled (235.48s)
--- PASS: TestAccAPIGatewayStage_cache (454.77s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/apigateway 819.540s
@alpacamybags118 Thanks for the contribution 🎉 👏. |
This functionality has been released in v4.6.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. Thank you! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Community Note
Closes #22866
This PR fixes the issue described in #22866 . The logic for waiting for the cache has been updated to do the following:
cache_cluster_enabled
is truecache_cluster_enabled
is enabled, orcache_cluster_size
is updated whencache_cluster_enabled
is already trueI wrote a new acceptance test for these scenarios but it may be redundant with the existing tests. I can adjust as needed.
Output from acceptance testing: