Comment # 44 on bug 1165780 from
(In reply to Fabian Vogt from comment #42)
> (In reply to Franck Bui from comment #41)
> > The change doesn't seem to be available in TW, anything wrong happened ?
> 
> The sr got declined: https://build.opensuse.org/request/show/823587
> 
> For reference, this is my comment which lead to the decline:
> 
> > Deleting the [Install] section is a noop, it will only break later calls to "systemctl enable btrfsmaintenance-refresh.service". Existing systems will not be impacted by that. In fact, the removal of After=local-fs.target might even break it.
> >
> > Is that intentional?
> >
> > I recommend updating systemd-presets-common-SUSE instead to not explicitly enable btrfsmaintenance-refresh.service.
> 
> To double check, I just updated btrfsmaintenance to the 0.5 version from the
> devel prj in a VM and btrfsmaintenance-refresh.service still runs on boot as
> the .service file is still linked to
> /etc/systemd/system/multi-user.target.wants/.

btrfsmaintenance-refresh.service was NEVER INTENDED to be enabled or installed.

btrfsmaintenance-refresh.path triggers btrfsmaintenance-refresh.service upon
changes of /etc/sysconfig/btrfsmaintenance

Correct settings are:

# systemctl list-unit-files btrfsmaintenance-refresh.*
UNIT FILE                        STATE   
btrfsmaintenance-refresh.path    enabled      
btrfsmaintenance-refresh.service static

# systemctl list-units btrfsmaintenance-refresh.*
  UNIT                          LOAD   ACTIVE SUB     DESCRIPTION               
  btrfsmaintenance-refresh.path loaded active waiting Watch
/etc/sysconfig/btrfsmaintenance

# systemctl cat btrfsmaintenance-refresh.service
# /etc/systemd/system/btrfsmaintenance-refresh.service
[Unit]
Description=Update cron periods from /etc/sysconfig/btrfsmaintenance

[Service]
ExecStart=/usr/share/btrfsmaintenance/btrfsmaintenance-refresh-cron.sh
systemd-timer
Type=oneshot


You are receiving this mail because: