-
Notifications
You must be signed in to change notification settings - Fork 36
Mount entire host filesystem as read-only to AWF container #10182
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
|
💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️ |
|
💫 TO BE CONTINUED... Smoke Claude failed! Our hero faces unexpected challenges... |
|
🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation... |
|
📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident... |
|
@copilot please address this run failure https://github.com/githubnext/gh-aw/actions/runs/21053692908/job/60545299930#step:34:1 |
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Fixed in 90e3e4e. The root cause was Docker's limitation: Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
Smoke Test Results Last 2 merged PRs:
✅ GitHub MCP - List PRs Status: PASS
|
|
GitHub MCP ✅ Fix expression extraction false positive in bash grep pattern; Fix blog author link styling and shorten AI author names
|
Smoke Test Results - Copilot EngineLast 2 merged PRs:
Test Results:
Overall Status: PARTIAL PASS (4/5 tests passed) Note: Go tooling not installed in this workflow environment (expected limitation). cc: @GH_AW_GITHUB_ACTOR
|
invalid specification: destination can't be '/'Root Cause Analysis
The original implementation added
--mount /:/:roto mount the host filesystem into the AWF container. However, Docker doesn't allow binding to/as the container destination path:Resolution
Reverted all changes from the previous commits. The root filesystem mounting feature would require a different approach, possibly:
/hostor/hostfsOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.
Changeset
destination can't be '/'). Restored prior engine and test changes and removed the invalid mount.