From 55633ebf93c1105e7ac668e6edbba8edaf414584 Mon Sep 17 00:00:00 2001
From: Mark Goddard <mark@stackhpc.com>
Date: Thu, 21 Mar 2019 10:23:30 +0000
Subject: [PATCH] Fix neutron rolling upgrade

Services were being passed as a JSON list, then iterated over in the
neutron-server container's extend_start.sh script like this:

['neutron-server'
'neutron-fwaas'
'neutron-vpnaas']

I'm not actually sure why we have to specify services explicitly, it
seems liable to break if we have other plugins that need migrating.

Change-Id: Ic8ce595793cbe0772e44c041246d5af3a9471d44
---
 ansible/roles/neutron/tasks/rolling_upgrade.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ansible/roles/neutron/tasks/rolling_upgrade.yml b/ansible/roles/neutron/tasks/rolling_upgrade.yml
index f826c6a9b..5ed0168f7 100644
--- a/ansible/roles/neutron/tasks/rolling_upgrade.yml
+++ b/ansible/roles/neutron/tasks/rolling_upgrade.yml
@@ -19,7 +19,7 @@
       KOLLA_UPGRADE:
       NEUTRON_DB_EXPAND:
       KOLLA_CONFIG_STRATEGY: "{{ config_strategy }}"
-      NEUTRON_ROLLING_UPGRADE_SERVICES: "{{ neutron_rolling_upgrade_services }}"
+      NEUTRON_ROLLING_UPGRADE_SERVICES: "{{ neutron_rolling_upgrade_services | join(' ') }}"
     image: "{{ neutron_server.image }}"
     labels:
       UPGRADE:
@@ -76,7 +76,7 @@
       KOLLA_UPGRADE:
       NEUTRON_DB_CONTRACT:
       KOLLA_CONFIG_STRATEGY: "{{ config_strategy }}"
-      NEUTRON_ROLLING_UPGRADE_SERVICES: "{{ neutron_rolling_upgrade_services }}"
+      NEUTRON_ROLLING_UPGRADE_SERVICES: "{{ neutron_rolling_upgrade_services | join(' ') }}"
     image: "{{ neutron_server.image }}"
     labels:
       UPGRADE:
-- 
GitLab