[SOLVED] 23.05 upgrade : nixos-rebuild switch fails

hansemschnokeloch · June 1, 2023, 1:14pm

Hi,

I need some help.
I tried to upgrade to 23.05, but switch to new configuration fails.
Build is successfull but nixos-rebuild boot or nixos-rebuild switch fails with this error msg

sudo nixos-rebuild boot 
building Nix...
building the system configuration...
Traceback (most recent call last):
  File "/nix/store/y8pzl5487as5vi1594r39fi9wpjby2pc-systemd-boot", line 341, in <module>
    main()
  File "/nix/store/y8pzl5487as5vi1594r39fi9wpjby2pc-systemd-boot", line 262, in main
    installed_out = subprocess.check_output(["/nix/store/75wxj2a3c0pdbf46bzmff8qr9vbjm5y1-systemd-253.3/bin/bootctl", "--esp-path=/boot/efi", "status"], universal_newlines=True)
  File "/nix/store/95cxzy2hpizr23343b8bskl4yacf4b3l-python3-3.10.11/lib/python3.10/subprocess.py", line 421, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/nix/store/95cxzy2hpizr23343b8bskl4yacf4b3l-python3-3.10.11/lib/python3.10/subprocess.py", line 526, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/nix/store/75wxj2a3c0pdbf46bzmff8qr9vbjm5y1-systemd-253.3/bin/bootctl', '--esp-path=/boot/efi', 'status']' returned non-zero exit status 1.
warning: error(s) occurred while switching to the new configuration

hansemschnokeloch · June 1, 2023, 3:28pm

In case someone has the same issue you can bypass it with
sudo nixos-rebuild switch --install-bootloader

The issue is reported on github nixos/systemd-boot: exit status 1 nixos-rebuild fails · Issue #223579 · NixOS/nixpkgs · GitHub

sirphobos · June 2, 2023, 1:01pm

the mentioned systemd fix seem to have been merged on 30th of march, and yet the issue is still there

hansemschnokeloch · June 2, 2023, 1:22pm

in the comments of the fix there’s a link to this PR that isn’t merged yet

github.com/NixOS/nixpkgs

systemd: 253.3 -> 253.4

NixOS:staging ← flokli:systemd-253.4

opened 07:12AM - 03 May 23 UTC

flokli

+30 -183

This allows us to drop our fsck-look-for-fsck-binary-not-just-in- sbin.patch, as… it was upstreamed. Changelog: ``` 5ed087fa46 generators: skip private tmpfs if /tmp does not exist 93143b6d6a test: replace sleep with timeout 881382685e test-network: add workaround for bug in iproute2 v6.2.0 abf9e916ad coredumpctl: add --file/--root/--image to bash completion dd349a0ede coredumpctl: fix bash completion matching 120342b62d test: match all messages with the FILE field e0da5c9bc6 test: add tests for "systemctl stop" vs triggering by path unit c1542a967b test: create temporary units under /run 03f2a8921e core/path: do not enqueue new job in .trigger_notify callback 674591e6af core/path: align table 0413fb7de9 test: add a couple of tests for systemd-pstore de41e55c7d pstore: avoid opening the dmesg.txt file if not requested 37c212dbd7 pstore: explicitly set the base when converting record ID daee48adbb test: dont use anchor char '$' to match a part of a string 53ac14a054 core/transaction: use hashmap_remove_value() to make not remove job with same ID 0258760397 resolved: adjust message about credentials 8f19911bc3 fuzz-journal-remote: fix potential fd-leak df1e479d4e fuzz-journal-remote: remove temporary files on exit 0d745e2de3 hwdb: update to 46b8c3f5b297ac034f2d024c1f3d84ad2c17f410 df9d1d9bb2 sd-journal: make journal_file_copy_entry() return earlier 3bc2553cfc sd-journal: copy boot ID 45b045880c sd-journal: tighten variable scope 3821e3ea07 journal: Don't try to write garbage if journal entry is corrupted 4eedc4711a test: add test case of negative match for SYMLINK and TAG cd795f9abc udev-rules: fix negative match rule for SYMLINK and TAG a25e2ef992 core: fix property getter method for NFileDescriptorStore bus property eec30e3143 repart: always take BSD lock when whole block device is opened 50ab96e442 bootctl: clean up handling of files with no version information 9d97c8d423 mkosi: disable centos 8 build c603dae241 mkosi: disable key check for Fedora builds 724a50fb01 mkfs-util: do not pass -quiet to mksquashfs 43d194392f test: use setpriv instead of su for user switch from root ba683eb48c test: wrap mkfs.*/mksquashfs/mkswap binaries when running w/ ASan fdcd1807ff test: bump the D-Bus related timeouts to 120s 4f8b2abf69 coredump filter: add mask for 'all' using UINT32_MAX, not UINT64_MAX 021bb972ff coredump filter: fix stack overflow with =all 3fd444c048 build(deps): bump github/super-linter from 4.9.7 to 4.10.1 a19396c73b cryptenroll: fix a memory leak 083ede1482 test: tell dfuzzer to skip Reexecute() ae12c1380b portablectl: add --extension to bash completion b1ecfe3fe7 man: /usr/lib/systemd/random-seed -> /usr/lib/systemd/systemd-random-seed 8895ccaaa8 cryptsetup-fido2: Depend on libcryptsetup c6e957d02d test: use idiomatic bash loop iteration 26e181e94e testsuite-54: drop unnecessary pipe d2c738341b testsuite-70: drop unnecessary env f3abd451dd test: drop uses of "&& { echo 'unexpected success'; exit 1; }" 59243061f6 man: fix LogControl1 manpage example 04983c2b00 pam: cache sd-bus separately per module 0045d952b5 pam_systemd_home: clean up sd-bus when called about something else's user c50ec75e1e testsuite-04: remove unnecessary conditional 5a8987794e man: clarify sd_bus_default b9af9a320e man: add working example to LogControl1 manpage 4d2b5338ac detect-virt: add message at debug level 749a6d9959 dissect: let's check for crypto_LUKS before fstype allowlist check 1aa6171081 ratelimit: handle counter overflows somewhat sanely 5ff63b8507 man: try to make clearer that /var/ is generally not available in /usr/lib/systemd/system-shutdown/ callouts 2be23f69ee dissect-image: issue BLKFLSBUF before probing an fs at block device offset != 0 7b437659b1 list: fix double evaluation ffbb75aa46 mountpoint-util: check /proc is mounted on failure 14eb49b5eb test: prefix the transient unit with test- to make coverage runs happy 980954d2cf kmod-setup: bypass heavy virtio-rng check if we are not running in a VM anyway 567a1a6fd8 kmod-setup: use STARTSWITH_SET() where appropriate d37f06f96f creds: make available to all ExecStartPre= and ExecStart= processes d15f907b5b user-util:remove duplicate includes cedea4cb7e virt: Further improve detection of EC2 metal instances 826662680b string-util: add strstrafter() ac721c88af test: add a couple of tests with invalid UTF-8 characters 9c8d8719e4 test: add a simple test for getenv_path_list() a9c73150ac test: add a couple of basic sanity tests for the security verb 06a70861bc test: add a couple of basic sanity tests for timedatectl def6c37a19 shared: add a missing include 79e23f618f test: add tests for uuid/uint64 specifiers 3ee1839c19 fsck: look for fsck binary not just in /sbin eab75a8591 test: stop the test unit when it's not needed anymore f86ec34958 Synposis and description of networkctl man page reflecting only part of its functionality (#27264) fffcebc4bb core/main: fix a typo for --log-target f152cdabae test: add some tests for RuntimeMaxSec 999f48558b scope: do not disable timer event source when state is SCOPE_RUNNING 430861fc96 Fix cross-reference of manual for LogsDirectory 91953109ec pid1: fix coredump_filter setting fa8d33bb37 Uphold/StopWhenUnneeded/BindsTo: requeue when job finishes 6fc08d8407 Uphold/StopWhenUnneeded/BindsTo: add retry timer on rate limit 1fb4ae32b0 man: add util-linux to the package list for Fedora container 841146f243 man: link to Fedora 37 465edc1230 systemctl: suppress error for try-* if unit is masked 7102925d1a ci: drop checkout from release workflow 167c01688f ci: don't run release wf on `systemd-security` bda5c892a8 shell-completion: add --xml-interface option of busctl to the rules 6265430ca9 busctl: add --xml-interface to the help message d26fd71d1a test: update description 35a6460a2f test: systemd-analyze blame should succeed now ef10974c66 analyze: make blame command work even the default target not reached dc2facf61d ci: add permissions to make a release 4c65c644d6 test/test-functions: fix typo in install_suse_systemd() fca5a45a59 test: install symlinks with valid targets on SUSE and Debian d18037b8ff localed: fix invalid free after shifting pointers using strstrip 93ac024b7e test: bump the timeout for non-qemu runs to 90s 283b7b4159 test: enable the systemd-resolved unit in TEST-75 6179141124 man/systemd-mount: Clearify documentation about --bind-device b2e1dabbeb resolve: change DNS_PACKET_UNICAST_SIZE_LARGE_MAX to 1232 (#27171) 16dc17d68c man: netdev: Clarify wireguard IPv6 endpoint format 0558c490a6 test: use kbd-mode-map we ship in TEST-73-LOCALE 64ef6ccd4f ci: do one build with no tpm/p11kit/fido2 018461aaf0 man: mention -o option for systemd-journal-remote 31c7f6d0d1 manager: remove transient unit directory during startup 49c6965946 core: a more informative error when SetProperties/StartTransientUnit fails 649e335bc1 journald: fix log message eda7bf237f Added unit test for strv_env_name_is_valid() function listed in env-util.c (#27100) 0430078cfb man: restore description of ConditionControlGroupController=v1|v2 0d9c2c270b test: set ReadWritePaths= for test-.services when built w/ coverage 384fec2622 core: skip deps on oomd if v2 or memory unavailable 2950b4ebf6 test: fixed negative checks in TEST-70-TPM2. Use in-line error handling rather than redirections. Follow up on #27020 786649c904 test: make make_addresses() actually return the addresses 5e3ac73017 coverage: add a wrapper for execveat() 8b1cc644c5 man: add example for sd_bus_call_method 382e53977c man: further shorten print-unit-path example 960f05945c man: link up new online coredump docs from man page edfca36727 tree-wide: reset optind to 0 when GNU extensions in optstring are used 91ff21962d test-kernel-install: several cleanups 9943f2af3d units: let's establish the coredump socket before writting core_pattern sysctl dbb1b9c2c8 test: do not remove state directory on failure 29cfb05183 test: fix shellcheck warnings in test-sysusers.sh 18afac6e90 man: fix shellcheck warning for html.in 4629419038 added more test cases 05ae9e276c test: fix regexp in testsuite-74.mount.sh 295012f7fa test: drop extraneous bracket in testsuite-74.mount.sh ff7040b193 busctl: also assume --full if not writing to terminal 00977a8e74 busctl: use size_t for set size 802fded9a5 busctl: do not truncate property values when --full e400a62a92 oomd: add inline comments with param names 4067ec52f4 test: add more testcases for rm_rf() 201830df21 rm-rf: also chmod() directory if it cannot be opened d91f7eb0fb rm-rf: mask file mode with 07777 when passed to chmod() 80417f90b0 rm-rf: fix errno handling ``` ###### Things done - Built on platform(s) - [x] x86_64-linux - [ ] aarch64-linux - [ ] x86_64-darwin - [ ] aarch64-darwin - [ ] For non-Linux: Is `sandbox = true` set in `nix.conf`? (See [Nix manual](https://nixos.org/manual/nix/stable/command-ref/conf-file.html)) - [ ] Tested, as applicable: - [NixOS test(s)](https://nixos.org/manual/nixos/unstable/index.html#sec-nixos-tests) (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests)) - and/or [package tests](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests) - or, for functions and "core" functionality, tests in [lib/tests](https://github.com/NixOS/nixpkgs/blob/master/lib/tests) or [pkgs/test](https://github.com/NixOS/nixpkgs/blob/master/pkgs/test) - made sure NixOS tests are [linked](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages - [ ] Tested compilation of all packages that depend on this change using `nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"`. Note: all changes have to be committed, also see [nixpkgs-review usage](https://github.com/Mic92/nixpkgs-review#usage) - [ ] Tested basic functionality of all binary files (usually in `./result/bin/`) - [23.05 Release Notes (or backporting 22.11 Release notes)](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#generating-2305-release-notes) - [ ] (Package updates) Added a release notes entry if the change is major or breaking - [ ] (Module updates) Added a release notes entry if the change is significant - [ ] (Module addition) Added a release notes entry if adding a new NixOS module - [x] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).

It’s not quite clear to me if this PR will fix it

arianvp · June 2, 2023, 7:07pm

The PR should fix it. Will be merged soon and back ported. Sorry for the inconvenience

sirphobos · June 8, 2023, 9:45am

I see the fix was merged into staging-23.05 pretty fast, but it’s still not in nixos-23.05 branch.

I’m curious, how long does it take on average for such fix to arrive in “stable” branch?

NobbZ · June 8, 2023, 9:59am

From staging to channel usually takes between 2 and 4 weeks, depending on where in the staging next cycle we currently are.

sirphobos · June 12, 2023, 3:55pm

for anyone interested: that commit finally made it into nixos-23.05 branch

edit: I’m surprised that commit did not make it into nixos-unstable though…

sirphobos · June 14, 2023, 2:54pm

sorry for late follow up - the update did not fix the error

for me --install-bootloader flag to nixos-rebuild does not help avoiding the error, so i cannot update to 23.05

sirphobos · June 15, 2023, 6:32pm

looks like i’m not alone: nixos/systemd-boot: exit status 1 nixos-rebuild fails · Issue #223579 · NixOS/nixpkgs · GitHub