Skip to content

fix(scale-set): implement review followups - #5472

Open
edersonbrilhante wants to merge 14 commits into
feat-scale-set-terraformfrom
fix/scale-set-review-followups
Open

edersonbrilhante wants to merge 14 commits into
feat-scale-set-terraformfrom
fix/scale-set-review-followups

Conversation

@edersonbrilhante

Copy link
Copy Markdown
Contributor

Description

This PR addresses the review feedback from PR #5299 and documents the experimental scale-set provider.

Changes include:

  • Remove SSM write-back from the read-only controller task role.
  • Require an explicit controller image and recommend immutable digests.
  • Scope private ECR permissions to the repository and enable them only for private ECR images.
  • Align scale-set CloudWatch log retention with the repository default of 180 days.
  • Rename scale-set logging input from kms_key_arn to kms_key_id.
  • Preserve logger error redaction by passing raw Error objects.
  • Remove duplicate startup logging.
  • Document HTTPS egress trade-offs and GitHub Meta API ranges.
  • Update the pinned harden-runner version.
  • Add the scale-set architecture, lifecycle, security boundaries, configuration defaults, migration guidance, and example navigation.
  • Document that the TypeScript controller owns GitHub scale-set API operations; Terraform provisions only the AWS controller infrastructure.
  • Add an ADR describing scale-set runtime ownership and lifecycle behavior.

Related Issues

Fix #5470 #5471

@edersonbrilhante
edersonbrilhante requested review from a team as code owners September 23, 2026 22:12
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

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

Scanned Files

  • .github/workflows/lambda.yml

@edersonbrilhante edersonbrilhante changed the title Fix/scale set review followups fix(scale-set): implement review followups Sep 23, 2026
Comment thread modules/orchestration-providers/scale-set/validations.tf Outdated

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