Commit d3e10f97 authored by Geoff Simmons's avatar Geoff Simmons

Wait more carefully for complete undeployment on test runs.

Call helm uninstall with --wait, and where necessary, wait for viking
service pods to be undeployed.

This minimizes errors during the sequential e2e test run -- the
controller may still be working on deletion of the resources of the
previous test when resources of the next test are added.
parent 386f6e43
...@@ -30,6 +30,7 @@ mkpath := $(abspath $(lastword $(MAKEFILE_LIST))) ...@@ -30,6 +30,7 @@ mkpath := $(abspath $(lastword $(MAKEFILE_LIST)))
mkdir := $(dir $(mkpath)) mkdir := $(dir $(mkpath))
CHARTDIR=$(mkdir)/../../../charts CHARTDIR=$(mkdir)/../../../charts
TESTDIR=$(mkdir)/../../../test
GEN_SECRET=head -c32 /dev/urandom | base64 | tr +/ -_ GEN_SECRET=head -c32 /dev/urandom | base64 | tr +/ -_
...@@ -127,12 +128,13 @@ verify: ...@@ -127,12 +128,13 @@ verify:
$(mkdir)/verify.sh $(mkdir)/verify.sh
undeploy-helm: undeploy-helm:
@helm uninstall other-ingress --namespace other @helm uninstall --wait other-ingress --namespace other
@helm uninstall tea-ingress --namespace cafe @helm uninstall --wait tea-ingress --namespace cafe
@helm uninstall coffee-ingress --namespace cafe @helm uninstall --wait coffee-ingress --namespace cafe
@helm uninstall viking-service-cafe --namespace cafe @helm uninstall --wait viking-service-cafe --namespace cafe
@helm uninstall viking-service-system --namespace kube-system @helm uninstall --wait viking-service-system --namespace kube-system
@kubectl delete -f namespace.yaml @kubectl delete -f namespace.yaml
$(TESTDIR)/wait_delete.sh app=varnish-ingress kube-system
undeploy-kubectl: undeploy-kubectl:
@kubectl delete -f other-ingress.yaml @kubectl delete -f other-ingress.yaml
...@@ -151,6 +153,7 @@ undeploy-kubectl: ...@@ -151,6 +153,7 @@ undeploy-kubectl:
@kubectl delete -f coffee.yaml @kubectl delete -f coffee.yaml
@kubectl delete -f rbac.yaml @kubectl delete -f rbac.yaml
@kubectl delete -f namespace.yaml @kubectl delete -f namespace.yaml
$(TESTDIR)/wait_delete.sh app=varnish-ingress kube-system
undeploy: undeploy:
......
...@@ -116,10 +116,10 @@ verify: ...@@ -116,10 +116,10 @@ verify:
$(mkdir)/verify.sh $(mkdir)/verify.sh
undeploy-helm: undeploy-helm:
@helm uninstall other-ingress --namespace other @helm uninstall --wait other-ingress --namespace other
@helm uninstall tea-ingress --namespace tea @helm uninstall --wait tea-ingress --namespace tea
@helm uninstall coffee-ingress --namespace coffee @helm uninstall --wait coffee-ingress --namespace coffee
@helm uninstall viking-service-clusterwide --namespace kube-system @helm uninstall --wait viking-service-clusterwide -n kube-system
@kubectl delete -f namespace.yaml @kubectl delete -f namespace.yaml
$(TESTDIR)/wait_delete.sh app=varnish-ingress kube-system $(TESTDIR)/wait_delete.sh app=varnish-ingress kube-system
......
...@@ -60,7 +60,7 @@ wait: ...@@ -60,7 +60,7 @@ wait:
$(TESTDIR)/wait.sh app=varnish-ingress $(TESTDIR)/wait.sh app=varnish-ingress
uninstall-helm: uninstall-helm:
@helm uninstall viking-ingress @helm uninstall --wait viking-ingress
undeploy-helm: uninstall-helm wait undeploy-helm: uninstall-helm wait
......
...@@ -53,7 +53,7 @@ wait: ...@@ -53,7 +53,7 @@ wait:
$(TESTDIR)/wait.sh app=varnish-ingress $(TESTDIR)/wait.sh app=varnish-ingress
uninstall: uninstall:
@helm uninstall viking-ingress-change-secret @helm uninstall --wait viking-ingress-change-secret
undeploy: uninstall wait undeploy: uninstall wait
......
...@@ -62,7 +62,7 @@ uninstall-kubectl: ...@@ -62,7 +62,7 @@ uninstall-kubectl:
uninstall-helm: uninstall-helm:
@kubectl delete -f other-ingress.yaml @kubectl delete -f other-ingress.yaml
@helm uninstall viking-ingress-delete-secret @helm uninstall --wait viking-ingress-delete-secret
undeploy-kubectl: uninstall-kubectl wait undeploy-kubectl: uninstall-kubectl wait
......
...@@ -55,7 +55,7 @@ wait: ...@@ -55,7 +55,7 @@ wait:
$(TESTDIR)/wait.sh app=varnish-ingress $(TESTDIR)/wait.sh app=varnish-ingress
uninstall: uninstall:
@helm uninstall haproxy-tmo @helm uninstall --wait haproxy-tmo
undeploy: uninstall wait undeploy: uninstall wait
......
...@@ -71,7 +71,7 @@ wait: ...@@ -71,7 +71,7 @@ wait:
$(TESTDIR)/wait.sh app=varnish-ingress $(TESTDIR)/wait.sh app=varnish-ingress
uninstall: uninstall:
@helm uninstall viking-ingress @helm uninstall --wait viking-ingress
undeploy: uninstall wait undeploy: uninstall wait
endif endif
......
...@@ -55,7 +55,7 @@ wait: ...@@ -55,7 +55,7 @@ wait:
$(TESTDIR)/wait.sh app=varnish-ingress $(TESTDIR)/wait.sh app=varnish-ingress
uninstall: uninstall:
@helm uninstall viking-ingress-pathtypes @helm uninstall --wait viking-ingress-pathtypes
undeploy: uninstall wait undeploy: uninstall wait
endif endif
......
...@@ -112,9 +112,9 @@ verify: ...@@ -112,9 +112,9 @@ verify:
$(mkdir)/verify.sh cafe.vtc $(mkdir)/verify.sh cafe.vtc
undeploy: undeploy:
@helm uninstall --namespace probe-cfg viking-ingress-probe-cfg @helm uninstall --wait --namespace probe-cfg viking-ingress-probe-cfg
@helm uninstall --namespace probe-cfg viking-service-probe-cfg @helm uninstall --wait --namespace probe-cfg viking-service-probe-cfg
@helm uninstall --namespace probe-cfg viking-controller-probe-cfg @helm uninstall --wait --namespace probe-cfg viking-controller-probe-cfg
@kubectl delete -f namespace.yaml @kubectl delete -f namespace.yaml
endif endif
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment