fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/33068 )

Change subject: fixup: contrib/jenkins: create workspace.tar.xz on error
......................................................................

fixup: contrib/jenkins: create workspace.tar.xz on error

cat-testlogs.sh does "exit 1", so no workspace.tar.xz is created.
Call this script after archiving the workspace.

Change-Id: Ibcb842f32418e66a186d6b21bb5861cf4a0b7c4a
Fixes: 799d972132 "contrib/jenkins: create workspace.tar.xz on error"
Related: OS#5665
---
M contrib/jenkins.sh
1 file changed, 15 insertions(+), 3 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved




diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index fc3eb12..c5161ef 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -14,14 +14,12 @@
 fi

 exit_tar_workspace() {
-       cat-testlogs.sh
-
        if [ "$IS_MASTER_BUILD" = "1" ]; then
                tar -cJf "/tmp/workspace.tar.xz" "$base"
                mv /tmp/workspace.tar.xz "$base"
        fi

-       exit 1
+       cat-testlogs.sh
 }

 set -ex

--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/33068
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ibcb842f32418e66a186d6b21bb5861cf4a0b7c4a
Gerrit-Change-Number: 33068
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to