diff options
author | Bruce Forstall <brucefo@microsoft.com> | 2019-02-04 13:30:12 -0800 |
---|---|---|
committer | Bruce Forstall <brucefo@microsoft.com> | 2019-02-04 13:46:48 -0800 |
commit | 34003a54a06c3530d34676a54a35c9a6b55eec33 (patch) | |
tree | 4a992ed19877bfbed4b6a7835f2c5402f50ba7a5 /netci.groovy | |
parent | 9aa68da7d3dcd24761d03c4f41720194ab3638b8 (diff) | |
download | coreclr-34003a54a06c3530d34676a54a35c9a6b55eec33.tar.gz coreclr-34003a54a06c3530d34676a54a35c9a6b55eec33.tar.bz2 coreclr-34003a54a06c3530d34676a54a35c9a6b55eec33.zip |
Always archive build logs
Even if the build fails. Add archiving in more cases where it was missing.
Diffstat (limited to 'netci.groovy')
-rwxr-xr-x | netci.groovy | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/netci.groovy b/netci.groovy index a03a0cbd64..00d42d14c3 100755 --- a/netci.groovy +++ b/netci.groovy @@ -2244,6 +2244,9 @@ def static calculateBuildCommands(def newJob, def scenario, def branch, def isPR Utilities.addXUnitDotNETResults(newJob, 'bin/**/TestRun*.xml', true) } } + + // Archive the logs, even if the build failed (which is when they are most interesting). + Utilities.addArchival(newJob, "bin/Logs/*.log,bin/Logs/*.wrn,bin/Logs/*.err", "", /* doNotFailIfNothingArchived */ true, /* archiveOnlyIfSuccessful */ false) break case 'arm': case 'arm64': @@ -2293,6 +2296,9 @@ def static calculateBuildCommands(def newJob, def scenario, def branch, def isPR // Add archival. Utilities.addArchival(newJob, "bin/Product/**,bin/tests/tests.zip", "bin/Product/**/.nuget/**") } + + // Archive the logs, even if the build failed (which is when they are most interesting). + Utilities.addArchival(newJob, "bin/Logs/*.log,bin/Logs/*.wrn,bin/Logs/*.err", "", /* doNotFailIfNothingArchived */ true, /* archiveOnlyIfSuccessful */ false) break default: println("Unknown architecture: ${architecture}"); @@ -2414,6 +2420,9 @@ def static calculateBuildCommands(def newJob, def scenario, def branch, def isPR Utilities.addXUnitDotNETResults(newJob, "${workspaceRelativeFxRoot}/artifacts/bin/**/testResults.xml") } } + + // Archive the logs, even if the build failed (which is when they are most interesting). + Utilities.addArchival(newJob, "bin/Logs/*.log,bin/Logs/*.wrn,bin/Logs/*.err", "", /* doNotFailIfNothingArchived */ true, /* archiveOnlyIfSuccessful */ false) break case 'armem': // Emulator cross builds for ARM runs on Tizen currently @@ -2537,8 +2546,8 @@ def static calculateBuildCommands(def newJob, def scenario, def branch, def isPR Utilities.addArchival(newJob, "${testArtifactsTgzFileName}", "") } - // Archive the build logs from both product and test builds. - Utilities.addArchival(newJob, "bin/Logs/*.log,bin/Logs/*.wrn,bin/Logs/*.err", "") + // Archive the logs, even if the build failed (which is when they are most interesting). + Utilities.addArchival(newJob, "bin/Logs/*.log,bin/Logs/*.wrn,bin/Logs/*.err", "", /* doNotFailIfNothingArchived */ true, /* archiveOnlyIfSuccessful */ false) // We need to clean up the build machines; the docker build leaves newly built files with root permission, which // the cleanup task in Jenkins can't remove. |