From Barak Korren <bkorren(a)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(a)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(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: David Caro <david(a)dcaro.es>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>