<div dir="ltr"><div dir="ltr">Hi!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 24, 2021 at 2:38 PM Niels Thykier <<a href="mailto:niels@thykier.net" target="_blank">niels@thykier.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Control: reassign -1 init-system-helpers<br>
<br>
Reassigning to init-system-helpers, quoting in full for the<br>
init-system-helpers maintainer's sake.<br>
<br>
On Tue, 16 Feb 2021 21:46:28 +0100 Niels Thykier <<a href="mailto:niels@thykier.net" target="_blank">niels@thykier.net</a>> wrote:<br>
> Luca Boccassi:<br>
> > Source: debhelper<br>
> > Priority: wishlist<br>
> > Tags: bookworm<br>
> > X-Debbugs-CC: <a href="mailto:pkg-systemd-maintainers@lists.alioth.debian.org" target="_blank">pkg-systemd-maintainers@lists.alioth.debian.org</a><br>
> > <br>
> > Dear Maintainer(s),<br>
> > <br>
> > systemd v248 will ship with a new dbus/systemctl interface to mark a<br>
> > unit as needing reload/restart, and a new dbus/systemctl to queue all<br>
> > reload/restart jobs in a single command.<br>
> > The RPM packaging/scripts are changing to use this instead of custom<br>
> > batching.<br>
> > <br>
> > I'm opening this ticket to explore whether it would be<br>
> > possible/good/desirable to switch the debhelper tools to use this as<br>
> > well in the future.<br>
> > <br>
> > It looks like this:<br>
> > <br>
> > systemctl set-property foo.service Markers=needs-restart<br>
> > systemctl set-property bar.service Markers=needs-reload<br>
> > ...<br>
> > systemctl reload-or-restart --marked<br>
> > <br>
> > (or equivalent DBUS calls)<br>
> > <br>
> > References:<br>
> > <br>
> > <a href="https://github.com/systemd/systemd/commit/ff68472a20c208121b69ea13586f3105a219bc14" rel="noreferrer" target="_blank">https://github.com/systemd/systemd/commit/ff68472a20c208121b69ea13586f3105a219bc14</a><br>
> > <a href="https://github.com/systemd/systemd/commit/c9615f73521986b3607b852c139036d58973043c" rel="noreferrer" target="_blank">https://github.com/systemd/systemd/commit/c9615f73521986b3607b852c139036d58973043c</a><br>
> > <a href="https://github.com/systemd/systemd/pull/18481/commits" rel="noreferrer" target="_blank">https://github.com/systemd/systemd/pull/18481/commits</a><br>
> > <br>
> > The advantage being, you can mark a unit inline, but then batch the<br>
> > actual jobs later/asynchronously.<br>
> > <br>
> > Note that, given the interface has just been merged and is not yet<br>
> > released, there is scope for improvements if there are debhelper-<br>
> > specific concerns to address, given the feature first use is the RPM's<br>
> > side of things.<br>
> > <br>
> > Thoughts?<br>
> > <br>
> <br>
> By the sound of it, this is something that might need to go into the<br>
> init system helpers that debhelper invokes in the maintscripts (possibly<br>
> with a trigger in the systemd side to do the batch reload/restart).<br>
> <br>
> @Michael: What is your view?<br>
><br></blockquote><div><br></div><div>Not Michael, but if I can offer my thoughts, this is not something that could be done in i-s-h (or at least, not something that we could do transparently). This is because presumably there are postinst scripts that assume post-debhelper-block the new version is already running. So I believe what would be needed is:</div><div><br></div><div>1. Support in i-s-h to activate this mode, probably via a new flag.</div><div>2. Support in debhelper to enable/disable this new method.</div><div>3. A trigger in systemd to do the final reload. </div><div><br></div><div>I believe requiring the new daemon version to be already updated by postinst time would be quite rare, so I think point 2 could be enabled by default in a new compat level.</div><div><br></div><div>Thoughts?</div></div><div><br></div>-- <br><div dir="ltr"><br>Saludos,<br>Felipe Sateler</div></div>