search

fedora-infra / mirrormanager2

Rewrite of the MirrorManager application in Flask and SQLAlchemy
https://mirrormanager.fedoraproject.org
GNU General Public License v2.0
63 stars 46 forks source link

Skip disabled repositories in the overview #190

Closed adrianreber closed 7 years ago

adrianreber commented 7 years ago

MirrorManager still picks up old and archived versions of repositories. This uses the existing disabled flag to ignore those repositories in the overview.

See https://fedorahosted.org/fedora-infrastructure/ticket/4625

Signed-off-by: Adrian Reber adrian@lisas.de

puiterwijk commented 7 years ago

It looks good to me, but just broke production, so maybe let's verify what happened there first :)

adrianreber commented 7 years ago

This change doesn't seem to be the direct cause for the broken mirrorlist in production. But setting the rawhide repository to disabled for a certain arch, disables all rawhide repositories.So this code change is not wrong but the database field 'disabled' has another meaning. Closing this PR as it cannot be used.