Migrate lightweight workflows to ubuntu-slim runners#4831
Conversation
Unit Test Results0 tests 0 ✅ 0s ⏱️ Results for commit e2f22a0. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Pull request overview
This PR migrates lightweight GitHub Actions workflows to ubuntu-slim runners (1 vCPU, 5GB RAM, 15min timeout) for improved startup performance and reduced costs. The changes target short-running workflows that use only JavaScript actions and have no Docker dependencies.
Changes:
- Migrated 4 workflows/jobs to ubuntu-slim runners: PR author verification, comment parsing, documentation deployment, and branch deployment preparation
- Updated CHANGELOG.md to document the migration
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/pr_comment_bot.yml |
Migrates comment parsing job to ubuntu-slim (destroy jobs intentionally kept on ubuntu-latest due to timeout constraints) |
.github/workflows/flag_external_pr.yml |
Migrates PR author verification job to ubuntu-slim |
.github/workflows/deploy_tre_branch.yml |
Migrates branch preparation/hash computation job to ubuntu-slim |
.github/workflows/build_docs.yml |
Migrates documentation build and deployment to ubuntu-slim |
CHANGELOG.md |
Documents the workflow migration under ENHANCEMENTS section |
|
/test-force-approve |
|
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit 90a9f11) (in response to this comment from @tamirkamara) |
Co-authored-by: tamirkamara <26870601+tamirkamara@users.noreply.github.com>
…imes Co-authored-by: tamirkamara <26870601+tamirkamara@users.noreply.github.com>
Co-authored-by: tamirkamara <26870601+tamirkamara@users.noreply.github.com>
Co-authored-by: tamirkamara <26870601+tamirkamara@users.noreply.github.com>
90a9f11 to
e2f22a0
Compare
|
/test-force-approve |
|
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit e2f22a0) (in response to this comment from @tamirkamara) |
What is being addressed
GitHub's ubuntu-slim runners (1 vCPU, 5GB RAM, 15min timeout, container-based) offer faster startup and lower costs. Current workflows uniformly use ubuntu-latest regardless of actual resource needs or Docker requirements.
How is this addressed
Migrated 4 workflows to ubuntu-slim:
flag_external_pr.yml- PR author verificationpr_comment_bot.yml- comment parsing job only (not destroy jobs - may exceed timeout)build_docs.yml- mkdocs build/deploydeploy_tre_branch.yml- hash computationSelection criteria: <5min execution, JavaScript actions only, no Docker dependencies.
Not migrated due to Docker container action incompatibilities:
test_results.yml- usesEnricoMi/publish-unit-test-result-action@v2(Docker)build_docker_images.yml- usesdocker/build-push-actionbuild_validation_develop.yml- usessuper-linter(Docker)cli-package.yml- uses devcontainer (Docker)Not migrated due to timeout constraints:
clean_validation_envs.yml- 30min timeout exceeds ubuntu-slim 15min limitpr_comment_bot.ymldestroy jobs - Azure RG deletion timingAction type verification:
actions/checkout,actions/setup-python,actions/github-script,azure/login,LouisBrunner/checks-actionEnricoMi/publish-unit-test-result-action@v2Updated CHANGELOG.md with PR reference (#4831).
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.