autounattend: use ForceShutdownNow instead of FirstLogonCommands shutdown #11

Open
astro wants to merge 0 commits from astro/wfvm:forceshutdownnow into master
astro commented 2 years ago
Collaborator

this moves the return a little earlier in the installation process.
it is less of a hack, and less problematic with custom defaultUser
settings.

this moves the return a little earlier in the installation process. it is less of a hack, and less problematic with custom `defaultUser` settings.
astro added 1 commit 2 years ago
54d9f41a6d autounattend: use ForceShutdownNow instead of FirstLogonCommands shutdown
this moves the return a little earlier in the installation process.
it is less of a hack, and less problematic with custom `defaultUser`
settings.
sb10q commented 2 years ago
Owner

@astro I don't feel qualified to comment on the technical side of things here. If this is tested and works, let's merge it.

@astro I don't feel qualified to comment on the technical side of things here. If this is tested and works, let's merge it.
astro commented 2 years ago
Poster
Collaborator

As this patch shifts the OOBE pass to the next layer (DisablePasswordExpiry), that will run longer. I found it helpful to increase timeout on slower machines.

@parthy Care to test wfvm pull requests in your environment?

As this patch shifts the OOBE pass to the next layer (`DisablePasswordExpiry`), that will run longer. I found it helpful to increase [timeout](https://git.m-labs.hk/M-Labs/wfvm/src/commit/ec54c9bf9b55403c859f10cdeee9370f8d64e69e/wfvm/utils.nix#L34) on slower machines. @parthy Care to test wfvm pull requests in your environment?
parthy commented 2 years ago

Unfortunately, due to the change in ec54c9bf9b, our pipeline fails to run on this:

qemu-system-x86_64: QXL VGA not available

Unfortunately, due to the change in https://git.m-labs.hk/M-Labs/wfvm/commit/ec54c9bf9b55403c859f10cdeee9370f8d64e69e, our pipeline fails to run on this: `qemu-system-x86_64: QXL VGA not available`
astro commented 2 years ago
Poster
Collaborator

qemu-system-x86_64: QXL VGA not available

Do you run qemu that is built w/o support for -vga qxl? That should be fixable.

> `qemu-system-x86_64: QXL VGA not available` Do you run qemu that is built w/o support for `-vga qxl`? That should be fixable.
parthy commented 2 years ago

I'm not running any custom qemu. I think the build process just picks pkgs.qemu, or not? Is it the fact that I am running the build in a pure server environment? (GitLab pipeline, Hydra)

I'm not running any custom qemu. I think the build process just picks `pkgs.qemu`, or not? Is it the fact that I am running the build in a pure server environment? (GitLab pipeline, Hydra)
astro commented 2 years ago
Poster
Collaborator

@parthy The issue is that the build process contains qemu_test which seems to lack QXL by default now.

diff --git wfvm/utils.nix wfvm/utils.nix
index eb81579..e4927f7 100644
--- wfvm/utils.nix
+++ wfvm/utils.nix
@@ -2,7 +2,7 @@
 
 rec {
   # qemu_test is a smaller closure only building for a single system arch
-  qemu = pkgs.qemu_test;
+  qemu = pkgs.qemu;
 
   mkQemuFlags = extraFlags: [
     "-enable-kvm"
@parthy The issue is that the build process contains qemu_test which seems to lack QXL by default now. ```patch diff --git wfvm/utils.nix wfvm/utils.nix index eb81579..e4927f7 100644 --- wfvm/utils.nix +++ wfvm/utils.nix @@ -2,7 +2,7 @@ rec { # qemu_test is a smaller closure only building for a single system arch - qemu = pkgs.qemu_test; + qemu = pkgs.qemu; mkQemuFlags = extraFlags: [ "-enable-kvm" ```
This branch is already included in the target branch. There is nothing to merge.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b astro-forceshutdownnow master
git pull forceshutdownnow

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff astro-forceshutdownnow
git push origin master
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/wfvm#11
Loading…
There is no content yet.