reload apache instead of restarting it
authorPeter Palfrader <peter@palfrader.org>
Sat, 6 Aug 2016 17:43:19 +0000 (19:43 +0200)
committerPeter Palfrader <peter@palfrader.org>
Sat, 6 Aug 2016 17:43:19 +0000 (19:43 +0200)
29 files changed:
modules/apache2/manifests/config.pp
modules/apache2/manifests/site.pp
modules/munin/manifests/master.pp
modules/roles/manifests/buildd_master.pp
modules/roles/manifests/contributors.pp
modules/roles/manifests/dbmaster.pp
modules/roles/manifests/debtags.pp
modules/roles/manifests/dgit_browse.pp
modules/roles/manifests/dgit_git.pp
modules/roles/manifests/ftp_master.pp
modules/roles/manifests/init.pp
modules/roles/manifests/jenkins.pp
modules/roles/manifests/keyring.pp
modules/roles/manifests/lists.pp
modules/roles/manifests/nm.pp
modules/roles/manifests/piuparts.pp
modules/roles/manifests/release.pp
modules/roles/manifests/rtmaster.pp
modules/roles/manifests/search_frontend.pp
modules/roles/manifests/security_master.pp
modules/roles/manifests/security_tracker.pp
modules/roles/manifests/sso.pp
modules/roles/manifests/static_mirror.pp
modules/roles/manifests/syncproxy.pp
modules/roles/manifests/tracker.pp
modules/roles/manifests/udd.pp
modules/roles/manifests/vote.pp
modules/roles/manifests/wiki.pp
modules/roles/manifests/www_master.pp

index 4faa33f..d185e42 100644 (file)
@@ -22,14 +22,14 @@ define apache2::config (
                                ensure  => $ensure,
                                content => $content,
                                require => Package['apache2'],
-                               notify  => Service['apache2'],
+                               notify  => Exec['service apache2 reload'],
                        }
                } else {
                        file { "/etc/apache2/conf.d/${name}":
                                ensure  => $ensure,
                                source  => $source,
                                require => Package['apache2'],
-                               notify  => Service['apache2'],
+                               notify  => Exec['service apache2 reload'],
                        }
                }
        } else {
@@ -38,14 +38,14 @@ define apache2::config (
                                ensure  => $ensure,
                                content => $content,
                                require => Package['apache2'],
-                               notify  => Service['apache2'],
+                               notify  => Exec['service apache2 reload'],
                        }
                } else {
                        file { "/etc/apache2/conf-available/${name}.conf":
                                ensure  => $ensure,
                                source  => $source,
                                require => Package['apache2'],
-                               notify  => Service['apache2'],
+                               notify  => Exec['service apache2 reload'],
                        }
                }
 
@@ -57,7 +57,7 @@ define apache2::config (
                file { "/etc/apache2/conf-enabled/${name}.conf":
                        ensure => $link_ensure,
                        target => "../conf-available/${name}.conf",
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        }
 }
index 46ec1bf..2a7257b 100644 (file)
@@ -35,14 +35,14 @@ define apache2::site (
                        ensure  => $ensure,
                        content => $content,
                        require => Package['apache2'],
-                       notify  => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        } else {
                file { $target:
                        ensure  => $ensure,
                        source  => $source,
                        require => Package['apache2'],
-                       notify  => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        }
 
@@ -53,7 +53,7 @@ define apache2::site (
 
                file { "/etc/apache2/sites-enabled/${name}":
                        ensure => absent,
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        }
 
@@ -61,12 +61,12 @@ define apache2::site (
                file { $symlink:
                        ensure => link,
                        target => $link_target,
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        } else {
                file { $symlink:
                        ensure => absent,
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        }
 }
index ca221b8..5dc0bbc 100644 (file)
@@ -10,7 +10,7 @@ class munin::master {
        }
 
        ssl::service { 'munin.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
        file { '/etc/munin/munin-conf.d':
                ensure  => directory,
index 03d06f8..63570ef 100644 (file)
@@ -1,6 +1,6 @@
 class roles::buildd_master {
        ssl::service { 'buildd.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 
        file { '/etc/ssh/userkeys/wb-buildd.more':
index 54863e2..d0ed1fb 100644 (file)
@@ -1,5 +1,5 @@
 class roles::contributors {
        ssl::service { 'contributors.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index 111b4f1..56b7433 100644 (file)
@@ -13,7 +13,7 @@ class roles::dbmaster {
        $rabbit_password = $roles::pubsub::parameters::rabbit_password
 
        ssl::service { 'db.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 
        roles::pubsub::config { 'generate':
index 5032809..b11e757 100644 (file)
@@ -3,7 +3,7 @@ class roles::debtags {
        package { 'libapache2-mod-wsgi': ensure => installed, }
 
        ssl::service { 'debtags.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 
index 0d2ea14..72ceebd 100644 (file)
@@ -1,6 +1,6 @@
 class roles::dgit_browse {
        ssl::service { 'browse.dgit.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 
@@ -13,7 +13,7 @@ class roles::dgit_browse {
        }
        file { '/etc/apache2/conf-enabled/cgit.conf':
                ensure => absent,
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 
        apache2::site { '010-browse.dgit.debian.org':
index be5b05c..866d8cb 100644 (file)
@@ -1,6 +1,6 @@
 class roles::dgit_git {
        ssl::service { 'git.dgit.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 
index 5f04386..a76e068 100644 (file)
@@ -6,6 +6,6 @@ class roles::ftp_master {
        }
 
        ssl::service { 'ftp-master.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index e11d1ae..be8f3ab 100644 (file)
@@ -19,7 +19,7 @@ class roles {
        if has_role('nagiosmaster') {
        #       include nagios::server
                ssl::service { 'nagios.debian.org':
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        }
 
@@ -43,11 +43,11 @@ class roles {
 
        if has_role('bugs_base') {
                ssl::service { 'bugs.debian.org':
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        }
        if has_role('bugs_master') {
-               ssl::service { 'bugs-master.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'bugs-master.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('ftp_master') {
@@ -57,12 +57,12 @@ class roles {
 
        if has_role('api.ftp-master') {
                ssl::service { 'api.ftp-master.debian.org':
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                }
        }
 
        if has_role('manpages') {
-               ssl::service { 'manpages.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'manpages.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('security_mirror') {
@@ -89,7 +89,7 @@ class roles {
        }
 
        if has_role('people') {
-               ssl::service { 'people.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'people.debian.org': notify  => Exec['service apache2 reload'], key => true, }
                onion::service { 'people.debian.org': port => 80, target_address => 'people.debian.org', target_port => 80, }
        }
 
@@ -103,7 +103,7 @@ class roles {
        }
 
        if has_role('cgi.d.o') {
-               ssl::service { 'cgi.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'cgi.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('keyring') {
@@ -232,19 +232,19 @@ class roles {
        }
 
        if has_role('packages') {
-               ssl::service { 'packages.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'packages.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('qamaster') {
-               ssl::service { 'qa.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'qa.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('packagesqamaster') {
-               ssl::service { 'packages.qa.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'packages.qa.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('gobby_debian_org') {
-               ssl::service { 'gobby.debian.org': notify => Service['apache2'], key => true, tlsaport => [443, 6523], }
+               ssl::service { 'gobby.debian.org': notify  => Exec['service apache2 reload'], key => true, tlsaport => [443, 6523], }
        }
 
        if has_role('search_backend') {
@@ -266,7 +266,7 @@ class roles {
        }
 
        if has_role('veyepar.debian.org') {
-               ssl::service { 'veyepar.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'veyepar.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('httpredir') {
@@ -278,24 +278,24 @@ class roles {
        }
 
        if has_role('planet_search') {
-               ssl::service { 'planet-search.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'planet-search.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('i18n.d.o') {
-               ssl::service { 'i18n.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'i18n.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('l10n.d.o') {
-               ssl::service { 'l10n.debian.org': notify => Service['apache2'], key => true, }
+               ssl::service { 'l10n.debian.org': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('dedup.d.n') {
-               ssl::service { 'dedup.debian.net': notify => Service['apache2'], key => true, }
+               ssl::service { 'dedup.debian.net': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('pet.d.n') {
-               ssl::service { 'pet.debian.net': notify => Service['apache2'], key => true, }
-               ssl::service { 'pet-devel.debian.net': notify => Service['apache2'], key => true, }
+               ssl::service { 'pet.debian.net': notify  => Exec['service apache2 reload'], key => true, }
+               ssl::service { 'pet-devel.debian.net': notify  => Exec['service apache2 reload'], key => true, }
        }
 
        if has_role('ports-master') {
index 1264e8b..e2374bc 100644 (file)
@@ -8,6 +8,6 @@ class roles::jenkins {
        }
 
        ssl::service { 'jenkins.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index eb331e0..379fae2 100644 (file)
@@ -4,7 +4,7 @@ class roles::keyring {
        }
 
        ssl::service { 'keyring.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 }
index d501244..ace2b25 100644 (file)
@@ -1,6 +1,6 @@
 class roles::lists {
        ssl::service { 'lists.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 
        dnsextras::tlsa_record{ 'tlsa-mailport':
index 4cdc3b3..f792de4 100644 (file)
@@ -1,5 +1,5 @@
 class roles::nm {
        ssl::service { 'nm.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index fbd69ef..5dfd1eb 100644 (file)
@@ -1,6 +1,6 @@
 class roles::piuparts {
        ssl::service { 'piuparts.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 }
index 084e80b..cc9e668 100644 (file)
@@ -1,5 +1,5 @@
 class roles::release {
        ssl::service { 'release.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index b1c7487..c6dfbbe 100644 (file)
@@ -1,5 +1,5 @@
 class roles::rtmaster {
        ssl::service { 'rt.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index 4d66c1d..dc1861d 100644 (file)
@@ -1,6 +1,6 @@
 class roles::search_frontend {
        ssl::service { 'search.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 }
index ae3c471..a96dbec 100644 (file)
@@ -1,7 +1,7 @@
 class roles::security_master {
 
        ssl::service { 'security-master.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 
index 1210c7e..abc0960 100644 (file)
@@ -1,5 +1,5 @@
 class roles::security_tracker {
        ssl::service { 'security-tracker.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index 890a0ba..4075993 100644 (file)
@@ -1,5 +1,5 @@
 class roles::sso {
        ssl::service { 'sso.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index 53b7b8a..178d59c 100644 (file)
@@ -78,40 +78,40 @@ class roles::static_mirror {
                content => template('roles/apache-www.debian.org.erb'),
        }
 
-       ssl::service { 'dsa.debian.org'      : ensure => "ifstatic", notify => Service['apache2'], }
-       ssl::service { 'www.debian.org'      : ensure => "ifstatic", notify => Service['apache2'], }
-       ssl::service { 'rtc.debian.org'      : ensure => "ifstatic", notify => Service['apache2'], }
-       ssl::service { 'd-i.debian.org'      : ensure => "ifstatic", notify => Service['apache2'], }
+       ssl::service { 'dsa.debian.org'      : ensure => "ifstatic", notify  => Exec['service apache2 reload'], }
+       ssl::service { 'www.debian.org'      : ensure => "ifstatic", notify  => Exec['service apache2 reload'], }
+       ssl::service { 'rtc.debian.org'      : ensure => "ifstatic", notify  => Exec['service apache2 reload'], }
+       ssl::service { 'd-i.debian.org'      : ensure => "ifstatic", notify  => Exec['service apache2 reload'], }
 
        # do
-       ssl::service { 'appstream.debian.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'backports.debian.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'bits.debian.org'               : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'blends.debian.org'             : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'lintian.debian.org'            : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'onion.debian.org'              : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'release.debian.org'            : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'security-team.debian.org'      : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'www.ports.debian.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
+       ssl::service { 'appstream.debian.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'backports.debian.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'bits.debian.org'               : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'blends.debian.org'             : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'lintian.debian.org'            : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'onion.debian.org'              : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'release.debian.org'            : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'security-team.debian.org'      : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'www.ports.debian.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
        # dn
-       ssl::service { 'news.debian.net'               : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'micronews.debian.net'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debaday.debian.net'            : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'timeline.debian.net'           : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'wnpp-by-tags.debian.net'       : ensure => "ifstatic", notify => Service['apache2'], key => true, }
+       ssl::service { 'news.debian.net'               : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'micronews.debian.net'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debaday.debian.net'            : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'timeline.debian.net'           : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'wnpp-by-tags.debian.net'       : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
        # dc
-       ssl::service { 'debconf0.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debconf1.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debconf2.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debconf3.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debconf4.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debconf5.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debconf6.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'debconf7.debconf.org'          : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { '10years.debconf.org'           : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'es.debconf.org'                : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'fr.debconf.org'                : ensure => "ifstatic", notify => Service['apache2'], key => true, }
-       ssl::service { 'miniconf10.debconf.org'        : ensure => "ifstatic", notify => Service['apache2'], key => true, }
+       ssl::service { 'debconf0.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debconf1.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debconf2.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debconf3.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debconf4.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debconf5.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debconf6.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'debconf7.debconf.org'          : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { '10years.debconf.org'           : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'es.debconf.org'                : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'fr.debconf.org'                : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
+       ssl::service { 'miniconf10.debconf.org'        : ensure => "ifstatic", notify  => Exec['service apache2 reload'], key => true, }
 
        if has_role('static_mirror_onion') {
                if ! $onion_v4_addr {
index 202cabd..3101025 100644 (file)
@@ -37,7 +37,7 @@ class roles::syncproxy {
        if $::apache2 and $syncproxy_name != 'unknown' {
                include apache2::ssl
                ssl::service { "$syncproxy_name":
-                       notify => Service['apache2'],
+                       notify  => Exec['service apache2 reload'],
                        key => true,
                }
                apache2::site { '010-syncproxy.debian.org':
index 23d95d5..d9bce2d 100644 (file)
@@ -1,6 +1,6 @@
 class roles::tracker {
        ssl::service { 'tracker.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 }
index 73fc804..fe9abba 100644 (file)
@@ -1,5 +1,5 @@
 class roles::udd {
        ssl::service { 'udd.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index a214703..88e3920 100644 (file)
@@ -1,5 +1,5 @@
 class roles::vote {
        ssl::service { 'vote.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
        }
 }
index b5939dc..b6fb7e8 100644 (file)
@@ -1,6 +1,6 @@
 class roles::wiki {
        ssl::service { 'wiki.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
        rsync::site { 'wiki':
index 57e52ef..aabe5f2 100644 (file)
@@ -1,6 +1,6 @@
 class roles::www_master {
        ssl::service { 'www-master.debian.org':
-               notify => Service['apache2'],
+               notify  => Exec['service apache2 reload'],
                key => true,
        }
 }