⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@movence
Copy link
Contributor

@movence movence commented Jan 8, 2026

Description of the issue

GitHub Action for cleaning up test and lingering resources doesn't include SSM documents or Parameter Store entries

Description of changes

Add SSM doc/param clean up job

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests

running locally

go run clean_ssm_documents.go
2026/01/08 19:16:05 🔍 Searching for test SSM documents and parameters older than 24h0m0s (cutoff: 2026-01-07 19:16:05 UTC) in us-west-2 region
2026/01/08 19:16:05 👷 Creating 10 workers for document cleanup
2026/01/08 19:16:06 🔍 Scanned 1 batches | Found 49 documents in current batch
2026/01/08 19:16:07 🔍 Scanned 11 batches | Found 4 documents in current batch
2026/01/08 19:16:08 🔍 Scanned 20 batches | Found 34 documents in current batch
2026/01/08 19:16:08 📄 Total test documents processed: 0
2026/01/08 19:16:08 👷 Creating 10 workers for parameter cleanup
2026/01/08 19:16:08 🔍 Scanned 1 batches | Found 50 parameters in current batch
2026/01/08 19:16:10 🔍 Scanned 11 batches | Found 50 parameters in current batch
2026/01/08 19:16:13 🔍 Scanned 21 batches | Found 50 parameters in current batch
2026/01/08 19:16:16 🔍 Scanned 31 batches | Found 50 parameters in current batch
2026/01/08 19:16:21 🔍 Scanned 41 batches | Found 50 parameters in current batch
2026/01/08 19:16:21 🔍 Scanned 47 batches | Found 26 parameters in current batch
2026/01/08 19:16:21 ⚙️  Total test parameters processed: 0
2026/01/08 19:16:21 ✅ No test resources found to clean up

Requirements

Before commiting your code, please do the following steps.

  1. Run make fmt and make fmt-sh
  2. Run make lint

Integration Tests

To run integration tests against this PR, add the ready for testing label.

@movence movence requested a review from a team as a code owner January 8, 2026 19:24
@movence movence added the ready for testing Indicates this PR is ready for integration tests to run label Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for testing Indicates this PR is ready for integration tests to run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants