chore: allow to differenciate JDKs to build depending on Jenkins release line (#2178)

Author: lemeurherve

Jenkinsfile

@@ -113,6 +113,8 @@ stage('Build') {
         }
 
         if (!infra.isTrusted()) {
+            // This list can be updated with the following command:
+            // make show | jq -r '.target | keys[]' | sort
             def images = [
                 'alpine_jdk17',
                 'alpine_jdk21',

docker-bake.hcl

@@ -1,5 +1,9 @@
 ## Variables
-variable "jdks_to_build" {
+variable "jdks_to_build_for_lts" {
+  default = [17, 21]
+}
+
+variable "jdks_to_build_for_weekly" {
   default = [17, 21]
 }
 
@@ -94,7 +98,7 @@ variable "current_rhel" {
 ## Targets
 target "alpine" {
   matrix = {
-    jdk = jdks_to_build
+    jdk = jdks_to_build()
   }
   name       = "alpine_jdk${jdk}"
   dockerfile = "alpine/hotspot/Dockerfile"
@@ -114,7 +118,7 @@ target "alpine" {
 
 target "debian" {
   matrix = {
-    jdk     = jdks_to_build
+    jdk     = jdks_to_build()
     variant = debian_variants
   }
   name       = "${variant}_jdk${jdk}"
@@ -137,7 +141,7 @@ target "debian" {
 
 target "rhel" {
   matrix = {
-    jdk = jdks_to_build
+    jdk = jdks_to_build()
   }
   name       = "rhel_jdk${jdk}"
   dockerfile = "rhel/Dockerfile"
@@ -166,6 +170,21 @@ group "linux" {
 }
 
 ## Common functions
+# return true if JENKINS_VERSION is a Weekly (one sequence of digits with a trailing literal '.')
+function "is_jenkins_version_weekly" {
+  # If JENKINS_VERSION has more than one sequence of digits with a trailing literal '.', this is LTS
+  # 2.523 has only one sequence of digits with a trailing literal '.'
+  # 2.516.1 has two sequences of digits with a trailing literal '.'
+  params = []
+  result = length(regexall("[0-9]+[.]", JENKINS_VERSION)) < 2 ? true : false
+}
+
+# return the list of jdk to build depending on JENKINS_VERSION
+function "jdks_to_build" {
+  params = []
+  result = is_jenkins_version_weekly() ? jdks_to_build_for_weekly : jdks_to_build_for_lts
+}
+
 # return a tag prefixed by the Jenkins version
 function "_tag_jenkins_version" {
   params = [tag]
@@ -192,13 +211,10 @@ function "tag_lts" {
 
 # return WAR_URL if not empty, get.jenkins.io URL depending on JENKINS_VERSION release line otherwise
 function "war_url" {
-  # If JENKINS_VERSION has more than one sequence of digits with a trailing literal '.', this is LTS
-  # 2.523 has only one sequence of digits with a trailing literal '.'
-  # 2.516.1 has two sequences of digits with a trailing literal '.'
   params = []
   result = (notequal(WAR_URL, "")
     ? WAR_URL
-    : (length(regexall("[0-9]+[.]", JENKINS_VERSION)) < 2
+    : (is_jenkins_version_weekly()
       ? "https://get.jenkins.io/war/${JENKINS_VERSION}/jenkins.war"
   : "https://get.jenkins.io/war-stable/${JENKINS_VERSION}/jenkins.war"))
 }