Enable archive pattern support 99/4999/1
authorThanh Ha <thanh.ha@linuxfoundation.org>
Sat, 27 May 2017 14:49:49 +0000 (10:49 -0400)
committerThanh Ha <thanh.ha@linuxfoundation.org>
Sat, 27 May 2017 15:23:37 +0000 (11:23 -0400)
lftools deploy supports archiving using patterns via the -p argument.
Pass in this argument if the env variable ARCHIVE_ARTIFACTS is defined.
If variable is not defined or blank then a blank string gets passed in
which still works.

Change-Id: I4d096ea9ad1b0c93864d37c6188f02f7e32f6228
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
README.md
shell/logs-deploy.sh

index 3bad6b0..4c21ec6 100644 (file)
--- a/README.md
+++ b/README.md
@@ -132,3 +132,18 @@ Optional parameters:
 
 **branch**: is the git branch to build from.
 **jjb-version**: is the version of JJB to install in the build minion.
+
+## Archiving logs in Jobs
+
+There are 2 ways supported for archiving log information:
+
+1) Job creates $WORKSPACE/archives directory and places logs there
+
+In this method the entire archives directory will be pushed to the log server
+in the same structure as configured in the archives directory.
+
+2) Via job variable ARCHIVE_ARTIFACTS using globstar patterns.
+
+In this method a job can define a globstar for example ``**/*.log`` which then
+causes the archive script to do a globstar search for that pattern and archives
+any files it finds matching.
index fa31f2b..6dc8028 100644 (file)
@@ -15,6 +15,7 @@ set -eu -o pipefail
 
 set -x  # Trace commands for this script to make debugging easier.
 
+ARCHIVE_ARTIFACTS="${ARCHIVE_ARTIFACTS:-}"
 LOGS_SERVER="${LOGS_SERVER:-None}"
 
 if [ "${LOGS_SERVER}" == 'None' ]
@@ -27,7 +28,7 @@ else
     NEXUS_PATH="${SILO}/${JENKINS_HOSTNAME}/${JOB_NAME}/${BUILD_NUMBER}"
     BUILD_URL="${BUILD_URL}"
 
-    lftools deploy archives "$NEXUS_URL" "$NEXUS_PATH" "$WORKSPACE"
+    lftools deploy archives -p "$ARCHIVE_ARTIFACTS" "$NEXUS_URL" "$NEXUS_PATH" "$WORKSPACE"
     lftools deploy logs "$NEXUS_URL" "$NEXUS_PATH" "$BUILD_URL"
 
     set +x  # Disable trace since we no longer need it.