Skip to content

Conversation

@matthewjmarangoni
Copy link
Contributor

@matthewjmarangoni matthewjmarangoni commented Dec 16, 2025

This PR addresses an issue a user noticed where the instructions resulted in a broken deployment.

Issue

Instances based off the pushed image would do nothing but reboot as shown by the logs. Of note the uploaded image was 37MB.

Changes

  • The primary fix is the addition of a flag specifying that the initrd filesystem is EROFS.
  • The headless image memory allocation is increased from 1024MB as it was insufficient.
  • Variables are added to run-unikernel.sh scripts that allow adjusting the volimport path prefix and the VCPUS parameter.
    • The volimport prefix environment variable might be unnecessary but was preferred as the original references onkernel. The official version seems to work and is the default.
  • The README is improved.

Both headless and headful images were built and run. The headful image ran with WebRTC enabled.


Note

Package unikernels with EROFS, increase headless memory, add VCPUS and VOLIMPORT_PREFIX envs, and refine README unikernel instructions.

  • Unikernel build
    • Add --rootfs-type erofs to images/chromium-headless/build-unikernel.sh and images/chromium-headful/build-unikernel.sh.
  • Unikernel run
    • images/chromium-headless/run-unikernel.sh:
      • Memory: -M increased from 1024 to 3072.
      • vCPUs configurable via VCPUS (default 1).
    • images/chromium-headful/run-unikernel.sh:
      • vCPUs configurable via VCPUS (default 4).
      • Volume import image prefix configurable via VOLIMPORT_PREFIX (defaults to onkernel).
  • Docs
    • README.md: Update Unikraft instructions—split secret exports, use explicit IMAGE=... images/.../{build,run}-unikernel.sh commands, add headful run alternative with VOLIMPORT_PREFIX, and note VCPUS tunable.

Written by Cursor Bugbot for commit d390334. This will update automatically on new commits. Configure here.

Copy link
Contributor

@rgarcia rgarcia left a comment

Choose a reason for hiding this comment

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

@rgarcia rgarcia merged commit 9993a27 into onkernel:main Dec 16, 2025
3 of 5 checks passed
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