Skip to content

Commit 5c7d373

Browse files
committed
Fix task ordering in rolling upgrade tests
The configuration of the upgraded cluster task was missing a dependency on the stopping of the second old node in the cluster. In some cases (e.g., --parallel) Gradle would then try to run the configuration of a node in the upgraded cluster before it had even configured the old nodes in the cluster. Relates #28036
1 parent 8b2659d commit 5c7d373

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

qa/rolling-upgrade/build.gradle

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,11 +70,10 @@ for (Version version : wireCompatVersions) {
7070
finalizedBy "${baseName}#oldClusterTestCluster#node0.stop"
7171
}
7272

73-
Task upgradedClusterTest = tasks.create(name: "${baseName}#upgradedClusterTest", type: RestIntegTestTask) {
74-
dependsOn(mixedClusterTestRunner, "${baseName}#oldClusterTestCluster#node0.stop")
75-
}
73+
Task upgradedClusterTest = tasks.create(name: "${baseName}#upgradedClusterTest", type: RestIntegTestTask)
7674

7775
configure(extensions.findByName("${baseName}#upgradedClusterTestCluster")) {
76+
dependsOn mixedClusterTestRunner, "${baseName}#oldClusterTestCluster#node0.stop"
7877
distribution = 'zip'
7978
clusterName = 'rolling-upgrade'
8079
unicastTransportUri = { seedNode, node, ant -> mixedClusterTest.nodes.get(0).transportUri() }

0 commit comments

Comments
 (0)