From 2cd58b8fb0799d2c00a35545daa0eaae2a5b2bc5 Mon Sep 17 00:00:00 2001 From: Niels Boehm Date: Tue, 17 Jul 2018 17:25:47 +0200 Subject: [PATCH] Notify not only cinder service, but also Apache Apache needs to be notified as well in case the API is being served in that context, otherwise config changes would not take effect without manual intervention. Change-Id: If5e7c14d59843ee7c813eb1df8d7131e002dbd22 --- manifests/api.pp | 1 + releasenotes/notes/restart-apache-6b5d1e4ef37b7ff6.yaml | 5 +++++ 2 files changed, 6 insertions(+) create mode 100644 releasenotes/notes/restart-apache-6b5d1e4ef37b7ff6.yaml diff --git a/manifests/api.pp b/manifests/api.pp index ef9ade2a..e281847b 100644 --- a/manifests/api.pp +++ b/manifests/api.pp @@ -262,6 +262,7 @@ class cinder::api ( enable => false, tag => ['cinder-service'], } + Service <| title == 'httpd' |> { tag +> 'cinder-service' } # we need to make sure cinder-api/eventlet is stopped before trying to start apache Service['cinder-api'] -> Service[$service_name] diff --git a/releasenotes/notes/restart-apache-6b5d1e4ef37b7ff6.yaml b/releasenotes/notes/restart-apache-6b5d1e4ef37b7ff6.yaml new file mode 100644 index 00000000..0791d5c5 --- /dev/null +++ b/releasenotes/notes/restart-apache-6b5d1e4ef37b7ff6.yaml @@ -0,0 +1,5 @@ +--- +fixes: + - | + Notify Apache in case the API is served in that context when the API is + supposed to be restarted, for instance due to configuration changes.