Skip to content

Update runner-doctor catalog with B29, B30, and C10 - #8070

Merged
lpcox merged 4 commits into
mainfrom
copilot/update-runner-doctor-lessons
Sep 4, 2026
Merged

Update runner-doctor catalog with B29, B30, and C10#8070
lpcox merged 4 commits into
mainfrom
copilot/update-runner-doctor-lessons

Conversation

Copilot AI commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Adds three newly identified self-hosted/GHEC failure-mode lessons to the shared catalog, workflow playbook, and portable runner-doctor agent:

  • B29 — Container workdir mount gaps: Documents codex-engine context-rebuild failures caused by an expected workspace path missing from the chroot.
  • B30 — Startup diagnostics: Documents pre-egress failures that previously left no Squid logs and caused awf logs summary to report no sources.
  • C10 — Fine-grained PAT authentication: Documents incorrect Copilot Authorization schemes, GHEC model-discovery gaps, and legacy placeholder credential handling.

Each entry includes remediation, probes, citations, and the corresponding quick-lookup signal. The workflow alignment tests and generated lock metadata are updated to keep all catalog copies synchronized.

Example diagnostic mappings:

context-rebuild circuit breaker tripped + failed cd → B29
no log sources found after startup failure → B30
wrong Copilot Authorization scheme with github_pat_... → C10

Copilot AI changed the title [WIP] Update Runner Doctor with new self-hosted failure-mode lessons Update runner-doctor catalog with B29, B30, and C10 Sep 3, 2026
Copilot AI requested a review from lpcox September 3, 2026 20:37
@lpcox
lpcox marked this pull request as ready for review September 3, 2026 23:06
Copilot AI balanced review requested due to automatic review settings September 3, 2026 23:06

Copilot AI 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.

🟡 Changes recommended

The portable catalog accidentally replaces C9 with C10, leaving its C9 references unresolved.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates runner-doctor knowledge for three recently fixed AWF failure modes.

Changes:

  • Adds B29, B30, and C10 diagnostics and remediation.
  • Synchronizes workflow and portable-agent guidance.
  • Updates alignment tests and generated metadata.
File summaries
File Description
scripts/ci/self-hosted-runner-doctor-workflow.test.ts Tests new catalog entries.
.github/workflows/shared/self-hosted-failure-modes.md Adds canonical failure-mode records.
.github/workflows/self-hosted-runner-doctor.md Adds diagnostic mappings and playbook guidance.
.github/workflows/self-hosted-runner-doctor.lock.yml Updates generated body hash.
.github/agents/self-hosted-runner-doctor.md Updates portable catalog and playbook.
Review details
  • Files reviewed: 5/5 changed files
  • Comments generated: 1
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/agents/self-hosted-runner-doctor.md
@lpcox lpcox closed this Sep 4, 2026
@lpcox lpcox reopened this Sep 4, 2026
@lpcox

lpcox commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

@copilot address review feedback and rebase main

Copilot AI and others added 2 commits September 4, 2026 05:12
…-doctor-lessons

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

Copilot AI commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

@copilot address review feedback and rebase main

Restored the C9 row in the portable runner-doctor catalog and added an alignment assertion in commit 9f6833c. Main was merged into the branch (rather than rebased) so the update could be published without rewriting history.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

✅ Copilot review passed with no inline comments.

@copilot Add the ready-for-aw label to this PR to trigger agentic CI smoke tests.

@lpcox
lpcox merged commit a6a4ed9 into main Sep 4, 2026
23 of 25 checks passed
@lpcox
lpcox deleted the copilot/update-runner-doctor-lessons branch September 4, 2026 13:42
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.

🩺 Runner Doctor UpdateRunner Doctor: 3 new self-hosted/GHEC failure-mode lessons (2026-09-01 scan)

3 participants