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(a)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(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server