all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Maximiliano Sandoval <m.sandoval@proxmox.com>
To: Stoiko Ivanov <s.ivanov@proxmox.com>
Cc: pmg-devel@lists.proxmox.com
Subject: Re: [pmg-devel] [PATCH pmg-api v5 05/11] postinstall: add new group for shared functionality
Date: Mon, 25 Aug 2025 16:13:38 +0200	[thread overview]
Message-ID: <s8ott1vo5f1.fsf@proxmox.com> (raw)
In-Reply-To: <20250814021307.3bb402d9@rosa.proxmox.com> (Stoiko Ivanov's message of "Thu, 14 Aug 2025 02:13:07 +0200")

Stoiko Ivanov <s.ivanov@proxmox.com> writes:

> TIL: `systemd-sysusers` - thanks!
> looks fine - a short comment/link to why the flag to `dh` is (still?)
> needed would help me here

The `dh` flag was added in the compatibility level v14 which if I am not
wrong is still open for development, this means it has the feature has
to be enabled explicitly. See [1, 2] for more details.

[1] https://manpages.debian.org/trixie/debhelper/debhelper.7.en.html#COMPATIBILITY_LEVELS
[2] https://manpages.debian.org/trixie/debhelper/debhelper-compat-upgrade-checklist.7.en.html

>
> On Fri,  4 Apr 2025 15:14:32 +0200
> Maximiliano Sandoval <m.sandoval@proxmox.com> wrote:
>
>> A shared group named 'pmg' is introduced for processes that need to be
>> accessible from multiple processes like spamassassin, rrdcached or the
>> mail queue at /var/spool/pmg.
>> 
>> Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
>> ---
>>  debian/pmg-api.sysusers | 1 +
>>  debian/postinst         | 4 ++++
>>  debian/rules            | 2 +-
>>  3 files changed, 6 insertions(+), 1 deletion(-)
>>  create mode 100644 debian/pmg-api.sysusers
>> 
>> diff --git a/debian/pmg-api.sysusers b/debian/pmg-api.sysusers
>> new file mode 100644
>> index 00000000..a546c45f
>> --- /dev/null
>> +++ b/debian/pmg-api.sysusers
>> @@ -0,0 +1 @@
>> +g pmg             -               -
>> diff --git a/debian/postinst b/debian/postinst
>> index 708350ec..9af65ebf 100644
>> --- a/debian/postinst
>> +++ b/debian/postinst
>> @@ -49,6 +49,10 @@ migrate_apt_auth_conf() {
>>  }
>>  
>>  migrate_pmg_smtp_filter() {
>> +    systemd-sysusers
>> +
>> +    chown :pmg /var/lib/pmg
>> +
>>      pmgdb update >/dev/null 2>&1 &
>>  }
>>  
>> diff --git a/debian/rules b/debian/rules
>> index 3e150795..ea8f110c 100755
>> --- a/debian/rules
>> +++ b/debian/rules
>> @@ -13,7 +13,7 @@ include debian/rules.env
>>  export REPOID=${REPOID_GENERATED}
>>  
>>  %:
>> -	dh $@
>> +	dh $@ --with installsysusers
>>  
>>  override_dh_installsystemd:
>>  	dh_installsystemd --no-start --no-stop-on-upgrade \

-- 
Maximiliano


_______________________________________________
pmg-devel mailing list
pmg-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pmg-devel


  reply	other threads:[~2025-08-25 14:14 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-04 13:14 [pmg-devel] [PATCH pmg-api v5 01/11] pmgpolicy: move pid file into /run/pmgpolicy Maximiliano Sandoval
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 02/11] pmg-smtp-filter: move pid file into /run/pmg-smtp-filter Maximiliano Sandoval
2025-08-14  0:10   ` Stoiko Ivanov
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 03/11] config: store config lock in smtp-filter runtime dir Maximiliano Sandoval
2025-08-14  0:10   ` Stoiko Ivanov
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 04/11] create new users for the rule db Maximiliano Sandoval
2025-08-14  0:12   ` Stoiko Ivanov
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 05/11] postinstall: add new group for shared functionality Maximiliano Sandoval
2025-08-14  0:13   ` Stoiko Ivanov
2025-08-25 14:13     ` Maximiliano Sandoval [this message]
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 06/11] postinstall: make rrdcached be readable by the pmg group Maximiliano Sandoval
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 07/11] spamasassin: store files in dir managed by pmg Maximiliano Sandoval
2025-08-14  0:13   ` Stoiko Ivanov
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 08/11] mailqueue: make mail queue writable by pmg group Maximiliano Sandoval
2025-08-14  0:13   ` Stoiko Ivanov
2025-08-25 13:21     ` Maximiliano Sandoval
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 09/11] d/sysusers: add users for pmgpolicy and smtp-filter Maximiliano Sandoval
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 10/11] pmgpolicy: set SumplementaryGroups=systemd-journal Maximiliano Sandoval
2025-04-04 13:14 ` [pmg-devel] [PATCH pmg-api v5 11/11] fix #4926: run pmg-smtp-filter and pmgpolicy without root rights Maximiliano Sandoval
2025-08-14  0:09 ` [pmg-devel] [PATCH pmg-api v5 01/11] pmgpolicy: move pid file into /run/pmgpolicy Stoiko Ivanov
2025-08-25 12:01   ` Maximiliano Sandoval

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s8ott1vo5f1.fsf@proxmox.com \
    --to=m.sandoval@proxmox.com \
    --cc=pmg-devel@lists.proxmox.com \
    --cc=s.ivanov@proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal