From 5c19f9a5e0fc02ccba52f327c3ea5cedb632788d Mon Sep 17 00:00:00 2001 From: Maksim Malchuk Date: Mon, 7 Jun 2021 10:43:52 +0300 Subject: [PATCH] Add forgotten 'Restart container' handler for swift Since I0474324b60a5f792ef5210ab336639edf7a8cd9e swift role uses the new service-cert-copy role introduced in the I6351147ddaff8b2ae629179a9bc3bae2ebac9519 but the swift role itself doesn't contain the handler used in the service-cert-copy. Right now, restarting the swift container isn't necessary, but the handler should exist. Also we should fix the name of the service used. Closes-Bug: #1931097 Change-Id: I2d0615ce6914e1f875a2647c8a95b86dd17eeb22 Signed-off-by: Maksim Malchuk --- ansible/roles/swift/defaults/main.yml | 2 +- ansible/roles/swift/handlers/main.yml | 4 ++++ releasenotes/notes/bug-1931097-c94832ed2ed92c3a.yaml | 6 ++++++ 3 files changed, 11 insertions(+), 1 deletion(-) create mode 100644 ansible/roles/swift/handlers/main.yml create mode 100644 releasenotes/notes/bug-1931097-c94832ed2ed92c3a.yaml diff --git a/ansible/roles/swift/defaults/main.yml b/ansible/roles/swift/defaults/main.yml index ac4fcdb849..82b70bb576 100644 --- a/ansible/roles/swift/defaults/main.yml +++ b/ansible/roles/swift/defaults/main.yml @@ -2,7 +2,7 @@ project_name: "swift" swift_services: - swift-api: + swift-proxy-server: group: swift-proxy-server enabled: true haproxy: diff --git a/ansible/roles/swift/handlers/main.yml b/ansible/roles/swift/handlers/main.yml new file mode 100644 index 0000000000..b4e9116bf1 --- /dev/null +++ b/ansible/roles/swift/handlers/main.yml @@ -0,0 +1,4 @@ +--- +- name: Restart swift-proxy-server container + debug: + msg: "Noop. No need to restart the swift-proxy-server container" diff --git a/releasenotes/notes/bug-1931097-c94832ed2ed92c3a.yaml b/releasenotes/notes/bug-1931097-c94832ed2ed92c3a.yaml new file mode 100644 index 0000000000..e53f026938 --- /dev/null +++ b/releasenotes/notes/bug-1931097-c94832ed2ed92c3a.yaml @@ -0,0 +1,6 @@ +--- +fixes: + - | + Fixes Swift deploy when TLS enabled. + Added the missing handler and corrected the container name. + `LP#1931097 `__