Revert "want systemd-timesyncd from multi-user.target"
authorPeter Palfrader <peter@palfrader.org>
Sun, 8 Sep 2019 06:43:23 +0000 (08:43 +0200)
committerPeter Palfrader <peter@palfrader.org>
Sun, 8 Sep 2019 06:43:23 +0000 (08:43 +0200)
This reverts commit 443aa81b256b615c55d4fe987a556c663ad4589d.

By default, systemd-timesyncd.service is installed/wanted-by
sysinit.target.  We changed that to multi-user.target about three
years ago, but it's not clear why we did that.

Revert to the defaults and see if it blows up.  If yes, we have a chance
to find out exactly why we moved it.  If not, we have one less thing
that gets messed with at every point release.

modules/systemdtimesyncd/manifests/init.pp

index bc8fe5f..6a9a298 100644 (file)
@@ -7,16 +7,9 @@ class systemdtimesyncd {
                fail ( "No local timeservers configured for systemdtimesyncd." )
        } else {
                file { '/etc/systemd/system/sysinit.target.wants/systemd-timesyncd.service':
-                       ensure => 'absent',
-                       notify => Exec['systemctl daemon-reload'],
-               }
-               file { '/etc/systemd/system/multi-user.target.wants':
-                       ensure => 'directory',
-               }
-               file { '/etc/systemd/system/multi-user.target.wants/systemd-timesyncd.service':
-                       ensure => 'link',
-                       target => '/lib/systemd/system/systemd-timesyncd.service',
-                       notify => Exec['systemctl daemon-reload'],
+                               ensure => 'link',
+                               target => '/lib/systemd/system/systemd-timesyncd.service',
+                               notify => Exec['systemctl daemon-reload'],
                }
 
                service { 'systemd-timesyncd':