Skip to content

Scope ssm_document to SSM instances; add non-SSM OS variants - #748

Open
bhavya76 wants to merge 1 commit into
mainfrom
scope-ssm-document-to-ssm-instances
Open

Scope ssm_document to SSM instances; add non-SSM OS variants#748
bhavya76 wants to merge 1 commit into
mainfrom
scope-ssm-document-to-ssm-instances

Conversation

@bhavya76

@bhavya76 bhavya76 commented Aug 26, 2026

Copy link
Copy Markdown

What

Run the ssm_document integration test only on SSM-enabled instances, and add non-SSM variants of the OSes that previously had no SSM agent.

Why

ssm_document drives the CloudWatch agent via an SSM document and therefore requires a registered SSM agent. rocky-linux-8, debian-12 and debian-13 image recipes strip the SSM agent at bake time, so the readiness wait times out and the test fails during setup (never reaching the code under test).

Changes

  • Image recipes (out of band, EC2 Image Builder): rocky-linux-8, debian-12, debian-13 now keep the SSM agent (uninstallAfterBuild=false + install component), making SSM the default; new -withoutSSM recipes/AMIs preserve the non-SSM configuration.
  • Matrix: add rocky-linux-8-withoutSSM, debian-12-withoutSSM, debian-13-withoutSSM.
  • rocky-linux-8 AMI filter tightened so it resolves the SSM image, not the -withoutSSM image.
  • ssm_document: excludedOs for the three -withoutSSM variants.

ssm_document manages the CloudWatch agent through an SSM document, so it
needs the SSM agent present on the instance. A few integration AMIs
(rocky-linux-8, debian-12, debian-13) do not keep the SSM agent, so the
test's SSM-readiness wait times out on them.

SSM is now the default for those OSes (handled in the image recipes). Add
explicit -withoutSSM AMI variants to preserve non-SSM coverage, run
ssm_document on all SSM-capable instances, and exclude the -withoutSSM
variants. All other tests continue to run on every instance.

- add rocky-linux-8-withoutSSM, debian-12-withoutSSM, debian-13-withoutSSM rows
- tighten rocky-linux-8 AMI name filter so it does not match the -withoutSSM image
- exclude the -withoutSSM OSes from ./test/ssm_document
@bhavya76
bhavya76 marked this pull request as ready for review August 26, 2026 15:13
@bhavya76
bhavya76 requested a review from a team as a code owner August 26, 2026 15:13

@AshuSrv AshuSrv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we run this branch against the actual agent to test the changes

@bhavya76 bhavya76 added github_actions Pull requests that update GitHub Actions code and removed github_actions Pull requests that update GitHub Actions code labels Aug 26, 2026
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.

2 participants