-
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
add call_as support for stackset #22440
Conversation
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.
Welcome @chakch 👋
It looks like this is your first Pull Request submission to the Terraform AWS Provider! If you haven’t already done so please make sure you have checked out our CONTRIBUTING guide and FAQ to make sure your contribution is adhering to best practice and has all the necessary elements in place for a successful approval.
Also take a look at our FAQ which details how we prioritize Pull Requests for inclusion.
Thanks again, and welcome to the community! 😃
One of my colleges at work already tested this branch and it's working as expected, would it be possible to get this fix included in the next release? |
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=TestAccCloudFormationStackSet_ PKG=cloudformation
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/cloudformation/... -v -count 1 -parallel 20 -run='TestAccCloudFormationStackSet_' -timeout 180m
=== RUN TestAccCloudFormationStackSet_basic
=== PAUSE TestAccCloudFormationStackSet_basic
=== RUN TestAccCloudFormationStackSet_disappears
=== PAUSE TestAccCloudFormationStackSet_disappears
=== RUN TestAccCloudFormationStackSet_administrationRoleARN
=== PAUSE TestAccCloudFormationStackSet_administrationRoleARN
=== RUN TestAccCloudFormationStackSet_description
=== PAUSE TestAccCloudFormationStackSet_description
=== RUN TestAccCloudFormationStackSet_executionRoleName
=== PAUSE TestAccCloudFormationStackSet_executionRoleName
=== RUN TestAccCloudFormationStackSet_name
=== PAUSE TestAccCloudFormationStackSet_name
=== RUN TestAccCloudFormationStackSet_parameters
=== PAUSE TestAccCloudFormationStackSet_parameters
=== RUN TestAccCloudFormationStackSet_Parameters_default
acctest.go:66: this resource does not currently ignore unconfigured CloudFormation template parameters with the Default property
--- SKIP: TestAccCloudFormationStackSet_Parameters_default (0.00s)
=== RUN TestAccCloudFormationStackSet_Parameters_noEcho
acctest.go:66: this resource does not currently ignore CloudFormation template parameters with the NoEcho property
--- SKIP: TestAccCloudFormationStackSet_Parameters_noEcho (0.00s)
=== RUN TestAccCloudFormationStackSet_PermissionModel_serviceManaged
acctest.go:66: API does not support enabling Organizations access (in particular, creating the Stack Sets IAM Service-Linked Role)
--- SKIP: TestAccCloudFormationStackSet_PermissionModel_serviceManaged (0.00s)
=== RUN TestAccCloudFormationStackSet_tags
=== PAUSE TestAccCloudFormationStackSet_tags
=== RUN TestAccCloudFormationStackSet_templateBody
=== PAUSE TestAccCloudFormationStackSet_templateBody
=== RUN TestAccCloudFormationStackSet_templateURL
=== PAUSE TestAccCloudFormationStackSet_templateURL
=== CONT TestAccCloudFormationStackSet_basic
=== CONT TestAccCloudFormationStackSet_name
=== CONT TestAccCloudFormationStackSet_templateURL
=== CONT TestAccCloudFormationStackSet_tags
=== CONT TestAccCloudFormationStackSet_templateBody
=== CONT TestAccCloudFormationStackSet_parameters
=== CONT TestAccCloudFormationStackSet_description
=== CONT TestAccCloudFormationStackSet_executionRoleName
=== CONT TestAccCloudFormationStackSet_administrationRoleARN
=== CONT TestAccCloudFormationStackSet_disappears
--- PASS: TestAccCloudFormationStackSet_disappears (37.62s)
--- PASS: TestAccCloudFormationStackSet_basic (42.98s)
--- PASS: TestAccCloudFormationStackSet_templateBody (68.71s)
--- PASS: TestAccCloudFormationStackSet_administrationRoleARN (71.36s)
--- PASS: TestAccCloudFormationStackSet_executionRoleName (77.38s)
--- PASS: TestAccCloudFormationStackSet_templateURL (91.66s)
--- PASS: TestAccCloudFormationStackSet_name (91.67s)
--- PASS: TestAccCloudFormationStackSet_description (97.72s)
--- PASS: TestAccCloudFormationStackSet_tags (172.96s)
--- PASS: TestAccCloudFormationStackSet_parameters (208.59s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/cloudformation 215.811s
@chakch Thanks for the contribution 🎉 👏. |
thanks @ewbankkit and @chakch !! |
This functionality has been released in v4.1.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 #19736.
Supersedes #19903.