Skip to content

Commit

Permalink
fix(e2e): don't reuse namespace for helm upgrade test
Browse files Browse the repository at this point in the history
Signed-off-by: Marcin Skalski <[email protected]>
  • Loading branch information
Automaat committed Sep 24, 2024
1 parent a060222 commit a6ce972
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions test/e2e/helm/kuma_helm_upgrade_multizone.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ import (
)

func UpgradingWithHelmChartMultizone() {
namespace := "helm-upgrade-ns"
var global, zoneK8s, zoneUniversal Cluster
var globalCP ControlPlane

Expand All @@ -42,7 +43,7 @@ func UpgradingWithHelmChartMultizone() {
}()
go func() {
defer grp.Done()
Expect(zoneK8s.DeleteNamespace(TestNamespace)).To(Succeed())
Expect(zoneK8s.DeleteNamespace(namespace)).To(Succeed())
Expect(zoneK8s.DeleteKuma()).To(Succeed())
Expect(zoneK8s.DismissCluster()).To(Succeed())
}()
Expand Down Expand Up @@ -114,7 +115,7 @@ spec:
By("Sync DPPs from Zone to Global")
// when start test server on Zone
err = NewClusterSetup().
Install(NamespaceWithSidecarInjection(TestNamespace)).
Install(NamespaceWithSidecarInjection(namespace)).
Install(testserver.Install()).Setup(zoneK8s)
Expect(err).ToNot(HaveOccurred())

Expand Down

0 comments on commit a6ce972

Please sign in to comment.