Improve parallelism of testing jobs
Stop triggering jobs in wave and use throttle concurrent
build plugin instead
Change-Id: If6eb216184da03cf918108d3e90f646bb41feed1
diff --git a/test-salt-formulas-pipeline.groovy b/test-salt-formulas-pipeline.groovy
index 4422ca6..1494aca 100644
--- a/test-salt-formulas-pipeline.groovy
+++ b/test-salt-formulas-pipeline.groovy
@@ -3,7 +3,6 @@
* DEFAULT_GIT_REF
* DEFAULT_GIT_URL
* CREDENTIALS_ID
- * KITCHEN_TESTS_PARALLEL
*/
def common = new com.mirantis.mk.Common()
def gerrit = new com.mirantis.mk.Gerrit()
@@ -16,13 +15,6 @@
gerritRef = null
}
-def parallelGroupSize
-try {
- parallelGroupSize = Integer.valueOf(PARALLEL_GROUP_SIZE)
-} catch (MissingPropertyException e) {
- parallelGroupSize = 8
-}
-
def defaultGitRef, defaultGitUrl
try {
defaultGitRef = DEFAULT_GIT_REF
@@ -92,15 +84,9 @@
common.infoMsg("CUSTOM_KITCHEN_ENVS not empty. Running with custom enviroments: ${kitchenEnvs}")
}
if (kitchenEnvs != null && kitchenEnvs != '') {
- def acc = 0
def kitchenTestRuns = [:]
common.infoMsg("Found " + kitchenEnvs.size() + " environment(s)")
for (int i = 0; i < kitchenEnvs.size(); i++) {
- if (acc >= parallelGroupSize) {
- parallel kitchenTestRuns
- kitchenTestRuns = [:]
- acc = 0
- }
def testEnv = kitchenEnvs[i]
kitchenTestRuns[testEnv] = {
build job: "test-salt-formulas-env", parameters: [
@@ -112,11 +98,8 @@
[$class: 'StringParameterValue', name: 'SALT_VERSION', value: SALT_VERSION]
]
}
- acc++;
}
- if (acc != 0) {
- parallel kitchenTestRuns
- }
+ parallel kitchenTestRuns
} else {
common.warningMsg(".kitchen.yml file not found, no kitchen tests triggered.")
}