-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
tests/Synthetics: Fix invalid Lambda parameter in acceptance test failures #23394
Conversation
2a7f56b
to
14987fb
Compare
% make testacc TESTS=TestAccSyntheticsCanary_ PKG=synthetics
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/synthetics/... -v -count 1 -parallel 20 -run='TestAccSyntheticsCanary_' -timeout 180m
=== RUN TestAccSyntheticsCanary_basic
=== PAUSE TestAccSyntheticsCanary_basic
=== RUN TestAccSyntheticsCanary_artifactEncryption
=== PAUSE TestAccSyntheticsCanary_artifactEncryption
=== RUN TestAccSyntheticsCanary_runtimeVersion
=== PAUSE TestAccSyntheticsCanary_runtimeVersion
=== RUN TestAccSyntheticsCanary_startCanary
=== PAUSE TestAccSyntheticsCanary_startCanary
=== RUN TestAccSyntheticsCanary_StartCanary_codeChanges
=== PAUSE TestAccSyntheticsCanary_StartCanary_codeChanges
=== RUN TestAccSyntheticsCanary_s3
=== PAUSE TestAccSyntheticsCanary_s3
=== RUN TestAccSyntheticsCanary_run
=== PAUSE TestAccSyntheticsCanary_run
=== RUN TestAccSyntheticsCanary_runTracing
=== PAUSE TestAccSyntheticsCanary_runTracing
=== RUN TestAccSyntheticsCanary_vpc
=== PAUSE TestAccSyntheticsCanary_vpc
=== RUN TestAccSyntheticsCanary_tags
=== PAUSE TestAccSyntheticsCanary_tags
=== RUN TestAccSyntheticsCanary_disappears
=== PAUSE TestAccSyntheticsCanary_disappears
=== CONT TestAccSyntheticsCanary_basic
=== CONT TestAccSyntheticsCanary_run
=== CONT TestAccSyntheticsCanary_startCanary
=== CONT TestAccSyntheticsCanary_disappears
=== CONT TestAccSyntheticsCanary_StartCanary_codeChanges
=== CONT TestAccSyntheticsCanary_runtimeVersion
=== CONT TestAccSyntheticsCanary_vpc
=== CONT TestAccSyntheticsCanary_tags
=== CONT TestAccSyntheticsCanary_runTracing
=== CONT TestAccSyntheticsCanary_s3
=== CONT TestAccSyntheticsCanary_artifactEncryption
--- PASS: TestAccSyntheticsCanary_disappears (61.70s)
--- PASS: TestAccSyntheticsCanary_s3 (65.49s)
=== CONT TestAccSyntheticsCanary_StartCanary_codeChanges
canary_test.go:224: Step 3/3 error: Error running apply: exit status 1
Error: error stopping Synthetics Canary (tf-acc-test-jtkguqi7): ConflictException: Canary is in a state that can't be stopped: STOPPED
with aws_synthetics_canary.test,
on terraform_plugin_test.tf line 110, in resource "aws_synthetics_canary" "test":
110: resource "aws_synthetics_canary" "test" {
=== CONT TestAccSyntheticsCanary_startCanary
canary_test.go:176: Step 3/4 error: Error running apply: exit status 1
Error: error stopping Synthetics Canary (tf-acc-test-9nfyoxjh): ConflictException: Canary is in a state that can't be stopped: STOPPED
with aws_synthetics_canary.test,
on terraform_plugin_test.tf line 110, in resource "aws_synthetics_canary" "test":
110: resource "aws_synthetics_canary" "test" {
=== CONT TestAccSyntheticsCanary_StartCanary_codeChanges
testing_new.go:70: Error running post-test destroy, there may be dangling resources: exit status 1
Error: error stopping Synthetics Canary (tf-acc-test-jtkguqi7): ConflictException: Canary is in a state that can't be stopped: STOPPED
--- FAIL: TestAccSyntheticsCanary_StartCanary_codeChanges (74.97s)
=== CONT TestAccSyntheticsCanary_startCanary
testing_new.go:70: Error running post-test destroy, there may be dangling resources: exit status 1
Error: error stopping Synthetics Canary (tf-acc-test-9nfyoxjh): ConflictException: Canary is in a state that can't be stopped: STOPPED
--- FAIL: TestAccSyntheticsCanary_startCanary (75.10s)
--- PASS: TestAccSyntheticsCanary_artifactEncryption (93.06s)
--- PASS: TestAccSyntheticsCanary_basic (107.19s)
--- PASS: TestAccSyntheticsCanary_tags (110.31s)
--- PASS: TestAccSyntheticsCanary_runTracing (111.80s)
--- PASS: TestAccSyntheticsCanary_runtimeVersion (116.66s)
--- PASS: TestAccSyntheticsCanary_run (125.78s)
--- PASS: TestAccSyntheticsCanary_vpc (2151.63s)
FAIL
FAIL github.com/hashicorp/terraform-provider-aws/internal/service/synthetics 2156.169s
FAIL
make: *** [testacc] Error 1 |
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 🚀.
This functionality has been released in v4.4.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 #23391
Description
Output from acceptance testing: