Change in jenkins[master]: Made mock runner create logs in a safe place

Code Review gerrit at ovirt.org
Thu Dec 29 10:32:33 UTC 2016


>From Barak Korren <bkorren at redhat.com>:

Barak Korren has submitted this change and it was merged.

Change subject: Made mock runner create logs in a safe place
......................................................................


Made mock runner create logs in a safe place

Recently we made mock_runner.sh create logs in exported-artifacts to
make post-build collection scritps consistant. This can fail when the
script running inside mock tries to clean up exported-artifacts on it
own. In that case, mock_runner.sh can fail because the directories and
files it created are not longer there.

This patch moves log creation to a safe temporary directory, and moves
the logs to exported-artifacts only when mock_runner.sh is finished.

Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-992
Change-Id: I5f7f7756c3f44f4bdd6b5f6f40ee3b547b906fdb
Signed-off-by: Barak Korren <bkorren at redhat.com>
---
M automation/check-patch.sh
M mock_configs/mock_runner.sh
2 files changed, 18 insertions(+), 4 deletions(-)

Approvals:
  Barak Korren: Verified; Looks good to me, approved; Ready for merge
  Gil Shinar: Looks good to me
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/69281
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5f7f7756c3f44f4bdd6b5f6f40ee3b547b906fdb
Gerrit-PatchSet: 3
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: David Caro <david at dcaro.es>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list