SHA256
1
0
forked from pool/systemd

- Reexecute user managers on package updates.

For now we send signal to user instances to trigger their reexecution. It's
  asynchronous but it shouldn't cause any problem in practice and it's probably
  safer than triggering reexecution with "systemctl --user -M 1000@
  daemon-reexec" command. The latter command creates a new PAM session behind
  the scene bringing with it the known issue (upstream issue #8598) with
  "(sd-pam)" helper process when the PAM session is being closed.

OBS-URL: https://build.opensuse.org/package/show/Base:System/systemd?expand=0&rev=1393
This commit is contained in:
Franck Bui 2023-06-01 15:56:32 +00:00 committed by Git OBS Bridge
parent fdc1c67320
commit 51ccc3bb3e
4 changed files with 42 additions and 34 deletions

View File

@ -121,7 +121,24 @@ case "$command" in
fi
;;
user-reload-restart|user-reload|user-restart|user-reexec)
user-reexec)
if [ -n "$*" ]; then
echo >&2 "Unexpected arguments for '$command': $*"
exit 2
fi
[ -d /run/systemd/system ] || exit 0
# Reexecute user manager instances (if any). It is asynchronous but it
# shouldn't be a problem in practice because systemd main package is not
# shipping any user services currently. A problem would arise only if a
# new version of a user service relied on an option that would be only
# understood by the latest version of the user manager and the user unit
# would be restarted before the user manager get reexecuted.
systemctl kill --kill-who=main --signal=SIGRTMIN+25 "user@*.service"
;;
user-reload-restart|user-reload|user-restart)
if [ -n "$*" ]; then
echo >&2 "Unexpected arguments for '$command': $*"
exit 2
@ -131,14 +148,6 @@ case "$command" in
users=$(systemctl list-units 'user@*' --legend=no | sed -n -r 's/.*user@([0-9]+).service.*/\1/p')
if [[ "$command" =~ reexec ]]; then
for user in $users; do
SYSTEMD_BUS_TIMEOUT=15s \
systemctl --user -M "$user@" daemon-reexec &
done
wait
fi
if [[ "$command" =~ reload ]]; then
for user in $users; do
SYSTEMD_BUS_TIMEOUT=15s \

View File

@ -1,3 +1,15 @@
-------------------------------------------------------------------
Thu Jun 1 15:21:54 UTC 2023 - Franck Bui <fbui@suse.com>
- Reexecute user managers on package updates.
For now we send signal to user instances to trigger their reexecution. It's
asynchronous but it shouldn't cause any problem in practice and it's probably
safer than triggering reexecution with "systemctl --user -M 1000@
daemon-reexec" command. The latter command creates a new PAM session behind
the scene bringing with it the known issue (upstream issue #8598) with
"(sd-pam)" helper process when the PAM session is being closed.
-------------------------------------------------------------------
Tue May 9 14:25:04 UTC 2023 - Franck Bui <fbui@suse.com>

View File

@ -976,14 +976,18 @@ if [ $1 -gt 1 ]; then
fi
%post
# Make /etc/machine-id an empty file during package installation. On the first
# boot, machine-id is initialized and either committed (if /etc/ is writable) or
# the system/image runs with a transient machine ID, that changes on each boot
# (if the image is read-only). This is especially important for appliance builds
# to avoid an identical machine ID in all images.
if [ $1 -eq 1 ]; then
# Make /etc/machine-id an empty file during package installation. On the
# first boot, machine-id is initialized and either committed (if /etc/
# is writable) or the system/image runs with a transient machine ID,
# that changes on each boot (if the image is read-only). This is
# important for appliance builds to avoid an identical machine ID in all
# images.
touch %{_sysconfdir}/machine-id
chmod 444 %{_sysconfdir}/machine-id
# Persistent journal is the default
mkdir -p %{_localstatedir}/log/journal
fi
%if %{without bootstrap}
@ -993,25 +997,8 @@ pam-config --add --systemd || :
%endif
systemctl daemon-reexec || :
# Reexecute user manager instances (if any). It is asynchronous but it shouldn't
# be a problem in practice: a problem would arise only if the new version of a
# user service has a brand new option that is only understood by the latest
# version of the user manager and the user service is started before the user
# manager get reexecuted. But this case is very unlikely especially since we
# don't restart any user service for now.
#
# Before doing this, we unfortunately have to wait until users will reexec their
# user manager (by either rebooting or restarting their session) to a version
# that supports SIGRTMIN+25 otherwise sending the signal to an old version will
# kill the manager which means tearing down the user session.
#
# systemctl kill --kill-who=main --signal=SIGRTMIN+25 "user@*.service" || :
if [ $1 -eq 1 ]; then
# Persistent journal is the default
mkdir -p %{_localstatedir}/log/journal
fi
# Reexecute the user managers (if any)
%{_systemd_util_dir}/systemd-update-helper user-reexec || :
%if %{without filetriggers}
# During package installation, the followings are for config files shipped by

View File

@ -1,4 +1,4 @@
# -*- Mode: rpm-spec; indent-tabs-mode: nil -*- */
# -*- Mode: rpm-spec; indent-tabs-mode: nil -*-
# SPDX-License-Identifier: LGPL-2.1-or-later
#
# This file is part of systemd.