openstack/openstack-helm-infra
Code Issues Proposed changes

Remove helm status from deployment scripts for osh-infra-local-storage

Browse Source 
With the move to helm v3, helm status requires a namespace to be specified, but doing so breaks helm v2 compatability. This change removes the usage of helm serve in openstack-helm-infra's deployment scripts.

Change-Id: I507f6e786b5e35741030c500368638d586c99c12
This commit is contained in:
jayonlau 2021-10-13 12:46:22 -04:00
parent 391813463d
commit 962686763b
2 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
tools/deployment/osh-infra-local-storage/020-local-storage.sh
View File

@ -33,9 +33,6 @@ helm upgrade --install local-storage ./local-storage \
#NOTE: Wait for deploy #NOTE: Wait for deploy
./tools/deployment/common/wait-for-pods.sh osh-infra ./tools/deployment/common/wait-for-pods.sh osh-infra
#NOTE: Validate Deployment info
helm status local-storage
# Simple object validation # Simple object validation
kubectl describe sc local-storage kubectl describe sc local-storage
kubectl get pv kubectl get pv

3
tools/deployment/osh-infra-local-storage/040-prometheus.sh
View File

@ -29,9 +29,6 @@ helm upgrade --install prometheus ./prometheus \
#NOTE: Wait for deploy #NOTE: Wait for deploy
./tools/deployment/common/wait-for-pods.sh osh-infra ./tools/deployment/common/wait-for-pods.sh osh-infra
#NOTE: Validate Deployment info
helm status prometheus
# Delete the test pod if it still exists # Delete the test pod if it still exists
kubectl delete pods -l application=prometheus,release_group=prometheus,component=test --namespace=osh-infra --ignore-not-found kubectl delete pods -l application=prometheus,release_group=prometheus,component=test --namespace=osh-infra --ignore-not-found
helm test prometheus helm test prometheus