Merge PR 24 and 25 from develop (#26)

* changes (#24) * added missing --create-namespace * Update ds389.sh * Change DS389 Helm name change (#25) * name change * Update Chart.yaml with version increment The checking tool can't realize when you change the name it's a new helm chart --------- Co-authored-by: Raul Mahiques <[email protected]>
SUSE · Oct 31, 2024 · be7657b · be7657b
1 parent 086515a
commit be7657b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/charts/ds389/Chart.yaml b/charts/ds389/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v2
-name: ds389-helm-chart
+name: ds389
 description: A Helm chart for deploying the 389 Directory Server
-version: 1.0.0
+version: 1.0.1
 appVersion: "1.0"
 maintainers:
   - name: rmahique

diff --git a/scripts/authentication/ds389.sh b/scripts/authentication/ds389.sh
@@ -115,9 +115,9 @@ function ds389_install() {
   # installs the chart with default parameters
   if [[ -f values.yaml ]]
   then
-    helm upgrade --install ds389  --namespace ds389 suse-lab-setup/ds389 -f values.yaml
+    helm upgrade --install ds389  --namespace ds389 --create-namespace suse-lab-setup/ds389 -f values.yaml
   else
-    helm upgrade --install ds389  --namespace ds389 suse-lab-setup/ds389
+    helm upgrade --install ds389  --namespace ds389 --create-namespace suse-lab-setup/ds389
   fi
   sleep 60
   ds389_restrict_permissions "${_ldap_uri}" "${_ldap_basedn}" "${_admin_user}" "${_admin_pwd}"
@@ -130,6 +130,6 @@ function ds389_install() {
 #   ds389_uninstall
 #######################################
 function ds389_uninstall() {
-  helm uninstall ds389
+  helm uninstall ds389 -n ds389
   sleep 15 
 }