Change in jenkins[master]: Skipping mounts on the setup steps

dcaroest at redhat.com dcaroest at redhat.com
Tue Sep 1 11:22:47 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk 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: Max Kovgan <mvk 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>



More information about the Engine-commits mailing list