Skip to content

Commit 77239f2

Browse files
committed
flaky test: application failover with purgeMode gracefully when the GracePeriodSeconds is reach out
Signed-off-by: zhzhuang-zju <[email protected]>
1 parent 4b5aa6b commit 77239f2

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

test/e2e/suites/base/failover_test.go

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -306,6 +306,7 @@ var _ = ginkgo.Describe("application failover testing", func() {
306306
var policy *policyv1alpha1.PropagationPolicy
307307
var maxGroups, minGroups int
308308
var gracePeriodSeconds, tolerationSeconds int32
309+
var targetClusters []string
309310

310311
ginkgo.BeforeEach(func() {
311312
policyNamespace = testNamespace
@@ -315,6 +316,7 @@ var _ = ginkgo.Describe("application failover testing", func() {
315316
deployment = testhelper.NewDeployment(deploymentNamespace, deploymentName)
316317
maxGroups = 1
317318
minGroups = 1
319+
targetClusters = framework.ClusterNames()[:2]
318320

319321
policy = &policyv1alpha1.PropagationPolicy{
320322
ObjectMeta: metav1.ObjectMeta{
@@ -331,7 +333,7 @@ var _ = ginkgo.Describe("application failover testing", func() {
331333
},
332334
Placement: policyv1alpha1.Placement{
333335
ClusterAffinity: &policyv1alpha1.ClusterAffinity{
334-
ClusterNames: framework.ClusterNames()[:2],
336+
ClusterNames: targetClusters,
335337
},
336338
SpreadConstraints: []policyv1alpha1.SpreadConstraint{
337339
{
@@ -372,6 +374,9 @@ var _ = ginkgo.Describe("application failover testing", func() {
372374

373375
ginkgo.It("application failover with purgeMode gracefully when the application come back to healthy on the new cluster", func() {
374376
disabledClusters := framework.ExtractTargetClustersFromRB(controlPlaneClient, deployment.Kind, deployment.Namespace, deployment.Name)
377+
framework.WaitDeploymentPresentOnClustersFitWith(disabledClusters, deployment.Namespace, deployment.Name, func(deployment *appsv1.Deployment) bool {
378+
return framework.CheckDeploymentReadyStatus(deployment, *deployment.Spec.Replicas)
379+
})
375380
ginkgo.By("create an overridePolicy to make the application unhealthy", func() {
376381
overridePolicy := testhelper.NewOverridePolicyByOverrideRules(policyNamespace, policyName, []policyv1alpha1.ResourceSelector{
377382
{
@@ -445,6 +450,9 @@ var _ = ginkgo.Describe("application failover testing", func() {
445450
})
446451

447452
disabledClusters := framework.ExtractTargetClustersFromRB(controlPlaneClient, deployment.Kind, deployment.Namespace, deployment.Name)
453+
framework.WaitDeploymentPresentOnClustersFitWith(disabledClusters, deployment.Namespace, deployment.Name, func(deployment *appsv1.Deployment) bool {
454+
return framework.CheckDeploymentReadyStatus(deployment, *deployment.Spec.Replicas)
455+
})
448456
var beginTime time.Time
449457
ginkgo.By("create an overridePolicy to make the application unhealthy", func() {
450458
overridePolicy := testhelper.NewOverridePolicyByOverrideRules(policyNamespace, policyName, []policyv1alpha1.ResourceSelector{

0 commit comments

Comments
 (0)