createrepo_c.PackageIterator yeilds no packages when filelists.xml.gz is empty about createrepo_c HOT 6 CLOSED

@apatters As it turns out, I reported this to packagecloud.io a few months back (they handle hosting the gitlab repos) and managed to get them to fix the issue.

And as of yesterday it looks like gitlab updated their repos in such a way that the metadata finally got refreshed, so it probably ought to be working now.

from createrepo_c.

I tested it and it does ^^

So I would suggest we don't make this particular change, but on a related note it is still impossible to pass a null for filelists or changelog xml without causing a failure when it perhaps ought to be optional to parse them. So there's still room for improvement.

from createrepo_c.

@dralley I am confused. I last tested this a few weeks ago and the iterator was not returning any packages with an empty filelists.xml.gz file. We have deliberately configured PackageCloud to use the empty file for GitLab repos due to size issues as detailed above. What issue did PackageCloud fix? Should I retest?

from createrepo_c.

The file list file is no longer empty. It correctly lists entries for each package.

from createrepo_c.

And so it is. It looks like they are now showing packages but that the files in each package:

<?xml version="1.0" encoding="UTF-8"?>
<filelists xmlns="http://linux.duke.edu/metadata/filelists" packages="276">
  <package pkgid="3b9f99f37013e4eb88293d04b53137c86b0fd679" name="gitlab-ee" arc
h="x86_64">
    <version epoch="0" ver="12.10.0" rel="ee.0.el8"/>
  </package>
.
.
.
<package pkgid="18b3214a19206f9b0727bfb024ae6c1da21dc9eb" name="gitlab-ee" arch="x86_64">
    <version epoch="0" ver="15.5.4" rel="ee.0.el8"/>
  </package>
</filelists>

Which works fine with pulp 3. Thanks!

from createrepo_c.

FWIW I would think this can be closed now.

from createrepo_c.