Skip to content

Commit

Permalink
Reset numMachinesPerTest each loop
Browse files Browse the repository at this point in the history
Signed-off-by: Adam Brousseau <[email protected]>
  • Loading branch information
AdamBrousseau committed Oct 6, 2021
1 parent 925a668 commit fea3762
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pipelines/build/common/openjdk_build_pipeline.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,6 @@ class Build {
List dynamicList = buildConfig.DYNAMIC_LIST
List numMachines = buildConfig.NUM_MACHINES
def enableTestDynamicParallel = Boolean.valueOf(buildConfig.ENABLE_TESTDYNAMICPARALLEL)
def numMachinesPerTest = ''

testList.each { testType ->

Expand All @@ -334,10 +333,11 @@ class Build {

def jobParams = getAQATestJobParams(testType)
def parallel = 'None'
def numMachinesPerTest = ''

if (enableTestDynamicParallel && dynamicList.contains(testType)) {
numMachinesPerTest = numMachines.getAt(dynamicList.indexOf(testType))
if (!numMachinesPerTest) {
if (numMachinesPerTest) {
// see build configuration in jdk*_pipeline_config.groovy
// when numMachines is an array, its size should match the testLists size
throw new Exception("No number of machines provided for running ${testType} tests in parallel, numMachines: ${numMachines}!")
Expand Down

0 comments on commit fea3762

Please sign in to comment.