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

provider/aws: Retry ElastiCache cluster deletion when it's snapshotting #14700

Merged
merged 1 commit into from
May 20, 2017

Conversation

radeksimko
Copy link
Member

This is to address the following test failure:

=== RUN   TestAccAWSElasticacheCluster_snapshotsWithUpdates
--- FAIL: TestAccAWSElasticacheCluster_snapshotsWithUpdates (260.18s)
    testing.go:280: Step 1 error: Error applying: 1 error(s) occurred:
        
        * aws_elasticache_cluster.bar (destroy): 1 error(s) occurred:
        
        * aws_elasticache_cluster.bar: InvalidCacheClusterState: Can only delete cache clusters with state in: available, failed, incompatible-parameters, incompatible-network, restore-failed, recovering. Cluster tf-tvzyhadgqq has state: snapshotting.
            status code: 400, request id: 0eef8895-3d24-11e7-a41c-bbcf35279727

Test plan

make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSElasticacheCluster_'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/05/20 16:07:13 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSElasticacheCluster_ -timeout 120m
=== RUN   TestAccAWSElasticacheCluster_importBasic
--- PASS: TestAccAWSElasticacheCluster_importBasic (501.62s)
=== RUN   TestAccAWSElasticacheCluster_basic
--- PASS: TestAccAWSElasticacheCluster_basic (536.60s)
=== RUN   TestAccAWSElasticacheCluster_snapshotsWithUpdates
--- PASS: TestAccAWSElasticacheCluster_snapshotsWithUpdates (1063.32s)
=== RUN   TestAccAWSElasticacheCluster_decreasingCacheNodes
--- PASS: TestAccAWSElasticacheCluster_decreasingCacheNodes (1118.01s)
=== RUN   TestAccAWSElasticacheCluster_vpc
--- PASS: TestAccAWSElasticacheCluster_vpc (714.34s)
=== RUN   TestAccAWSElasticacheCluster_multiAZInVpc
--- PASS: TestAccAWSElasticacheCluster_multiAZInVpc (706.26s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	4640.196s

@stack72
Copy link
Contributor

stack72 commented May 20, 2017

LGTM! Merging to get in tonights' test run

@stack72 stack72 merged commit 60bae99 into master May 20, 2017
@stack72 stack72 deleted the b-aws-retry-ec-deletion branch May 20, 2017 21:42
@ghost
Copy link

ghost commented Apr 12, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants