diff --git a/resources/templates/minion-multibranch-pipeline-default.vm b/resources/templates/minion-multibranch-pipeline-default.vm index e55bc1d..1124220 100644 --- a/resources/templates/minion-multibranch-pipeline-default.vm +++ b/resources/templates/minion-multibranch-pipeline-default.vm @@ -1,11 +1,12 @@ -#foreach ( $library in $libraries ) -library(identifier: '$library', changelog: false) -#end +<% +libraries.each { library -> + println "library(identifier: '$library', changelog: false)" +} -#set( $runCheckout = $properties.getBool('RUN_STAGE_CHECKOUT') ) -#set( $runBuild = $properties.getBool('RUN_STAGE_BUILD') ) -#set( $runSonar = $properties.getBool('RUN_STAGE_SONAR') && $item.auditable ) -## +runCheckout = properties.getBool('RUN_STAGE_CHECKOUT') +runBuild = properties.getBool('RUN_STAGE_BUILD') +runSonar = properties.getBool('RUN_STAGE_SONAR') && item.auditable +%> Map defaultParams = [ SLAVE_NODE_LABEL : '$properties.getString('SLAVE_NODE_LABEL')', WORKSPACE : '$properties.getString('WORKSPACE')', diff --git a/resources/templates/minion-pipeline-default.vm b/resources/templates/minion-pipeline-default.vm index 0cef53b..aa778c6 100644 --- a/resources/templates/minion-pipeline-default.vm +++ b/resources/templates/minion-pipeline-default.vm @@ -1,17 +1,18 @@ -#foreach ( $library in $libraries ) -library(identifier: '$library', changelog: false) -#end +<% +libraries.each { library -> + println "library(identifier: '$library', changelog: false)" +} -#set( $runCheckout = $properties.getBool('RUN_STAGE_CHECKOUT') ) -#set( $runBuild = $properties.getBool('RUN_STAGE_BUILD') ) -#set( $runTest = $properties.getBool('RUN_STAGE_TEST') && $item.testable ) -#set( $runArchive = $properties.getBool('RUN_STAGE_ARCHIVE') && $item.archivable ) -## +runCheckout = properties.getBool('RUN_STAGE_CHECKOUT') +runBuild = properties.getBool('RUN_STAGE_BUILD') +runTest = properties.getBool('RUN_STAGE_TEST') && item.testable +runArchive = properties.getBool('RUN_STAGE_ARCHIVE') && item.archivable +%> Map defaultParams = [ - SLAVE_NODE_LABEL : '$properties.getString('SLAVE_NODE_LABEL')', - WORKSPACE : '$properties.getString('WORKSPACE')', - BUILD_DATA_ROOT_PATH: '$properties.getString('BUILD_DATA_ROOT_PATH')', - BUILD_DATA_FILE : '$properties.getString('BUILD_DATA_FILE')', + SLAVE_NODE_LABEL : '$properties.SLAVE_NODE_LABEL', + WORKSPACE : '$properties.WORKSPACE', + BUILD_DATA_ROOT_PATH: '$properties.BUILD_DATA_ROOT_PATH', + BUILD_DATA_FILE : '$properties.BUILD_DATA_FILE', RUN_STAGE_CHECKOUT : $runCheckout, RUN_STAGE_BUILD : $runBuild, diff --git a/src/org/hitachivantara/ci/jenkins/MinionHandler.groovy b/src/org/hitachivantara/ci/jenkins/MinionHandler.groovy index b745f72..bc6827e 100644 --- a/src/org/hitachivantara/ci/jenkins/MinionHandler.groovy +++ b/src/org/hitachivantara/ci/jenkins/MinionHandler.groovy @@ -12,6 +12,8 @@ import org.hitachivantara.ci.config.BuildData import org.hitachivantara.ci.config.ConfigurationException import org.hitachivantara.ci.config.ConfigurationMap import org.jenkinsci.plugins.workflow.cps.CpsScript +import groovy.text.SimpleTemplateEngine +import groovy.text.TemplateEngine import java.nio.file.Paths @@ -89,11 +91,11 @@ class MinionHandler { String rootFolderPath = getRootFolderPath() // use the defined template or the default ones - Map templateSource + String templateSource if(buildData.isSet(MINION_PIPELINE_TEMPLATE)){ - templateSource = [file: buildData.getString(MINION_PIPELINE_TEMPLATE)] + templateSource = steps.readFile(file: buildData.getString(MINION_PIPELINE_TEMPLATE), encoding: 'UTF-8') } else { - templateSource = [text: getDefaultPipelineTemplate()] + templateSource = getDefaultPipelineTemplate() } // add all the current build libraries @@ -101,22 +103,23 @@ class MinionHandler { workJobItems.each { JobItem item -> steps.log.debug "Creating minion job for ${item.jobID}" - - Map extraProps = [(BUILD_DATA_ROOT_PATH): getBuildDataPath(), - (BUILD_DATA_FILE) : getBuildDataFilename(item)] - - String script = steps.resolveTemplate(templateSource + [ - parameters: [ - libraries : libraries, - properties : buildData.buildProperties + extraProps, - item : item.export() - ] - ]) + + Map parameters = [ + libraries : libraries, + properties: new ConfigurationMap(buildData.buildProperties, [ + (BUILD_DATA_ROOT_PATH): getBuildDataPath(), + (BUILD_DATA_FILE) : getBuildDataFilename(item), + ]), + item: item.export() + ] + + TemplateEngine engine = new SimpleTemplateEngine() + Writable template = engine.createTemplate(templateSource).make(parameters) Map jobConfig = [ name: getJobName(item), folder: rootFolderPath, - script: script + script: template.toString() ] if (buildData.getBool(USE_MINION_MULTIBRANCH_JOBS)) { @@ -135,8 +138,8 @@ class MinionHandler { prMerge : item.prMerge ] ] - } - + } + steps.createJob(jobConfig) }