Skip to content

chore: use hosted GitHub runners#78

Open
yocaba wants to merge 1 commit into
mainfrom
chore/move-github-runner
Open

chore: use hosted GitHub runners#78
yocaba wants to merge 1 commit into
mainfrom
chore/move-github-runner

Conversation

@yocaba
Copy link
Copy Markdown
Contributor

@yocaba yocaba commented May 12, 2026

What

Relates to https://github.com/opendefensecloud/odd-internal/issues/28.

Testing

n/a

Checklist

  • Tests added/updated n/a
  • No breaking changes (or upgrade path documented above)
  • Readable commit history (squashed and cleaned up as desired)
  • AI code review considered and comments resolved

Summary by CodeRabbit

  • Chores
    • Updated internal CI/CD infrastructure configuration.

Note: This release contains only infrastructure updates with no user-facing changes.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 12, 2026

📝 Walkthrough

Walkthrough

This PR updates the GitHub Actions workflow to use ubuntu-24.04 as the runner environment for both the lint and test jobs, replacing the previous arc-scale-set runner. No job logic or step definitions were modified.

Changes

CI Runner Migration

Layer / File(s) Summary
Migrate CI jobs to ubuntu-24.04
.github/workflows/golang.yaml
The lint and test job runner environments are updated from arc-scale-set to ubuntu-24.04.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A bunny hops through CI with glee,
ubuntu-24 now for all to see,
No more arc-scale, just runners so lean,
Tests and lints on the latest machine! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete and missing required sections from the template. Add a concise one-sentence summary under 'What' that describes the change (not just the issue reference). Complete the 'Why' section explaining the motivation. Clarify testing approach. Mark the AI review checklist item as complete if applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: use hosted GitHub runners' accurately and concisely describes the main change: switching from arc-scale-set to ubuntu-24.04 hosted runners.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/move-github-runner

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 25735940915

Coverage remained the same at 27.101%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 476
Covered Lines: 129
Line Coverage: 27.1%
Coverage Strength: 0.72 hits per line

💛 - Coveralls

@yocaba
Copy link
Copy Markdown
Contributor Author

yocaba commented May 12, 2026

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 12, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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