Skip to content

refactor(ec2): decouple creation result from control plane - #5312

Open
edersonbrilhante wants to merge 1 commit into
refactor-ec2-runner-orchestration-boundaryfrom
refactor-ec2-orchestration-support
Open

refactor(ec2): decouple creation result from control plane#5312
edersonbrilhante wants to merge 1 commit into
refactor-ec2-runner-orchestration-boundaryfrom
refactor-ec2-orchestration-support

Conversation

@edersonbrilhante

@edersonbrilhante edersonbrilhante commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Description

Separates the result returned by shared EC2 runner creation from the control-plane CreateRunnerResult contract.

createEc2Runner() and Ec2RunnerResourceOperations.create() now return Ec2RunnerCreateResult, containing created instance IDs, the failed-instance count, and bounded AWS failure identifiers.

The control-plane adapter converts that result into its existing CreateRunnerResult and applies retryable or non-retryable classification using scaleErrors. The core contract and existing pool and scale-up behavior remain unchanged.

Test Plan

  • Compute-provider suite: 14 files, 295 tests passed.
  • Control-plane suite: 14 files, 345 tests passed.
  • Control-plane TypeScript type-check passed.
  • ESLint and Prettier passed for changed EC2 TypeScript files.
  • git diff --check and pre-commit checks passed.

Related Issues

Depends on #5301.

@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 refactor-ec2-orchestration-support branch 2 times, most recently from c452fb1 to 6a5790d Compare August 27, 2026 20:15
@edersonbrilhante edersonbrilhante changed the title refactor(ec2): support runner orchestration refactor(ec2): decouple runner creation results Aug 27, 2026
@edersonbrilhante
edersonbrilhante force-pushed the refactor-ec2-orchestration-support branch from 6a5790d to 3179711 Compare August 27, 2026 20:35
@edersonbrilhante edersonbrilhante changed the title refactor(ec2): decouple runner creation results refactor(ec2): decouple creation result from control plane Aug 27, 2026
@edersonbrilhante
edersonbrilhante marked this pull request as ready for review August 27, 2026 20:44
@edersonbrilhante
edersonbrilhante requested a review from a team as a code owner August 27, 2026 20:44
@edersonbrilhante
edersonbrilhante force-pushed the refactor-ec2-orchestration-support branch from 3179711 to 0d1969f Compare August 27, 2026 21:00
@edersonbrilhante
edersonbrilhante force-pushed the refactor-ec2-orchestration-support branch from 0d1969f to f990720 Compare August 28, 2026 11:03
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.

1 participant