fedora-infra / mirrormanager2

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

umdl: fix 'modular' repository detection #221

Closed adrianreber closed 7 years ago

adrianreber commented 7 years ago

The 'modular' repository detection from PR #220 needed some fixes.

The repository detection was not entirely correct (some ifs at the wrong level) and a newly merged feature to not create repositories for ('Cloud', 'Server', 'Workstation') blocked the creation of the modular Server repository (PR #219).

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

pypingou commented 7 years ago

👍 for me

adrianreber commented 7 years ago

@pypingou thanks

@ausil: With this I get the following in staging:

# curl -s 'http://localhost/metalink?repo=modular-bikeshed-server&arch=x86_64&country=global' | grep fedorapr
        <url protocol="https" type="https" location="US" preference="98" >https://dl.fedoraproject.org/pub/fedora/linux/modular/development/bikeshed/Server/x86_64/os/repodata/repomd.xml</url>
        <url protocol="http" type="http" location="US" preference="98" >http://dl.fedoraproject.org/pub/fedora/linux/modular/development/bikeshed/Server/x86_64/os/repodata/repomd.xml</url>

I will release version 0.8.3 with this included, try it once more in staging and then try to get a freeze exception for the production systems.