-
Notifications
You must be signed in to change notification settings - Fork 0
Jrepp - Self Hosted Runners #206
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
User request: "convert ci and release to use a self managed runner" Migrated from GitHub-hosted to self-managed runners: - ci.yml: All 17 ubuntu-latest instances → self-hosted - release.yml: All ubuntu-latest instances → self-hosted - Preserved macos-latest for macOS builds (conservative approach) Benefits: - Cost optimization for self-managed infrastructure - Better control over build environment and caching - Consistent runner configuration across workflows Note: macOS runners remain on GitHub-hosted (macos-latest) as self-hosted macOS runners require additional setup. Co-Authored-By: Claude <noreply@anthropic.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates all GitHub Actions workflow jobs from GitHub-hosted ubuntu-latest runners to self-hosted runners. The changes affect both the CI and release workflows, transitioning the entire build, test, and deployment infrastructure to self-hosted runner infrastructure.
Key Changes:
- All job runner configurations changed from
ubuntu-latesttoself-hosted - Affects both
.github/workflows/ci.ymland.github/workflows/release.yml - No changes to job logic, timeouts, or dependencies
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/release.yml | Updates all 8 jobs (proto generation, binary builds, image builds, release creation, etc.) to use self-hosted runners |
| .github/workflows/ci.yml | Updates all 18 jobs (matrix generation, linting, building, testing, coverage, status checks) to use self-hosted runners |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automatically approving PR from repo owner
🧪 CI InsightsHere's what we observed from your CI run for 35f5218. 🟢 All jobs passed!But CI Insights is watching 👀 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #206 +/- ##
===========================================
+ Coverage 30.59% 64.01% +33.42%
===========================================
Files 8 21 +13
Lines 572 2971 +2399
===========================================
+ Hits 175 1902 +1727
- Misses 397 943 +546
- Partials 0 126 +126
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
## Summary Reverts the self-hosted runners configuration from PR #206 and restores GitHub-hosted runners (ubuntu-latest) for all CI workflows. **Changes**: - `.github/workflows/ci.yml`: All 17 jobs changed from `self-hosted` back to `ubuntu-latest` - `.github/workflows/release.yml`: All 8 jobs changed from `self-hosted` back to `ubuntu-latest` **Rationale**: Reverting to standard GitHub-hosted runners for improved reliability and maintainability. ## Test plan - [ ] Verify CI workflow runs successfully on GitHub-hosted runners - [ ] Verify release workflow configuration is valid - [ ] Confirm all required checks pass
User request: "look at all local branches for unmerged commits, create PRs if they are found by first merging origin/main and submitting the commit data"
This branch contains 3 unmerged commit(s). Final aggressive resolution applied.
Co-Authored-By: Claude noreply@anthropic.com