Skip to content

ci: use dedicated release runners - #645

Open
aidandaly24 wants to merge 1 commit into
aws:mainfrom
aidandaly24:fix/release-runner
Open

ci: use dedicated release runners#645
aidandaly24 wants to merge 1 commit into
aws:mainfrom
aidandaly24:fix/release-runner

Conversation

@aidandaly24

Copy link
Copy Markdown
Contributor

Summary

Moves the manually triggered release preparation workflow to the dedicated aws-release-4-core GitHub-hosted runner group. This prevents release preparation from waiting behind the shared ubuntu-latest pool.

The release-publish.yml workflow is intentionally unchanged because it is triggered by pull_request: closed; the dedicated runner is restricted to release workflows triggered by workflow_dispatch or pushes to main.

Testing

Parsed both release workflows as YAML, verified every job in release-prepare.yml uses aws-release-4-core, and verified the pull-request-triggered publish workflow does not use the dedicated runner. Source tests were not run because this change only updates workflow runner labels.

Security

No permissions, triggers, dependencies, or secrets changed.

@aidandaly24
aidandaly24 requested a review from a team August 20, 2026 01:15
@github-actions github-actions Bot added the size/xs PR size: XS label Aug 20, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 20, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 20, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@b981f7e). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #645   +/-   ##
=======================================
  Coverage        ?   88.92%           
=======================================
  Files           ?      120           
  Lines           ?    10202           
  Branches        ?     1563           
=======================================
  Hits            ?     9072           
  Misses          ?      749           
  Partials        ?      381           
Flag Coverage Δ
unittests 88.92% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tejaskash tejaskash left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the release-runner changes. Release preparation and validation move consistently to the dedicated runner, while the pull-request-triggered publish workflow remains on the shared runner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants