Ensure evaluation pipeline config load resets groovy binding before t…

…he load (#790) * Ensure evaluation pipeline config loaded correctly Signed-off-by: Andrew Leonard <[email protected]> * Ensure evaluation pipeline config loaded correctly Signed-off-by: Andrew Leonard <[email protected]> * Ensure evaluation pipeline config loaded correctly Signed-off-by: Andrew Leonard <[email protected]> --------- Signed-off-by: Andrew Leonard <[email protected]>
adoptium · Aug 24, 2023 · 1ded474 · 1ded474
1 parent c103819
commit 1ded474
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 7 deletions.
diff --git a/pipelines/build/regeneration/evaluation_pipeline_generator.groovy b/pipelines/build/regeneration/evaluation_pipeline_generator.groovy
@@ -198,11 +198,16 @@ node('worker') {
                     }
                     config.put('targetConfigurations', targetEvaluation.targetConfigurations)
 
+                    println "[INFO] JDK${javaVersion}: loaded targetEvaluation configuration:"
+                    println JsonOutput.prettyPrint(JsonOutput.toJson(targetEvaluation))
+
                     // if has a triggerSchedule_evaluation variable set then use it or default to '0 0 31 2 0'/never run
                     if (enablePipelineSchedule.toBoolean()){
-                        config.put('pipelineSchedule', targetEvaluation.triggerSchedule_evaluation)
-                    } else { // empty to not run
-                        config.put('pipelineSchedule', '')
+                        try {
+                            config.put('pipelineSchedule', targetEvaluation.triggerSchedule_evaluation)
+                        } catch (Exception ex) {
+                            config.put('pipelineSchedule', '0 0 31 2 0')
+                        }
                     }
 
                     // hack as jenkins groovy does not seem to allow us to check if disableJob exists
@@ -248,9 +253,11 @@ node('worker') {
                     }
                     config.PIPELINE = "evaluation-openjdk${javaVersion}-pipeline"
                     if (enablePipelineSchedule.toBoolean()) {
-                        config.put('pipelineSchedule', targetEvaluation.triggerSchedule_weekly_evaluation)
-                    } else { // empty string will never run
-                        config.put('pipelineSchedule', '')
+                        try {
+                            config.put('pipelineSchedule', targetEvaluation.triggerSchedule_weekly_evaluation)
+                        } catch (Exception ex) {
+                            config.put('pipelineSchedule', '0 0 31 2 0')
+                        }
                     }
                     config.put('targetConfigurations', targetEvaluation.targetConfigurations) // explicit set it to make things clear
                     config.weekly_release_scmReferences = targetEvaluation.weekly_evaluation_scmReferences
@@ -278,6 +285,13 @@ node('worker') {
                     }
                     // add into list
                     generatedPipelines.add(config['JOB_NAME'])
+
+                    // config.load() loads into the current groovy binding, and returns "this", so we need to reset variables before next load of targetEvaluation
+                    targetEvaluation.targetConfigurations = {} 
+                    targetEvaluation.triggerSchedule_evaluation = '0 0 31 2 0'
+                    targetEvaluation.triggerSchedule_weekly_evaluation = '0 0 31 2 0'
+                    targetEvaluation.weekly_evaluation_scmReferences = {} 
+                    targetEvaluation.disableJob = false
                 }
             })
 

diff --git a/pipelines/jobs/configurations/jdk17u_evaluation.groovy b/pipelines/jobs/configurations/jdk17u_evaluation.groovy
@@ -31,7 +31,7 @@ targetConfigurations = [
 ]
 
 // if set to empty string then it wont get triggered
-triggerSchedule_pevaluation = 'TZ=UTC\n30 23 * * 2,4'
+triggerSchedule_evaluation = 'TZ=UTC\n30 23 * * 2,4'
 // if set to empty string then it wont get triggered
 triggerSchedule_weekly_evaluation = 'TZ=UTC\n05 12 * * 7'