ci: move jobs off dead Depot runners to GitHub-hosted - #39
Merged
Conversation
Seven jobs never run — they queue until GitHub cancels them at 24h. Bumps test/lint timeouts 15m->30m for the smaller GitHub runners.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Unblocks CI on
develop. Infra-only — no application code touched.Problem
Seven jobs never run on this branch. They aren't failing — they sit
queuedforever untilGitHub cancels them at the 24-hour mark, which is why they surface as
failwith a duration of24h0m0s:runs-ontest-unit-coverdepot-ubuntu-24.04-16test-fuzzdepot-ubuntu-24.04-4Run golangci-lintdepot-ubuntu-24.04-8jsonrpc-compatibility-testdepot-ubuntu-22.04-8Analyze (go / javascript / python)depot-ubuntu-24.04-8depot-*is a custom label served only by Depot runners. GitHub hasnothing to dispatch these jobs to unless Depot is connected — behaviourally identical to a
self-hosted label with no runner online. Depot did work here historically (
system-testsucceeded2026-06-16 and 2026-06-27), but every run since ~2026-07-27 is stuck, so access lapsed
account-side. The labels are inherited from upstream
cosmos/evm, not something this fork chose.Consequence: the Go unit suite, lint, fuzz, CodeQL and JSON-RPC compat have never validated this
branch. Concretely, PR #37 adds two new test files that have never executed in CI.
Change
Point those jobs at GitHub-hosted runners, which are free and unmetered for public repositories:
GitHub's free runners are 4-core where Depot's were 4–16, so timeouts sized for the bigger
boxes need headroom — otherwise this trades queued forever for timeout:
Makefile: test/lint timeouts15m → 30m(COMMON_COVER_ARGS,test-unit,test-evmd,test-unit-cover,test-all, and golangci's own--timeout)lint.yml: golangcitimeout-minutes: 15 → 30jsonrpc-compatibility-testkeeps its existing cap;test-unit-cover/test-fuzzhave no jobcap and inherit GitHub's 360 min default.
Relationship to #36
This is the same fix #36 applied to the v0.7 line (now
develop-0.7). It was never on the v0.6line, which is now
develop. This branch additionally coverscodeql-analysis.yml, which #36 didnot touch.
Alternative worth trying first
Depot offers a free tier for open-source projects and this repo qualifies. If someone
reconnects Depot under that plan, everything goes green with zero changes and the fork's
workflows stay byte-identical to upstream — which keeps future upstream merges clean. This PR is
the fallback if that isn't viable; it's a small revert either way.
Validation
The real proof is this PR's own checks: the five jobs above should go from
24h0m0s / failtoactually executing. Please confirm they run green before merging — if
test-unit-covernow failsfor a genuine reason, that's a pre-existing breakage this change has surfaced, not caused.