fix: Update namespace in the helm chart (#337)

**Reason for Change**: - Update the namespace to use the release namespace. **Requirements** - [ ] added unit tests and e2e tests (if applicable). **Issue Fixed**: Fixes #336 **Notes for Reviewers**: --------- Signed-off-by: Heba <[email protected]> Co-authored-by: ishaansehgal99 <[email protected]>
kaito-project · Apr 4, 2024 · fde6369 · fde6369
1 parent 77aa95b
commit fde6369
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/charts/kaito/workspace/templates/clusterrole_binding.yaml b/charts/kaito/workspace/templates/clusterrole_binding.yaml
@@ -11,4 +11,4 @@ roleRef:
 subjects:
 - kind: ServiceAccount
   name: {{ include "kaito.fullname" . }}-sa
-  namespace: {{ include "kaito.fullname" . }}
+  namespace: {{ .Release.Namespace }}
diff --git a/charts/kaito/workspace/templates/nvidia-device-plugin-ds.yaml b/charts/kaito/workspace/templates/nvidia-device-plugin-ds.yaml
@@ -63,7 +63,7 @@ apiVersion: scheduling.k8s.io/v1
 kind: PriorityClass
 metadata:
   name: high-priority-nonpreempting
-  namespace: {{ include "kaito.fullname" . }}
+  namespace: {{ .Release.Namespace }}
   labels:
     {{- include "kaito.labels" . | nindent 4 }}
 value: 1000000

diff --git a/charts/kaito/workspace/templates/role_binding.yaml b/charts/kaito/workspace/templates/role_binding.yaml
@@ -12,4 +12,4 @@ roleRef:
 subjects:
 - kind: ServiceAccount
   name: {{ include "kaito.fullname" . }}-sa
-  namespace: {{ include "kaito.fullname" . }}
+  namespace: {{ .Release.Namespace }}
diff --git a/charts/kaito/workspace/templates/webhooks.yaml b/charts/kaito/workspace/templates/webhooks.yaml
@@ -10,7 +10,7 @@ webhooks:
     clientConfig:
       service:
         name: {{ include "kaito.fullname" . }}
-        namespace: {{ include "kaito.fullname" . }}
+        namespace: {{ .Release.Namespace }}
         port: {{ .Values.webhook.port }}
     failurePolicy: Fail
     sideEffects: None