fix(helm): remove namespace template to resolve installation conflicts (vllm-project#651)

* fix(helm): remove namespace template to resolve installation conflicts

The namespace.yaml template with lookup function causes installation
failures when the namespace already exists. Removing this template
allows Helm's --create-namespace flag to handle namespace creation
properly, avoiding conflicts with existing namespaces.

Fixes the error: namespaces "vllm-semantic-router-system" already exists

Signed-off-by: bitliu <[email protected]>

* fix(ci): update helm-ci workflow to remove namespace check

Remove Namespace from required resources check since namespace.yaml
template has been removed. The namespace is now managed by Helm's
--create-namespace flag instead of being part of the chart templates.

Signed-off-by: bitliu <[email protected]>

* fix(ci): update validate-chart.sh to remove namespace checks

Remove namespace.yaml from required files list and Namespace from
required resources check in the validation script. The namespace
is now managed by Helm's --create-namespace flag.

Signed-off-by: bitliu <[email protected]>

---------

Signed-off-by: bitliu <[email protected]>

Author: Xunzhuo

.github/workflows/helm-ci.yml

@@ -111,7 +111,6 @@ jobs:
         run: |
           echo "::group::Verify Required Resources"
           required_resources=(
-            "Namespace"
             "ServiceAccount"
             "PersistentVolumeClaim"
             "ConfigMap"
@@ -128,6 +127,7 @@ jobs:
             fi
           done
           echo "All required resources found"
+          echo "Note: Namespace is managed by Helm's --create-namespace flag"
           echo "::endgroup::"
 
       - name: Upload templates as artifacts

deploy/helm/semantic-router/templates/namespace.yaml

@@ -119,7 +119,6 @@ required_files=(
     "templates/deployment.yaml"
     "templates/service.yaml"
     "templates/configmap.yaml"
-    "templates/namespace.yaml"
     "templates/pvc.yaml"
     "templates/serviceaccount.yaml"
     "templates/ingress.yaml"
@@ -146,7 +145,6 @@ echo ""
 # Test 7: Validate generated resources
 log_info "Validating generated Kubernetes resources..."
 resource_types=(
-    "Namespace"
     "ServiceAccount"
     "PersistentVolumeClaim"
     "ConfigMap"
@@ -162,6 +160,7 @@ for resource in "${resource_types[@]}"; do
         exit 1
     fi
 done
+log_info "Note: Namespace is managed by Helm's --create-namespace flag"
 echo ""
 
 # Test 8: Validate Chart.yaml