Change in jenkins[master]: Modify cleanup_slave.sh not to remove docker DMs

Code Review gerrit at ovirt.org
Sun Aug 27 07:34:36 UTC 2017


>From Barak Korren <bkorren at redhat.com>:

Barak Korren has submitted this change and it was merged. ( https://gerrit.ovirt.org/80806 )

Change subject: Modify cleanup_slave.sh not to remove docker DMs
......................................................................


Modify cleanup_slave.sh not to remove docker DMs

This patch supposed to fix a bug where after removal
of all device mappings that loop devices, docker
fails to detect that it's device was removed.

Instead of removing all device mappings and loop devices,
we test if the device pool docker uses actually known by
devicemapper. If it was deleted, we need to restart docker
after removing /var/lib/docker.

Jira: https://ovirt-jira.atlassian.net/browse/OVIRT-1600

Change-Id: If72c360bd717d001cd24d9e9514ea83ef812936e
Signed-off-by: Daniel Belenky <dbelenky at redhat.com>
---
M jobs/confs/shell-scripts/cleanup_slave.sh
1 file changed, 55 insertions(+), 3 deletions(-)

Approvals:
  Barak Korren: Looks good to me, approved; Ready for merge
  Daniel Belenky: Verified; Ready for review
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/80806
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If72c360bd717d001cd24d9e9514ea83ef812936e
Gerrit-PatchSet: 6
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Daniel Belenky <dbelenky at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren at redhat.com>
Gerrit-Reviewer: Daniel Belenky <dbelenky at redhat.com>
Gerrit-Reviewer: Evgheni Dereveanchin <ederevea at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>
Gerrit-Reviewer: Yuval Turgeman <yturgema at redhat.com>


More information about the Engine-commits mailing list