Skip to content

docs(proof): security-baseline-new-accounts tested against a real org - #20

Merged
DustyStudy merged 1 commit into
mainfrom
docs/security-baseline-new-accounts-proof
Sep 23, 2026
Merged

DustyStudy merged 1 commit into
mainfrom
docs/security-baseline-new-accounts-proof

Conversation

@DustyStudy

Copy link
Copy Markdown
Owner

Ran member-baseline's StackSet for real against a real AWS Organization (management account applying to a test OU), verified GuardDuty/Security Hub/Config in the target member account via direct API reads (not just Terraform's output), then tore it down and reverified everything was gone.

Two gaps the run surfaced, both fixed here:

  • README Prerequisites was incomplete. CreateStackSet also needs cloudformation:ActivateOrganizationsAccess in the calling account, separate from the Organizations-side trusted access already documented. Failed with ValidationError: You must enable organizations access to operate a service managed stack set until this was added.
  • Deprecated Terraform argument. aws_cloudformation_stack_set_instance's region is deprecated in AWS provider v6; switched to stack_set_instance_region.

Full claims/evidence table, what running it for real found, and what this run does not prove (new-account auto-deployment, multi-region, DELEGATED_ADMIN, GovCloud, failure/rollback, cost) is in docs/PROOF.md, matching the format used in aws-orgseed's proof.

Ran the member-baseline StackSet for real: applied via Terraform from the
management account against a test OU, verified GuardDuty/Security
Hub/Config came up in the target account, then tore it down and
reverified everything was gone.

Found and fixed two gaps the proof surfaced:
- README's Prerequisites was missing cloudformation:ActivateOrganizations
  Access; CreateStackSet fails without it even with Organizations trusted
  access already enabled.
- aws_cloudformation_stack_set_instance's region argument is deprecated
  in AWS provider v6; switched to stack_set_instance_region.

See docs/PROOF.md for the claims/evidence table, what running it for real
found, and what this run does not prove.
@DustyStudy
DustyStudy merged commit a007645 into main Sep 23, 2026
3 checks passed
@DustyStudy
DustyStudy deleted the docs/security-baseline-new-accounts-proof branch September 23, 2026 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant