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 <dcaro...@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 <dcaro...@redhat.com>
Gerrit-Reviewer: Barak Korren <bkor...@redhat.com>
Gerrit-Reviewer: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Paz Dangur <pdan...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Sharon Naftaly <snaft...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to