[WIP] Fix awmg-mcpg container-attach race in Smoke Aider - #50443
Conversation
|
@copilot fastuuid must be predownloaded/prebuilt |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Fixed in 4fbcbd0 (well, latest push): |
There was a problem hiding this comment.
Pull request overview
Mitigates Smoke Aider’s fastuuid installation failure by forcing a prebuilt wheel; the MCP gateway race fix is not changed here.
Changes:
- Preinstalls
fastuuid0.14.0 as binary-only. - Regenerates the Smoke Aider workflow.
- Incidentally enables unrelated Smoke Crush routing.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/shared/aider.md |
Adds binary-only fastuuid installation. |
.github/workflows/smoke-aider.lock.yml |
Regenerates the compiled Aider workflow. |
.github/workflows/agentic_commands.yml |
Adds Smoke Crush command routing. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 3/3 changed files
- Comments generated: 2
- Review effort level: Balanced
| # /smoke-copilot-sdk -> smoke-copilot-sdk [issue_comment,issues,pull_request,pull_request_comment] reaction=eyes | ||
| # /smoke-copilot-small -> smoke-copilot-small [issue_comment,issues,pull_request,pull_request_comment] reaction=eyes | ||
| # /smoke-create-cross-repo-pr -> smoke-create-cross-repo-pr [issue_comment,issues,pull_request,pull_request_comment] reaction=eyes | ||
| # /smoke-crush -> smoke-crush [issue_comment,issues,pull_request,pull_request_comment] reaction=eyes |
| # fastuuid only ships manylinux wheels for CPython; installing it explicitly first | ||
| # ensures pip resolves the prebuilt wheel instead of falling back to a source build | ||
| # that would require Cargo/crates.io network access. | ||
| python3 -m pip install --quiet --user --disable-pip-version-check --only-binary=:all: fastuuid==0.14.0 |
|
🎉 This pull request is included in a new release. Release: |
fastuuidbuild and add compiler coverage