Skip to content

refactor(multi-runner): consume effective configuration - #5342

Open
edersonbrilhante wants to merge 15 commits into
use-resolved-config-consumersfrom
replace-legacy-config-consumers
Open

refactor(multi-runner): consume effective configuration#5342
edersonbrilhante wants to merge 15 commits into
use-resolved-config-consumersfrom
replace-legacy-config-consumers

Conversation

@edersonbrilhante

Copy link
Copy Markdown
Contributor

Description

  • Route AMI housekeeper, SSM, webhook, runner-binaries, termination-watcher, queue, and runner consumers through local.effective_config.
  • Pass resolved SSM housekeeper artifacts through the runners module so lane ZIP/S3 precedence is preserved.
  • Keep stable v1 compatibility in the translation adapter and add v2 test fixtures for effective global artifacts and settings.

Test Plan

  • terraform fmt -recursive
  • terraform validate in modules/multi-runner — passed with existing ami-housekeeper deprecation warnings
  • terraform test in modules/multi-runner — 6 passed, 0 failed
  • Pre-commit hooks passed: Terraform fmt, Terraform validate with TFLint, and merge-conflict checks

Related Issues

@github-actions

github-actions Bot commented Sep 3, 2026

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 replace-legacy-config-consumers branch 2 times, most recently from 033527a to cf49c59 Compare September 3, 2026 18:14
@edersonbrilhante
edersonbrilhante marked this pull request as ready for review September 3, 2026 18:32
@edersonbrilhante
edersonbrilhante requested a review from a team as a code owner September 3, 2026 18:32
@edersonbrilhante
edersonbrilhante force-pushed the replace-legacy-config-consumers branch from 29643bc to 2c7202c Compare September 3, 2026 19:02
@edersonbrilhante
edersonbrilhante force-pushed the replace-legacy-config-consumers branch from 2c7202c to e54c5dd Compare September 3, 2026 19:30
@edersonbrilhante
edersonbrilhante force-pushed the replace-legacy-config-consumers branch from e54c5dd to c8bbc3d Compare September 3, 2026 19:45
@edersonbrilhante
edersonbrilhante force-pushed the replace-legacy-config-consumers branch from c22c7a7 to 3f04604 Compare September 3, 2026 20:12
@edersonbrilhante
edersonbrilhante force-pushed the replace-legacy-config-consumers branch from 3f04604 to 27337b9 Compare September 3, 2026 21:11
@edersonbrilhante
edersonbrilhante requested a review from a team as a code owner September 4, 2026 17:59
@edersonbrilhante
edersonbrilhante force-pushed the replace-legacy-config-consumers branch from 7b89929 to a555fdd Compare September 4, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant