nix-info: init #30397

grahamc · 2017-10-13T20:57:26Z

Motivation for this change

Create a one-liner output to help diagnose systems on IRC:

grahamc@Morbo> ./result/bin/nix-help
system: "x86_64-linux", multi-user? yes, version: nix-env (Nix) 1.11.15, channels(root): "nixos-17.09.1547.2231575a7d", channels(grahamc): "fizzbuzz-17.03.1775.56da88a298", nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Things done

Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)
- NixOS
- macOS
- other Linux distributions
Tested execution of all binary files (usually in ./result/bin/)
Fits CONTRIBUTING.md.

LnL7 · 2017-10-13T21:08:20Z

$ nix-help
system: "x86_64-darwin", multi-user? yes, version: nix-env (Nix) 1.12pre5619_346aeee1, channels(root): "", channels(lnl): "latest", nixpkgs: ~/.nix-defexpr/nixpkgs

LnL7 · 2017-10-13T21:11:10Z

pkgs/tools/nix/help/help.sh

+}
+
+desc_multi_user() {
+    if test -e /nix/var/nix/daemon-socket; then


I just realised that this will probably always exist on darwin, unless nix was installed before the multi-user installer.

I feel okay with this, but if there is a good fix / improvement I'd be happy to do that

What about also checking test $NIX_REMOTE = daemon and printing unknown instead if the results are inconsistent

grahamc · 2017-10-13T21:54:08Z

grahamc@Morbo> ./result/bin/nix-help -m

system: "x86_64-linux"
multi-user?: daemon
version: nix-env (Nix) 1.11.15
channels(root): "nixos-17.09.1547.2231575a7d"
channels(grahamc): "fizzbuzz-17.03.1775.56da88a298"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Mic92 · 2017-10-13T22:24:20Z

How about naming it nix-info?

grahamc · 2017-10-13T22:38:55Z

done, @Mic92

grahamc · 2017-10-13T22:51:18Z

I included nix-info in to the default closure per @LnL7's suggestion.

grahamc · 2017-10-14T01:54:22Z

Two outputs:

$ ./result/bin/nix-info

system: "x86_64-linux", multi-user?: daemon, version: nix-env (Nix) 1.11.15, channels(root): "nixos-17.09.1547.2231575a7d", channels(grahamc): "fizzbuzz-17.03.1775.56da88a298", nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

$ ./result/bin/nix-info -m

system: "x86_64-linux"
uname -srvm: Linux 4.9.53 #1-NixOS SMP Thu Oct 5 07:44:17 UTC 2017 x86_64
multi-user?: daemon
sandbox: build-use-sandbox = true
version: nix-env (Nix) 1.11.15
channels(root): "nixos-17.09.1547.2231575a7d"
channels(grahamc): "fizzbuzz-17.03.1775.56da88a298"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

zimbatm

nice!

zimbatm · 2017-10-14T11:24:55Z

It might be useful to include source /etc/os-release && echo $VERSION to the report. Most linux systems are migrating to this new standard: https://www.freedesktop.org/software/systemd/man/os-release.html

grahamc · 2017-10-14T12:19:11Z

Hmm I was thinking u name would be sufficient for that, as it provides the distro they’re on, and I’m not certain the utility of the specific distro version. Except for NixOS, which we can tell from the channel. Does that make sense? Would it make more sense to drop uname and use $VERSION instead?

…

On Sat, Oct 14, 2017 at 7:24 AM zimbatm ***@***.***> wrote: It might be useful to include source /etc/os-release && echo $VERSION to the report. Most linux systems are migrating to this new standard: https://www.freedesktop.org/software/systemd/man/os-release.html — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#30397 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAErrEz3s-s3ollJ18vnPysv9JugcLaVks5ssJoJgaJpZM4P5C35> .

grahamc · 2017-10-15T02:01:53Z

Updated: it now performs conclusive sandboxing and multi-user testing by running nix builds.

zimbatm

Great! Works for me on NixOS.

zimbatm · 2017-10-16T08:34:13Z

If this change was back-ported to 17.09 (and 17.03?) then it can be used for issues reporting more reliably.

edolstra · 2017-10-16T09:35:53Z

Wouldn't it be better to include something like this in Nix itself?

Mic92 · 2017-10-16T11:24:23Z

On the long term maybe, but it could take a while until we have a new nix version.

edolstra · 2017-10-16T11:52:02Z

I don't think adding a separate command is the way to go, because then we would need to tell people to install a separate package just to get some trivial bits of info that Nix can easily provide. (E.g. why run nix-build several times when you can just print the value of settings.sandboxMode.)

IMHO it would be better to expand the output of nix-store --version -v.

grahamc · 2017-10-16T12:04:47Z

@edolstra the reason I didn't propose this in core Nix is the lifecycle. If we determine there is something useless in this command, or more important information, it takes just a couple days to update it. I would prefer it be in core Nix, but I think it would be helpful to mature the tool for a bit. That said, I agree: making the nix tool better at describing itself is a good idea, and we could use that information to remove some of the sillier things this tool does to work.

zimbatm · 2017-10-16T18:44:09Z

I thought that it was obvious that it would be integrated into nix in the long run. Nixpkgs is much more amenable to change and experiment by the community. Once the command has stabilized it will be easy enough to implement an optimized version inside of nix directly.

grahamc · 2017-10-20T12:04:48Z

I'll send a PR updating the issue template soon.

LnL7 reviewed Oct 13, 2017

View reviewed changes

grahamc force-pushed the nix-help branch 2 times, most recently from dbfdd67 to 8dc07e5 Compare October 13, 2017 21:53

grahamc force-pushed the nix-help branch from 8dc07e5 to a9c33ff Compare October 13, 2017 22:38

grahamc changed the title ~~nix-help: init~~ nix-info: init Oct 13, 2017

grahamc force-pushed the nix-help branch 2 times, most recently from e0757d5 to cc2f1d6 Compare October 14, 2017 00:12

zimbatm approved these changes Oct 14, 2017

View reviewed changes

grahamc force-pushed the nix-help branch 2 times, most recently from 7ca954d to 74f39be Compare October 14, 2017 21:48

grahamc added 2 commits October 14, 2017 22:01

nix-info: init

d9d2c4a

nix-info: include in nixos' default system-path, nixpkgs's channel

b16f3b2

grahamc force-pushed the nix-help branch from 5f19c6d to b16f3b2 Compare October 15, 2017 02:01

zimbatm approved these changes Oct 16, 2017

View reviewed changes

zimbatm added the 9.needs: port to stable A PR needs a backport to the stable release. label Oct 16, 2017

grahamc merged commit ab8b14c into NixOS:master Oct 17, 2017

grahamc deleted the nix-help branch October 17, 2017 01:37

This was referenced Nov 26, 2017

Technical Details section should be easier to fill out, more comprehensive, and usable in IRC #27758

Closed

Automate the “Technical details” section of the issue template #29932

Closed

samueldr removed the 9.needs: port to stable A PR needs a backport to the stable release. label Apr 17, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

nix-info: init #30397

nix-info: init #30397

grahamc commented Oct 13, 2017 •

edited

LnL7 commented Oct 13, 2017

LnL7 Oct 13, 2017

grahamc Oct 13, 2017

LnL7 Oct 13, 2017

grahamc commented Oct 13, 2017

Mic92 commented Oct 13, 2017

grahamc commented Oct 13, 2017

grahamc commented Oct 13, 2017 •

edited

grahamc commented Oct 14, 2017 •

edited

zimbatm left a comment

zimbatm commented Oct 14, 2017

grahamc commented Oct 14, 2017 via email

grahamc commented Oct 15, 2017

zimbatm left a comment

zimbatm commented Oct 16, 2017

edolstra commented Oct 16, 2017

Mic92 commented Oct 16, 2017 •

edited

edolstra commented Oct 16, 2017

grahamc commented Oct 16, 2017

zimbatm commented Oct 16, 2017

grahamc commented Oct 20, 2017

nix-info: init #30397

nix-info: init #30397

Conversation

grahamc commented Oct 13, 2017 • edited

Motivation for this change

Things done

LnL7 commented Oct 13, 2017

LnL7 Oct 13, 2017

Choose a reason for hiding this comment

grahamc Oct 13, 2017

Choose a reason for hiding this comment

LnL7 Oct 13, 2017

Choose a reason for hiding this comment

grahamc commented Oct 13, 2017

Mic92 commented Oct 13, 2017

grahamc commented Oct 13, 2017

grahamc commented Oct 13, 2017 • edited

grahamc commented Oct 14, 2017 • edited

zimbatm left a comment

Choose a reason for hiding this comment

zimbatm commented Oct 14, 2017

grahamc commented Oct 14, 2017 via email

grahamc commented Oct 15, 2017

zimbatm left a comment

Choose a reason for hiding this comment

zimbatm commented Oct 16, 2017

edolstra commented Oct 16, 2017

Mic92 commented Oct 16, 2017 • edited

edolstra commented Oct 16, 2017

grahamc commented Oct 16, 2017

zimbatm commented Oct 16, 2017

grahamc commented Oct 20, 2017

grahamc commented Oct 13, 2017 •

edited

grahamc commented Oct 13, 2017 •

edited

grahamc commented Oct 14, 2017 •

edited

Mic92 commented Oct 16, 2017 •

edited