Skip to content

Commit 429de8c

Browse files
fix test-case timing issue
1 parent 0d81530 commit 429de8c

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

Jenkinsfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ boolean isNightly() {
238238
}
239239

240240
String getVersion() {
241-
def prefix = "4.2.0-dev1"
241+
def prefix = "4.2.0"
242242
def candidateName = ""
243243
if(isNightly()) {
244244
def timestamp = new Date().format("yyyy-MM-dd")

test/cluster/rack_revision_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ var _ = Describe(
105105

106106
Eventually(func() bool {
107107
return checkBothRevisionsExist(k8sClient, ctx, clusterNamespacedName, versionV1, versionV2)
108-
}, 5*time.Minute, 10*time.Second).Should(BeTrue())
108+
}, 10*time.Minute, 10*time.Second).Should(BeTrue())
109109

110110
// Manually delete old StatefulSet
111111
By("Manually deleting old StatefulSet during migration")
@@ -131,11 +131,12 @@ var _ = Describe(
131131
aeroCluster := createDummyClusterWithRackRevision(clusterNamespacedName, versionV1, 6)
132132
Expect(DeployCluster(k8sClient, ctx, aeroCluster)).ToNot(HaveOccurred())
133133

134+
By("Changing rack revision to trigger migration")
134135
updatedCluster := changeRackRevision(k8sClient, ctx, clusterNamespacedName)
135136

136137
Eventually(func() bool {
137138
return checkBothRevisionsExist(k8sClient, ctx, clusterNamespacedName, versionV1, versionV2)
138-
}, 5*time.Minute, 5*time.Second).Should(BeTrue())
139+
}, 10*time.Minute, 10*time.Second).Should(BeTrue())
139140

140141
updatedCluster.Spec.Size = 2
141142

@@ -344,7 +345,7 @@ func testRackRevisionChangeWithStorageUpdate(
344345
// Both rack revision StatefulSets should exist during migration
345346
Eventually(func() bool {
346347
return checkBothRevisionsExist(k8sClient, ctx, clusterNamespacedName, fromRevision, toRevision)
347-
}, 5*time.Minute, 10*time.Second).Should(BeTrue())
348+
}, 10*time.Minute, 10*time.Second).Should(BeTrue())
348349

349350
By("Waiting for the migration to complete")
350351

0 commit comments

Comments
 (0)