Fix Helm test, values linting (#2726)

* add test annotations, rename Signed-off-by: chipzoller <[email protected]> * linting of comments Signed-off-by: chipzoller <[email protected]> * reorder Signed-off-by: chipzoller <[email protected]> * Use existing RBAC Signed-off-by: chipzoller <[email protected]> --------- Signed-off-by: chipzoller <[email protected]>
kubecost · Nov 2, 2023 · f395421 · f395421
1 parent 11bc1a6
commit f395421
Show file tree

Hide file tree

Showing 2 changed files with 64 additions and 93 deletions.
diff --git a/cost-analyzer/templates/tests/basic-health.yaml b/cost-analyzer/templates/tests/basic-health.yaml
@@ -1,3 +1,4 @@
+---
 apiVersion: v1
 kind: Pod
 metadata:
@@ -6,7 +7,7 @@ metadata:
   annotations:
     {{- include "kubecost.test.annotations" . | nindent 4 }}
 spec:
-  serviceAccountName: tester
+  serviceAccountName: {{ template "cost-analyzer.serviceAccountName" . }}
   restartPolicy: Never
   containers:
   - name: test-kubecost
@@ -27,33 +28,3 @@ spec:
           echo "Failed to fetch Kubecost configuration. Response was $response"
           exit 1
         fi
----
-apiVersion: rbac.authorization.k8s.io/v1
-kind: Role
-metadata:
-  name: test-role
-  namespace: {{ .Release.Namespace }}
-rules:
-- apiGroups: [""]
-  resources: ["services"]
-  verbs: ["list"]
----
-apiVersion: rbac.authorization.k8s.io/v1
-kind: RoleBinding
-metadata:
-  name: test-rolebinding
-  namespace: {{ .Release.Namespace }}
-subjects:
-- kind: ServiceAccount
-  name: tester
-  namespace: {{ .Release.Namespace }}
-roleRef:
-  kind: Role
-  name: test-role
-  apiGroup: rbac.authorization.k8s.io
----
-apiVersion: v1
-kind: ServiceAccount
-metadata:
-  name: tester
-  namespace: {{ .Release.Namespace }}