Skip to content

feat(storage): add runner config consumer - #5358

Merged
edersonbrilhante merged 4 commits into
mainfrom
runner-config-consumer
Sep 8, 2026
Merged

feat(storage): add runner config consumer#5358
edersonbrilhante merged 4 commits into
mainfrom
runner-config-consumer

Conversation

@edersonbrilhante

@edersonbrilhante edersonbrilhante commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Description

Adds the provider-neutral runner-config consumer and its AWS SSM implementation. The consumer polls the configured per-runner token path until a one-time JIT or registration configuration is available, applies bounded per-call and overall timeouts, and supports cancellation during polling and backoff.

After reading a configuration it deletes the consumed parameter with bounded retries, classifies retryable provider failures, validates the storage context and environment-derived limits, and exposes safe failure behavior to compute providers. The old environment-based factory remains as a deprecated compatibility alias while callers migrate to createRunnerConfigConsumer.

Test Plan

  • Added/updated consumer tests for polling, timeout, cancellation, retry, deletion, and error sanitization.
  • git diff --check passed.
  • Runtime Yarn tests could not be run locally because the repository's pinned Yarn launcher is unavailable in this environment; CI should provide the full test result.

Related Issues

@github-actions

github-actions Bot commented Sep 4, 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 runner-config-consumer branch 3 times, most recently from 1c8ca59 to 4b2e53c Compare September 4, 2026 12:31
@edersonbrilhante
edersonbrilhante marked this pull request as ready for review September 7, 2026 13:54
@edersonbrilhante
edersonbrilhante requested a review from a team as a code owner September 7, 2026 13:54
guicaulada
guicaulada previously approved these changes Sep 8, 2026
guicaulada
guicaulada previously approved these changes Sep 8, 2026
Base automatically changed from github-app-credentials-storage to main September 8, 2026 19:48
@edersonbrilhante
edersonbrilhante merged commit 138309c into main Sep 8, 2026
7 checks passed
@edersonbrilhante
edersonbrilhante deleted the runner-config-consumer branch September 8, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants