Skip to content

fix: bypass AWF entrypoint in rootless artifact permission repair#6356

Merged
lpcox merged 1 commit into
copilot/rootless-artifact-permission-repairfrom
copilot/fix-with-copilot
Jul 18, 2026
Merged

fix: bypass AWF entrypoint in rootless artifact permission repair#6356
lpcox merged 1 commit into
copilot/rootless-artifact-permission-repairfrom
copilot/fix-with-copilot

Conversation

Copilot AI commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

fixArtifactPermissionsForRootless ran docker without --entrypoint sh, causing AWF's entrypoint.sh to execute instead of the intended shell. That entrypoint waits 30s for an iptables-init container that never starts in the repair context, hanging then failing every rootless permission fix attempt.

Changes

  • src/artifact-permissions.ts

    • Add --entrypoint sh flag to docker run; move sh from positional arg after imageRef to -c flag argument
    • Capture stdout alongside stderr for error reporting — the entrypoint timeout message writes to stdout, not stderr
  • src/artifact-permissions.test.ts

    • Add test: logs stdout when stderr is empty and command writes only to stdout
    • Update mount-path assertion to verify --entrypoint sh and -c 'chown...' are present
  • src/artifact-preservation-errors.test.ts

    • Update arrayContaining assertion to require --entrypoint, sh, and the -c command string

Before / after:

- imageRef,
- 'sh',
- '-c',
+ '--entrypoint', 'sh',
+ ...
+ imageRef,
+ '-c',
  'chown -R "$TUID:$TGID" /fix 2>/dev/null; chmod -R a+rwX /fix',

Copilot AI changed the title [WIP] Fix issue with Copilot integration fix: bypass AWF entrypoint in rootless artifact permission repair Jul 18, 2026
Copilot finished work on behalf of lpcox July 18, 2026 05:52
Copilot AI requested a review from lpcox July 18, 2026 05:52
@lpcox
lpcox marked this pull request as ready for review July 18, 2026 14:42
Copilot AI review requested due to automatic review settings July 18, 2026 14:42

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 wasn't able to review any files in this pull request.

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.

3 participants