[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#36957] [PATCH v2] machine: Allow non-root users to deploy.
From: |
Jakob L. Kreuze |
Subject: |
[bug#36957] [PATCH v2] machine: Allow non-root users to deploy. |
Date: |
Thu, 08 Aug 2019 16:24:47 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Hey Ricardo,
Ricardo Wurmus <address@hidden> writes:
> Perhaps also wrap “NOPASSWD” in @code{…}.
Got it, thanks!
> This is a comment for future changes only: currently, we can assume that
> the remote machine already runs Guix System. In the future “guix
> deploy” should probably also be able to initialize a system. In that
> case “sudo” may have to be searched on the target or otherwise be
> provided.
Ah, that's a good point. I'd imagine that would involve changing a few
other things with how the REPL is spawned, too.
> (What happens if /run/setuid-programs/sudo is not available on the
> target machine?)
I'm a bit short on time before boarding this flight, so I can't test it
out at the moment, but I'm pretty sure the "failed to run..." message
condition would be thrown. I'll check and get back to you.
> I’m just stumbling upon “socket-name”. “/var/guix” is not guaranteed to
> be the localstatedir. It would be better to use (guix config) to
> determine the configured value.
>
> This doesn’t block this patch, of course, but it would be good to change
> this in the future.
Right, yeah. I may submit a separate patch for it shortly since it
should be a simple change.
Regards,
Jakob
signature.asc
Description: PGP signature
[bug#36957] [PATCH] machine: Allow non-root users to deploy., Christopher Lemmer Webber, 2019/08/07
- [bug#36957] [PATCH v2] machine: Allow non-root users to deploy., Jakob L. Kreuze, 2019/08/07
- [bug#36957] [PATCH v2] machine: Allow non-root users to deploy., Ricardo Wurmus, 2019/08/08
- [bug#36957] [PATCH v2] machine: Allow non-root users to deploy.,
Jakob L. Kreuze <=
- [bug#36957] [PATCH v2] machine: Allow non-root users to deploy., Christopher Lemmer Webber, 2019/08/14
- [bug#36957] [PATCH 0/5] Consolidated patches for guix deploy, Jakob L. Kreuze, 2019/08/15
- [bug#36957] [PATCH 1/5] machine: Allow non-root users to deploy., Jakob L. Kreuze, 2019/08/15
- [bug#36957] [PATCH 2/5] machine: Implement 'roll-back-machine'., Jakob L. Kreuze, 2019/08/15
- [bug#36957] [PATCH 3/5] machine: Automatically authorize the coordinator's signing key., Jakob L. Kreuze, 2019/08/15
- [bug#36957] [PATCH 4/5] doc: Add description of 'build-locally?'., Jakob L. Kreuze, 2019/08/15
- [bug#36957] [PATCH 5/5] remote: Use (%daemon-socket-uri) rather than hard-coded path., Jakob L. Kreuze, 2019/08/15
- bug#36957: [PATCH 5/5] remote: Use (%daemon-socket-uri) rather than hard-coded path., Christopher Lemmer Webber, 2019/08/15
[bug#36957] [PATCH 1/5] machine: Allow non-root users to deploy., Ricardo Wurmus, 2019/08/15
[bug#36957] [PATCH 1/5] machine: Allow non-root users to deploy., Jakob L. Kreuze, 2019/08/15