Change in jenkins[master]: Move generic cleanup steps to cleanup_slave.sh

Code Review gerrit at ovirt.org
Tue Jul 11 06:38:35 UTC 2017


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

Barak Korren has submitted this change and it was merged. ( https://gerrit.ovirt.org/79103 )

Change subject: Move generic cleanup steps to cleanup_slave.sh
......................................................................


Move generic cleanup steps to cleanup_slave.sh

We had several non-mock cleanup steps in 'mock_cleanup.sh'. This patch
moves them to their right place in 'cleanup_slave.sh'.

This patch also:
* Fixes libvirt cleanup code in cleanup_slave.sh
* Makes the job fail if something failed in cleanup_slave.sh

Change-Id: I1560f0171a614f654a245b4f2d42d9830813d9b1
Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-1507
Signed-off-by: Barak Korren <bkorren at redhat.com>
---
M jobs/confs/shell-scripts/cleanup_slave.sh
M jobs/confs/shell-scripts/mock_cleanup.sh
2 files changed, 58 insertions(+), 65 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1560f0171a614f654a245b4f2d42d9830813d9b1
Gerrit-PatchSet: 4
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: Daniel Belenky <dbelenky at redhat.com>
Gerrit-Reviewer: Evgheni Dereveanchin <ederevea at redhat.com>
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>


More information about the Engine-commits mailing list