Skip to content

ci: run check-skip on blacksmith with GitHub-hosted fallback#7219

Merged
PastaPastaPasta merged 1 commit intodashpay:v23.1.xfrom
thepastaclaw:ci/check-skip-blacksmith
Mar 14, 2026
Merged

ci: run check-skip on blacksmith with GitHub-hosted fallback#7219
PastaPastaPasta merged 1 commit intodashpay:v23.1.xfrom
thepastaclaw:ci/check-skip-blacksmith

Conversation

@thepastaclaw
Copy link

Issue

The check-skip job runs on ubuntu-latest (GitHub-hosted only). When all GitHub runners are busy, this lightweight job stalls — blocking the entire CI pipeline since every other job depends on it.

Fix

Use vars.RUNNER_AMD64 with a fallback to ubuntu-24.04:

runs-on: ${{ vars.RUNNER_AMD64 || 'ubuntu-24.04' }}
  • dashpay org: RUNNER_AMD64 is set to a Blacksmith runner label → uses Blacksmith
  • External forks: Variable not set → falls back to GitHub-hosted ubuntu-24.04

This is the same pattern already used for build jobs in the repo.

The check-skip job uses ubuntu-latest (GitHub-hosted only), which stalls
when all GitHub runners are busy — blocking the entire CI pipeline.

Use vars.RUNNER_AMD64 (set to blacksmith in the dashpay org) with a
fallback to ubuntu-24.04 for external forks that don't have the variable
configured.
@UdjinM6 UdjinM6 added this to the 24 milestone Mar 12, 2026
@github-actions
Copy link

✅ No Merge Conflicts Detected

This PR currently has no conflicts with other open PRs.

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK d02243c

@PastaPastaPasta PastaPastaPasta merged commit 366090d into dashpay:v23.1.x Mar 14, 2026
38 of 42 checks passed
@PastaPastaPasta
Copy link
Member

Whoops; this was merged into v23.1.x 🙈

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.

3 participants