Skip to content

Commit 86cd6c1

Browse files
committed
[Build] Fix export of 'TEST_CONFIGURATIONS_EXPECTED' in SSH execution
Because in the 'Collect Result' jobs the publish.xml ANT script is executed remotely on the download-server via SSH the environment variables defined in the shell on the build-server are not available. As a workaround, define the 'testsConfigExpected' property explicitly as command-line argument. Also rename 'TESTS_CONFIGURATIONS_EXPECTED' to 'TEST_CONFIGURATIONS_EXPECTED' to avoid a double-plural. Follow-up to #2693
1 parent 5f0e4ca commit 86cd6c1

File tree

7 files changed

+8
-5
lines changed

7 files changed

+8
-5
lines changed

JenkinsJobs/Builds/I_build.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ spec:
8888
MAVEN_OPTS = "-Xmx6G"
8989
CJE_ROOT = "${WORKSPACE}/eclipse.platform.releng.aggregator/eclipse.platform.releng.aggregator/cje-production"
9090
logDir = "$CJE_ROOT/buildlogs"
91-
TESTS_CONFIGURATIONS_EXPECTED = \'''' + TEST_CONFIGURATIONS.collect{c ->
91+
TEST_CONFIGURATIONS_EXPECTED = \'''' + TEST_CONFIGURATIONS.collect{c ->
9292
'ep' + MAJOR + MINOR + 'I-unit-' + c.os + '-' + c.arch + '-java' + c.javaVersion + '_' + c.os + '.' + c.ws + '.' + c.arch + '_' + c.javaVersion
9393
}.join(',') + ''''
9494
}

JenkinsJobs/Releng/collectPerfResults.groovy

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,7 @@ ssh [email protected] ${javaCMD} -jar ${launcherJar} -n
124124
-Djob=${triggeringJob} \\
125125
-DbuildID=${buildID} \\
126126
-DeclipseStream=${STREAM} \\
127+
"-DtestsConfigExpected=${TEST_CONFIGURATIONS_EXPECTED}" \\
127128
-DEBuilderDir=${workspace}
128129
129130
#Delete Workspace

JenkinsJobs/Releng/collectResults.groovy

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@ ssh [email protected] ${javaCMD} -jar ${launcherJar} -n
110110
-Djob=${triggeringJob} \\
111111
-DbuildID=${buildID} \\
112112
-DeclipseStream=${STREAM} \\
113+
"-DtestsConfigExpected=${TEST_CONFIGURATIONS_EXPECTED}" \\
113114
-DEBuilderDir=${workspace}
114115
115116

JenkinsJobs/YBuilds/Y_build.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ spec:
8484
MAVEN_OPTS = "-Xmx6G"
8585
CJE_ROOT = "${WORKSPACE}/eclipse.platform.releng.aggregator/eclipse.platform.releng.aggregator/cje-production"
8686
logDir = "$CJE_ROOT/buildlogs"
87-
TESTS_CONFIGURATIONS_EXPECTED = \'''' + TEST_CONFIGURATIONS.collect{c ->
87+
TEST_CONFIGURATIONS_EXPECTED = \'''' + TEST_CONFIGURATIONS.collect{c ->
8888
'ep' + MAJOR + MINOR + 'Y-unit-' + c.os + '-' + c.arch + '-java' + c.javaVersion + '_' + c.os + '.' + c.ws + '.' + c.arch + '_' + c.javaVersion
8989
}.join(',') + ''''
9090
}

JenkinsJobs/YBuilds/collectYbuildResults.groovy

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ ssh [email protected] ${javaCMD} -jar ${launcherJar} -n
111111
-Djob=${triggeringJob} \\
112112
-DbuildID=${buildID} \\
113113
-DeclipseStream=${STREAM} \\
114+
"-DtestsConfigExpected=${TEST_CONFIGURATIONS_EXPECTED}" \\
114115
-DEBuilderDir=${workspace}
115116
116117
#Delete Workspace

cje-production/mbscripts/mb010_createEnvfiles.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,4 +78,4 @@ fn-addToPropFiles ECLIPSE_BUILDER_DIR "\"$CJE_ROOT/$AGG_DIR/eclipse.platform.rel
7878
fn-addToPropFiles PLATFORM_PRODUCTS_DIR "\"$CJE_ROOT/$AGG_DIR/eclipse.platform.releng.tychoeclipsebuilder/eclipse.platform.repository/target/products\""
7979
fn-addToPropFiles PLATFORM_REPO_DIR "\"$CJE_ROOT/$AGG_DIR/eclipse.platform.releng.tychoeclipsebuilder/eclipse.platform.repository/target/repository\""
8080
fn-addToPropFiles PLATFORM_TARGET_DIR "\"$CJE_ROOT/$AGG_DIR/eclipse.platform.releng.tychoeclipsebuilder/eclipse.platform.repository/target\""
81-
fn-addToPropFiles TESTS_CONFIGURATIONS_EXPECTED "\"${TESTS_CONFIGURATIONS_EXPECTED}\""
81+
fn-addToPropFiles TEST_CONFIGURATIONS_EXPECTED "\"${TEST_CONFIGURATIONS_EXPECTED}\""

cje-production/scripts/publish.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@
182182
substring="-perf-" />
183183
</condition>
184184
<!-- else normal unit tests configs -->
185-
<property name="testsConfigExpected" value="${env.TESTS_CONFIGURATIONS_EXPECTED}"/>
185+
<property name="testsConfigExpected" value="${env.TEST_CONFIGURATIONS_EXPECTED}"/>
186186

187187
<condition
188188
property="expectedConfigFilename"
@@ -257,7 +257,7 @@
257257
</loadresource>
258258
<echo message="eclipseStreamService: ${eclipseStreamService}"/>
259259

260-
<property environment="env" />
260+
<property environment="env"/>
261261

262262
</target>
263263

0 commit comments

Comments
 (0)