Skip to content

Revert "docker: make sure nix config check works"#13702

Merged
Mic92 merged 1 commit intoNixOS:masterfrom
h0nIg:revert-13351-push-rrzylpqynznw
Aug 6, 2025
Merged

Revert "docker: make sure nix config check works"#13702
Mic92 merged 1 commit intoNixOS:masterfrom
h0nIg:revert-13351-push-rrzylpqynznw

Conversation

@h0nIg
Copy link
Copy Markdown
Contributor

@h0nIg h0nIg commented Aug 6, 2025

Reverts #13351

-> list of installed profiles is lost

$ docker run -ti nixos/nix:2.30.2
bash-5.2# ls -lsa /root/
total 16
4 drwxr-xr-x 3 root root 4096 Jan  1  1980 .
4 drwxr-xr-x 1 root root 4096 Aug  6 18:26 ..
4 lrwxrwxrwx 1 root root   74 Jan  1  1980 .nix-channels -> /nix/store/wb9abgrwawwmdpdkj8jx9j4cnpp93b7h-base-system/root/.nix-channels
4 drwxr-xr-x 2 root root 4096 Jan  1  1980 .nix-defexpr
bash-5.2#
bash-5.2# nix profile list --extra-experimental-features nix-command
bash-5.2#
$ docker run -ti nixos/nix:2.29.1
bash-5.2# ls -lsa /root/
total 16
4 drwxr-xr-x 3 root root 4096 Jan  1  1980 .
4 drwxr-xr-x 1 root root 4096 Aug  6 18:31 ..
4 lrwxrwxrwx 1 root root   74 Jan  1  1980 .nix-channels -> /nix/store/gmkw6bw21s67hf01c93k39wf0dsp9gy7-base-system/root/.nix-channels
4 drwxr-xr-x 2 root root 4096 Jan  1  1980 .nix-defexpr
0 lrwxrwxrwx 1 root root   29 Jan  1  1980 .nix-profile -> /nix/var/nix/profiles/default

bash-5.2# nix profile list --extra-experimental-features nix-command
Name:               bash-interactive
Store paths:        /nix/store/41pi0jiyvlhxgpfhxzvb44w1skc8yp4z-bash-interactive-5.2p37

Name:               coreutils-full
Store paths:        /nix/store/9rf6b6s2cs8r62srk00arcydf8xd59jy-coreutils-full-9.7

.....

@h0nIg h0nIg requested a review from edolstra as a code owner August 6, 2025 18:34
@Mic92
Copy link
Copy Markdown
Member

Mic92 commented Aug 6, 2025

Yeah, that seems worse than having nix config check work

@Mic92 Mic92 enabled auto-merge August 6, 2025 18:38
@drupol
Copy link
Copy Markdown
Contributor

drupol commented Aug 6, 2025

Do you have any idea on what to do to have nix config check working ?

@Mic92 Mic92 merged commit 1376e27 into NixOS:master Aug 6, 2025
15 checks passed
@h0nIg
Copy link
Copy Markdown
Contributor Author

h0nIg commented Oct 15, 2025

@Mic92 may i ask for a backport into 2.30 series please? i dont have permissions to set tags

@internal-nix-ci
Copy link
Copy Markdown

Successfully created backport PR for 2.30-maintenance:

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

Labels

backport 2.30-maintenance Automatically creates a PR against the branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants