Re: Freeze break request: Move mirrorlists from Forking to Threading

+1 Stephen John Smoogen escreveu em dom, 18/10/2015 às 23:34 : > +1 from smooge. > > On 18 October 2015 at 16:21, Kevin Fenzi wrote: > > +1 here. > > > > kevin > > > > ___ > > infrastructure mailing list > >

Re: Freeze break request: Fix host netblocks lookup in threaded mirrorlist

> Looks ok. Would it be ok to put comments in the future about what the > change does since it doesn't seem to match exactly what the above > comment says? [Or did I misread that ocmment?] > > > > > commit 8a9d389bc77fe49086458df7ca03f9192a711e79 > > Author: Patrick Uiterwijk

Re: Freeze break request: Fix host netblocks lookup in threaded mirrorlist

Looks ok. Would it be ok to put comments in the future about what the change does since it doesn't seem to match exactly what the above comment says? [Or did I misread that ocmment?] On 19 October 2015 at 17:41, Patrick Uiterwijk wrote: > -BEGIN PGP SIGNED MESSAGE-

Freeze break request: Fix host netblocks lookup in threaded mirrorlist

-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi all, After some discussion with Matt on my pull request to make mirrorlist use threads instead of forks, I found one issue with the current code that breaks the ASN-based lookup: it changes the in-memory version of the ASN lookup tree. Could I

Re: Freeze break request: Fix host netblocks lookup in threaded mirrorlist

+1 here kevin pgp1ch8Xtx5jt.pgp Description: OpenPGP digital signature ___ infrastructure mailing list infrastructure@lists.fedoraproject.org http://lists.fedoraproject.org/admin/infrastructure@lists.fedoraproject.org

Re: Freeze break request: Fix host netblocks lookup in threaded mirrorlist

So, I deployed this to some servers first to test, but they were again falling under the load (since they were having to do a lot of deep copies, and that took more CPU time than I had expected). As such, I reverted this patch, and will be looking at a better way to fix this. With kind regards,

Wiki change - Infra-apprentice

I added the following line and linked page to the Fedora Infra wiki Apprentice Page ; *"See the List of hosts available to Infrastructure Apprentices