Add runner-doctor B26 artifact download failure mode - #7667
Conversation
There was a problem hiding this comment.
Pull request overview
Adds B26 guidance for network-isolated artifact downloads failing on Azure Blob redirects.
Changes:
- Documents B26 diagnosis and remediation.
- Adds doctor quick-lookup guidance.
- Adds alignment assertions.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/shared/self-hosted-failure-modes.md |
Adds B26 catalog and lookup entries. |
.github/workflows/self-hosted-runner-doctor.md |
Adds B26 triage guidance. |
scripts/ci/self-hosted-runner-doctor-workflow.test.ts |
Adds B26 documentation assertions. |
Review details
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 3/3 changed files
- Comments generated: 3
- Review effort level: Balanced
| // B26 new failure mode (network-isolation artifact ZIP download via blob storage egress) | ||
| expect(shared).toContain('| B26 | In `--network-isolation` mode'); | ||
| expect(shared).toContain('`error connecting to productionresultssa*.blob.core.windows.net`'); | ||
| expect(shared).toContain('`http_access allow from_cli_proxy cli_proxy_artifact_storage`'); | ||
| expect(shared).toContain('github/gh-aw#54371, github/gh-aw-firewall#7615, github/gh-aw-firewall#7635'); | ||
| expect(shared).toContain('| `error connecting to productionresultssa*.blob.core.windows.net` from `gh run download`/artifact ZIP fetch in `--network-isolation` mode | B26'); |
|
@copilot address review feedback |
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
|
✅ Copilot review passed with no inline comments. @copilot Add the |
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (2 files)
✨ New Files (1 files)
Coverage comparison generated by |
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤
|
|
🔌 Smoke Services — All services reachable! ✅
|
|
📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅
|
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded.
|
|
✅ Build Test Suite completed successfully!
|
|
❌ Contribution Check failed. Please review the logs for details.
|
|
🚀 Security Guard has started processing this pull request |
|
❌ Smoke Gemini reports failed. Facets need polishing...
|
|
❌ Smoke Copilot BYOK AOAI (api-key) reports failed. AOAI BYOK (api-key) mode investigation needed...
|
|
✅ Smoke Claude passed
|
|
✅ Smoke Copilot BYOK AOAI (Entra) completed. Copilot AOAI BYOK (Entra) mode operational. 🔓
|
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 Warning Firewall blocked 3 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "msfeed2.pkgs.visualstudio.com"
- "msfeed25.pkgs.visualstudio.com"
- "registry.npmjs.org"See Network Configuration for more information.
|
|
🛡️ Smoke Copilot Network Isolation confirmed the egress allowlist is enforced. ✅ Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "example.com"See Network Configuration for more information.
|
Smoke Test: Claude Engine Validation
Overall result: PASS
|
|
Smoke Test Results —
Overall: PASS
|
|
EGRESS_RESULT allow=pass deny=pass ✅ Allowed domain (github.com): reachable (HTTP 200) Overall status: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "example.com"See Network Configuration for more information.
|
|
Smoke Test: Copilot BYOK (Direct) Mode — PASS ✅
Running in direct BYOK mode (
|
|
Smoke Test: Docker Sbx — PR by
Overall: PASS
|
|
Smoke Test Results: GitHub Actions Services Connectivity
Overall: FAIL —
|
Chroot Version Comparison
Overall: FAILED — Node.js version mismatch between host and chroot environment.
|
|
Recent PR titles:
GitHub read smoke: ❌ Warning Firewall blocked 3 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "msfeed2.pkgs.visualstudio.com"
- "msfeed25.pkgs.visualstudio.com"
- "registry.npmjs.org"See Network Configuration for more information.
|
|
feat: support filesystem allowWrite with Cloud Hypervisor
|
OTel Tracing Smoke Test Results
Overall: All scenarios pass or are expected-pending. No regression detected in the OTel tracing integration.
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — PASS All clones, installs/builds, and test suites completed successfully across Bun, C++, Deno, .NET, Go, Java, Node.js, and Rust. Java required a
|
Runner Doctor needed coverage for
--network-isolationartifact ZIP downloads failing whenghfollows GitHub’s redirect to Azure Blob storage from inside cli-proxy.Failure-mode catalog
Doctor quick lookup
Alignment coverage