Skip to content

autoinstall-terminal: fail on error instead of reporting a false success - #268

Open
T2an wants to merge 1 commit into
cloud-gouv:mainfrom
T2an:fix/autoinstall-terminal-missing-error-handling
Open

T2an wants to merge 1 commit into
cloud-gouv:mainfrom
T2an:fix/autoinstall-terminal-missing-error-handling

Conversation

@T2an

@T2an T2an commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Bumped into "Installation is complete. You can now reboot in the installed system." after a failed install.

Nothing after the disk-mount check checked its own exit status, so a failing nixos-install (or Secure Boot enrollment, or key provisioning) still ended with "Installation is complete." Wraps that section in a subshell with set -e and checks its exit code before reporting success. The install log is still saved to the target disk either way.

Nothing after the disk-mount check checked its own exit status, so a
failing nixos-install (or Secure Boot enrollment, or key provisioning)
still ended with "Installation is complete." Wraps that section in a
subshell with set -e and checks its exit code before reporting success;
the install log is still saved to the target disk either way.
@rlahfa-dinum rlahfa-dinum added the status: awaiting smoke testing Awaiting basic smoke testing on the Sécurix team side. label Sep 22, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: awaiting smoke testing Awaiting basic smoke testing on the Sécurix team side.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants