If you don't mind filing an issue about it in our tracker, we'll take a look.
https://pulp.plan.io/projects/pulp_rpm/issues/new It should be an easy fix. If you or anyone else reading this is interested in an easy code contribution to pulp, this would be a great candidate! Aside from yum's code, did they cite any other specific documentation? I would be very interested in seeing that. Michael On Fri, Mar 4, 2016 at 1:30 AM, Vaclav Adamec <[email protected]> wrote: > So they response is quite ... unhelpful, based on some documenation > for repomd.xml seems that revision can be also null (number or null), any > chance to "fix" this on Pulp side or is it something which Pulp really use > for repo sync ? > > #/usr/lib/python2.7/site-packages/pulp_rpm/plugins/importers/yum/repomd/metadata.py > +168 > > if element.tag == REVISION_TAG: > try: > self.revision = int(element.text) > > > > On Thu, Mar 3, 2016 at 6:40 AM, Vaclav Adamec <[email protected] > > wrote: > >> Revision is empty element: <revision /> >> >> http://pastebin.com/raw/3YCq7dTN >> >> I'll get in touch with Artifactory support, if it's possible to fix this >> on their side. Thanks for help >> >> Vasek >> >> On Wed, Mar 2, 2016 at 10:26 PM, Michael Hrivnak <[email protected]> >> wrote: >> >>> Interesting. Can you provide the contents of >>> https://artifactory-master.test.com/artifactory/yum-local/test-repo/6/x86_64/repodata/repomd.xml >>> ? >>> >>> It should have an element like this: <revision>1456870296</revision> >>> >>> But based on the error you're seeing, I suspect it does not, or that it >>> is empty. Unfortunately, we haven't been able to identify a documented >>> schema for this file (or any of the other yum repo metadata files), so it >>> is difficult to know what is guaranteed or required to be present. >>> >>> Here is an example that does have the revision element, near the top: >>> >>> >>> https://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/zoo/repodata/repomd.xml >>> >>> Michael >>> >>> On Wed, Mar 2, 2016 at 10:34 AM, Vaclav Adamec < >>> [email protected]> wrote: >>> >>>> Pulp reports any issue with metadata, but I can use this repository >>>> directly in yum, without any issues. >>>> >>>> http://pastebin.com/x1p2zme3 >>>> >>>> Any advice ? >>>> >>>> Vasek >>>> >>>> >>>> On Tue, Mar 1, 2016 at 4:30 PM, Sean Myers <[email protected]> >>>> wrote: >>>> >>>>> On 02/29/2016 11:11 AM, Vaclav Adamec wrote: >>>>> > Hi, >>>>> > it is possible to feed pulp repo from authenticated source ? eg >>>>> something >>>>> > like this: >>>>> > >>>>> > pulp-admin rpm repo create --repo-id=test-repo-x86_64-live >>>>> > --display-name="TEST REPO x86_64 (Live)" --description="TEST REPO >>>>> x86_64 >>>>> > (Live)" --feed=" >>>>> > >>>>> https://login:[email protected]/artifactory/yum-local/test-repo/6/x86_64/ >>>>> " >>>>> > --serve-http=True --serve-https=False >>>>> > --relative-url=/test-repo/6/x86_64/live >>>>> > >>>>> > or >>>>> > >>>>> > pulp-admin rpm repo create --repo-id=test-repo-x86_64-live >>>>> > --display-name="TEST REPO x86_64 (Live)" --description="TEST REPO >>>>> x86_64 >>>>> > (Live)" --feed=" >>>>> > >>>>> https://login:[email protected]/artifactory/yum-local/test-repo/6/x86_64/ >>>>> " >>>>> > --serve-http=True --serve-https=False >>>>> > --relative-url=/test-repo/6/x86_64/live >>>>> > --basicauth-user=login --basicauth-pass=pass >>>>> > >>>>> > I'm getting: >>>>> > >>>>> > pulp-admin rpm repo sync run --repo-id=test-repo-x86_64-live >>>>> > >>>>> > Task Failed >>>>> > int() argument must be a string or a number, not 'NoneType' >>>>> > >>>>> > >>>>> > Via curl I can download metadata without any issues, no ssl problems >>>>> at >>>>> > all. Any clue ? >>>>> >>>>> I don't know what's causing that error, but a couple things in those >>>>> URLs might be confusing pulp. Some tracebacks related to the error >>>>> would >>>>> probably help to troubleshoot. >>>>> >>>>> https://pulp.readthedocs.org/en/latest/user-guide/troubleshooting.html >>>>> >>>>> Here are some guesses that might help in the meantime... >>>>> >>>>> The second method looks most correct. You don't need the "login:pass@" >>>>> out of the feed URL. If I remember right, pulp ignores them. >>>>> Regardless, >>>>> using --basicauth-user and --basicauth-pass is the supported way to get >>>>> basic auth working. Also, --relative-url should be a relative URL, and >>>>> not have a leading slash. I don't see anything else that would anger >>>>> the >>>>> pulp gods of validation, but my eyes are notoriously bad regex parsers. >>>>> Try this out: >>>>> >>>>> pulp-admin rpm repo create --repo-id=test-repo-x86_64-live >>>>> --display-name="TEST REPO x86_64 (Live)" --description="TEST REPO >>>>> x86_64 >>>>> (Live)" >>>>> --feed=" >>>>> https://artifactory-master.test.com/artifactory/yum-local/test-repo/6/x86_64/ >>>>> " >>>>> --serve-http=True --serve-https=False >>>>> --relative-url=test-repo/6/x86_64/live --basicauth-user=login >>>>> --basicauth-pass=pass >>>>> >>>>> The equals signs in args aren't strictly needed (but also aren't >>>>> hurting >>>>> anything), so this should also work: >>>>> >>>>> pulp-admin rpm repo create --repo-id=test-repo-x86_64-live >>>>> --display-name="TEST REPO x86_64 (Live)" --description="TEST REPO >>>>> x86_64 >>>>> (Live)" >>>>> --feed=" >>>>> https://artifactory-master.test.com/artifactory/yum-local/test-repo/6/x86_64/ >>>>> " >>>>> --serve-http true --serve-https false >>>>> --relative-url=test-repo/6/x86_64/live --basicauth-user login >>>>> --basicauth-pass pass >>>>> >>>>> >>>>> >>>> >>>> >>>> -- >>>> -- May the fox be with you ... >>>> /\ >>>> (~( >>>> ) ) /\_/\ >>>> (_=---_(@ @) >>>> ( \ / >>>> /|/----\|\ V >>>> " " " " >>>> >>>> >>>> >>>> _______________________________________________ >>>> Pulp-list mailing list >>>> [email protected] >>>> https://www.redhat.com/mailman/listinfo/pulp-list >>>> >>> >>> >> >> >> -- >> -- May the fox be with you ... >> /\ >> (~( >> ) ) /\_/\ >> (_=---_(@ @) >> ( \ / >> /|/----\|\ V >> " " " " >> >> >> > > > -- > -- May the fox be with you ... > /\ > (~( > ) ) /\_/\ > (_=---_(@ @) > ( \ / > /|/----\|\ V > " " " " > > > > _______________________________________________ > Pulp-list mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/pulp-list >
_______________________________________________ Pulp-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/pulp-list
