[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#67288] [PATCH] services: laminar: Add configuration option for supp
From: |
Ludovic Courtès |
Subject: |
[bug#67288] [PATCH] services: laminar: Add configuration option for supplementary groups |
Date: |
Sat, 25 Nov 2023 16:25:22 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Hi,
"Thompson, David" <dthompson2@worcester.edu> skribis:
> I started using Laminar CI for my personal server, but I had trouble
> with the current system service. My server is configured to only allow
> members of the "git" group access to the Git repositories, so the CI
> job running as the "laminar" user couldn't do anything useful. This
> patch adds a new configuration field for a list of supplementary
> groups to be used for the "laminar" user and the service process.
Cc’ing Arun and Chris, who know better than me. Is this a problem they
worked around so far?
> From ed62d885a5493f64779bc9c2a9b9978af8f61824 Mon Sep 17 00:00:00 2001
> Message-ID:
> <ed62d885a5493f64779bc9c2a9b9978af8f61824.1700423610.git.dthompson2@worcester.edu>
> From: David Thompson <dthompson2@worcester.edu>
> Date: Sun, 19 Nov 2023 14:46:52 -0500
> Subject: [PATCH] services: laminar: Add configuration option for supplementary
> groups.
>
> * gnu/services/ci (<laminar-configuration>)[supplemental-groups]: New field.
> (laminar-shepherd-service): Exec laminard with supplementary groups.
> (laminar-account): Add supplementary groups to laminar user.
> * doc/guix.texi (Laminar): Document new configuration field.
[...]
> +@item @code{supplementary-groups} (default: @code{()})
> +Supplementary groups for the Laminar user account.
Perhaps mention the “git” group example you gave above?
Otherwise looks pretty harmless to me.
Ludo’.