[I/Y-Build] Streamline environment variable computation and export

Avoid need for common-functions.shsource and buildproperties.shsource
Replace createEnvfiles script
Remove quotes around values in buildproperties files.

Author: HannesWell

JenkinsJobs/Builds/build.jenkinsfile

@@ -1,5 +1,7 @@
 @groovy.transform.Field
 def BUILD = null
+@groovy.transform.Field
+def BUILD_PROPERTIES = null
 
 pipeline {
 	options {
@@ -36,11 +38,13 @@ pipeline {
 		CJE_ROOT = "${WORKSPACE}/cje-production"
 		SCRIPTS = "${WORKSPACE}/scripts"
 		AGG_DIR = "${CJE_ROOT}/gitCache/eclipse.platform.releng.aggregator"
+		TMP_DIR = 'tmp'
 		logDir = "$CJE_ROOT/buildlogs"
 
 		PRODUCTS_DIR = "${AGG_DIR}/products"
 		SITES_DIR = "${AGG_DIR}/sites"
 		PLATFORM_REPO_DIR = "${SITES_DIR}/eclipse-platform-repository/target/repository"
+		ECLIPSE_BUILDER_DIR = "${AGG_DIR}/eclipse.platform.releng.tychoeclipsebuilder"
 
 		DROP_DIR = "${CJE_ROOT}/siteDir/eclipse/downloads/drops4"
 		EQUINOX_DROP_DIR = "${CJE_ROOT}/siteDir/equinox/drops"
@@ -58,29 +62,24 @@ pipeline {
 
 					def job = utilities.matchPattern('Job name', "${JOB_BASE_NAME}", [ /(?<type>[IY])-build-(?<major>\d).(?<minor>\d+)/ ])
 					// Load build properties and configuration from the properties and configuration file
-					assignEnvVariable('BUILD_TYPE', job.type)
-					def configurations = readJSON(file: "${WORKSPACE}/JenkinsJobs/buildConfigurations.json")
-					BUILD = configurations["${BUILD_TYPE}"]
-					assignEnvVariable('BUILD_TYPE_NAME', BUILD.typeName)
+					def int epochSeconds = currentBuild.startTimeInMillis.intdiv(1000)
+					def TIMESTAMP = sh(script: """TZ="America/New_York" date +%Y%m%d-%H%M --date='@${epochSeconds}'""", returnStdout: true)
+					BUILD = readJSON(file: 'JenkinsJobs/buildConfigurations.json')["${job.type}"]
+					BUILD_PROPERTIES = [
+						TIMESTAMP: TIMESTAMP,
+						BUILD_TYPE: job.type,
+						BUILD_TYPE_NAME: BUILD.typeName,
+						BUILD_ID: "${job.type}${TIMESTAMP}",
+					]
+					BUILD_PROPERTIES += readProperties(file: "${CJE_ROOT}/buildproperties.txt")
+					BUILD_PROPERTIES.TEST_CONFIGURATIONS_EXPECTED = BUILD.tests.collect{ c ->
+						"${testJobName(c)}_${c.os}.${c.ws}.${c.arch}_${c.javaVersion}"
+					}.join(',')
+					
+					// Export build properties
+					BUILD_PROPERTIES.each{ key, value -> assignEnvVariable(key, value) }
 					assignEnvVariable('GIT_SUBMODULE_BRANCHES', BUILD.branches ? BUILD.branches.collect{ name, branch -> "${name}:${branch}" }.join(',') : '')
-					assignEnvVariable('TEST_NAME_PREFIX', "ep${job.major}${job.minor}${BUILD_TYPE}-unit")
-					assignEnvVariable('TEST_CONFIGURATIONS_EXPECTED', BUILD.tests.collect{ c ->
-						"${TEST_NAME_PREFIX}-${c.os}-${c.arch}-java${c.javaVersion}_${c.os}.${c.ws}.${c.arch}_${c.javaVersion}"
-					}.join(','))
 				}
-				dir("${CJE_ROOT}") {
-					sh '''#!/bin/bash -xe
-						chmod +x mbscripts/*
-						chmod +x $CJE_ROOT/scripts/notarizeMacApp.sh
-						
-						./mbscripts/mb010_createEnvfiles.sh ${CJE_ROOT}/buildproperties.shsource 2>&1
-					'''
-				}
-				exportPropertiesAsEnvironmentVariables("${CJE_ROOT}/buildproperties.properties", [
-					'BUILD_ID',
-					'STREAM', 'RELEASE_VER',
-					'PREVIOUS_RELEASE_ID', 'PREVIOUS_RELEASE_VER',
-				])
 			}
 		}
 		stage('Clone and tag Build Inputs') {
@@ -118,7 +117,6 @@ pipeline {
 						// Try to find the last tag of the current build type that is available as a promoted build
 						// by checking the most recent 5 tags and seeing if an update site for it exists
 						sh '''#!/bin/bash -xe
-							source $CJE_ROOT/buildproperties.shsource
 							updateSiteRootPath=${EP_ECLIPSE_UPDATES}/${STREAMMajor}.${STREAMMinor}-${BUILD_TYPE}-builds
 							lastTagList=$(git tag --list "${BUILD_TYPE}*" | tail -n5 | tac)
 							for lt in ${lastTagList}; do
@@ -148,8 +146,6 @@ pipeline {
 					// Create git tags and push changes
 					sshagent (['github-bot-ssh']) {
 						sh '''#!/bin/bash -xe
-							source $CJE_ROOT/buildproperties.shsource
-							
 							function remoteURL {
 								git config remote.origin.url | sed --expression 's,https://github.com/,git@github.com:,'
 							}
@@ -181,8 +177,8 @@ pipeline {
 					def exe = utilities.installDownloadableTool('eclipse', eclipsePlatformLatestRelease) + '/eclipse'
 					assignEnvVariable('BASE_BUILDER_ECLIPSE_EXE', "${exe} -nosplash --launcher.suppressErrors")
 					sh '''
-						source $CJE_ROOT/buildproperties.shsource
 						mkdir -p $CJE_ROOT/$TMP_DIR
+						#TODO: How to treat old releases of p2-analyzers on maintanance branches?
 						$BASE_BUILDER_ECLIPSE_EXE \
 							-debug -consolelog -data $CJE_ROOT/$TMP_DIR/workspace-toolsinstall \
 							-application org.eclipse.equinox.p2.director \
@@ -202,8 +198,6 @@ pipeline {
 			steps {
 				dir("${AGG_DIR}") {
 					sh '''#!/bin/bash -xe
-						source $CJE_ROOT/buildproperties.shsource
-						
 						mvn clean install -pl :eclipse-sdk-prereqs,:org.eclipse.jdt.core.compiler.batch \
 							-Dmaven.repo.local=${LOCAL_REPO} -DlocalEcjVersion=99.99 \
 							-DcompilerBaselineMode=disable -DcompilerBaselineReplace=none
@@ -223,6 +217,13 @@ pipeline {
 							2>&1 | tee ${logDir}/mb220_buildSdkPatch.sh.log
 							# Capture the build log for the subsequent scan for comparator errors
 					'''
+					script {
+						def mavenProperties = readProperties(file: 'eclipse-platform-parent/target/mavenproperties.properties')
+						mavenProperties.each{ key, value -> assignEnvVariable(key, value) }
+						BUILD_PROPERTIES += mavenProperties
+						// Store complete build-properties
+						writeFile(file: "${CJE_ROOT}/buildproperties.properties", text: BUILD_PROPERTIES.collect{ key, value -> "${key}=${value}"}.join('\n'))
+					}
 				}
 			}
 		}
@@ -232,13 +233,8 @@ pipeline {
 			}
 			steps {
 				sh '''#!/bin/bash -xe
-					source $CJE_ROOT/scripts/common-functions.shsource
-					source $CJE_ROOT/buildproperties.shsource
-					
 					# Gather maven properties
 					mkdir -p ${DROP_DIR}/$BUILD_ID
-					cp ${AGG_DIR}/eclipse-platform-parent/target/mavenproperties.properties ${DROP_DIR}/$BUILD_ID/mavenproperties.properties
-					comparatorRepo=$(grep '^comparator.repo=' ${DROP_DIR}/${BUILD_ID}/mavenproperties.properties | cut -d'=' -f2-)
 
 					# Gather artifactcomparisons
 					pushd ${AGG_DIR}
@@ -251,9 +247,10 @@ pipeline {
 					#TODO: Generally try to avoid the need to capture the build log in a file/for teeing
 					cp ${logDir}/mb220_buildSdkPatch.sh.log ${DROP_DIR}/$BUILD_ID/buildlogs/
 
+					#TODO: when outputting comparator error logs, add the baseline repository in the metadata (even on non-failure?)
 					java \
 						-DbuildDirectory=${DROP_DIR}/$BUILD_ID \
-						-DcomparatorRepo=${comparatorRepo} \
+						-DcomparatorRepo=${COMPARATOR_BASELINE} \
 						-Djava.io.tmpdir=$CJE_ROOT/$TMP_DIR \
 						${WORKSPACE}/scripts/releng/ComparatorSummaryExtractor.java
 
@@ -262,13 +259,15 @@ pipeline {
 					logSize=$(stat --format='%s' ${comparatorLog})
 					if [[ ${logSize} -gt ${comparatorLogMinimumSize} ]]; then
 						echo -e "DEBUG: found logsize greater an minimum. preparing message using ${link}"
-						fn-write-property COMPARATOR_ERRORS_SUBJECT "\\"- Comparator Errors Found\\""
+						echo "- Comparator Errors Found" > comparatorErrorsSubject
 					else
 						echo -e "DEBUG: comparator logSize of $logSize was not greater than comparatorLogMinimumSize of ${comparatorLogMinimumSize}"
-						fn-write-property COMPARATOR_ERRORS_SUBJECT "\\"\\""
+						touch comparatorErrorsSubject
 					fi
 				'''
-				exportPropertiesAsEnvironmentVariables("${CJE_ROOT}/buildproperties.properties", [ 'COMPARATOR_ERRORS_SUBJECT' ])
+				script { // Export subject to signal comparator errors, if any
+					assignEnvVariable('COMPARATOR_ERRORS_SUBJECT', readFile("${WORKSPACE}/comparatorErrorsSubject").trim())
+				}
 			}
 		}
 		stage('Promote results') {
@@ -287,7 +286,6 @@ pipeline {
 								}
 								sh '''#!/bin/bash -xe
 									cp $CJE_ROOT/buildproperties.* ${DROP_DIR}/${BUILD_ID}
-									source $CJE_ROOT/buildproperties.shsource
 
 									mkdir -p ${DROP_DIR}/${BUILD_ID}/testresults/consolelogs
 
@@ -323,6 +321,7 @@ pipeline {
 
 									notarizeLogDir=$CJE_ROOT/notarizeLog
 									mkdir -p $notarizeLogDir
+									chmod +x ${CJE_ROOT}/scripts/notarizeMacApp.sh
 									pushd ${DROP_DIR}/${BUILD_ID}
 									for macProduct in *.dmg; do
 										(/bin/bash ${CJE_ROOT}/scripts/notarizeMacApp.sh ${macProduct} > $notarizeLogDir/${macProduct}.log 2>&1)&
@@ -388,7 +387,7 @@ pipeline {
 										-DbuildId=$BUILD_ID \
 										-DbuildLabel=$BUILD_ID \
 										-DbuildWorkingArea=${AGG_DIR} \
-										-DpreviousBaseURL=https://$DOWNLOAD_HOST/eclipse/downloads/drops4/$PREVIOUS_RELEASE_ID/eclipse-SDK-$PREVIOUS_RELEASE_VER-win32-x86_64.zip \
+										-DpreviousBaseURL=https://download.eclipse.org/eclipse/downloads/drops4/$PREVIOUS_RELEASE_ID/eclipse-SDK-$PREVIOUS_RELEASE_VER-win32-x86_64.zip \
 										-DpreviousBaselineName=Eclipse-SDK-$PREVIOUS_RELEASE_VER \
 										-DpreviousBaselineFilename=eclipse-SDK-$PREVIOUS_RELEASE_VER-win32-x86_64.zip \
 										-Djava.io.tmpdir=$CJE_ROOT/$TMP_DIR \
@@ -449,7 +448,6 @@ pipeline {
 								}
 								sh '''#!/bin/bash -xe
 									cp $CJE_ROOT/buildproperties.* ${EQUINOX_DROP_DIR}/${BUILD_ID}
-									source $CJE_ROOT/buildproperties.shsource
 
 									# Gather Equinox Starter Kit
 									for file in ${PRODUCTS_DIR}/equinox-starterkit/target/products/*; do
@@ -460,6 +458,7 @@ pipeline {
 
 									notarizeLogDir=$CJE_ROOT/notarizeEqLog
 									mkdir -p $notarizeLogDir
+									chmod +x ${CJE_ROOT}/scripts/notarizeMacApp.sh
 									pushd ${EQUINOX_DROP_DIR}/${BUILD_ID}
 									(/bin/bash ${CJE_ROOT}/scripts/notarizeMacApp.sh EclipseRT-OSGi-StarterKit-$BUILD_ID-macosx-cocoa-x86_64.dmg > $notarizeLogDir/equinoxX64.log 2>&1)&
 									popd
@@ -522,7 +521,6 @@ pipeline {
 					steps {
 						sshagent(['projects-storage.eclipse.org-bot-ssh']) {
 							sh '''#!/bin/bash -xe
-								source $CJE_ROOT/buildproperties.shsource
 								repoPath=${EP_ECLIPSE_UPDATES}/${STREAMMajor}.${STREAMMinor}-${BUILD_TYPE}-builds/${BUILD_ID}
 								ssh genie.releng@projects-storage.eclipse.org mkdir -p ${repoPath}
 								scp -r ${PLATFORM_REPO_DIR}/. genie.releng@projects-storage.eclipse.org:${repoPath}/.
@@ -580,7 +578,7 @@ pipeline {
 			steps {
 				script {
 					for (c in BUILD.tests) {
-						build job: "${BUILD.testsFolder}/${TEST_NAME_PREFIX}-${c.os}-${c.arch}-java${c.javaVersion}", wait: false, parameters: [
+						build job: "${BUILD.testsFolder}/${testJobName(c)}", wait: false, parameters: [
 							string(name: 'buildId', value: "${BUILD_ID}"),
 							string(name: 'testAgent', value: "${c.agent}"),
 						]
@@ -650,10 +648,7 @@ def assignEnvVariable(String name, String value) {
 	utilities.assignEnvVariable(name, value)
 }
 
-def exportPropertiesAsEnvironmentVariables(String propertiesFile, Collection<String> exportedVariables){
-	def properties = readProperties(file: propertiesFile, charset: 'UTF-8')
-	for (envVar in exportedVariables) {
-		def value = properties[envVar].trim()
-		assignEnvVariable(envVar, value.startsWith('"') && value.endsWith('"') ? value.substring(1, value.length() - 1) : value)
-	}
+@NonCPS
+def testJobName(Map<String, String> c) {
+	return "ep${BUILD_PROPERTIES.STREAMMajor}${BUILD_PROPERTIES.STREAMMinor}${BUILD_PROPERTIES.BUILD_TYPE}-unit-${c.os}-${c.arch}-java${c.javaVersion}"
 }

JenkinsJobs/Builds/markBuild.jenkinsfile

@@ -77,5 +77,5 @@ pipeline {
 def readBuildProperty(String name) {
 	def buildPropertiesURL = "https://download.eclipse.org/eclipse/downloads/drops4/${buildId}/buildproperties.properties"
 	def buildProperties = readProperties(text: sh(script: "curl --fail ${buildPropertiesURL}", returnStdout: true))
-	return buildProperties[name].replace('"','') // Remove surrounding quotes
+	return buildProperties[name]
 }

JenkinsJobs/Cleanup/cleanupBuilds.jenkinsfile

@@ -43,9 +43,9 @@ pipeline {
 					utilities = load "JenkinsJobs/shared/utilities.groovy"
 					utilities.setDryRun(false)
 					def buildProperties = readProperties(file: 'cje-production/buildproperties.txt')
-					devVersionMajor = buildProperties.STREAMMajor.replace('"','').toInteger() // Remove surrounding quotes
-					devVersionMinor = buildProperties.STREAMMinor.replace('"','').toInteger() // Remove surrounding quotes
-					devVersionService = buildProperties.STREAMService.replace('"','').toInteger() // Remove surrounding quotes
+					devVersionMajor = buildProperties.STREAMMajor.toInteger()
+					devVersionMinor = buildProperties.STREAMMinor.toInteger()
+					devVersionService = buildProperties.STREAMService.toInteger()
 				}
 			}
 		}

JenkinsJobs/Releng/prepareNextDevCycle.jenkinsfile

@@ -47,13 +47,12 @@ pipeline {
 					assignEnvVariable('NEXT_RELEASE_VERSION_MINOR', nextVersion.minor)
 
 					echo "PREVIOUS_RELEASE_CANDIDATE_ID: ${PREVIOUS_RELEASE_CANDIDATE_ID}"
-					def buildPropertiesTxt = sh(script: "curl --fail https://download.eclipse.org/eclipse/downloads/drops4/${PREVIOUS_RELEASE_CANDIDATE_ID}/buildproperties.txt", returnStdout: true)
-					def buildProperties = readProperties(text: buildPropertiesTxt)
-					assignEnvVariable('IS_JAVA_RELEASE_IMMINENT', !buildProperties.NEXT_JAVA_RELEASE_DATE.replace('"','').isEmpty()) // Remove surrounding quotes
+					def buildProperties = utilities.loadBuildProperties("${PREVIOUS_RELEASE_CANDIDATE_ID}")
+					assignEnvVariable('IS_JAVA_RELEASE_IMMINENT', !buildProperties.NEXT_JAVA_RELEASE_DATE.trim().isEmpty()) // Remove surrounding quotes
 
 					def previousRC = utilities.matchBuildIdentifier("${PREVIOUS_RELEASE_CANDIDATE_ID}", { iBuild ->
-						iBuild.major = buildProperties.STREAMMajor.replace('"','') // Remove surrounding quotes
-						iBuild.minor = buildProperties.STREAMMinor.replace('"','') // Remove surrounding quotes
+						iBuild.major = buildProperties.STREAMMajor
+						iBuild.minor = buildProperties.STREAMMinor
 						assignEnvVariable('PREVIOUS_RELEASE_CANDIDATE_TAG', "${PREVIOUS_RELEASE_CANDIDATE_ID}")
 						assignEnvVariable('PREVIOUS_RELEASE_CANDIDATE_GIT_TAG', "${PREVIOUS_RELEASE_CANDIDATE_ID}")
 					}, { sBuild ->
@@ -153,12 +152,12 @@ pipeline {
 		stage('Update build scripts') {
 			steps {
 				replaceAllInFile('cje-production/buildproperties.txt', [
-					"RELEASE_VER=\".*\"" : "RELEASE_VER=\"${NEXT_RELEASE_VERSION}\"",
-					"STREAM=\".*\"" : "STREAM=\"${NEXT_RELEASE_VERSION}.0\"",
-					"STREAMMajor=\".*\"" : "STREAMMajor=\"${NEXT_RELEASE_VERSION_MAJOR}\"",
-					"STREAMMinor=\".*\"" : "STREAMMinor=\"${NEXT_RELEASE_VERSION_MINOR}\"",
-					"ECLIPSE_RUN_REPO=\".*\"" : "ECLIPSE_RUN_REPO=\"https://download.eclipse.org/eclipse/updates/${NEXT_RELEASE_VERSION}-I-builds/\"",
-					"NEXT_JAVA_RELEASE_DATE=\".*\"" : "NEXT_JAVA_RELEASE_DATE=\"${NEXT_JAVA_RELEASE_DATE}\"",
+					'RELEASE_VER=.*' : "RELEASE_VER=${NEXT_RELEASE_VERSION}",
+					'STREAM=.*' : "STREAM=${NEXT_RELEASE_VERSION}.0",
+					'STREAMMajor=.*' : "STREAMMajor=${NEXT_RELEASE_VERSION_MAJOR}",
+					'STREAMMinor=.*' : "STREAMMinor=${NEXT_RELEASE_VERSION_MINOR}",
+					'ECLIPSE_RUN_REPO=.*' : "ECLIPSE_RUN_REPO=https://download.eclipse.org/eclipse/updates/${NEXT_RELEASE_VERSION}-I-builds/",
+					'NEXT_JAVA_RELEASE_DATE=.*' : "NEXT_JAVA_RELEASE_DATE=${NEXT_JAVA_RELEASE_DATE}",
 				])
 				replaceInFile('products/eclipse-platform/platform.p2.inf', [
 					"${PREVIOUS_RELEASE_VERSION} Release" : "${NEXT_RELEASE_VERSION} Release",
@@ -197,10 +196,10 @@ pipeline {
 						"-DnewPrevious-release.baseline=https://download.eclipse.org/eclipse/updates/${PREVIOUS_RELEASE_VERSION}-I-builds/${PREVIOUS_RELEASE_CANDIDATE_I_BUILD}/"
 				'''
 				replaceAllInFile('cje-production/buildproperties.txt', [
-					'PREVIOUS_RELEASE_VER=".*"' : "PREVIOUS_RELEASE_VER=\"${PREVIOUS_RELEASE_CANDIDATE_TAG}\"",
-					'PREVIOUS_RELEASE_REPO_ID=".*"' : "PREVIOUS_RELEASE_REPO_ID=\"${PREVIOUS_RELEASE_VERSION}-I-builds\"",
-					'BASEBUILD_ID=".*"' : "BASEBUILD_ID=\"${PREVIOUS_RELEASE_CANDIDATE_I_BUILD}\"",
-					'PREVIOUS_RELEASE_ID=".*"' : "PREVIOUS_RELEASE_ID=\"${PREVIOUS_RELEASE_CANDIDATE_ID}\"",
+					'PREVIOUS_RELEASE_VER=.*' : "PREVIOUS_RELEASE_VER=${PREVIOUS_RELEASE_CANDIDATE_TAG}",
+					'PREVIOUS_RELEASE_REPO_ID=.*' : "PREVIOUS_RELEASE_REPO_ID=${PREVIOUS_RELEASE_VERSION}-I-builds",
+					'BASEBUILD_ID=.*' : "BASEBUILD_ID=${PREVIOUS_RELEASE_CANDIDATE_I_BUILD}",
+					'PREVIOUS_RELEASE_ID=.*' : "PREVIOUS_RELEASE_ID=${PREVIOUS_RELEASE_CANDIDATE_ID}",
 				])
 
 				gitCommitAllExcludingSubmodules("Move previous version to ${PREVIOUS_RELEASE_CANDIDATE_TAG} in build scripts")
@@ -431,7 +430,7 @@ def githubAPI = null
 
 @NonCPS
 def assignEnvVariable(String name, Object value) {
-	utilities.assignEnvVariable(name, value)
+	utilities.assignEnvVariable(name, value?.toString())
 }
 
 @NonCPS

JenkinsJobs/Releng/promoteBuild.jenkinsfile

@@ -76,11 +76,10 @@ pipeline {
 					})
 					assignEnvVariable('REPO_ID', "I${build.date}-${build.time}")
 
-					def buildPropertiesTxt = sh(script: "curl --fail https://download.eclipse.org/eclipse/downloads/drops4/${DROP_ID}/buildproperties.txt", returnStdout: true)
-					def buildProperties = readProperties(text: buildPropertiesTxt)
-					assignEnvVariable('BUILD_MAJOR', buildProperties.STREAMMajor.replace('"','')) // Remove surrounding quotes
-					assignEnvVariable('BUILD_MINOR', buildProperties.STREAMMinor.replace('"','')) // Remove surrounding quotes
-					assignEnvVariable('BUILD_SERVICE', buildProperties.STREAMService.replace('"','')) // Remove surrounding quotes
+					def buildProperties = utilities.loadBuildProperties("${DROP_ID}")
+					assignEnvVariable('BUILD_MAJOR', buildProperties.STREAMMajor)
+					assignEnvVariable('BUILD_MINOR', buildProperties.STREAMMinor)
+					assignEnvVariable('BUILD_SERVICE', buildProperties.STREAMService)
 
 					if ("${CHECKPOINT}" ==~ /M\d+([a-z])?/ || "${CHECKPOINT}" ==~ /RC\d+([a-z])?/) { // milestone or RC promotion
 						assignEnvVariable('DL_TYPE', 'S')
@@ -231,10 +230,10 @@ pipeline {
 								-DnewPrevious-release.baseline="${RELEASE_P2_REPOSITORY}"
 						'''
 						utilities.replaceAllInFile('cje-production/buildproperties.txt', [
-							'PREVIOUS_RELEASE_VER=".*?"' : "PREVIOUS_RELEASE_VER=\"${BUILD_MAJOR}.${BUILD_MINOR}\"",
-							'PREVIOUS_RELEASE_REPO_ID=".*?"' : "PREVIOUS_RELEASE_REPO_ID=\"${BUILD_MAJOR}.${BUILD_MINOR}\"",
-							'BASEBUILD_ID=".*?"' : "BASEBUILD_ID=\"${DL_DROP_ID}\"",
-							'PREVIOUS_RELEASE_ID=".*?"' : "PREVIOUS_RELEASE_ID=\"${DL_DROP_ID}\"",
+							'PREVIOUS_RELEASE_VER=.*' : "PREVIOUS_RELEASE_VER=${BUILD_MAJOR}.${BUILD_MINOR}",
+							'PREVIOUS_RELEASE_REPO_ID=.*' : "PREVIOUS_RELEASE_REPO_ID=${BUILD_MAJOR}.${BUILD_MINOR}",
+							'BASEBUILD_ID=.*' : "BASEBUILD_ID=${DL_DROP_ID}",
+							'PREVIOUS_RELEASE_ID=.*' : "PREVIOUS_RELEASE_ID=${DL_DROP_ID}",
 						])
 						utilities.modifyJSON('JenkinsJobs/buildConfigurations.json') { builds ->
 							// Remove old I/Y-build jobs for this release
@@ -256,7 +255,7 @@ pipeline {
 								-DnewCbi-ecj-version="${ecjVersion}"
 						"""
 						utilities.replaceAllInFile('cje-production/buildproperties.txt', [
-							'ECLIPSE_RUN_REPO="https://download.eclipse.org/eclipse/updates/.*"' : "ECLIPSE_RUN_REPO=\"${RELEASE_P2_REPOSITORY}\"",
+							'ECLIPSE_RUN_REPO=https://download.eclipse.org/eclipse/updates/.*' : "ECLIPSE_RUN_REPO=${RELEASE_P2_REPOSITORY}",
 						])
 						utilities.gitCommitAllExcludingSubmodules("Update ${MAINTENANCE_BRANCH} branch with release version for ${BUILD_MAJOR}_${BUILD_MINOR}+ changes")
 					}

JenkinsJobs/Releng/publishPromotedBuild.jenkinsfile

@@ -57,14 +57,8 @@ pipeline {
 					} else {
 						error "RELEASE_TYPE, ${RELEASE_TYPE}, did not match any expected pattern."
 					}
-					
-					def eclipseParentPOM = sh(script: "curl --fail https://raw.githubusercontent.com/eclipse-platform/eclipse.platform.releng.aggregator/refs/tags/${buildId}/eclipse-platform-parent/pom.xml", returnStdout: true)
-					def yearMatcher = eclipseParentPOM =~ /<releaseYear>(?<year>\d+)<\/releaseYear>/
-					def monthMatcher = eclipseParentPOM =~ /<releaseMonth>(?<month>\d+)<\/releaseMonth>/
-					if (!yearMatcher.find() || !monthMatcher.find()) {
-						error "Eclipse Parent POM does not contain releaseYear or releaseMonth"
-					}
-					assignEnvVariable('TRAIN_NAME', yearMatcher.group('year') + '-' + monthMatcher.group('month'))
+					def buildProperties = utilities.loadBuildProperties(buildId)
+					assignEnvVariable('TRAIN_NAME', "${buildProperties.RELEASE_YEAR}-${buildProperties.RELEASE_MONTH}")
 					if (!env.TRAIN_NAME) {
 						error "TRAIN_NAME is empty."
 					}