Good news! A fix has already been done and will be released with 2.8.0. https://pulp.plan.io/issues/1263
Michael On Fri, Mar 4, 2016 at 9:13 AM, Michael Hrivnak <[email protected]> wrote: > 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
