Skip to content

Fix copilot-setup-steps workflow#1390

Merged
jonathanpeppers merged 4 commits intomainfrom
jonathanpeppers/fix-copilot-setup-steps
Apr 1, 2026
Merged

Fix copilot-setup-steps workflow#1390
jonathanpeppers merged 4 commits intomainfrom
jonathanpeppers/fix-copilot-setup-steps

Conversation

@jonathanpeppers
Copy link
Copy Markdown
Member

@jonathanpeppers jonathanpeppers commented Apr 1, 2026

Fixes several issues with the copilot-setup-steps workflow:

  • Permission denied on /mnt: Use sudo mkdir and sudo chmod to create build directories on the secondary disk
  • Broken symlinks: Rename externals-link/generated-link to externals/generated so the build actually uses the secondary disk
  • Slow disk space logging: Simplify log-disk-space action to just df -h — the du commands were scanning the entire filesystem (~1m45s per call)
  • PR trigger: Add pull_request trigger so changes to the workflow are validated before merge

The workflow failed with 'mkdir: cannot create directory: Permission
denied' when creating directories on /mnt. Use sudo to create the
directories and chmod to make them writable.

Also added push/pull_request triggers so the workflow runs on PRs that
edit it.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 1, 2026 21:20
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the copilot-setup-steps GitHub Actions workflow to both (1) run automatically when the workflow file changes and (2) fix permissions when creating build directories under /mnt on hosted runners.

Changes:

  • Add push and pull_request triggers filtered to .github/workflows/copilot-setup-steps.yml.
  • Create /mnt build directories using sudo and apply writable permissions.

jonathanpeppers and others added 3 commits April 1, 2026 16:25
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The symlinks were created as externals-link and generated-link, but
the build and cache steps use ./externals and ./generated. Rename so
the secondary disk is actually used.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The du commands scan the entire filesystem, taking ~1m45s per call.
Replace with just 'df -h' which returns instantly. Remove the
detailed input option and its callers.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jonathanpeppers jonathanpeppers force-pushed the jonathanpeppers/fix-copilot-setup-steps branch from 0d66b89 to b902531 Compare April 1, 2026 21:30
@jonathanpeppers jonathanpeppers changed the title Fix copilot-setup-steps: sudo for /mnt dirs, add PR trigger Fix copilot-setup-steps workflow Apr 1, 2026
@jonathanpeppers jonathanpeppers merged commit d51d219 into main Apr 1, 2026
2 of 3 checks passed
@jonathanpeppers jonathanpeppers deleted the jonathanpeppers/fix-copilot-setup-steps branch April 1, 2026 21:47
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.

2 participants