Skip to content

various cleanup #59

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.

Sign up for GitHub

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

Merged
rgarcia merged 1 commit into from
Aug 25, 2025

fix permissions in headless; run as kernel user everywhere

ef877ba
Select commit
Loading
Failed to load commit list.
Merged

various cleanup #59

fix permissions in headless; run as kernel user everywhere
ef877ba
Select commit
Loading
Failed to load commit list.
Cursor / Cursor Bugbot completed Aug 22, 2025 in 4m 4s

Bugbot Review

Bugbot Analysis Progress (4m 5s elapsed)

✅ Gathered PR context (2s)
✅ Analyzed code changes (0s)
✅ Completed bug detection — 1 potential bug found (3m 59s)
✅ Validation and filtering completed (0s)
✅ Posted analysis results — 1 bug reported (3s)
✅ Analysis completed successfully (0s)

Final Result: Bugbot completed review and found 1 potential issue

Request ID: serverGenReqId_a041091d-d6b3-4aed-a4b6-94b6d758ba98

Details

Bug: Dockerfile Configuration Mismatch Causes Errors

The headful Dockerfile is missing ENV WITHDOCKER=true, unlike the headless image. This causes the wrapper.sh script to incorrectly attempt to mount /dev/shm and enable scale-to-zero functionality, which are operations intended for non-Docker environments. This can lead to /dev/shm mounting errors and unintended scale-to-zero behavior.

images/chromium-headful/Dockerfile#L169-L183

https://github.com/onkernel/kernel-images/blob/ef877babfa4ef95c95666cce9d41c800b22da3d5/images/chromium-headful/Dockerfile#L169-L183

images/chromium-headful/run-docker.sh#L30-L44

https://github.com/onkernel/kernel-images/blob/ef877babfa4ef95c95666cce9d41c800b22da3d5/images/chromium-headful/run-docker.sh#L30-L44

Fix in Cursor Fix in Web

View more details on Cursor