On Mon, May 18, 2015 at 09:11:11AM -0600, Kevin Fenzi wrote: > On Mon, 18 May 2015 13:03:23 +0200 > Adrian Reber <adrian@xxxxxxxx> wrote: > > > On Mon, May 18, 2015 at 11:04:45AM +0200, Pierre-Yves Chibon wrote: > > > On Mon, May 18, 2015 at 09:23:06AM +0200, Adrian Reber wrote: > > > > I am not sure if this is necessary, but searching for > > > > 'publiclist fedora 20' returns following link: > > > > > > > > http://mirrors.fedoraproject.org/publiclist/Fedora/20/ > > > > > > > > which is currently not correctly rewritten/redirected to > > > > > > > > https://admin.fedoraproject.org/mirrormanager/mirrors/Fedora/20 > > > > > > > > The following patch fixes the rewrite (tested in staging). For > > > > full MM1 compatibility additional code changes are required: > > > > > > > > https://github.com/fedora-infra/mirrormanager2/pull/76 > > > > > > > > Without the above PR URLs like > > > > > > > > http://mirrors.fedoraproject.org/publiclist/Fedora > > > > http://mirrors.fedoraproject.org/publiclist/EPEL > > > > > > > > are not correctly rewritten/redirected. So I am not sure if this > > > > should also be applied as a hotfix. > > > > > > > > Can I get two +1 for the apache rewrite changes? And maybe some > > > > feedback if we need the changes from the PR mentioned above. > > > > > > > > > > i agree with the change proposed below, +1 > > > > > > However, we could backport the pull-request above on the top of 1.0 > > > and release a fixed RPM or we could cut a 1.1 release but there are > > > more changes in the pipes: > > > > > > * Include the background header image in the fedora template > > > directly (instead of calling it from koji) > > > * Mark always up to date hosts as up to date, even when unreachable > > > * Rework the crawler with the introduction of the canary mode > > > (among other) > > > * Adjust spec file to follow the systemd packaging guidelines > > > * Offer possibility to sort by product (that's the change described > > > above) > > > > > > The first two changes seem harmless to me, the last two are also ok > > > but I fear the change in the crawler might be a little big to put > > > in a freeze-break. > > > > > > So this considered, I guess I am more in favor of backporting the > > > change as a patch and do a 1.0.0-2 RPM release. > > > > I would also not deploy the crawler changes during the freeze. I am > > also in favour of backporting the mentioned changes (without the > > crawler changes) > > +1 to just the rewrite and changes needed to make it work. So the apache configuration change has been pushed and I just pushed a new RPM, -2 release fixing the application side of it. https://admin.fedoraproject.org/mirrormanager/ looks good :) Thanks, Pierre
Attachment:
pgpF20Ouv2oU6.pgp
Description: PGP signature
_______________________________________________ infrastructure mailing list infrastructure@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/infrastructure
