aws-samples / aws-security-reference-architecture-examples

Example solutions demonstrating how to implement patterns within the AWS Security Reference Architecture guide using CloudFormation (including Customizations for AWS Control Tower) and Terraform.
Other
965 stars 235 forks source link

[BUG] sra-macie-org-main-ssm.json includes a pSRAStagingS3BucketName parameter that should be read from SSM parameters #223

Open julian-price opened 2 months ago

julian-price commented 2 months ago

Community Note

Describe the bug

When deploying Macie (via the CfCT in my case, but the same issue should happen via CloudFormation CLI), the SSM JSON parameters file (sra-macie-org-main-ssm.json) includes a value that should be read from it's SSM parameter and therefore not be included in the JSON file.

To Reproduce

Steps to reproduce the behavior:

  1. Copy the sra-macie-org-main-ssm.json file from the SRA solution to the parameters directory in the custom-control-tower-configuration CodeCommit repo.
  2. Copy the sra-macie-org-main-ssm.yaml file from the SRA solution to the templates directory in the custom-control-tower-configuration CodeCommit repo.
  3. Add an entry to the manifest.yaml file to deploy the solution.
  4. Commit changes to kick off the pipeline
  5. The Macie org solution fails to deploy with an error
    Parameter pSRAStagingS3BucketName should either have input value or default value

Expected behavior

The Macie org solution deploys without errors.

Deployment Environment (please complete the following information)

Additional context

Removing the parameter from the JSON file allows the deployment to proceed successfully.