Merge pull request #28 from SUSE/develop

Merge PR 24 and 25 from develop
SUSE · Oct 31, 2024 · 167cc7c · 167cc7c
2 parents 086515a + 44dfd87
commit 167cc7c
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 
 }