Skip to content

fix(ci): migrate self-hosted to ubuntu-latest for PR safety (ADR-FM-027 D3) - #91

Open
hummbl-dev wants to merge 1 commit into
mainfrom
fix/devin/ci-ubuntu-latest
Open

fix(ci): migrate self-hosted to ubuntu-latest for PR safety (ADR-FM-027 D3)#91
hummbl-dev wants to merge 1 commit into
mainfrom
fix/devin/ci-ubuntu-latest

Conversation

@hummbl-dev

Copy link
Copy Markdown
Collaborator

Summary

  • Migrates runs-on from [self-hosted, Linux, X64, wsl, <repo>] to ubuntu-latest
  • Eliminates P1 security exposure: pull_request trigger was running untrusted PR code on self-hosted runners
  • Per GitHub guidance, self-hosted runners should never be used on public repos with PR triggers
  • ubuntu-latest is ephemeral (destroyed after each job), eliminating the risk

Context

ADR-FM-027 D3 Phase 1 — Fleet CI runner security remediation. 6 public repos under hummbl-dev-org had pull_request + self-hosted in ci.yml. This PR fixes one of them.

Test plan

  • CI passes on this PR (runs on ubuntu-latest, confirming the migration works)
  • No test failures related to runner environment change

…27 D3)

Eliminates P1 security exposure: pull_request trigger was running untrusted PR code on self-hosted runners. Per GitHub guidance, self-hosted runners should never be used on public repos with PR triggers. ubuntu-latest is ephemeral and eliminates the risk.

@hummbl-agent hummbl-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Operator-directed batch triage merge.

@hummbl-agent
hummbl-agent enabled auto-merge (squash) August 2, 2026 05:30
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.

2 participants