Skip to content

Run pull request tests on Blacksmith - #300

Merged
webdevtodayjason merged 2 commits into
mainfrom
fable/ci-blacksmith
Oct 5, 2026
Merged

webdevtodayjason merged 2 commits into
mainfrom
fable/ci-blacksmith

Conversation

@webdevtodayjason

@webdevtodayjason webdevtodayjason commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • move the pull request and main branch test job from ubuntu-latest to blacksmith-4vcpu-ubuntu-2404
  • make live cluster-secret reloads recognize the atomic file replacements used by NodeConfig.save() on Blacksmith filesystems
  • preserve the three Spark image jobs because they run only for tags or workflow_dispatch
  • audit the other branches identified in the 2026-10-05 hosted-runner report

Verification

  • rg -n 'runs-on:' .github/workflows/*.yml
  • no GitHub-hosted runs-on labels remain
  • Ruff passes
  • both cluster-secret rotation regression tests pass locally
  • full local Pytest on macOS with Python 3.14: 3,250 passed, 2 skipped, 1 xfailed, and one unrelated cache-order failure in tests/test_reachable_urls.py; that test passed on both Blacksmith attempts
  • the Tests gate must pass on Blacksmith before merge

Preserved self-hosted Spark jobs

These jobs use [self-hosted, dgx-spark, aarch64] and are not triggered by pull requests:

  • .github/workflows/publish-image.yml: v* tags and workflow_dispatch
  • .github/workflows/publish-train-image.yml: train-v* tags and workflow_dispatch
  • .github/workflows/publish-whisper-image.yml: whisper-v* tags and workflow_dispatch

The lead can decide separately whether to migrate these release jobs.

Branch audit

To delete

Merged PR branches:

Abandoned branch with no PR and no commit in 30 days:

  • tools/benchchat, last commit 2026-09-04

No branches were deleted as part of this PR.

Open PR branches, rebase later

Related repository audit

getainode/ainode-desktop has one flagged branch: main.

@webdevtodayjason
webdevtodayjason merged commit 30111c1 into main Oct 5, 2026
2 checks passed
@webdevtodayjason
webdevtodayjason deleted the fable/ci-blacksmith branch October 5, 2026 20:32
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