openstack/openstack-helm-infra
Code Issues Proposed changes

Remove helm status from deployment scripts for keystone-auth

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: I4cf135dc9852506cc2e853c9faa8544b7b2e2fae
This commit is contained in:
jayonlau 2021-10-13 12:33:07 -04:00
parent 74b79700c0
commit 248a280293
3 changed files with 0 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tools/deployment/keystone-auth/020-ingress.sh
View File

@ -32,9 +32,6 @@ helm upgrade --install ingress-kube-system ./ingress \
#NOTE: Wait for deploy #NOTE: Wait for deploy
./tools/deployment/common/wait-for-pods.sh kube-system ./tools/deployment/common/wait-for-pods.sh kube-system
#NOTE: Display info
helm status ingress-kube-system
#NOTE: Deploy namespace ingress #NOTE: Deploy namespace ingress
for NAMESPACE in openstack; do for NAMESPACE in openstack; do
helm upgrade --install ingress-${NAMESPACE} ./ingress \ helm upgrade --install ingress-${NAMESPACE} ./ingress \
@ -44,7 +41,4 @@ for NAMESPACE in openstack; do
#NOTE: Wait for deploy #NOTE: Wait for deploy
./tools/deployment/common/wait-for-pods.sh ${NAMESPACE} ./tools/deployment/common/wait-for-pods.sh ${NAMESPACE}
#NOTE: Display info
helm status ingress-${NAMESPACE}
done done

3
tools/deployment/keystone-auth/060-mariadb.sh
View File

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

4
tools/deployment/keystone-auth/070-keystone.sh
View File

@ -28,9 +28,6 @@ helm upgrade --install ldap ./ldap \
${OSH_INFRA_EXTRA_HELM_ARGS} \ ${OSH_INFRA_EXTRA_HELM_ARGS} \
${OSH_INFRA_EXTRA_HELM_ARGS_LDAP} ${OSH_INFRA_EXTRA_HELM_ARGS_LDAP}
./tools/deployment/common/wait-for-pods.sh openstack
helm status ldap
# Install Keystone # Install Keystone
cd ${OSH_PATH} cd ${OSH_PATH}
make keystone make keystone
@ -42,7 +39,6 @@ helm upgrade --install keystone ${OSH_PATH}/keystone \
${OSH_EXTRA_HELM_ARGS_KEYSTONE} ${OSH_EXTRA_HELM_ARGS_KEYSTONE}
./tools/deployment/common/wait-for-pods.sh openstack ./tools/deployment/common/wait-for-pods.sh openstack
helm status keystone
# Testing basic functionality # Testing basic functionality
export OS_CLOUD=openstack_helm export OS_CLOUD=openstack_helm