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.
