Skip to content

Expand self-hosted runner diagnostics for GHES, orphaned networks, and ARC/DinD logs - #9207

Merged
lpcox merged 3 commits into
mainfrom
copilot/runner-doctor-updates-2026-09-27
Sep 29, 2026
Merged

lpcox merged 3 commits into
mainfrom
copilot/runner-doctor-updates-2026-09-27

Conversation

Copilot AI commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Updates runner-doctor coverage for three newly identified failure modes: GHES /meta responses in gh-proxy mode, orphaned awf-net networks, and read-only ARC/DinD streaming-log paths.

  • Diagnostics catalog

    • Document C5’s partial /meta shim mitigation and remaining upstream limitation.
    • Document B27’s orphan-network recreation behavior and actionable inspection probe.
    • Add A28 for writable sandbox/agent streaming-log paths, including unresolved gh-aw compiler work.
  • Doctor workflows

    • Mirror catalog, lookup, probe, and unresolved-item updates in the shared workflow and portable agent.
    • Add regression assertions ensuring all three representations remain aligned.
  • Generated artifacts

    • Regenerate the self-hosted runner doctor lock with the repository’s pinned compiler.

Example orphan-network probe:

docker network inspect awf-net --format '{{json .Options}}'

An empty options map without com.docker.network.bridge.name identifies the affected orphaned network.

Copilot AI changed the title [WIP] Update runner doctor documentation for 2026-09-27 Expand self-hosted runner diagnostics for GHES, orphaned networks, and ARC/DinD logs Sep 29, 2026
Copilot AI requested a review from lpcox September 29, 2026 19:00
@lpcox
lpcox marked this pull request as ready for review September 29, 2026 19:31
Copilot AI balanced review requested due to automatic review settings September 29, 2026 19:32

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.

Copilot review overview

🟡 Changes recommended

Canonical registry outputs and B27/C5 records are incomplete, the A28 probe violates the read-only contract, and portable routing is missing entries.

Review effort: Balanced
Findings: 1 High severity · 1 Medium severity · 2 Low severity

Open (4)
What changed in this PR

Expands runner-doctor diagnostics for ARC/DinD streaming logs, orphaned networks, and GHES metadata handling.

Changes:

  • Adds A28 and updates B27/C5 guidance.
  • Mirrors diagnostics across workflow and portable-agent artifacts.
  • Updates regression assertions and generated workflow metadata.
File Description
scripts/​ci/​self-hosted-runner-doctor-workflow.test.ts Extends alignment assertions.
docs/​diagnostics/​findings/​runner/​A28.json Adds canonical A28 finding.
.github/​workflows/​shared/​self-hosted-failure-modes.md Updates shared diagnostic catalog.
.github/​workflows/​self-hosted-runner-doctor.md Adds probes and symptom routing.
.github/​workflows/​self-hosted-runner-doctor.lock.yml Updates generated workflow hash.
.github/​agents/​self-hosted-runner-doctor.md Updates portable diagnostic guidance.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +1 to +3
{
"$schema": "../../schema.json",
"id": "A28",
Comment thread scripts/ci/self-hosted-runner-doctor-workflow.test.ts
- `error mounting "/dev/null" to .../.npmrc: create mountpoint ...: read-only file system` on `arc-dind` persisting even after upgrading past github/gh-aw-firewall#7998 (A23's fix), where the credential mountpoint is missing under a declared-`rw` home bind backed by a genuinely read-only directory → A24 (`pruneUnmountableCredentialOverlays` only checked declared bind mode, never real filesystem writability for `rw`-declared covers; fixed in github/gh-aw-firewall#8086)
- `docker network connect --alias <name> awf-net <service_container>` is needed for raw-protocol GitHub Actions `services:` containers under `runner.topology: arc-dind` → A25 (service container must join `awf-net` for direct protocol access while the agent stays isolated; documented in github/gh-aw-firewall#8085)
- `a network with name awf-net exists but was not created for project` → B27 (orphaned fixed-name `awf-net` from a prior run on a persistent self-hosted runner; fixed in github/gh-aw-firewall#7817)
- `a network with name awf-net exists but was not created for project`, or a missing bridge in legacy iptables mode → B27 (inspect `docker network inspect awf-net --format '{{json .Options}}'`; an empty options map without `com.docker.network.bridge.name` identifies an unoccupied orphan; fixed in github/gh-aw-firewall#9130)
- `error mounting "/dev/null" to .../.npmrc: create mountpoint ...: read-only file system` on `arc-dind` persisting even after upgrading past github/gh-aw-firewall#7998 (A23's fix), where the credential mountpoint is missing under a declared-`rw` home bind backed by a genuinely read-only directory → A24 (`pruneUnmountableCredentialOverlays` only checked declared bind mode, never real filesystem writability for `rw`-declared covers; fixed in github/gh-aw-firewall#8086)
- `docker network connect --alias <name> awf-net <service_container>` is needed for raw-protocol GitHub Actions `services:` containers under `runner.topology: arc-dind` → A25 (service container must join `awf-net` for direct protocol access while the agent stays isolated; documented in github/gh-aw-firewall#8085)
- `a network with name awf-net exists but was not created for project` → B27 (orphaned fixed-name `awf-net` from a prior run on a persistent self-hosted runner; fixed in github/gh-aw-firewall#7817)
- `a network with name awf-net exists but was not created for project`, or a missing bridge in legacy iptables mode → B27 (inspect `docker network inspect awf-net --format '{{json .Options}}'`; an empty options map without `com.docker.network.bridge.name` identifies an unoccupied orphan; fixed in github/gh-aw-firewall#9130)
@lpcox

lpcox commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator

@copilot address review feedback

Copilot AI commented Sep 29, 2026

Copy link
Copy Markdown
Contributor Author

@copilot address review feedback

Addressed the review feedback in commit 6f94611. The A28 probe is read-only, B27 now checks for attached containers, and generated diagnostics plus portable routing parity are updated.

@github-actions

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.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation Preview

Documentation has been built for this PR.

Download preview artifact

To view locally:

  1. Download the docs-preview-pr-9207 artifact from the workflow run
  2. Unzip and open index.html in your browser

Built from commit 906d32e

@github-actions

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 92.66% 92.67% 📈 +0.01%
Statements 91.15% 91.16% ➡️ +0.01%
Functions 89.18% 89.18% ➡️ +0.00%
Branches 84.29% 84.29% ➡️ +0.00%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

❌ Smoke Copilot BYOK AOAI (Entra) reports failed. AOAI BYOK (Entra) mode investigation needed...

🪪 BYOK (AOAI Entra) report filed by Smoke Copilot BYOK AOAI (Entra)

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

🔌 Smoke Services — All services reachable! ✅

🔌 Service connectivity validated by Smoke Services

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • clients2.google.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "clients2.google.com"

See Network Configuration for more information.

🔮 The oracle has spoken through Smoke Codex

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Smoke Cloud Hypervisor completed. Cloud Hypervisor + Copilot passed.

Cloud Hypervisor + Copilot smoke test by Smoke Cloud Hypervisor

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

📰 BREAKING: Report filed by Smoke Copilot

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Security Guard has started processing this pull request

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓

🔑 BYOK report filed by Smoke Copilot BYOK

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded.

Tested by Smoke Chroot

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

✅ Build Test Suite completed successfully!

Generated by Build Test Suite for #9207

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅

📡 OTel tracing validated by Smoke OTel Tracing

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

❌ Smoke Copilot BYOK AOAI (api-key) reports failed. AOAI BYOK (api-key) mode investigation needed...

🔑 BYOK (AOAI api-key) report filed by Smoke Copilot BYOK AOAI (api-key)

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

❌ Smoke Gemini reports failed. Facets need polishing...

💎 Faceted by Smoke Gemini

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

🛡️ Smoke Copilot Network Isolation confirmed the egress allowlist is enforced. ✅

🛡️ Egress verdict from Smoke Copilot Network Isolation

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

❌ Smoke Claude failed

Generated by Smoke Claude for #9207

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Copilot — PASS ✅

  • MCP: ✅ (latest PR: "Use routing-capable images in model-routing smoke")
  • github.com: ✅ (HTTP 200)
  • File write/read: ✅
    cc @Copilot @lpcox

📰 BREAKING: Report filed by Smoke Copilot
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK (Direct) Mode ✅

Test Result
GitHub MCP connectivity ✅
github.com (HTTP 200) ✅
File write/read ✅
BYOK inference path ✅

Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY → api-proxy → api.githubcopilot.com)

Status: PASS

🔑 BYOK report filed by Smoke Copilot BYOK
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Cloud Hypervisor + Copilot

  1. list_pull_requests (merged, limit 1): PASS — call succeeded, returned data
  2. curl https://github.com: PASS (200)
  3. Write/read /tmp/gh-aw/agent/smoke-cloud-hypervisor-*.txt: PASS
  4. curl (example.com/redacted) (expect blocked): PASS (000)

All checks PASS.

Cloud Hypervisor + Copilot smoke test by Smoke Cloud Hypervisor
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

OTEL smoke test

  • ✅ S1 Module: otel.js loads (exports startRequestSpan, setTokenAttributes, setBudgetAttributes, endSpan, endSpanError, shutdown, isEnabled, etc.)
  • ✅ S2 Tests: 3 OTEL suites, 68/68 passed
  • ✅ S3 Env forwarding: trace context in env-passthrough.ts; OTEL vars in api-proxy-env-config.ts
  • ✅ S4 Token tracker: onUsage hook present in token-tracker-http.js
  • ⚠️ S5 Diagnostics: no exported spans checked (post-step not run yet); not treated as failure

📡 OTel tracing validated by Smoke OTel Tracing
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

EGRESS_RESULT allow=pass deny=pass

  • ✅ Allowed domain (api.github.com): HTTP 403 status returned (reachable)
  • ✅ Blocked domain (example.com): blocked (TLS cert error from proxy)

Overall: PASS — cc @lpcox

🛡️ Egress verdict from Smoke Copilot Network Isolation
Add label ready-for-aw to run again

@github-actions github-actions Bot added smoke-copilot-network-isolation Copilot network-isolation egress smoke test smoke-services labels Sep 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Services smoke test

  • Redis PING: ✅ PONG
  • pg_isready: ✅ accepting connections
  • psql SELECT 1: ✅ 1

Overall: PASS

🔌 Service connectivity validated by Smoke Services
Add label ready-for-aw to run again

@lpcox
lpcox deployed to aoai-model September 29, 2026 19:59 — with GitHub Actions Active
@lpcox
lpcox deployed to aoai-model September 29, 2026 19:59 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

Chroot Version Comparison

Runtime Host Version Chroot Version Match?
Python Python 3.12.14 Python 3.12.14 ✅
Node.js v24.21.0 v22.23.2 ❌
Go go1.22.12 go1.22.12 ✅

Result: Not all tests passed (Node.js version mismatch), so the smoke-chroot label was not added.

Tested by Smoke Chroot
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia ❌ N/A ❌ FAIL
Bun hono ❌ N/A ❌ FAIL
C++ fmt ✅ N/A ✅ PASS
C++ json ✅ N/A ✅ PASS
Deno oak ❌ N/A ❌ FAIL
Deno std ❌ N/A ❌ FAIL
.NET hello-world ✅ N/A ✅ PASS
.NET json-parse ✅ N/A ✅ PASS
Go color ✅ 1/1 passed ✅ PASS
Go env ✅ 1/1 passed ✅ PASS
Go uuid ✅ 1/1 passed ✅ PASS
Java gson ✅ 1/1 passed ✅ PASS
Java caffeine ✅ 1/1 passed ✅ PASS
Node.js clsx ✅ pass ✅ PASS
Node.js execa ✅ pass ✅ PASS
Node.js p-limit ✅ pass ✅ PASS
Rust fd ✅ 1/1 passed ✅ PASS
Rust zoxide ✅ 1/1 passed ✅ PASS

Overall: 6/8 ecosystems passed — FAIL

Failure details

  • Bun / Deno: The install scripts (curl ... | bash / | sh from bun.sh and deno.land) were blocked by the sandbox with "Permission denied and could not request permission from user". Bun and Deno were not installed and no tests were run.
  • Java (passed with a workaround): ~/.m2/repository was not writable ("Could not create local repository"), so I used -Dmaven.repo.local=/tmp/gh-aw/agent/m2.
  • All 8 repositories cloned successfully.

Generated by Build Test Suite for #9207 · copilot · auto · 29.1 AIC · ⊞ 11.8K · ◷
Add label ready-for-aw to run again

@lpcox
lpcox merged commit 8f9b21b into main Sep 29, 2026
128 of 135 checks passed
@lpcox
lpcox deleted the copilot/runner-doctor-updates-2026-09-27 branch September 29, 2026 21:41

This branch was successfully deployed

1 active deployment
aoai-model — 6f94611d Deployed Sep 29, 2026 by lpcox via conclusion #1823
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🩺 Runner Doctor Updatescan 2026-09-27: GHES /meta shim, orphaned awf-net, arc-dind streaming log

3 participants