CentOS / container-pipeline-service

Code, infrastructure and deployment backend for the CentOS Container Pipeline backing up build system for registry.centos.org
https://registry.centos.org
GNU General Public License v3.0
49 stars 27 forks source link

Disable registry cleanup #557

Closed dharmit closed 6 years ago

dharmit commented 6 years ago

This fix is meant to prevent automated image deletion from internal registry and r.c.o till the time we fix #551

dharmit commented 6 years ago

@bamachrn would disabling the internal registry images deletion require manual intervention at regular interval? We might fill up the openshift node with images, right?

bamachrn commented 6 years ago

yes, I see it now, we are actually pruning it. so current images wont get deleted any way. may be we can skip disabling it?

dharmit commented 6 years ago

I'm confused. Do you want me to remove that part from the PR and just keep it limited to undoing #514?

bamachrn commented 6 years ago

yes, sorry for the confusion. I thought that is also deleting the current images.

bamachrn commented 6 years ago

LGTM, merging