From Barak Korren <bkorren@redhat.com>:
Barak Korren has submitted this change and it was merged. ( https://gerrit.ovirt.org/77751 ) Change subject: cleanup_slave: Fixed the cleanup of docker images ...................................................................... cleanup_slave: Fixed the cleanup of docker images We can't cleanup docker images by image ID as we might remove 'whitelisted' images. For example, a job that tags the official centos image with custom:tag, then we'll search for all the images which doesn't belong to the whitelisted repositories, and remove them (referring them by image id) - this will cause the whitelisted image to be removed. The new cleanup, is referring images by it's repository:tag, and will not remove whitelisted images even if the repository name was retagged by another image during the run. Change-Id: I3c780cd5b7bc329e5715a1c7d9340340ac073cac --- M jobs/confs/shell-scripts/cleanup_slave.sh 1 file changed, 2 insertions(+), 7 deletions(-) Approvals: Barak Korren: Looks good to me, approved Jenkins CI: Passed CI tests Daniel Belenky: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/77751 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c780cd5b7bc329e5715a1c7d9340340ac073cac Gerrit-PatchSet: 11 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Daniel Belenky <dbelenky@redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk@redhat.com> Gerrit-Reviewer: Barak Korren <bkorren@redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky@redhat.com> Gerrit-Reviewer: Evgheni Dereveanchin <ederevea@redhat.com> Gerrit-Reviewer: Eyal Edri <eedri@redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi@redhat.com>