Skip to content

Commit

Permalink
fix(e2e): don't reuse namespace for helm upgrade test (#11539)
Browse files Browse the repository at this point in the history
---------

Signed-off-by: Marcin Skalski <skalskimarcin33@gmail.com>
  • Loading branch information
Automaat authored Sep 25, 2024
1 parent 27b3498 commit 7d80a5a
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 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,8 +115,8 @@ spec:
By("Sync DPPs from Zone to Global")
// when start test server on Zone
err = NewClusterSetup().
Install(NamespaceWithSidecarInjection(TestNamespace)).
Install(testserver.Install()).Setup(zoneK8s)
Install(NamespaceWithSidecarInjection(namespace)).
Install(testserver.Install(testserver.WithNamespace(namespace))).Setup(zoneK8s)
Expect(err).ToNot(HaveOccurred())

Eventually(func(g Gomega) (int, error) {
Expand Down

0 comments on commit 7d80a5a

Please sign in to comment.