From 88575904c096a50731dff8563010a594129a23d9 Mon Sep 17 00:00:00 2001
From: Michael Polenchuk <mpolenchuk@mirantis.com>
Date: Tue, 24 Sep 2019 10:44:59 +0400
Subject: [PATCH] [neutron] Handle ML2 mechanism drivers

* add l2pop driver only if vxlan mode is enabled
* use network:backend list as drivers source

Change-Id: If513c2664d465f53ff3756c27ecab0fc519f6a50
---
 neutron/templates/configmap-etc.yaml | 12 +++---------
 neutron/values.yaml                  |  1 -
 2 files changed, 3 insertions(+), 10 deletions(-)

diff --git a/neutron/templates/configmap-etc.yaml b/neutron/templates/configmap-etc.yaml
index 0e06750107..96dae79451 100644
--- a/neutron/templates/configmap-etc.yaml
+++ b/neutron/templates/configmap-etc.yaml
@@ -141,16 +141,10 @@ just set it along with nova_metadata_host.
 {{- end -}}
 
 {{- if empty $envAll.Values.conf.plugins.ml2_conf.ml2.mechanism_drivers -}}
-{{- $_ := set $envAll.Values "__mechanism_drivers" ( list "l2population" ) }}
-{{- if ( has "openvswitch" $envAll.Values.network.backend ) -}}
-{{ $__mechanism_drivers := append $envAll.Values.__mechanism_drivers "openvswitch" }}
-{{- $_ := set $envAll.Values "__mechanism_drivers" $__mechanism_drivers }}
+{{- if (contains "vxlan" $envAll.Values.conf.plugins.ml2_conf.ml2.tenant_network_types) -}}
+{{- $_ := set $envAll.Values "__mechanism_drivers" (append $envAll.Values.network.backend "l2population") -}}
 {{- end -}}
-{{- if ( has "linuxbridge" $envAll.Values.network.backend ) -}}
-{{ $__mechanism_drivers := append $envAll.Values.__mechanism_drivers "linuxbridge" }}
-{{- $_ := set $envAll.Values "__mechanism_drivers" $__mechanism_drivers }}
-{{- end -}}
-{{- $_ := set $envAll.Values.conf.plugins.ml2_conf.ml2 "mechanism_drivers" $envAll.Values.__mechanism_drivers -}}
+{{- $_ := set $envAll.Values.conf.plugins.ml2_conf.ml2 "mechanism_drivers" ($envAll.Values.__mechanism_drivers | default $envAll.Values.network.backend | uniq) -}}
 {{- end -}}
 
 {{- if empty .Values.conf.neutron.DEFAULT.bind_port -}}
diff --git a/neutron/values.yaml b/neutron/values.yaml
index bdda754643..65a2a0ee9e 100644
--- a/neutron/values.yaml
+++ b/neutron/values.yaml
@@ -91,7 +91,6 @@ labels:
 
 network:
   # provide what type of network wiring will be used
-  # possible options: openvswitch, linuxbridge, sriov
   backend:
     - openvswitch
   # NOTE(Portdirect): Share network namespaces with the host,