microvm.nix icon indicating copy to clipboard operation
microvm.nix copied to clipboard

cloud-hypervisor: add platformOEMStrings and extraPlatformOpts.

Open Ramblurr opened this issue 11 months ago • 1 comments

The cloud-hypervisor command line interface unfortunately doesn't support multiple instances of the same arg with a different value, so we have to resort to these extra module options rather than using extraArgs.

To make matters even worse, the --platform argument (of which there can be only one), is overloaded with different types of sub-args that also need to be provided multiple times.

This commit allows the operator to add oem strings (for example to pass systemd credentials), as well as raw platform options as needed.

Ramblurr avatar Feb 18 '25 12:02 Ramblurr

NixCI is ready to run on this PR. Maintainer: Comment nix-ci run to run now.

nix-ci-app[bot] avatar Feb 18 '25 12:02 nix-ci-app[bot]

Hi again @astro, I want to pick this little project back up.. so I'm going to be reviving these PRs today :) Please see my comments to your comments!.

I've just pushed new commits that should address your concerns.

[!NOTE]
I added the refactor as a new commit, but I expect this to be squashed during the merge with the newest commit's commit message. I wanted to keep the old extraPlatformOpts around in case we changed our minds.

Ramblurr avatar Jul 16 '25 07:07 Ramblurr

@SuperSandro2000 I've pushed a new commit that rebases onto main to pickup your latest PR, and also addresses your comments. Let me know if there is anything else.

Ramblurr avatar Jul 18 '25 07:07 Ramblurr

fixed the merge conflict

Ramblurr avatar Jul 18 '25 13:07 Ramblurr

Let's try this 🎉 thanks

SuperSandro2000 avatar Jul 22 '25 08:07 SuperSandro2000