Skip to content

chore: Run Domus CI on self hosted runners (#114) - #115

Merged
efraespada merged 7 commits into
developfrom
chore/114-run-domus-ci-on-self-hosted-runners
Nov 12, 2026
Merged

efraespada merged 7 commits into
developfrom
chore/114-run-domus-ci-on-self-hosted-runners

Conversation

@efraespada

@efraespada efraespada commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Routes the repository-owned CI Check workflow to the organization’s self-hosted runners and runs amd64 and ARM64 image validation on an Apple Silicon runner. Job-scoped Docker resources and serialized browser fixtures help isolate concurrent runs.

What changed

  • Routes all CI Check jobs to the vypdev runner group and triggers the workflow on branch pushes.
  • Builds and exercises amd64 and ARM64 images with platform-specific Docker settings and immutable Core image digests.
  • Scopes image tags, Docker configuration, and Buildx builders to each job; scopes browser fixture cleanup to the job’s image.
  • Serializes browser suites that share fixture ports and documents runner prerequisites and workflow behavior.

Validation

  • Supplied evidence reports 60 local workflow and browser bridge tests passed, along with Ruff and the documentation check.
  • Supplied evidence reports Docker preflights for both Core manifests and independent labeling and cleanup of two fixture containers.

Review notes

  • Verify a full CI Check run for runner placement and capacity, required dependencies, and complete browser and migration coverage on both architectures. The issue says the cause of existing queued runs is unconfirmed.

Closes #114

@docs-page

docs-page Bot commented Nov 12, 2026

Copy link
Copy Markdown

To preview the documentation for this pull request, visit the following URL:

docs.page/vypdev/domus~115

Documentation is deployed and generated using docs.page

@vypbot vypbot added size: L Label to indicate a task of size L 55% Progress: 55% state:ai-processing A Copilot agent is analyzing or working on the issue or change. labels Nov 12, 2026
@vypbot vypbot changed the title Run Domus CI on self-hosted runners chore: Run Domus CI on self hosted runners (#114) Nov 12, 2026
@vypbot
vypbot self-requested a review November 12, 2026 08:31
@vypbot vypbot added this to vypdev Nov 12, 2026
@vypbot vypbot moved this to In Progress in vypdev Nov 12, 2026
@vypbot
vypbot changed the base branch from develop to master November 12, 2026 08:31
@efraespada efraespada linked an issue Nov 12, 2026 that may be closed by this pull request
1 task done
@vypbot
vypbot changed the base branch from master to develop November 12, 2026 08:31
@vypbot vypbot added 60% Progress: 60% and removed 55% Progress: 55% labels Nov 12, 2026

@vypbot vypbot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Current status: All findings originating in this review are resolved.
Last reconciled on c6a6958. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 1e67ac7 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • high: Fork PRs can bypass the self-hosted runner guard — .github/workflows/ci_check.yml:20

Comment thread .github/workflows/ci_check.yml Resolved
@vypbot

vypbot commented Nov 12, 2026 •

Copy link
Copy Markdown

Bugbot: review complete

Current status: No active findings on c6a6958.

Pull request · Verified commit · Workflow run

@vypbot vypbot added state:changes-requested Review identified changes that are required. state:awaiting-issue-author The next action requires more information or changes from the issue author. and removed state:ai-processing A Copilot agent is analyzing or working on the issue or change. labels Nov 12, 2026
@vypbot vypbot added 45% Progress: 45% state:ai-processing A Copilot agent is analyzing or working on the issue or change. and removed 60% Progress: 60% labels Nov 12, 2026

@vypbot vypbot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Current status: All findings originating in this review are resolved.
Last reconciled on c6a6958. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 27ffe95 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • medium: Fork pull requests receive no upstream CI run — .github/workflows/ci_check.yml:4

Comment thread .github/workflows/ci_check.yml Resolved
@vypbot vypbot removed the state:ai-processing A Copilot agent is analyzing or working on the issue or change. label Nov 12, 2026
@efraespada

Copy link
Copy Markdown
Member Author

/copilot dismiss "ci_check.yml:4:fork PRs receive no upstream CI run"

@vypbot vypbot removed the state:changes-requested Review identified changes that are required. label Nov 12, 2026
@vypbot vypbot added state:reviewing A pull request is being reviewed. and removed state:awaiting-issue-author The next action requires more information or changes from the issue author. labels Nov 12, 2026
@vypbot

vypbot commented Nov 12, 2026

Copy link
Copy Markdown

No matching findings were found. Nothing was dismissed.

@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@vypbot vypbot added state:ai-processing A Copilot agent is analyzing or working on the issue or change. state:ready The change is ready for human approval or merge. state:awaiting-maintainer The next action requires a maintainer response or approval. and removed state:reviewing A pull request is being reviewed. state:ai-processing A Copilot agent is analyzing or working on the issue or change. labels Nov 12, 2026
@vypbot vypbot added size: XL Label to indicate a task of size XL 60% Progress: 60% state:ai-processing A Copilot agent is analyzing or working on the issue or change. and removed size: L Label to indicate a task of size L 45% Progress: 45% labels Nov 12, 2026
@vypbot vypbot added 55% Progress: 55% and removed 60% Progress: 60% state:ai-processing A Copilot agent is analyzing or working on the issue or change. labels Nov 12, 2026
@efraespada

Copy link
Copy Markdown
Member Author

The maintainer approved a temporary exception for Branch Sync during the GitHub Actions queue incident. Run 43095218150 remains queued with zero jobs, and an earlier Branch Sync timed out waiting for it.

The project CI, Copilot Verification and Copilot Review passed on c6a69588674d72e30300f2ca98518799ddcae1e3. The exception applies to the blocked Branch Sync run during this incident.

@efraespada
efraespada merged commit ff6b64f into develop Nov 12, 2026
16 of 17 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in vypdev Nov 12, 2026
@vypbot vypbot added state:verified The change has passed Copilot verification. and removed state:ready The change is ready for human approval or merge. state:awaiting-maintainer The next action requires a maintainer response or approval. labels Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

55% Progress: 55% size: XL Label to indicate a task of size XL state:verified The change has passed Copilot verification.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

🔧🧑‍💻 - Run Domus CI on self hosted runners

2 participants