Change in jenkins[master]: Doing proper mock cleanup

dcaroest at redhat.com dcaroest at redhat.com
Tue Dec 9 18:51:19 UTC 2014


David Caro has submitted this change and it was merged.

Change subject: Doing proper mock cleanup
......................................................................


Doing proper mock cleanup

Now it checks if there are any mounted dirs inside the chroot and tries to
umount before removing. It will fail if unable to umount.

Also the cleanup has been put as post-build task, so it should always run.
Changed the logs tarball extension to more standard tgz

Change-Id: I20cb54fb968556f3d1bdcf3a9ab14e826c56b352
Signed-off-by: David Caro <dcaroest at redhat.com>
---
M jobs/confs/shell-scripts/mock_cleanup.sh
M jobs/confs/shell-scripts/mock_setup.sh
M jobs/confs/yaml/builders/mock_install.yaml
M jobs/confs/yaml/builders/mock_rpm.yaml
A jobs/confs/yaml/publishers/mock-cleanup.yaml
M jobs/confs/yaml/templates/create-rpms_mock.yaml
M jobs/confs/yaml/templates/install-rpm-sanity.yaml
7 files changed, 134 insertions(+), 12 deletions(-)

Approvals:
  David Caro: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/35597
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I20cb54fb968556f3d1bdcf3a9ab14e826c56b352
Gerrit-PatchSet: 9
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Paz Dangur <pdangur at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Sharon Naftaly <snaftaly at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list