-
Notifications
You must be signed in to change notification settings - Fork 9k
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
r/aws_route53_health_check - suppress diff when expanded ipv6 address is the same #12305
r/aws_route53_health_check - suppress diff when expanded ipv6 address is the same #12305
Conversation
Verified acceptance tests: $ make testacc TEST=./aws TESTARGS='-run=TestAccAWSRoute53HealthCheck_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSRoute53HealthCheck_ -timeout 120m
=== RUN TestAccAWSRoute53HealthCheck_basic
=== PAUSE TestAccAWSRoute53HealthCheck_basic
=== RUN TestAccAWSRoute53HealthCheck_tags
=== PAUSE TestAccAWSRoute53HealthCheck_tags
=== RUN TestAccAWSRoute53HealthCheck_withSearchString
=== PAUSE TestAccAWSRoute53HealthCheck_withSearchString
=== RUN TestAccAWSRoute53HealthCheck_withChildHealthChecks
=== PAUSE TestAccAWSRoute53HealthCheck_withChildHealthChecks
=== RUN TestAccAWSRoute53HealthCheck_withHealthCheckRegions
=== PAUSE TestAccAWSRoute53HealthCheck_withHealthCheckRegions
=== RUN TestAccAWSRoute53HealthCheck_IpConfig
=== PAUSE TestAccAWSRoute53HealthCheck_IpConfig
=== RUN TestAccAWSRoute53HealthCheck_Ipv6Config
=== PAUSE TestAccAWSRoute53HealthCheck_Ipv6Config
=== RUN TestAccAWSRoute53HealthCheck_CloudWatchAlarmCheck
=== PAUSE TestAccAWSRoute53HealthCheck_CloudWatchAlarmCheck
=== RUN TestAccAWSRoute53HealthCheck_withSNI
=== PAUSE TestAccAWSRoute53HealthCheck_withSNI
=== RUN TestAccAWSRoute53HealthCheck_disappears
=== PAUSE TestAccAWSRoute53HealthCheck_disappears
=== CONT TestAccAWSRoute53HealthCheck_basic
=== CONT TestAccAWSRoute53HealthCheck_Ipv6Config
=== CONT TestAccAWSRoute53HealthCheck_disappears
=== CONT TestAccAWSRoute53HealthCheck_IpConfig
=== CONT TestAccAWSRoute53HealthCheck_withHealthCheckRegions
=== CONT TestAccAWSRoute53HealthCheck_withChildHealthChecks
=== CONT TestAccAWSRoute53HealthCheck_withSearchString
=== CONT TestAccAWSRoute53HealthCheck_tags
=== CONT TestAccAWSRoute53HealthCheck_withSNI
=== CONT TestAccAWSRoute53HealthCheck_CloudWatchAlarmCheck
--- PASS: TestAccAWSRoute53HealthCheck_disappears (17.66s)
--- PASS: TestAccAWSRoute53HealthCheck_withHealthCheckRegions (22.79s)
--- PASS: TestAccAWSRoute53HealthCheck_IpConfig (23.30s)
--- PASS: TestAccAWSRoute53HealthCheck_withChildHealthChecks (23.91s)
--- PASS: TestAccAWSRoute53HealthCheck_CloudWatchAlarmCheck (28.14s)
--- PASS: TestAccAWSRoute53HealthCheck_Ipv6Config (36.35s)
--- PASS: TestAccAWSRoute53HealthCheck_basic (36.36s)
--- PASS: TestAccAWSRoute53HealthCheck_withSearchString (37.08s)
--- PASS: TestAccAWSRoute53HealthCheck_withSNI (53.14s)
--- PASS: TestAccAWSRoute53HealthCheck_tags (53.26s)
PASS
ok github.com/terraform-providers/terraform-provider-aws/aws 53.292s |
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.
One suggestion about the tests, but otherwise LGTM 🚀
--- PASS: TestAccAWSRoute53HealthCheck_disappears (9.15s)
--- PASS: TestAccAWSRoute53HealthCheck_IpConfig (18.18s)
--- PASS: TestAccAWSRoute53HealthCheck_CloudWatchAlarmCheck (18.94s)
--- PASS: TestAccAWSRoute53HealthCheck_basic (22.69s)
--- PASS: TestAccAWSRoute53HealthCheck_withChildHealthChecks (27.64s)
--- PASS: TestAccAWSRoute53HealthCheck_withSearchString (27.71s)
--- PASS: TestAccAWSRoute53HealthCheck_Ipv6Config (29.90s)
--- PASS: TestAccAWSRoute53HealthCheck_withSNI (31.01s)
--- PASS: TestAccAWSRoute53HealthCheck_withHealthCheckRegions (35.22s)
--- PASS: TestAccAWSRoute53HealthCheck_tags (35.68s)
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.
There's a test failure in TestAccAWSRoute53HealthCheck_basic
now.
Check: resource.ComposeTestCheckFunc( | ||
testAccCheckRoute53HealthCheckExists(resourceName), | ||
resource.TestCheckResourceAttr(resourceName, "failure_threshold", "5"), | ||
resource.TestCheckResourceAttr(resourceName, "invert_healthcheck", "false"), | ||
), | ||
PlanOnly: true, |
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.
This test changes values, so leaving it as a Check
would be good.
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.
oops, didnt mean to change that.
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 🚀
--- PASS: TestAccAWSRoute53HealthCheck_disappears (8.41s)
--- PASS: TestAccAWSRoute53HealthCheck_IpConfig (13.69s)
--- PASS: TestAccAWSRoute53HealthCheck_withHealthCheckRegions (14.79s)
--- PASS: TestAccAWSRoute53HealthCheck_Ipv6Config (19.80s)
--- PASS: TestAccAWSRoute53HealthCheck_CloudWatchAlarmCheck (22.49s)
--- PASS: TestAccAWSRoute53HealthCheck_withChildHealthChecks (23.64s)
--- PASS: TestAccAWSRoute53HealthCheck_withSearchString (27.91s)
--- PASS: TestAccAWSRoute53HealthCheck_basic (32.58s)
--- PASS: TestAccAWSRoute53HealthCheck_tags (35.41s)
--- PASS: TestAccAWSRoute53HealthCheck_withSNI (37.42s)
This has been released in version 2.55.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 for triage. Thanks! |
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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Closes #11069
Release note for CHANGELOG:
Output from acceptance testing: