Merge pull request #207 from Vacant2333/fix-role

fix: remove the unused env and add the required roles

Author: jwcesign

charts/karpenter/templates/deployment.yaml

@@ -50,8 +50,6 @@ spec:
               value: "{{ .Values.controller.healthProbe.port }}"
             - name: CLUSTER_ID
               value: {{ .Values.controller.settings.clusterID }}
-            - name: CLUSTER_CNI
-              value: {{ .Values.controller.settings.clusterCNI }}
             - name: TELEMETRY_SHARE
               value: "{{ .Values.controller.settings.telemetryShare }}"
             - name: KUBERNETES_MIN_VERSION

charts/karpenter/templates/role.yaml

@@ -37,3 +37,44 @@ rules:
   - apiGroups: [""]
     resources: ["secrets"]
     verbs: ["get", "list", "watch"]
+---
+apiVersion: rbac.authorization.k8s.io/v1
+kind: RoleBinding
+metadata:
+  name: {{ include "karpenter.fullname" . }}-lease
+  namespace: kube-node-lease
+  labels:
+    {{- include "karpenter.labels" . | nindent 4 }}
+  {{- with .Values.additionalAnnotations }}
+  annotations:
+    {{- toYaml . | nindent 4 }}
+  {{- end }}
+roleRef:
+  apiGroup: rbac.authorization.k8s.io
+  kind: Role
+  name: {{ include "karpenter.fullname" . }}-lease
+subjects:
+  - kind: ServiceAccount
+    name: {{ template "karpenter.serviceAccountName" . }}
+    namespace: {{ .Release.Namespace }}
+---
+apiVersion: rbac.authorization.k8s.io/v1
+kind: Role
+metadata:
+  name: {{ include "karpenter.fullname" . }}-lease
+  namespace: kube-node-lease
+  labels:
+    {{- include "karpenter.labels" . | nindent 4 }}
+  {{- with .Values.additionalAnnotations }}
+  annotations:
+    {{- toYaml . | nindent 4 }}
+  {{- end }}
+rules:
+  # Read
+  - apiGroups: ["coordination.k8s.io"]
+    resources: ["leases"]
+    verbs: ["get", "list", "watch"]
+  # Write
+  - apiGroups: ["coordination.k8s.io"]
+    resources: ["leases"]
+    verbs: ["delete"]

charts/karpenter/values.yaml

@@ -60,8 +60,6 @@ controller:
   affinity: {}
   tolerations: []
   settings:
-    # -- The network cni used by the cluster.
-    clusterCNI: terway-eniip
     # -- The external kubernetes cluster id for new nodes to connect with.
     clusterID: ""
     # -- The VM memory overhead as a percent that will be subtracted from the total memory for all instance types. The value of `0.075` equals to 7.5%.