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