David Caro has submitted this change and it was merged.
Change subject: Skipping mounts on the setup steps
......................................................................
Skipping mounts on the setup steps
I've found out that in some cases (quite rare) if the setup fails, it
will delete $HOME, that includes the mounted dir right now, so adding
the mounts only when running the shell.
It avoids from the packages installed from modifying the mounted dirs
(see the bug url).
Also added some missing local declarations to prepare_chroot
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1258255
Change-Id: I3fffbab5b86ab2e348f36f4bf0b226009f3a9af2
Signed-off-by: David Caro <dcaroest(a)redhat.com>
---
M mock_configs/mock_runner.sh
1 file changed, 30 insertions(+), 13 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me; Ready for merge
Anton Marchukov: Looks good to me
David Caro: Verified; Passed CI tests; Ready for review
--
To view, visit
https://gerrit.ovirt.org/45546
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3fffbab5b86ab2e348f36f4bf0b226009f3a9af2
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaroest(a)redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(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: Max Kovgan <mvk(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>