Revert "Fix calico-node in etcd mode (kubernetes-sigs#10438)"

This reverts commit 29ea790. Check that CI still catches that bug with node-etcd-client
VannTen · Jan 11, 2024 · aca9b0d · aca9b0d
1 parent 3a8f921
commit aca9b0d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/roles/network_plugin/calico/templates/calico-config.yml.j2 b/roles/network_plugin/calico/templates/calico-config.yml.j2
@@ -36,6 +36,12 @@ data:
           {% if calico_datastore == "kdd" %}
             "datastore_type": "kubernetes",
             "nodename": "__KUBERNETES_NODE_NAME__",
+          {% else %}
+          {% if cloud_provider is defined %}
+            "nodename": "{{ calico_kubelet_name.stdout }}",
+            {% else %}
+            "nodename": "{{ calico_baremetal_nodename }}",
+          {% endif %}
           {% endif %}
             "type": "calico",
             "log_level": "info",
@@ -102,4 +108,3 @@ data:
         }
       ]
     }
-
diff --git a/roles/network_plugin/calico/templates/calico-node.yml.j2 b/roles/network_plugin/calico/templates/calico-node.yml.j2
@@ -96,13 +96,6 @@ spec:
             # Prevents the container from sleeping forever.
             - name: SLEEP
               value: "false"
-{% if calico_datastore == "etcd" %}
-            - name: ETCD_ENDPOINTS
-              valueFrom:
-                configMapKeyRef:
-                  name: calico-config
-                  key: etcd_endpoints
-{% endif %}
 {% if calico_datastore == "kdd" %}
             # Set the hostname based on the k8s node name.
             - name: KUBERNETES_NODE_NAME