diff --git a/.github/workflows/e2e-1.30.yaml b/.github/workflows/e2e-1.30.yaml index ab31fabf67..2218eb20c9 100644 --- a/.github/workflows/e2e-1.30.yaml +++ b/.github/workflows/e2e-1.30.yaml @@ -171,7 +171,7 @@ jobs: - name: Run E2E Tests run: | export KUBECONFIG=/home/runner/.kube/config - tools/hack/run-kruise-e2e-test.sh --focus '\[apps\] SidecarSet' --print-info --disable-parallel + tools/hack/run-kruise-e2e-test.sh --focus '\[apps\] SidecarSet' --print-info ephemeraljob: runs-on: ubuntu-20.04 @@ -231,7 +231,7 @@ jobs: - name: Run E2E Tests run: | export KUBECONFIG=/home/runner/.kube/config - tools/hack/run-kruise-e2e-test.sh --focus '\[policy\] PodUnavailableBudget' --print-info --disable-parallel + tools/hack/run-kruise-e2e-test.sh --focus '\[policy\] PodUnavailableBudget' --print-info clonesetAndInplace: runs-on: ubuntu-20.04 diff --git a/test/e2e/apps/sidecarset.go b/test/e2e/apps/sidecarset.go index 671d505a6e..9b2b81adc0 100644 --- a/test/e2e/apps/sidecarset.go +++ b/test/e2e/apps/sidecarset.go @@ -24,13 +24,14 @@ import ( "strconv" "time" + v1core "k8s.io/client-go/kubernetes/typed/core/v1" + appsv1alpha1 "github.com/openkruise/kruise/apis/apps/v1alpha1" kruiseclientset "github.com/openkruise/kruise/pkg/client/clientset/versioned" "github.com/openkruise/kruise/pkg/control/sidecarcontrol" "github.com/openkruise/kruise/pkg/util" "github.com/openkruise/kruise/pkg/util/configuration" "github.com/openkruise/kruise/test/e2e/framework" - v1core "k8s.io/client-go/kubernetes/typed/core/v1" "github.com/onsi/ginkgo" "github.com/onsi/gomega" @@ -47,6 +48,7 @@ import ( ) var _ = SIGDescribe("SidecarSet", func() { + framework.TestContext.DeleteNamespaceOnFailure = false f := framework.NewDefaultFramework("sidecarset") var ns string var c clientset.Interface @@ -66,7 +68,7 @@ var _ = SIGDescribe("SidecarSet", func() { framework.DumpDebugInfo(c, ns) } framework.Logf("Deleting all SidecarSet in cluster") - tester.DeleteSidecarSets() + tester.DeleteSidecarSets(ns) tester.DeleteDeployments(ns) }) framework.ConformanceIt("pods don't have matched sidecarSet", func() { @@ -652,14 +654,14 @@ var _ = SIGDescribe("SidecarSet", func() { })*/ }) - framework.KruiseDescribe("SidecarSet Upgrade functionality [SidecarSeUpgrade]", func() { + framework.KruiseDescribe("SidecarSet Upgrade functionality [SidecarSetUpgrade]", func() { ginkgo.AfterEach(func() { if ginkgo.CurrentGinkgoTestDescription().Failed { framework.DumpDebugInfo(c, ns) } framework.Logf("Deleting all SidecarSet in cluster") - tester.DeleteSidecarSets() + tester.DeleteSidecarSets(ns) tester.DeleteDeployments(ns) }) diff --git a/test/e2e/apps/sidecarset_hotupgrade.go b/test/e2e/apps/sidecarset_hotupgrade.go index cb66da0670..e1a825b239 100644 --- a/test/e2e/apps/sidecarset_hotupgrade.go +++ b/test/e2e/apps/sidecarset_hotupgrade.go @@ -53,7 +53,7 @@ var _ = SIGDescribe("SidecarSet", func() { framework.DumpDebugInfo(c, ns) } framework.Logf("Deleting all SidecarSet in cluster") - tester.DeleteSidecarSets() + tester.DeleteSidecarSets(ns) tester.DeleteDeployments(ns) }) diff --git a/test/e2e/framework/sidecarset_utils.go b/test/e2e/framework/sidecarset_utils.go index bcfd468bd7..5d79a0b403 100644 --- a/test/e2e/framework/sidecarset_utils.go +++ b/test/e2e/framework/sidecarset_utils.go @@ -18,14 +18,16 @@ package framework import ( "context" + "fmt" "time" + "k8s.io/utils/ptr" + appsv1alpha1 "github.com/openkruise/kruise/apis/apps/v1alpha1" kruiseclientset "github.com/openkruise/kruise/pkg/client/clientset/versioned" "github.com/openkruise/kruise/pkg/control/sidecarcontrol" "github.com/openkruise/kruise/pkg/util" webhookutil "github.com/openkruise/kruise/pkg/webhook/util" - "k8s.io/utils/ptr" "github.com/onsi/gomega" apps "k8s.io/api/apps/v1" @@ -58,7 +60,8 @@ func (s *SidecarSetTester) NewBaseSidecarSet(ns string) *appsv1alpha1.SidecarSet APIVersion: "apps.kruise.io/v1alpha1", }, ObjectMeta: metav1.ObjectMeta{ - Name: "test-sidecarset", + + Name: fmt.Sprintf("test-sidecarset-%s", ns), Labels: map[string]string{ "app": "sidecar", }, @@ -226,7 +229,7 @@ func (s *SidecarSetTester) CreateDeployment(deployment *apps.Deployment) { s.WaitForDeploymentRunning(deployment) } -func (s *SidecarSetTester) DeleteSidecarSets() { +func (s *SidecarSetTester) DeleteSidecarSets(ns string) { sidecarSetList, err := s.kc.AppsV1alpha1().SidecarSets().List(context.TODO(), metav1.ListOptions{}) if err != nil { Logf("List sidecarSets failed: %s", err.Error()) @@ -234,6 +237,9 @@ func (s *SidecarSetTester) DeleteSidecarSets() { } for _, sidecarSet := range sidecarSetList.Items { + if sidecarSet.Namespace != ns { + continue + } s.DeleteSidecarSet(&sidecarSet) } } diff --git a/test/e2e/policy/podunavailablebudget.go b/test/e2e/policy/podunavailablebudget.go index f683d8fc3a..615a213876 100644 --- a/test/e2e/policy/podunavailablebudget.go +++ b/test/e2e/policy/podunavailablebudget.go @@ -65,7 +65,7 @@ var _ = SIGDescribe("PodUnavailableBudget", func() { tester.DeletePubs(ns) tester.DeleteDeployments(ns) tester.DeleteCloneSets(ns) - sidecarTester.DeleteSidecarSets() + sidecarTester.DeleteSidecarSets(ns) }) ginkgo.It("PodUnavailableBudget selector no matched pods", func() {