Skip to content

ci: run go and buf jobs on github-hosted runners - #363

Merged
baktun14 merged 1 commit into
mainfrom
ci/github-hosted-go-and-buf-jobs
Sep 28, 2026
Merged

baktun14 merged 1 commit into
mainfrom
ci/github-hosted-go-and-buf-jobs

Conversation

@baktun14

@baktun14 baktun14 commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

📝 Description

Moves the Go and buf jobs to GitHub-hosted runners, where they stop failing for reasons outside the code. Also points setup-go at go/go.sum so its module cache restores.

🔧 Purpose of the Change

  • Other: CI reliability

📌 Related Issues

✅ Checklist

  • I've updated relevant documentation
  • Code follows Akash Network's style guide
  • I've added/updated relevant unit tests
  • Dependencies have been properly updated
  • I agree and adhered to the Contribution Guidelines

📎 Notes for Reviewers

@coderabbitai

coderabbitai Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: akash-network/chain-sdk/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: 1f26cfb7-4ea4-435e-bb95-5f645cfdfd45

📥 Commits

Reviewing files that changed from the base of the PR and between 9e151b6 and 8153368.

📒 Files selected for processing (3)
  • .github/workflows/breakage.yaml
  • .github/workflows/lint.yaml
  • .github/workflows/tests.yaml

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.


Walkthrough

CI workflow jobs changed from self-hosted runners to ubuntu-latest. Go setup steps now specify cache dependency paths for the Go and SDL modules.

Changes

CI Workflow Settings

Layer / File(s) Summary
Runner and cache settings
.github/workflows/breakage.yaml, .github/workflows/lint.yaml, .github/workflows/tests.yaml
Breakage, Go lint, proto lint, test, and coverage jobs now use ubuntu-latest. Go setup steps specify go/go.sum; SDL parity specifies go/sdl/go.sum.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: troian

Merge Risk: ⚪ Minimal · up to 81533

The Go and Buf-related CI jobs now use GitHub-hosted Ubuntu runners. The reviewed workflows set up their required tools, and no concrete incompatibility was found; the change appears ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the primary change: moving Go and Buf CI jobs to GitHub-hosted runners.
Description check ✅ Passed The description includes all required sections and explains the runner changes, cache configuration, related issue, checklist status, and reviewer notes.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

A rabbit checks the runners in a row
Hosted paths are set for Go to know
Cache clues point to sums in place
Workflows hop along at a steady pace
Ears perk up; the checks begin to go

Comment @coderabbitai help to get the list of available commands.

@baktun14
baktun14 force-pushed the ci/github-hosted-go-and-buf-jobs branch from 9e151b6 to f6801cb Compare September 25, 2026 11:56
These jobs fail intermittently on the self-hosted runners for reasons
outside the code, mostly rejected buf.build downloads and interrupted builds.

- run test, coverage, go lint, proto lint and the breakage check on
  ubuntu-latest
- point setup-go at go/go.sum (go/sdl/go.sum for sdl-parity) so its
  cache restores
@baktun14
baktun14 merged commit 8b760e9 into main Sep 28, 2026
11 of 12 checks passed
@baktun14
baktun14 deleted the ci/github-hosted-go-and-buf-jobs branch September 28, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants