aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJuan Antonio Osorio Robles <jaosorior@redhat.com>2016-10-17 09:47:34 +0300
committerJuan Antonio Osorio Robles <jaosorior@redhat.com>2016-10-17 09:47:34 +0300
commitbd5fd8c1095a6d66e94a6629a3be6bf5f81dc687 (patch)
tree0b5a9d29fa3e0c7323b78df8893a1503521399b4
parentdae21b71bb7c9e5a5661afc4e0653220c3ae10cf (diff)
Remove faulty migration logic to stop nova-api
The patch making nova run over httpd had added migration logic to stop nova-api, However, this doesn't work since nova-metadata is running over the same process. Now, the fact that is was running seems to be just luck, since the systemctl runs, then we start the service via the nova::api resource. So this is fragile in it's current state. This then removes the exec, as we don't need it for the migration. Change-Id: I4603b81d30a704b07eef461b3cdbfe164614b04f
-rw-r--r--manifests/profile/base/nova/api.pp13
1 files changed, 0 insertions, 13 deletions
diff --git a/manifests/profile/base/nova/api.pp b/manifests/profile/base/nova/api.pp
index 03706dc..c3e9ec4 100644
--- a/manifests/profile/base/nova/api.pp
+++ b/manifests/profile/base/nova/api.pp
@@ -39,19 +39,6 @@ class tripleo::profile::base::nova::api (
if $step >= 4 or ($step >= 3 and $sync_db) {
- # Manages the migration to Nova API in mod_wsgi with Apache.
- # - First update nova.conf with new parameters
- # - Stop nova-api process before starting apache to avoid binding error
- # - Start apache after configuring all vhosts
- exec { 'stop_nova-api':
- command => 'service openstack-nova-api stop',
- path => ['/usr/bin', '/usr/sbin'],
- onlyif => 'systemctl is-active openstack-nova-api',
- refreshonly => true,
- }
- Nova_config<||> ~> Exec['stop_nova-api']
- Exec['stop_nova-api'] -> Service['httpd']
-
class { '::nova::api':
service_name => 'httpd', # Temporary: will be moved to t-h-t
sync_db => $sync_db,