Re: [PATCH v10 0/3] maintenance: add support for systemd timers on Linux

["Lénaïc Huard" <lenaic@lhuard.fr>]

Le mercredi 8 septembre 2021, 13:44:26 CEST Derrick Stolee a écrit :
> On 9/7/2021 12:48 PM, Derrick Stolee wrote:
> > On 9/4/2021 4:54 PM, Lénaïc Huard wrote:
> >> Hello,
> >> 
> >> Please find hereafter my updated patchset to add support for systemd
> >> timers on Linux for the `git maintenance start` command.
> >> 
> >> The only changes compared to the previous version are fixes for the
> >> two typos in a comment that Ramsay Jones pointed out [1]
> >> 
> >> [1]
> >> https://lore.kernel.org/git/51246c10-fe0b-b8e5-cdc3-54bdc6c8054e@ramsayj
> >> ones.plus.com/> 
> > The changes in the most recent two versions look good to me.
> 
> I recently tested the 'seen' branch for an unrelated reason, but found
> that the t7900-maintenance.sh test failed for me. It was during test 34,
> 'start and stop Linux/systemd maintenance' with the following issue:
> 
>   + systemd-analyze verify systemd/user/git-maintenance@.service
>   Failed to create /user.slice/user-1000.slice/session-44.scope/init.scope
> control group: Permission denied Failed to initialize manager: Permission
> denied
> 
> Now, this test has the prereq SYSTEMD_ANALYZE, but for some reason this
> later command fails for permission issues. I'm running Ubuntu, if that
> helps.

Thank you for the feedback.

Could you please share which version of Ubuntu and which version of systemd 
you are using ?

I’ve just tried to start an Ubuntu Impish 21.10 which uses systemd 
248.3-1ubuntu3 and to test the `seen` git branch.

All tests of `t/t7900-maintenance.sh` passed including the one which is 
failing for you.

As `systemd-analyse verify` should only check a unit file validity [1], I 
wouldn’t expect it to fail on a cgroup manipulation.

[1] https://www.freedesktop.org/software/systemd/man/systemd-analyze.html#systemd-analyze%20verify%20FILE...

I tried to run 
systemd-analyze verify /etc/systemd/system/sshd.service
and it didn’t produce the error you mentioned but if I `strace` it, I can find:

mkdir("/sys/fs/cgroup/unified/user.slice/user-1000.slice/session-3.scope/
init.scope", 0755) = -1 EACCES (Permission denied)

This makes me think your version of systemd is wrongly considering this cgroup 
directory failure as fatal.
I’d like to know more precisely which versions are affected.

Kind regards,
Lénaïc.