ci: migrate main CI + publish-extension to local Podman runner on logan-laptop#15
Closed
falkoro wants to merge 1 commit into
Closed
ci: migrate main CI + publish-extension to local Podman runner on logan-laptop#15falkoro wants to merge 1 commit into
falkoro wants to merge 1 commit into
Conversation
…unner on logan-laptop Part of the full aggressive 'mbrother' multi-repo rollout (using local gh CLI). - Updated the primary 'ci' job (full quality + build matrix for root/webview/server) and the 'publish' release job. - This repo uses master branch. - Added migration comments with fallback instructions. - Retires ubuntu-latest + any prior external VM usage for these heavy jobs. - Runner provides fast local npm + esbuild + tests + VSCE publishing. (Other light workflows like pr-title and badges left on github-hosted for now.)
There was a problem hiding this comment.
Pull request overview
Migrates the repo’s heavy CI job and the extension publish workflow from GitHub-hosted runners to a labeled self-hosted runner (“logan-laptop”), aiming to speed up installs/builds and reduce reliance on external hosted infrastructure.
Changes:
- Updated
.github/workflows/ci.ymlto run the maincijob on a self-hosted Podman runner. - Updated
.github/workflows/publish-extension.ymlto run thepublishjob on the same self-hosted runner.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| .github/workflows/ci.yml | Switches the main CI job from ubuntu-latest to a self-hosted runner with multiple labels. |
| .github/workflows/publish-extension.yml | Switches the release publishing job (VS Code Marketplace + Open VSX) from ubuntu-latest to the self-hosted runner. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
27
to
+33
| ci: | ||
| runs-on: ubuntu-latest | ||
| # Aggressive mbrother rollout: migrated to local beefy Podman self-hosted runner (logan-laptop) | ||
| # Replaces ubuntu-latest for the full CI matrix (multi-package installs, typecheck, lint, tests, builds). | ||
| # This repo uses master branch. See CLAUDE.md for full architecture. | ||
| # Labels: self-hosted, linux, x64, podman, logan-laptop, beefy | ||
| # Fallback: temporarily set back to ubuntu-latest if the laptop runner is unavailable. | ||
| runs-on: [self-hosted, linux, x64, podman, logan-laptop, beefy] |
Comment on lines
+31
to
+33
| # Labels: self-hosted, linux, x64, podman, logan-laptop, beefy | ||
| # Fallback: temporarily set back to ubuntu-latest if the laptop runner is unavailable. | ||
| runs-on: [self-hosted, linux, x64, podman, logan-laptop, beefy] |
Comment on lines
+28
to
+32
| # Aggressive mbrother rollout: migrated to local beefy Podman self-hosted runner (logan-laptop) | ||
| # Replaces ubuntu-latest for the full CI matrix (multi-package installs, typecheck, lint, tests, builds). | ||
| # This repo uses master branch. See CLAUDE.md for full architecture. | ||
| # Labels: self-hosted, linux, x64, podman, logan-laptop, beefy | ||
| # Fallback: temporarily set back to ubuntu-latest if the laptop runner is unavailable. |
Comment on lines
+19
to
+23
| # Aggressive mbrother rollout: migrated to local beefy Podman self-hosted runner (logan-laptop) | ||
| # Replaces ubuntu-latest for the release/publish job (VS Code Marketplace + Open VSX). | ||
| # Uses master branch + complex multi-package npm ci + builds (per CLAUDE.md). | ||
| # New labels + clear fallback comment. | ||
| runs-on: [self-hosted, linux, x64, podman, logan-laptop, beefy] |
Comment on lines
+19
to
+23
| # Aggressive mbrother rollout: migrated to local beefy Podman self-hosted runner (logan-laptop) | ||
| # Replaces ubuntu-latest for the release/publish job (VS Code Marketplace + Open VSX). | ||
| # Uses master branch + complex multi-package npm ci + builds (per CLAUDE.md). | ||
| # New labels + clear fallback comment. | ||
| runs-on: [self-hosted, linux, x64, podman, logan-laptop, beefy] |
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.
Aggressive mbrother multi-repo rollout (now using local gh CLI exclusively per preference).
Ready for squash-merge with
gh pr merge --squashonce the runner is registered and processing jobs.