Comment # 16 on bug 1165780 from
(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=2932503#post2932503

https://forums.opensuse.org/showthread.php/539806-files-disappear-reappear-in-my-opt-folder-happening-on-2-machines-now?p=2932422#post2932422

https://forums.opensuse.org/showthread.php/539503-boot-efi-is-not-mounted?p=2930500#post2930500

> 
> > 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: