From 34003a54a06c3530d34676a54a35c9a6b55eec33 Mon Sep 17 00:00:00 2001 From: Bruce Forstall Date: Mon, 4 Feb 2019 13:30:12 -0800 Subject: Always archive build logs Even if the build fails. Add archiving in more cases where it was missing. --- netci.groovy | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'netci.groovy') 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. -- cgit v1.2.3