We have pretty slow connection from s390x koji which helped to uncover this part. Kojid downloads all files from repomd.xml (incl. filelists) which is really big. What we really want is just 'origin' (used by Koji only).
Signed-off-by: Igor Gnatenko ignatenkobrain@fedoraproject.org
FTR, I didn't test it since I don't really have place to do so :)
Testing this with some librepo code (outside of koji), patch should be correct. In Fedora it should save around 140M per architecture when repo changes.
pkgorigins = r.getinfo(librepo.LRR_YUM_REPOMD)['origin']['location_href'] koji.util.rmtree(tmpdir) … relpath = os.path.join(repodir, self.br_arch, pkgorigins) with koji.openRemoteFile(relpath, **opts) as fo:
This looks weird... Did anybody test this part of code ever?
rebased onto 1561150c9cbb6cd913f5cd92c4f9fe36763f0239
Ok, I found out why it works... I have updated patch to be even more effective.
:thumbsup:
rebased onto a66b4ddfb4399cdc03ccc6f66acd634c692ee88f
Nice one, I've created #1399 for reference.
Metadata Update from @tkopecek: - Pull-request tagged with: testing-ready
@tkopecek, what is preventing this from being merged?
@mikem is doing the merges. Can you review/merge it, please?
Commit 5d26852c fixes this pull-request
Pull-Request has been merged by mikem
We have pretty slow connection from s390x koji which helped to uncover
this part. Kojid downloads all files from repomd.xml (incl. filelists)
which is really big. What we really want is just 'origin' (used by Koji
only).
Signed-off-by: Igor Gnatenko ignatenkobrain@fedoraproject.org