Skip to content

chore(ci): run Cloudflare deploys on Blacksmith runners - #56818

Merged
thdxr merged 1 commit into
devfrom
blacksmith-deploys
Oct 18, 2026
Merged

thdxr merged 1 commit into
devfrom
blacksmith-deploys

Conversation

@thdxr

@thdxr thdxr commented Oct 18, 2026 •

Copy link
Copy Markdown
Member

Issue for this PR

Closes #

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Moves every deploy workflow that uses CLOUDFLARE_API_TOKEN from GitHub-hosted ubuntu-latest to blacksmith-4vcpu-ubuntu-2404: deploy, deploy-changelog, deploy-files, deploy-posts, deploy-update and deploy-www. deploy.yml previously used Blacksmith only on dev; now it uses Blacksmith on every branch.

We're adding a Blacksmith static egress IP so the Cloudflare token can be restricted to that IP. GitHub-hosted runners share thousands of address ranges with all GitHub users, so jobs that use the token have to run on Blacksmith first. publish.yml already does.

How did you verify your code works?

The only change is the runs-on label, which publish.yml and deploy.yml (on dev) already use. The deploy workflows will run on the new runners after merge.

Screenshots / recordings

N/A

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

— from 𝕺𝖕𝖊𝖓𝕮𝖔𝖉𝖊

@opencode-agent opencode-agent Bot 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.

Looks good. I checked the diff and searched .github/ for every use of CLOUDFLARE_API_TOKEN. With this change, every job that uses it runs on blacksmith-4vcpu-ubuntu-2404: the six deploy workflows here, plus publish.yml, which already did. No job using the token is left on ubuntu-latest, so restricting the token to the Blacksmith egress IP shouldn't break any deploy. This is a CI-only change, so there was nothing to run locally.

Public API: unchanged.

No concerns.

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