Skip to content

Commit

Permalink
test: Fix incorrect uninstall in K8sBandwidth
Browse files Browse the repository at this point in the history
The statement to uninstall Cilium in K8sBandwidth/AfterAll doesn't match
the statements to install it, meaning we could try to disable Cilium
even though we never installed it. This commit fixes it.

Signed-off-by: Paul Chaignon <paul@cilium.io>
  • Loading branch information
pchaigno authored and tklauser committed May 17, 2021
1 parent 80fbcac commit 15437e4
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions test/k8sT/Bandwidth.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ var _ = Describe("K8sBandwidthTest", func() {
AfterAll(func() {
_ = kubectl.Delete(demoYAML)
ExpectAllPodsTerminated(kubectl)

UninstallCiliumFromManifest(kubectl, ciliumFilename)
kubectl.CloseSSHClient()
})

Expand Down Expand Up @@ -87,6 +85,10 @@ var _ = Describe("K8sBandwidthTest", func() {
backgroundCancel()
})

AfterAll(func() {
UninstallCiliumFromManifest(kubectl, ciliumFilename)
})

waitForTestPods := func() {
podLabels := []string{
testDS10,
Expand Down

0 comments on commit 15437e4

Please sign in to comment.