Skip to content

Commit

Permalink
refactor(helm): update certificate approval command condition in NOTE…
Browse files Browse the repository at this point in the history
…S.txt (#130)

* fix: update certificate approval command condition in NOTES.txt

* refactor: remove SET_MULTIPLE env var from k8s cred mgr
  • Loading branch information
konstan authored Dec 9, 2024
1 parent 990c6f1 commit f3b82fa
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion helm/templates/NOTES.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ The NuvlaEdge has just been installed on your Kubernetes node {{ .Values.kuberne
for the certificate used to manage the NuvlaEdge instance please run the
following command as system admin of the K8s cluster:{{printf "\n\n" }}

{{- if .Values.SET_MULTIPLE -}}
{{- if eq .Values.credManager.csr_name "nuvlaedge-csr" -}}
kubectl certificate approve {{ .Values.credManager.csr_name }}{{ printf "-" }}{{ include "nuvlaedge.uuid" . }}{{printf "\n\n" }}
{{- else -}}
kubectl certificate approve {{ .Values.credManager.csr_name }}{{printf "\n\n" }}
Expand Down
2 changes: 0 additions & 2 deletions helm/templates/kubernetes-credentials-manager-job.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,6 @@ spec:
value: {{ .Values.NUVLAEDGE_UUID }}
- name: MY_NAMESPACE
value: {{ include "nuvlaedge.namespace" . }}
- name: SET_MULTIPLE
value: {{ .Values.SET_MULTIPLE }}
volumes:
- name: nuvlaedge-data
hostPath:
Expand Down

0 comments on commit f3b82fa

Please sign in to comment.