Skip to content

chore(ci): switch non-hot-path workflows from Blacksmith to ubuntu-latest#1134

Open
pyramation wants to merge 1 commit into
mainfrom
chore/downsize-ci-runners
Open

chore(ci): switch non-hot-path workflows from Blacksmith to ubuntu-latest#1134
pyramation wants to merge 1 commit into
mainfrom
chore/downsize-ci-runners

Conversation

@pyramation
Copy link
Copy Markdown
Contributor

Summary

Switch non-hot-path CI workflows from Blacksmith runners to ubuntu-latest to reduce Blacksmith usage. Only workflows that don't run on the PR critical path are affected:

Workflow Trigger Before After
docker.yaml workflow_dispatch only blacksmith-4vcpu-ubuntu-2404 ubuntu-latest
docker-constructive.yaml (build) push to main blacksmith-4vcpu-ubuntu-2404 + arm ubuntu-latest (arm via QEMU)
docker-constructive.yaml (manifest) push to main blacksmith-2vcpu-ubuntu-2404 ubuntu-latest
notify-e2e.yml push to main blacksmith-2vcpu-ubuntu-2404 ubuntu-latest

Hot-path jobs unchangedbuild, unit-tests, pg-tests, integration-tests, and examples-integration remain on Blacksmith for fast PR feedback.

Review & Testing Checklist for Human

  • Verify Docker builds still succeed on ubuntu-latest after next push to main (Docker + Buildx are pre-installed on GitHub runners)
  • Confirm notify-e2e still triggers constructive-hub E2E tests on push to main

Notes

  • docker-constructive.yaml previously used a native ARM runner (blacksmith-4vcpu-ubuntu-2404-arm). With ubuntu-latest, ARM builds will use QEMU emulation which is slower but still functional. If ARM build speed matters, this can be revisited.
  • All PR-blocking test workflows remain on Blacksmith — no impact on developer iteration speed.

Link to Devin session: https://app.devin.ai/sessions/f42f42528fb0465684159227efbef017
Requested by: @pyramation

…test

Docker builds (docker.yaml, docker-constructive.yaml) and notify-e2e
only run on push-to-main or workflow_dispatch — not on the PR critical
path. Switch them to ubuntu-latest to reduce Blacksmith usage.

Hot-path jobs (build, unit-tests, pg-tests, integration-tests) remain
on Blacksmith for fast PR feedback.
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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