Merge pull request #407 from datamattsson/chartfixes

Lint
hpe-storage · Dec 12, 2024 · d3804a2 · d3804a2
2 parents 2c06ea2 + c9ce4af
commit d3804a2
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 12 deletions.
diff --git a/helm/charts/hpe-csi-driver/templates/hpe-csi-node.yaml b/helm/charts/hpe-csi-driver/templates/hpe-csi-node.yaml
@@ -62,15 +62,15 @@ spec:
           env:
             - name: CSI_ENDPOINT
               value: unix:///csi/csi.sock
-            {{ if .Values.disableNodeConformance -}}
+            {{- if .Values.disableNodeConformance }}
             - name: DISABLE_NODE_CONFORMANCE
               value: "true"
             {{- end }}
-            {{ if .Values.disableNodeConfiguration -}}
+            {{- if .Values.disableNodeConfiguration }}
             - name: DISABLE_NODE_CONFIGURATION
               value: "true"
             {{- end }}
-            {{ if .Values.disableNodeMonitor -}}
+            {{- if .Values.disableNodeMonitor }}
             - name: DISABLE_NODE_MONITOR
               value: "true"
             {{- end }}
@@ -136,27 +136,27 @@ spec:
             - name: CHAP_SECRET_NAMESPACE
               value: {{ .Release.Namespace }}
             {{- end }}
-            {{ if .Values.disableNodeConformance -}}
+            {{- if .Values.disableNodeConformance }}
             - name: DISABLE_NODE_CONFORMANCE
               value: "true"
             {{- end }}
-            {{ if .Values.disableNodeConfiguration -}}
+            {{- if .Values.disableNodeConfiguration }}
             - name: DISABLE_NODE_CONFIGURATION
               value: "true"
             {{- end }}
-            {{ if .Values.disableNodeMonitor -}}
+            {{- if .Values.disableNodeMonitor }}
             - name: DISABLE_NODE_MONITOR
               value: "true"
             {{- end }}
-            {{ if .Values.maxVolumesPerNode -}}
+            {{- if .Values.maxVolumesPerNode }}
             - name: MAX_VOLUMES_PER_NODE
               value: "{{ .Values.maxVolumesPerNode }}"
             {{- end }}
             {{- if .Values.kubeletRootDir }}
             - name: KUBELET_ROOT_DIR
               value: {{ .Values.kubeletRootDir }}
             {{- end }}
-            {{ if .Values.disableNodeGetVolumeStats -}}
+            {{- if .Values.disableNodeGetVolumeStats }}
             - name: DISABLE_NODE_GET_VOLUMESTATS
               value: "true"
             {{- end }}

diff --git a/helm/charts/hpe-csi-driver/templates/hpe-csi-rbac.yaml b/helm/charts/hpe-csi-driver/templates/hpe-csi-rbac.yaml
@@ -205,7 +205,6 @@ roleRef:
   apiGroup: rbac.authorization.k8s.io
 
 ---
-
 # Resizer must be able to work with end point in current namespace
 # if (and only if) leadership election is enabled
 kind: Role
@@ -366,7 +365,6 @@ rules:
     verbs: ["get", "list"]
 
 ---
-
 kind: ClusterRoleBinding
 apiVersion: rbac.authorization.k8s.io/v1
 metadata:
@@ -416,7 +414,6 @@ subjects:
     name: hpe-csi-controller-sa
     # replace with non-default namespace name
     namespace: {{ .Release.Namespace }}
-
 roleRef:
   kind: ClusterRole
   name: hpe-csi-mutator-role
@@ -445,7 +442,6 @@ subjects:
   - kind: ServiceAccount
     name: hpe-csi-controller-sa
     namespace: {{ .Release.Namespace }}
-
 roleRef:
   kind: Role
   name: hpe-csi-mutator-cfg