move logic for overriding CCM into kustomize instead of separate helm…

…ChartProxies (#298) * move logic for overriding CCM into kustomize instead of separate helmChartProxies
linode · May 1, 2024 · 5180d4c · 5180d4c
1 parent 3bc1f07
commit 5180d4c
Show file tree

Hide file tree

Showing 10 changed files with 58 additions and 98 deletions.
diff --git a/...es/addons/provider-linode/linode-ccm.yaml → templates/addons/ccm-linode/ccm-linode.yaml b/...es/addons/provider-linode/linode-ccm.yaml → templates/addons/ccm-linode/ccm-linode.yaml
@@ -4,9 +4,8 @@ metadata:
   name: ${CLUSTER_NAME}-linode-cloud-controller-manager
 spec:
   clusterSelector:
-    matchExpressions:
-      - {key: vpcless, operator: DoesNotExist}
-      - {key: ccm, operator: In, values: ["${CLUSTER_NAME}-linode"]}
+    matchLabels:
+      ccm: "${CLUSTER_NAME}-linode"
   repoURL: https://linode.github.io/linode-cloud-controller-manager/
   chartName: ccm-linode
   namespace: kube-system

diff --git a/...addons/provider-linode/kustomization.yaml → ...ates/addons/ccm-linode/kustomization.yaml b/...addons/provider-linode/kustomization.yaml → ...ates/addons/ccm-linode/kustomization.yaml
@@ -1,5 +1,4 @@
 apiVersion: kustomize.config.k8s.io/v1beta1
 kind: Kustomization
 resources:
-    - linode-ccm.yaml
-    - linode-ccm-vpcless.yaml
+    - ccm-linode.yaml
diff --git a/templates/addons/cilium/cilium-ipv6.yaml b/templates/addons/cilium/cilium-ipv6.yaml
diff --git a/templates/addons/cilium/cilium-vxlan.yaml b/templates/addons/cilium/cilium-vxlan.yaml
diff --git a/templates/addons/cilium/cilium.yaml b/templates/addons/cilium/cilium.yaml
@@ -4,10 +4,8 @@ metadata:
   name: ${CLUSTER_NAME}-cilium
 spec:
   clusterSelector:
-    matchExpressions:
-      - {key: ipv6, operator: DoesNotExist}
-      - {key: vxlan, operator: DoesNotExist}
-      - {key: cni, operator: In, values: ["${CLUSTER_NAME}-cilium"]}
+    matchLabels:
+      cni: "${CLUSTER_NAME}-cilium"
   repoURL: https://helm.cilium.io/
   chartName: cilium
   namespace: kube-system

diff --git a/templates/addons/cilium/kustomization.yaml b/templates/addons/cilium/kustomization.yaml
@@ -2,5 +2,3 @@ apiVersion: kustomize.config.k8s.io/v1beta1
 kind: Kustomization
 resources:
     - cilium.yaml
-    - cilium-ipv6.yaml
-    - cilium-vxlan.yaml
diff --git a/templates/addons/provider-linode/linode-ccm-vpcless.yaml b/templates/addons/provider-linode/linode-ccm-vpcless.yaml
diff --git a/templates/flavors/default/kustomization.yaml b/templates/flavors/default/kustomization.yaml
@@ -6,7 +6,7 @@ resources:
   - kubeadmControlPlane.yaml
   - ../../addons/cilium
   - ../../addons/csi-driver-linode
-  - ../../addons/provider-linode
+  - ../../addons/ccm-linode
   - ../../addons/cluster-resource-set
 patches:
   - target:

diff --git a/templates/flavors/dual-stack/kustomization.yaml b/templates/flavors/dual-stack/kustomization.yaml
@@ -14,8 +14,6 @@ patches:
       kind: Cluster
       metadata:
         name: ${CLUSTER_NAME}
-        labels:
-          ipv6: "true"
       spec:
         clusterNetwork:
           pods:
@@ -48,3 +46,23 @@ patches:
             controllerManager:
               extraArgs:
                 node-cidr-mask-size-ipv6: "96"
+  - target:
+      kind: HelmChartProxy
+      name: .*-cilium
+    patch: |-
+      - op: replace
+        path: /spec/valuesTemplate
+        value: |
+          bgpControlPlane:
+            enabled: true
+          ipv6:
+            enabled: true
+          ipam:
+            mode: kubernetes
+          k8s:
+            requireIPv4PodCIDR: true
+          hubble:
+            relay:
+              enabled: true
+            ui:
+              enabled: true
diff --git a/templates/flavors/vpcless/kustomization.yaml b/templates/flavors/vpcless/kustomization.yaml
@@ -4,6 +4,38 @@ resources:
   - ../default
 
 patches:
+  - target:
+      kind: HelmChartProxy
+      name: .*-cilium
+    patch: |-
+      - op: replace
+        path: /spec/valuesTemplate
+        value: |
+          bgpControlPlane:
+            enabled: true
+          ipam:
+            mode: kubernetes
+          k8s:
+            requireIPv4PodCIDR: true
+          hubble:
+            relay:
+              enabled: true
+            ui:
+              enabled: true
+
+  - target:
+      kind: HelmChartProxy
+      name: .*-linode-cloud-controller-manager
+    patch: |-
+      - op: replace
+        path: /spec/valuesTemplate
+        value: |
+          secretRef:
+            name: "linode-token-region"
+          image:
+            pullPolicy: IfNotPresent
+
+
   - target:
       kind: LinodeVPC
     patch: |-