fix linter complaints, add empty ci values test

rstudio · Nov 15, 2023 · 978e4a2 · 978e4a2
1 parent f51d84b
commit 978e4a2
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/charts/posit-chronicle/ci/empty-values.yaml b/charts/posit-chronicle/ci/empty-values.yaml
diff --git a/charts/posit-chronicle/templates/_helpers.tpl b/charts/posit-chronicle/templates/_helpers.tpl
@@ -61,7 +61,7 @@ Generate selector labels for  various resources
 app: chronicle-server
 {{- end -}}
 
-{{- define "posit-chronicle.serviceaccount.selectorLabels" -}}
+{{- define "posit-chronicle.service.selectorLabels" -}}
 {{- range $key,$value := $.Values.serviceaccount.selectorLabels -}}
 {{ $key }}: {{ $value | quote }}
 {{ end }}

diff --git a/charts/posit-chronicle/templates/serviceaccount.yaml b/charts/posit-chronicle/templates/serviceaccount.yaml
@@ -1,5 +1,5 @@
 ---
-{{- if $.Values.serviceaccount.enabled -}}
+{{- if .Values.serviceaccount.enabled -}}
 apiVersion: v1
 kind: ServiceAccount
 metadata:
@@ -9,4 +9,3 @@ metadata:
   annotations:
     {{ include "posit-chronicle.serviceaccount.annotations" . | indent 4 }}
 {{- end -}}
----
diff --git a/charts/posit-chronicle/templates/stateful-set.yaml b/charts/posit-chronicle/templates/stateful-set.yaml
@@ -9,12 +9,12 @@ spec:
   replicas: {{ .Values.replicas }}
   selector:
     matchLabels:
-      {{- include "posit-chronicle.selectorLabels" . | nindent 6 }}
+      {{- include "posit-chronicle.pod.selectorLabels" . | nindent 6 }}
   template:
     metadata:
       labels:
         {{- include "posit-chronicle.pod.labels" . | nindent 8 }}
-        {{- include "posit-chronicle.selectorLabels" . | indent 8 }}
+        {{- include "posit-chronicle.pod.selectorLabels" . | indent 8 }}
       annotations:
         {{- include "posit-chronicle.pod.annotations" . | indent 8 }}
     spec:

diff --git a/charts/posit-chronicle/templates/svc.yaml b/charts/posit-chronicle/templates/svc.yaml
@@ -4,12 +4,13 @@ kind: Service
 metadata:
   name: chronicle-server
   labels:
-    {{ include "posit-chronicle.service.labels" . | indent 4 }}
+    {{- include "posit-chronicle.service.labels" . | nindent 4 }}
+    {{- include "posit-chronicle.service.selectorLabels" . | indent 4 }}
   annotations:
     {{ include "posit-chronicle.service.annotations" . | indent 4 }}
 spec:
   selector:
-    {{- include "posit-chronicle.selectorLabels" . | nindent 5 }}
+    {{- include "posit-chronicle.service.selectorLabels" . | indent 4 }}
   ports:
   - port: {{ .Values.service.rest.port }}
     targetPort: {{ .Values.service.rest.targetPort }}