Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Commit 95a1fef

Browse files
committed
Change corefx jit stress scenario names due to git complaining too long
file name
1 parent 5ef759d commit 95a1fef

File tree

1 file changed

+15
-14
lines changed

1 file changed

+15
-14
lines changed

netci.groovy

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@ class Constants {
4444
'jitstressregs3' : ['COMPlus_JitStressRegs' : '3'], 'jitstressregs4' : ['COMPlus_JitStressRegs' : '4'],
4545
'jitstressregs8' : ['COMPlus_JitStressRegs' : '8'], 'jitstressregs0x10' : ['COMPlus_JitStressRegs' : '0x10'],
4646
'jitstressregs0x80' : ['COMPlus_JitStressRegs' : '0x80'],
47-
'corefx_jitstress1' : ['COMPlus_JitStress' : '1'],
48-
'corefx_jitstress2' : ['COMPlus_JitStress' : '2']]
47+
'fxjs1' : ['COMPlus_JitStress' : '1'],
48+
'fxjs2' : ['COMPlus_JitStress' : '2']]
4949
// This is the basic set of scenarios
5050
def static basicScenarios = ['default', 'pri1', 'ilrt']
5151
// This is the set of configurations
@@ -68,7 +68,7 @@ def static setMachineAffinity(def job, def os, def architecture) {
6868
}
6969

7070
def static isCorefxTesting(def scenario) {
71-
return scenario.substring(0,6) == 'corefx'
71+
return scenario.substring(0,2) == 'fx'
7272
}
7373

7474
// Generates the string for creating a file that sets environment variables
@@ -208,8 +208,8 @@ def static addTriggers(def job, def isPR, def architecture, def os, def configur
208208
case 'forcerelocs':
209209
case 'jitstress1':
210210
case 'jitstress2':
211-
case 'corefx_jitstress1':
212-
case 'corefx_jitstress2':
211+
case 'fxjs1':
212+
case 'fxjs2':
213213
assert (os == 'Windows_NT') || (os in Constants.crossList)
214214
Utilities.addPeriodicTrigger(job, '@daily')
215215
break
@@ -319,8 +319,8 @@ def static addTriggers(def job, def isPR, def architecture, def os, def configur
319319
Utilities.addGithubPRTrigger(job, "${os} ${architecture} ${configuration} Build and Test (Jit - JitStressRegs=0x80)",
320320
"(?i).*test\\W+${os}\\W+${scenario}.*")
321321
break
322-
case 'corefx_jitstress1':
323-
case 'corefx_jitstress2':
322+
case 'fxjs1':
323+
case 'fxjs2':
324324
// No Linux support is needed now
325325
break
326326
default:
@@ -410,12 +410,12 @@ def static addTriggers(def job, def isPR, def architecture, def os, def configur
410410
Utilities.addGithubPRTrigger(job, "${os} ${architecture} ${configuration} Build and Test (Jit - JitStressRegs=0x80)",
411411
"(?i).*test\\W+${os}\\W+${scenario}.*")
412412
break
413-
case 'corefx_jitstress1':
413+
case 'fxjs1':
414414
assert (os == 'Windows_NT') || (os in Constants.crossList)
415415
Utilities.addGithubPRTrigger(job, "${os} ${architecture} ${configuration} Build and Test (Jit - CoreFx JitStress=1)",
416416
"(?i).*test\\W+${os}\\W+${scenario}.*")
417417
break
418-
case 'corefx_jitstress2':
418+
case 'fxjs2':
419419
assert (os == 'Windows_NT') || (os in Constants.crossList)
420420
Utilities.addGithubPRTrigger(job, "${os} ${architecture} ${configuration} Build and Test (Jit - CoreFx JitStress=2)",
421421
"(?i).*test\\W+${os}\\W+${scenario}.*")
@@ -600,7 +600,8 @@ combinedScenarios.each { scenario ->
600600
case 'x86':
601601

602602
if (scenario == 'default' || Constants.jitStressModeScenarios.containsKey(scenario)) {
603-
buildCommands += "build.cmd ${lowerConfiguration} ${architecture}"
603+
buildOpts = enableCorefxTesting ? 'skiptests' : ''
604+
buildCommands += "build.cmd ${lowerConfiguration} ${architecture} ${buildOpts}"
604605
}
605606

606607
// For Pri 1 tests, we must shorten the output test binary path names.
@@ -629,7 +630,7 @@ combinedScenarios.each { scenario ->
629630
if (Constants.jitStressModeScenarios.containsKey(scenario)) {
630631
if (enableCorefxTesting) {
631632
// Sync to corefx repo
632-
buildCommands += "git clone https://github.com/dotnet/corefx corefx"
633+
buildCommands += "git clone https://github.com/dotnet/corefx fx"
633634

634635
def setEnvVar = ''
635636
def envVars = Constants.jitStressModeScenarios[scenario]
@@ -638,7 +639,7 @@ combinedScenarios.each { scenario ->
638639
}
639640

640641
// Run corefx testing
641-
buildCommands += "cd corefx && call \"C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\vcvarsall.bat\" x86 ${setEnvVar} && Build.cmd /p:ConfigurationGroup=Release /p:WithCategories=\"InnerLoop;OuterLoop\" /p:BUILDTOOLS_OVERRIDE_RUNTIME=%WORKSPACE%\\bin\\Product\\Windows_NT.x64.Checked /p:TestWithLocalLibraries=true"
642+
buildCommands += "cd fx && call \"C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\vcvarsall.bat\" x86 ${setEnvVar} && Build.cmd /p:ConfigurationGroup=Release /p:WithCategories=\"InnerLoop;OuterLoop\" /p:BUILDTOOLS_OVERRIDE_RUNTIME=%WORKSPACE%\\bin\\Product\\Windows_NT.x64.Checked /p:TestWithLocalLibraries=true"
642643
}
643644
else {
644645
def stepScriptLocation = "%WORKSPACE%\\bin\\tests\\SetStressModes.bat"
@@ -678,10 +679,10 @@ combinedScenarios.each { scenario ->
678679
// Archive only result xml files since corefx/bin/tests is very large around 10 GB.
679680

680681
// For windows, pull full test results and test drops for x86/x64
681-
Utilities.addArchival(newJob, "corefx/bin/test/**/testResults.xml")
682+
Utilities.addArchival(newJob, "fx/bin/test/**/testResults.xml")
682683

683684
if (architecture == 'x64' || !isPR) {
684-
Utilities.addXUnitDotNETResults(newJob, 'corefx/bin/tests/**/testResults.xml')
685+
Utilities.addXUnitDotNETResults(newJob, 'fx/bin/tests/**/testResults.xml')
685686
}
686687
}
687688

0 commit comments

Comments
 (0)