Change in jenkins[master]: Fix cleanup_docker

Code Review gerrit at ovirt.org
Sun Jul 30 07:15:59 UTC 2017


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

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

Change subject: Fix cleanup_docker
......................................................................


Fix cleanup_docker

cleanup_docker failed when it encountered images (mostly layers)
without repo or tag.
This patch fixes the cleanup bug by addressing the images by
the image id instead of repo/tag.

Also moved a global constant (whitelisted repositories) to
the top of the file where it belongs.

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

Change-Id: Ic6086eff1a211d8d4154b65b41d0a3a77d52e9f6
---
M jobs/confs/shell-scripts/cleanup_slave.sh
1 file changed, 6 insertions(+), 5 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic6086eff1a211d8d4154b65b41d0a3a77d52e9f6
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: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>


More information about the Engine-commits mailing list