-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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_launch_configuration: No default for associate_public_ip_address
#25450
r/aws_launch_configuration: No default for associate_public_ip_address
#25450
Conversation
…_address'. Acceptance test output: % make testacc TESTARGS='-run=TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_' PKG=autoscaling ACCTEST_PARALLELISM=2 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/autoscaling/... -v -count 1 -parallel 2 -run=TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_ -timeout 180m === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull launch_configuration_test.go:698: Step 1/2 error: Check failed: Check 4/4 error: i-04282ae7bd585f800 has public IP address; got false, expected true --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull (175.54s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue --- FAIL: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull (221.09s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue (186.39s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue (147.57s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse (160.34s) --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse (185.05s) FAIL FAIL github.com/hashicorp/terraform-provider-aws/internal/service/autoscaling 550.974s FAIL make: *** [testacc] Error 1
…c_ip_address' (amd mark as Computed). Acceptance test output: % make testacc TESTARGS='-run=TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_' PKG=autoscaling ACCTEST_PARALLELISM=2 ==> Checking that code complies with gofmt requirements... TF_ACC=1 go test ./internal/service/autoscaling/... -v -count 1 -parallel 2 -run=TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_ -timeout 180m === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse === RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue === PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull (157.31s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull (183.34s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse (154.65s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue (196.07s) === CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue (142.93s) --- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse (149.59s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/autoscaling 506.726s
==> Checking source code with golangci-lint...
==> Checking source code with providerlint... |
associate_public_ip_address
associate_public_ip_address
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 TESTARGS='-run=TestAccAutoScalingLaunchConfiguration_' PKG=autoscaling ACCTEST_PARALLELISM=3
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/autoscaling/... -v -count 1 -parallel 3 -run=TestAccAutoScalingLaunchConfiguration_ -timeout 180m
=== RUN TestAccAutoScalingLaunchConfiguration_basic
=== PAUSE TestAccAutoScalingLaunchConfiguration_basic
=== RUN TestAccAutoScalingLaunchConfiguration_disappears
=== PAUSE TestAccAutoScalingLaunchConfiguration_disappears
=== RUN TestAccAutoScalingLaunchConfiguration_Name_generated
=== PAUSE TestAccAutoScalingLaunchConfiguration_Name_generated
=== RUN TestAccAutoScalingLaunchConfiguration_namePrefix
=== PAUSE TestAccAutoScalingLaunchConfiguration_namePrefix
=== RUN TestAccAutoScalingLaunchConfiguration_withBlockDevices
=== PAUSE TestAccAutoScalingLaunchConfiguration_withBlockDevices
=== RUN TestAccAutoScalingLaunchConfiguration_withInstanceStoreAMI
=== PAUSE TestAccAutoScalingLaunchConfiguration_withInstanceStoreAMI
=== RUN TestAccAutoScalingLaunchConfiguration_RootBlockDevice_amiDisappears
=== PAUSE TestAccAutoScalingLaunchConfiguration_RootBlockDevice_amiDisappears
=== RUN TestAccAutoScalingLaunchConfiguration_RootBlockDevice_volumeSize
=== PAUSE TestAccAutoScalingLaunchConfiguration_RootBlockDevice_volumeSize
=== RUN TestAccAutoScalingLaunchConfiguration_encryptedRootBlockDevice
=== PAUSE TestAccAutoScalingLaunchConfiguration_encryptedRootBlockDevice
=== RUN TestAccAutoScalingLaunchConfiguration_withSpotPrice
=== PAUSE TestAccAutoScalingLaunchConfiguration_withSpotPrice
=== RUN TestAccAutoScalingLaunchConfiguration_withVPCClassicLink
=== PAUSE TestAccAutoScalingLaunchConfiguration_withVPCClassicLink
=== RUN TestAccAutoScalingLaunchConfiguration_withIAMProfile
=== PAUSE TestAccAutoScalingLaunchConfiguration_withIAMProfile
=== RUN TestAccAutoScalingLaunchConfiguration_withGP3
=== PAUSE TestAccAutoScalingLaunchConfiguration_withGP3
=== RUN TestAccAutoScalingLaunchConfiguration_encryptedEBSBlockDevice
=== PAUSE TestAccAutoScalingLaunchConfiguration_encryptedEBSBlockDevice
=== RUN TestAccAutoScalingLaunchConfiguration_metadataOptions
=== PAUSE TestAccAutoScalingLaunchConfiguration_metadataOptions
=== RUN TestAccAutoScalingLaunchConfiguration_EBS_noDevice
=== PAUSE TestAccAutoScalingLaunchConfiguration_EBS_noDevice
=== RUN TestAccAutoScalingLaunchConfiguration_userData
=== PAUSE TestAccAutoScalingLaunchConfiguration_userData
=== RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull
=== PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull
=== RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse
=== PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse
=== RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue
=== PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue
=== RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull
=== PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull
=== RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse
=== PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse
=== RUN TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue
=== PAUSE TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue
=== CONT TestAccAutoScalingLaunchConfiguration_basic
=== CONT TestAccAutoScalingLaunchConfiguration_withGP3
=== CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse
--- PASS: TestAccAutoScalingLaunchConfiguration_withGP3 (30.41s)
=== CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue
--- PASS: TestAccAutoScalingLaunchConfiguration_basic (31.37s)
=== CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse
--- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigFalse (157.16s)
=== CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull
--- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigFalse (140.87s)
=== CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue
--- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigTrue (143.68s)
=== CONT TestAccAutoScalingLaunchConfiguration_RootBlockDevice_amiDisappears
--- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigTrue (121.10s)
=== CONT TestAccAutoScalingLaunchConfiguration_withIAMProfile
--- PASS: TestAccAutoScalingLaunchConfiguration_withIAMProfile (40.71s)
=== CONT TestAccAutoScalingLaunchConfiguration_withVPCClassicLink
ec2_classic.go:56: this test can only run in EC2-Classic, platforms available in us-east-1: ["VPC"]
--- SKIP: TestAccAutoScalingLaunchConfiguration_withVPCClassicLink (0.43s)
=== CONT TestAccAutoScalingLaunchConfiguration_withSpotPrice
--- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetTrueConfigNull (179.02s)
=== CONT TestAccAutoScalingLaunchConfiguration_encryptedRootBlockDevice
--- PASS: TestAccAutoScalingLaunchConfiguration_withSpotPrice (27.32s)
=== CONT TestAccAutoScalingLaunchConfiguration_RootBlockDevice_volumeSize
--- PASS: TestAccAutoScalingLaunchConfiguration_encryptedRootBlockDevice (27.14s)
=== CONT TestAccAutoScalingLaunchConfiguration_EBS_noDevice
--- PASS: TestAccAutoScalingLaunchConfiguration_EBS_noDevice (28.31s)
=== CONT TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull
--- PASS: TestAccAutoScalingLaunchConfiguration_RootBlockDevice_volumeSize (59.78s)
=== CONT TestAccAutoScalingLaunchConfiguration_userData
--- PASS: TestAccAutoScalingLaunchConfiguration_userData (57.90s)
=== CONT TestAccAutoScalingLaunchConfiguration_namePrefix
--- PASS: TestAccAutoScalingLaunchConfiguration_namePrefix (28.90s)
=== CONT TestAccAutoScalingLaunchConfiguration_withInstanceStoreAMI
--- PASS: TestAccAutoScalingLaunchConfiguration_withInstanceStoreAMI (29.25s)
=== CONT TestAccAutoScalingLaunchConfiguration_withBlockDevices
--- PASS: TestAccAutoScalingLaunchConfiguration_RootBlockDevice_amiDisappears (371.28s)
=== CONT TestAccAutoScalingLaunchConfiguration_Name_generated
--- PASS: TestAccAutoScalingLaunchConfiguration_AssociatePublicIPAddress_subnetFalseConfigNull (157.49s)
=== CONT TestAccAutoScalingLaunchConfiguration_disappears
--- PASS: TestAccAutoScalingLaunchConfiguration_withBlockDevices (28.58s)
=== CONT TestAccAutoScalingLaunchConfiguration_metadataOptions
--- PASS: TestAccAutoScalingLaunchConfiguration_disappears (23.38s)
=== CONT TestAccAutoScalingLaunchConfiguration_encryptedEBSBlockDevice
--- PASS: TestAccAutoScalingLaunchConfiguration_Name_generated (28.27s)
--- PASS: TestAccAutoScalingLaunchConfiguration_metadataOptions (29.47s)
--- PASS: TestAccAutoScalingLaunchConfiguration_encryptedEBSBlockDevice (57.33s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/autoscaling 631.217s
This functionality has been released in v4.20.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 #25244.
Closes #25252.
Relates #17695.
Removes the default value for
associate_public_ip_address
argument and marks it as Computed.This allows the pre-v4.17.0 behavior of no configured value triggering the API default behavior to be restored whilst also allowing a configured value of
false
to override the subnet's setting.