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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Barak Korren <[email protected]>
Gerrit-Reviewer: David Caro <[email protected]>
Gerrit-Reviewer: Eyal Edri <[email protected]>
Gerrit-Reviewer: Paz Dangur <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Sharon Naftaly <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to