Skip to content

Commit

Permalink
Limit docker label to linux special.system
Browse files Browse the repository at this point in the history
  • Loading branch information
sophia-guo committed Nov 5, 2024
1 parent bb6714e commit a2ffa73
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 9 deletions.
6 changes: 3 additions & 3 deletions pipelines/build/common/openjdk_build_pipeline.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -418,6 +418,8 @@ class Build {
testStages["${testType}"] = {
context.println "Running test: ${testType}"
context.stage("${testType}") {
def jobParams = getAQATestJobParams(testType)
def jobName = jobParams.TEST_JOB_NAME
def keep_test_reportdir = buildConfig.KEEP_TEST_REPORTDIR
def rerunIterations = '1'
if ("${testType}".contains('dev') || "${testType}".contains('external')) {
Expand All @@ -433,7 +435,7 @@ class Build {
DYNAMIC_COMPILE = true
}
def additionalTestLabel = buildConfig.ADDITIONAL_TEST_LABEL
if (testType == 'dev.openjdk' || testType == 'special.system') {
if (testType == 'dev.openjdk' || (testType == 'special.system' && jobName.contains('linux'))) {
context.println "${testType} need extra label sw.tool.docker"
if (additionalTestLabel == '') {
additionalTestLabel = 'sw.tool.docker'
Expand All @@ -452,9 +454,7 @@ class Build {
vendorTestBranches = buildConfig.BUILD_REF ?: vendorTestBranches
}

def jobParams = getAQATestJobParams(testType)

def jobName = jobParams.TEST_JOB_NAME
String helperRef = buildConfig.HELPER_REF ?: DEFAULTS_JSON['repository']['helper_ref']
def JobHelper = context.library(identifier: "openjdk-jenkins-helper@${helperRef}").JobHelper

Expand Down
4 changes: 2 additions & 2 deletions pipelines/jobs/configurations/jdk21u_pipeline_config.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ class Config21 {
arch : 'x64',
additionalNodeLabels: 'win2022&&vs2022',
test: [
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system']
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system', 'special.system']
],
buildArgs : [
'temurin' : '--create-jre-image --create-sbom --use-adoptium-devkit vs2022_redist_14.40.33807_10.0.26100.0'
Expand Down Expand Up @@ -138,7 +138,7 @@ class Config21 {
arch : 'aarch64',
additionalNodeLabels: 'xcode15.0.1',
test: [
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system']
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system', 'special.system']
],
buildArgs : [
'temurin' : '--create-jre-image --create-sbom'
Expand Down
6 changes: 4 additions & 2 deletions pipelines/jobs/configurations/jdk23u_pipeline_config.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ class Config23 {
arch : 'x64',
additionalNodeLabels: 'win2022&&vs2022',
test: [
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system']
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system', 'special.system']
],
buildArgs : [
'temurin' : '--create-jre-image --create-sbom --use-adoptium-devkit vs2022_redist_14.40.33807_10.0.26100.0'
Expand Down Expand Up @@ -138,7 +138,9 @@ class Config23 {
os : 'mac',
arch : 'aarch64',
additionalNodeLabels: 'xcode15.0.1',
test : 'default',
test: [
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system', 'special.system']
],
buildArgs : [
'temurin' : '--create-jre-image --create-sbom'
]
Expand Down
4 changes: 2 additions & 2 deletions pipelines/jobs/configurations/jdk24_pipeline_config.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ class Config24 {
arch : 'x64',
additionalNodeLabels: 'win2022&&vs2022',
test: [
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system']
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system', 'special.system']
],
buildArgs : [
'temurin' : '--create-jre-image --create-sbom --use-adoptium-devkit vs2022_redist_14.40.33807_10.0.26100.0'
Expand Down Expand Up @@ -139,7 +139,7 @@ class Config24 {
arch : 'aarch64',
additionalNodeLabels: 'xcode15.0.1',
test: [
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system']
weekly : ['sanity.openjdk', 'sanity.system', 'extended.system', 'sanity.perf', 'sanity.functional', 'extended.functional', 'extended.openjdk', 'extended.perf', 'special.functional', 'special.openjdk', 'dev.functional', 'dev.system', 'special.system']
],
buildArgs : [
'temurin' : '--create-jre-image --create-sbom'
Expand Down

0 comments on commit a2ffa73

Please sign in to comment.