Skip to content

Commit

Permalink
don't pass node label to windows installer job (#1025)
Browse files Browse the repository at this point in the history
  • Loading branch information
gdams authored May 3, 2024
1 parent 54ff2fb commit c82d8ce
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions pipelines/build/common/openjdk_build_pipeline.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -796,8 +796,6 @@ class Build {
We run two jobs if we have a JRE (see https://github.com/adoptium/temurin-build/issues/1751).
*/
private void buildWindowsInstaller(VersionInfo versionData, String filter, String category) {
def nodeFilter = "${buildConfig.TARGET_OS}&&wix"

def buildNumber = versionData.build

if (versionData.major == 8) {
Expand All @@ -813,6 +811,12 @@ class Build {
// Get version patch number if one is present
def patch_version = versionData.patch ?: 0

def INSTALLER_JVM = "${buildConfig.VARIANT}"
// if variant is temurin set param as hotpot
if (buildConfig.VARIANT == 'temurin') {
INSTALLER_JVM = 'hotspot'
}

// Execute installer job
def installerJob = context.build job: 'build-scripts/release/create_installer_windows',
propagate: true,
Expand All @@ -827,9 +831,8 @@ class Build {
context.string(name: 'PRODUCT_BUILD_NUMBER', value: "${buildNumber}"),
context.string(name: 'MSI_PRODUCT_VERSION', value: "${versionData.msi_product_version}"),
context.string(name: 'PRODUCT_CATEGORY', value: "${category}"),
context.string(name: 'JVM', value: "${buildConfig.VARIANT}"),
context.string(name: 'JVM', value: "${INSTALLER_JVM}"),
context.string(name: 'ARCH', value: "${INSTALLER_ARCH}"),
['$class': 'LabelParameterValue', name: 'NODE_LABEL', label: "${nodeFilter}"]
]
context.copyArtifacts(
projectName: 'build-scripts/release/create_installer_windows',
Expand Down

0 comments on commit c82d8ce

Please sign in to comment.