Skip to content

feat: support dedicated host ec2 overrides#5191

Open
edersonbrilhante wants to merge 4 commits into
mainfrom
dedicated-host-runinstances-overrides
Open

feat: support dedicated host ec2 overrides#5191
edersonbrilhante wants to merge 4 commits into
mainfrom
dedicated-host-runinstances-overrides

Conversation

@edersonbrilhante

Copy link
Copy Markdown
Contributor

Description

Adds support for applying supported dynamic EC2 override labels to dedicated-host runner launches that use RunInstances.

This maps the parsed ec2OverrideConfig into the RunInstances request for supported fields:

  • InstanceType
  • SubnetId
  • ImageId
  • Placement
  • BlockDeviceMappings

Fleet-only override fields such as InstanceRequirements, MaxPrice, Priority, and WeightedCapacity are intentionally not forwarded to RunInstances.

Also updates the dynamic label documentation to show which EC2 override labels work with EC2 Fleet and which work with dedicated-host launches.

Test Plan

  • Added unit test coverage for dedicated-host RunInstances overrides.
  • Verified supported EC2 override config values are passed to RunInstances.
  • Verified Fleet-only fields are not passed to RunInstances.
  • Ran yarn exec vitest run functions/control-plane/src/aws/runners.test.ts --config functions/control-plane/vitest.config.ts
  • Ran yarn exec eslint functions/control-plane/src/aws/runners.ts functions/control-plane/src/aws/runners.test.ts
  • Ran yarn workspace @aws-github-runner/control-plane build
  • Ran git diff --check

Related Issues

N/A

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

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

Scanned Files

None

@edersonbrilhante edersonbrilhante marked this pull request as ready for review June 29, 2026 20:47
@edersonbrilhante edersonbrilhante requested a review from a team as a code owner June 29, 2026 20:47
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