-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
New Resource: aws_auditmanager_control
#27857
Conversation
Community NoteVoting for Prioritization
For Submitters
|
d89cf76
to
79262b2
Compare
Converting back to draft to investigate use of plugin framework with computed set attributes. |
79262b2
to
14218c2
Compare
b472e13
to
cd1bc87
Compare
cd1bc87
to
a0f2fb5
Compare
9ec5a16
to
e5fbef2
Compare
e5fbef2
to
2ee7d24
Compare
2ee7d24
to
18af067
Compare
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=TestAccAuditManagerControl_' PKG=auditmanager ACCTEST_PARALLELISM=3
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/auditmanager/... -v -count 1 -parallel 3 -run=TestAccAuditManagerControl_ -timeout 180m
=== RUN TestAccAuditManagerControl_basic
=== PAUSE TestAccAuditManagerControl_basic
=== RUN TestAccAuditManagerControl_disappears
=== PAUSE TestAccAuditManagerControl_disappears
=== RUN TestAccAuditManagerControl_tags
=== PAUSE TestAccAuditManagerControl_tags
=== RUN TestAccAuditManagerControl_optional
=== PAUSE TestAccAuditManagerControl_optional
=== RUN TestAccAuditManagerControl_optionalSources
=== PAUSE TestAccAuditManagerControl_optionalSources
=== CONT TestAccAuditManagerControl_basic
=== CONT TestAccAuditManagerControl_optional
=== CONT TestAccAuditManagerControl_optionalSources
--- PASS: TestAccAuditManagerControl_basic (18.63s)
=== CONT TestAccAuditManagerControl_tags
--- PASS: TestAccAuditManagerControl_optional (28.74s)
=== CONT TestAccAuditManagerControl_disappears
--- PASS: TestAccAuditManagerControl_optionalSources (28.87s)
--- PASS: TestAccAuditManagerControl_disappears (11.25s)
--- PASS: TestAccAuditManagerControl_tags (37.13s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/auditmanager 60.287s
can be fixed by % aws auditmanager register-account
{
"status": "ACTIVE"
} Maybe we need an |
This is planned for a future phase, but I missed adding it to the original issue. It's been included now, and I may implement it sooner since this service is opt-in. |
This functionality has been released in v4.46.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. |
Description
aws_auditmanager_control
will allow practitioners to manage custom controls.Relations
Relates #17981
Depends on #28119
Output from Acceptance Testing