Skip to content

ci: fix scheduled link-check sweep crashing on ubuntu-24.04 sandbox restriction - #728

Merged
rfay merged 1 commit into
mainfrom
20260824_linkcheck
Aug 24, 2026
Merged

ci: fix scheduled link-check sweep crashing on ubuntu-24.04 sandbox restriction#728
rfay merged 1 commit into
mainfrom
20260824_linkcheck

Conversation

@rfay

@rfay rfay commented Aug 24, 2026

Copy link
Copy Markdown
Member

Summary

The scheduled/manual "Sweep all blog links" job runs linkspector directly via npx, unlike the PR job, which goes through umbrelladocs/action-linkspector. ubuntu-24.04 runners restrict unprivileged user namespaces via AppArmor, which Chromium's sandbox needs to launch; action-linkspector already works around this internally, but the bare npx invocation did not, so every scheduled sweep run crashed with "No usable sandbox!" instead of producing a report. Every pull_request run of this workflow passed while every schedule run failed with the identical crash, which pointed at the difference between the two jobs.

  • Add a step to the sweep job that disables the AppArmor user-namespace restriction before running linkspector, mirroring the workaround action-linkspector applies for the PR job.

Test plan

🤖 Developed with assistance from Claude Code

…estriction

The sweep job runs linkspector directly via npx, unlike the PR job which goes through action-linkspector. ubuntu-24.04 runners restrict unprivileged user namespaces via AppArmor, which Chromium's sandbox needs to launch; action-linkspector already works around this internally, but the bare npx invocation did not, so every scheduled/manual sweep run crashed with "No usable sandbox!" instead of producing a report.

## Summary
- Add a step to the sweep job that disables the AppArmor user-namespace restriction before running linkspector, mirroring the workaround action-linkspector applies for the PR job.

## Test plan
- [ ] Trigger the workflow manually (workflow_dispatch) and confirm the sweep job completes and produces JSON output instead of crashing.

🤖 Developed with assistance from [Claude Code](https://claude.ai/code)

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
@rfay

rfay commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

Ran this manually before opening PR, worked OK.

@rfay
rfay merged commit a61e021 into main Aug 24, 2026
7 checks passed
@rfay
rfay deleted the 20260824_linkcheck branch August 24, 2026 18:36
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.

1 participant