http://bugzilla.suse.com/show_bug.cgi?id=1165780 http://bugzilla.suse.com/show_bug.cgi?id=1165780#c16 --- Comment #16 from Karl Mistelberger <karl.mistelberger@nefkom.net> --- (In reply to Franck Bui from comment #13)
(In reply to Fabian Vogt from comment #10)
AFAICT it moves the PID reloading to happen only after local-fs.target, which mitigates bug 1137373. It's mostly for symmetry to btrfsmaintenance-refresh.service, which already has After=local-fs.service.
Since both units (.path and .service) have the default dependencies both units don't need "After=local-fs.service" as it's already implied.
Yet another claim which is not supported by evidence. I reiterate: removing After=local-fs.target from unit btrfsmaintenance-refresh.path results in btrfsmaintenance-refresh.service being triggered before local-fs.target is reached. This has been verified on one of my machines and has been confirmed independently by several users at https://forums.opensuse.org/forum.php, e.g.: https://forums.opensuse.org/showthread.php/539836-snapper-not-working?p=2932... https://forums.opensuse.org/showthread.php/539806-files-disappear-reappear-i... https://forums.opensuse.org/showthread.php/539503-boot-efi-is-not-mounted?p=...
It still reloads systemd at least three times during boot and when sysconfig changes.
This is what should be optimized IMHO.
https://github.com/kdave/btrfsmaintenance/pull/79 indeed fixes systemd reload during boot, but does not address factual changes of /etc/sysconfig/btrfsmaintenance. -- You are receiving this mail because: You are on the CC list for the bug.