[MediaWiki-commits] [Gerrit] operations/puppet[production]: openstack: disable notify temporarily

2017-11-27 Thread Rush (Code Review)
Rush has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/393600 )

Change subject: openstack: disable notify temporarily
..


openstack: disable notify temporarily

To be reverted post cleanup.  This is a precautionary measure.

Bug: T171494
Change-Id: I88300b1321b7286ce7c25c86703bd59630d58a97
---
M modules/openstack/manifests/designate/service.pp
M modules/openstack/manifests/glance/service.pp
M modules/openstack/manifests/horizon/service.pp
M modules/openstack/manifests/keystone/hooks.pp
M modules/openstack/manifests/keystone/service.pp
M modules/openstack/manifests/nova/compute/service.pp
M modules/openstack/manifests/nova/network/service.pp
M modules/openstack/manifests/nova/scheduler/service.pp
M modules/profile/manifests/openstack/base/pdns/recursor/service.pp
9 files changed, 0 insertions(+), 37 deletions(-)

Approvals:
  Rush: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/openstack/manifests/designate/service.pp 
b/modules/openstack/manifests/designate/service.pp
index 404cba2..b209f23 100644
--- a/modules/openstack/manifests/designate/service.pp
+++ b/modules/openstack/manifests/designate/service.pp
@@ -85,27 +85,23 @@
 group   => 'designate',
 mode=> '0440',
 content => 
template("openstack/${version}/designate/designate.conf.erb"),
-notify  => 
Service['designate-api','designate-sink','designate-central','designate-mdns','designate-pool-manager'],
 require => Package['designate-common'];
 '/etc/designate/api-paste.ini':
 content => 
template("openstack/${version}/designate/api-paste.ini.erb"),
 owner   => 'designate',
 group   => 'designate',
-notify  => 
Service['designate-api','designate-sink','designate-central'],
 require => Package['designate-api'],
 mode=> '0440';
 '/etc/designate/policy.json':
 source  => 
"puppet:///modules/openstack/${version}/designate/policy.json",
 owner   => 'designate',
 group   => 'designate',
-notify  => 
Service['designate-api','designate-sink','designate-central'],
 require => Package['designate-common'],
 mode=> '0440';
 '/etc/designate/rootwrap.conf':
 source  => 
"puppet:///modules/openstack/${version}/designate/rootwrap.conf",
 owner   => 'root',
 group   => 'root',
-notify  => 
Service['designate-api','designate-sink','designate-central'],
 require => Package['designate-common'],
 mode=> '0440';
 }
@@ -142,7 +138,6 @@
 group   => 'root',
 mode=> '0544',
 content => 
template('openstack/initscripts/designate-pool-manager.upstart.erb'),
-notify  => Service['designate-pool-manager'],
 }
 
 file {'/etc/init/designate-mdns.conf':
@@ -151,7 +146,6 @@
 group   => 'root',
 mode=> '0544',
 content => 
template('openstack/initscripts/designate-mdns.upstart.erb'),
-notify  => Service['designate-mdns'],
 }
 
 # include rootwrap.d entries
diff --git a/modules/openstack/manifests/glance/service.pp 
b/modules/openstack/manifests/glance/service.pp
index 5c2d4e8..375507a 100644
--- a/modules/openstack/manifests/glance/service.pp
+++ b/modules/openstack/manifests/glance/service.pp
@@ -40,21 +40,18 @@
 owner   => 'glance',
 group   => 'nogroup',
 mode=> '0440',
-notify  => Service['glance-api'],
 require => Package['glance'];
 '/etc/glance/glance-registry.conf':
 content => 
template("openstack/${version}/glance/glance-registry.conf.erb"),
 owner   => 'glance',
 group   => 'nogroup',
 mode=> '0440',
-notify  => Service['glance-registry'],
 require => Package['glance'];
 '/etc/glance/policy.json':
 source  => 
"puppet:///modules/openstack/${version}/glance/policy.json",
 owner   => 'root',
 group   => 'root',
 mode=> '0644',
-notify  => Service['glance-api'],
 require => Package['glance'];
 }
 
diff --git a/modules/openstack/manifests/horizon/service.pp 
b/modules/openstack/manifests/horizon/service.pp
index 91241d9..3856143 100644
--- a/modules/openstack/manifests/horizon/service.pp
+++ b/modules/openstack/manifests/horizon/service.pp
@@ -49,7 +49,6 @@
 group   => 'horizon',
 mode=> '0440',
 require => Package['openstack-dashboard'],
-notify  => [Service['apache2'], Exec['djangorefresh']],
 }
 
 # In the perfect future, Horizon policies will be the same
@@ -61,7 +60,6 @@
 group   => 'horizon',
 mode=> '0440',
 require => Package['openstack-dashboard']

[MediaWiki-commits] [Gerrit] operations/puppet[production]: openstack: disable notify temporarily

2017-11-27 Thread Rush (Code Review)
Rush has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/393600 )

Change subject: openstack: disable notify temporarily
..

openstack: disable notify temporarily

To be reverted post cleanup.  This is a precautionary measure.

Bug: T171494
Change-Id: I88300b1321b7286ce7c25c86703bd59630d58a97
---
M modules/openstack/manifests/designate/service.pp
M modules/openstack/manifests/glance/service.pp
M modules/openstack/manifests/horizon/service.pp
M modules/openstack/manifests/keystone/hooks.pp
M modules/openstack/manifests/keystone/service.pp
M modules/openstack/manifests/nova/compute/service.pp
M modules/openstack/manifests/nova/network/service.pp
M modules/openstack/manifests/nova/scheduler/service.pp
M modules/profile/manifests/openstack/base/pdns/recursor/service.pp
9 files changed, 0 insertions(+), 37 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/00/393600/1

diff --git a/modules/openstack/manifests/designate/service.pp 
b/modules/openstack/manifests/designate/service.pp
index 404cba2..b209f23 100644
--- a/modules/openstack/manifests/designate/service.pp
+++ b/modules/openstack/manifests/designate/service.pp
@@ -85,27 +85,23 @@
 group   => 'designate',
 mode=> '0440',
 content => 
template("openstack/${version}/designate/designate.conf.erb"),
-notify  => 
Service['designate-api','designate-sink','designate-central','designate-mdns','designate-pool-manager'],
 require => Package['designate-common'];
 '/etc/designate/api-paste.ini':
 content => 
template("openstack/${version}/designate/api-paste.ini.erb"),
 owner   => 'designate',
 group   => 'designate',
-notify  => 
Service['designate-api','designate-sink','designate-central'],
 require => Package['designate-api'],
 mode=> '0440';
 '/etc/designate/policy.json':
 source  => 
"puppet:///modules/openstack/${version}/designate/policy.json",
 owner   => 'designate',
 group   => 'designate',
-notify  => 
Service['designate-api','designate-sink','designate-central'],
 require => Package['designate-common'],
 mode=> '0440';
 '/etc/designate/rootwrap.conf':
 source  => 
"puppet:///modules/openstack/${version}/designate/rootwrap.conf",
 owner   => 'root',
 group   => 'root',
-notify  => 
Service['designate-api','designate-sink','designate-central'],
 require => Package['designate-common'],
 mode=> '0440';
 }
@@ -142,7 +138,6 @@
 group   => 'root',
 mode=> '0544',
 content => 
template('openstack/initscripts/designate-pool-manager.upstart.erb'),
-notify  => Service['designate-pool-manager'],
 }
 
 file {'/etc/init/designate-mdns.conf':
@@ -151,7 +146,6 @@
 group   => 'root',
 mode=> '0544',
 content => 
template('openstack/initscripts/designate-mdns.upstart.erb'),
-notify  => Service['designate-mdns'],
 }
 
 # include rootwrap.d entries
diff --git a/modules/openstack/manifests/glance/service.pp 
b/modules/openstack/manifests/glance/service.pp
index 5c2d4e8..375507a 100644
--- a/modules/openstack/manifests/glance/service.pp
+++ b/modules/openstack/manifests/glance/service.pp
@@ -40,21 +40,18 @@
 owner   => 'glance',
 group   => 'nogroup',
 mode=> '0440',
-notify  => Service['glance-api'],
 require => Package['glance'];
 '/etc/glance/glance-registry.conf':
 content => 
template("openstack/${version}/glance/glance-registry.conf.erb"),
 owner   => 'glance',
 group   => 'nogroup',
 mode=> '0440',
-notify  => Service['glance-registry'],
 require => Package['glance'];
 '/etc/glance/policy.json':
 source  => 
"puppet:///modules/openstack/${version}/glance/policy.json",
 owner   => 'root',
 group   => 'root',
 mode=> '0644',
-notify  => Service['glance-api'],
 require => Package['glance'];
 }
 
diff --git a/modules/openstack/manifests/horizon/service.pp 
b/modules/openstack/manifests/horizon/service.pp
index 91241d9..3856143 100644
--- a/modules/openstack/manifests/horizon/service.pp
+++ b/modules/openstack/manifests/horizon/service.pp
@@ -49,7 +49,6 @@
 group   => 'horizon',
 mode=> '0440',
 require => Package['openstack-dashboard'],
-notify  => [Service['apache2'], Exec['djangorefresh']],
 }
 
 # In the perfect future, Horizon policies will be the same
@@ -61,7 +60,6 @@
 group   => 'horizon',
 mode=> '0440',
 require => Package['openstack-da