Skip to content

fix(terraform): complete SSM provider wiring - #5473

Open
edersonbrilhante wants to merge 48 commits into
enable-all-tf-testsfrom
feat/dynamodb-storage-provider-terraform
Open

edersonbrilhante wants to merge 48 commits into
enable-all-tf-testsfrom
feat/dynamodb-storage-provider-terraform

Conversation

@edersonbrilhante

Copy link
Copy Markdown
Contributor

Description

Complete the SSM storage-provider wiring across runner configuration, webhook/orchestration modules, and Terraform test coverage. Rename the SSM housekeeper to runner-config-housekeeper, keep SSM cleanup environment and policies conditional, and reserve DynamoDB schema space without including the DynamoDB module.

Test Plan

  • Ran terraform init -backend=false and terraform test -test-directory=tests -no-color for all ten test-bearing modules in TEST_MODULES.
  • Result: 61 tests passed, 0 failed.
  • Confirmed the staged diff contains only Terraform/HCL files and .github/workflows/terraform.yml.
  • Confirmed no files under modules/storage-providers/aws/dynamodb/ are included.

The local TFLint hook was skipped for the commit because the cached AWS ruleset plugin failed its local handshake; Terraform formatting passed and the Terraform test matrix passed.

Related Issues

Not applicable.

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch 3 times, most recently from 0d029a5 to 589c8a2 Compare September 24, 2026 00:28
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from 39b16de to 2cb1ed3 Compare September 24, 2026 00:30
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from cde7246 to 07094af Compare September 24, 2026 00:35
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from 0561769 to 911eb5a Compare September 24, 2026 10:00
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from f1e017c to 9920e14 Compare September 24, 2026 11:55
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from 1b5a358 to 118cfa0 Compare September 24, 2026 12:44
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from c12149b to 4f71e3a Compare September 24, 2026 12:55
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from b8b87a8 to a4c0f5f Compare September 24, 2026 13:15
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from 8dc1163 to 96dcefb Compare September 24, 2026 13:24
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from 880fa14 to 17b2b32 Compare September 24, 2026 13:47
Comment thread scripts/migrate_multi_runner_state.py Outdated
@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from 24810de to ccaba65 Compare September 24, 2026 22:49
@edersonbrilhante
edersonbrilhante marked this pull request as ready for review September 25, 2026 01:04
@edersonbrilhante
edersonbrilhante requested review from a team as code owners September 25, 2026 01:04
@edersonbrilhante

Copy link
Copy Markdown
Contributor Author

@guicaulada ready for review.
Job testing drift in state between v1 and v2 is passing.
https://github.com/github-aws-runners/terraform-aws-github-runner/actions/runs/36078028628/job/107900892057?pr=5473

@edersonbrilhante
edersonbrilhante force-pushed the feat/dynamodb-storage-provider-terraform branch from 41a4365 to ad78451 Compare September 26, 2026 22:25
@edersonbrilhante
edersonbrilhante changed the base branch from main to enable-all-tf-tests September 27, 2026 01:15

This branch has not been deployed

No deployments
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