Use the same label with openvswitch for ovn-metadata agent
diff --git a/charts/neutron/templates/daemonset-ovn-metadata-agent.yaml b/charts/neutron/templates/daemonset-ovn-metadata-agent.yaml
index bc8bb74..1fa7f52 100644
--- a/charts/neutron/templates/daemonset-ovn-metadata-agent.yaml
+++ b/charts/neutron/templates/daemonset-ovn-metadata-agent.yaml
@@ -80,7 +80,7 @@
{{ tuple $envAll "neutron" | include "helm-toolkit.snippets.kubernetes_tolerations" | indent 6 }}
{{ end }}
nodeSelector:
- {{ .Values.labels.ovn.node_selector_key }}: {{ .Values.labels.ovn.node_selector_value }}
+ {{ .Values.labels.ovs.node_selector_key }}: {{ .Values.labels.ovs.node_selector_value }}
dnsPolicy: ClusterFirstWithHostNet
hostNetwork: true
{{- if or ( gt .Capabilities.KubeVersion.Major "1" ) ( ge .Capabilities.KubeVersion.Minor "10" ) }}
diff --git a/charts/neutron/values.yaml b/charts/neutron/values.yaml
index 2349460..122d912 100644
--- a/charts/neutron/values.yaml
+++ b/charts/neutron/values.yaml
@@ -77,9 +77,6 @@
# label that can apply to both control hosts
# and compute hosts, until we get more sophisticated
# with our daemonset scheduling
- ovn:
- node_selector_key: ovn
- node_selector_value: enabled
ovs:
node_selector_key: openvswitch
node_selector_value: enabled
diff --git a/charts/ovn/templates/daemonset-controller-gw.yaml b/charts/ovn/templates/daemonset-controller-gw.yaml
index ff76537..4e8dc12 100644
--- a/charts/ovn/templates/daemonset-controller-gw.yaml
+++ b/charts/ovn/templates/daemonset-controller-gw.yaml
@@ -49,7 +49,6 @@
dnsPolicy: {{ .Values.pod.dns_policy }}
nodeSelector:
{{ .Values.labels.ovn_controller_gw.node_selector_key }}: {{ .Values.labels.ovn_controller_gw.node_selector_value }}
- {{ .Values.labels.ovn.node_selector_key }}: {{ .Values.labels.ovn.node_selector_value }}
initContainers:
{{- tuple $envAll "ovn_controller_gw" list | include "helm-toolkit.snippets.kubernetes_entrypoint_init_container" | indent 8 }}
- name: controller-init
diff --git a/charts/ovn/templates/daemonset-controller.yaml b/charts/ovn/templates/daemonset-controller.yaml
index 3fd3f7b..94aec42 100644
--- a/charts/ovn/templates/daemonset-controller.yaml
+++ b/charts/ovn/templates/daemonset-controller.yaml
@@ -49,7 +49,6 @@
dnsPolicy: {{ .Values.pod.dns_policy }}
nodeSelector:
{{ .Values.labels.ovn_controller.node_selector_key }}: {{ .Values.labels.ovn_controller.node_selector_value }}
- {{ .Values.labels.ovn.node_selector_key }}: {{ .Values.labels.ovn.node_selector_value }}
initContainers:
{{- tuple $envAll "ovn_controller" list | include "helm-toolkit.snippets.kubernetes_entrypoint_init_container" | indent 8 }}
- name: controller-init
diff --git a/charts/ovn/templates/deployment-northd.yaml b/charts/ovn/templates/deployment-northd.yaml
index 1688062..e3afdd0 100644
--- a/charts/ovn/templates/deployment-northd.yaml
+++ b/charts/ovn/templates/deployment-northd.yaml
@@ -56,7 +56,6 @@
serviceAccountName: {{ $serviceAccountName }}
nodeSelector:
{{ .Values.labels.ovn_northd.node_selector_key }}: {{ .Values.labels.ovn_northd.node_selector_value }}
- {{ .Values.labels.ovn.node_selector_key }}: {{ .Values.labels.ovn.node_selector_value }}
initContainers:
{{- tuple $envAll "ovn_northd" list | include "helm-toolkit.snippets.kubernetes_entrypoint_init_container" | indent 8 }}
containers:
diff --git a/charts/ovn/templates/statefulset-ovsdb-nb.yaml b/charts/ovn/templates/statefulset-ovsdb-nb.yaml
index 1c13e0e..9380515 100644
--- a/charts/ovn/templates/statefulset-ovsdb-nb.yaml
+++ b/charts/ovn/templates/statefulset-ovsdb-nb.yaml
@@ -45,7 +45,6 @@
{{- tuple $envAll "ovn" "ovn-ovsdb-nb" | include "helm-toolkit.snippets.kubernetes_pod_anti_affinity" | indent 8 }}
nodeSelector:
{{ .Values.labels.ovn_ovsdb_nb.node_selector_key }}: {{ .Values.labels.ovn_ovsdb_nb.node_selector_value }}
- {{ .Values.labels.ovn.node_selector_key }}: {{ .Values.labels.ovn.node_selector_value }}
initContainers:
{{- tuple $envAll "ovn_ovsdb_nb" list | include "helm-toolkit.snippets.kubernetes_entrypoint_init_container" | indent 8 }}
containers:
diff --git a/charts/ovn/templates/statefulset-ovsdb-sb.yaml b/charts/ovn/templates/statefulset-ovsdb-sb.yaml
index d00689b..400b127 100644
--- a/charts/ovn/templates/statefulset-ovsdb-sb.yaml
+++ b/charts/ovn/templates/statefulset-ovsdb-sb.yaml
@@ -45,7 +45,6 @@
{{- tuple $envAll "ovn" "ovn-ovsdb-sb" | include "helm-toolkit.snippets.kubernetes_pod_anti_affinity" | indent 8 }}
nodeSelector:
{{ .Values.labels.ovn_ovsdb_sb.node_selector_key }}: {{ .Values.labels.ovn_ovsdb_sb.node_selector_value }}
- {{ .Values.labels.ovn.node_selector_key }}: {{ .Values.labels.ovn.node_selector_value }}
initContainers:
{{- tuple $envAll "ovn_ovsdb_sb" list | include "helm-toolkit.snippets.kubernetes_entrypoint_init_container" | indent 8 }}
containers:
diff --git a/charts/ovn/values.yaml b/charts/ovn/values.yaml
index 2f152dd..b0560ba 100644
--- a/charts/ovn/values.yaml
+++ b/charts/ovn/values.yaml
@@ -34,9 +34,6 @@
- image_repo_sync
labels:
- ovn:
- node_selector_key: ovn
- node_selector_value: enabled
ovn_ovsdb_nb:
node_selector_key: openstack-compute-node
node_selector_value: enabled
diff --git a/roles/kubernetes_node_labels/defaults/main.yml b/roles/kubernetes_node_labels/defaults/main.yml
index 5c427ff..6d1784d 100644
--- a/roles/kubernetes_node_labels/defaults/main.yml
+++ b/roles/kubernetes_node_labels/defaults/main.yml
@@ -18,21 +18,10 @@
{% set res = {} -%}
{% if inventory_hostname in groups.get('controllers', []) %}
{% set _ = res.update({'openstack-control-plane': 'enabled'}) %}
- {% if atmosphere_network_backend == 'openvswitch' %}
{% set _ = res.update({'openvswitch': 'enabled'}) %}
{% endif %}
- {% if atmosphere_network_backend == 'ovn' %}
- {% set _ = res.update({'ovn': 'enabled'}) %}
- {% endif %}
- {% endif %}
- atmosphere_network_backend
{% if inventory_hostname in groups.get('computes', []) %}
{% set _ = res.update({'openstack-compute-node': 'enabled'}) %}
- {% if atmosphere_network_backend == 'openvswitch' %}
{% set _ = res.update({'openvswitch': 'enabled'}) %}
{% endif %}
- {% if atmosphere_network_backend == 'ovn' %}
- {% set _ = res.update({'ovn': 'enabled'}) %}
- {% endif %}
- {% endif %}
{{ res }}