Bug 790218 - Repo discovery url must end in a trailing slash
Summary: Repo discovery url must end in a trailing slash
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: user-experience
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: Pradeep Kilambi
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-13 23:04 UTC by James Slagle
Modified: 2013-09-09 16:37 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:


Attachments (Terms of Use)

Description James Slagle 2012-02-13 23:04:22 UTC
If the url you use for repo discovery does not end in a trailing slash, no repos will be discovered.

[slagle@veeone ~]$ pulp-admin repo discovery -u https://cdn.redhat.com/content/dist/rhel/rhui/server/6/6Server/x86_64 --cert rcm-debug.pem --type yum
Discovering urls with yum metadata, This could take some time...
Number of Urls Discovered (/): 0
No repos discovered @ url location [https://cdn.redhat.com/content/dist/rhel/rhui/server/6/6Server/x86_64]

versus:

[slagle@veeone ~]$ pulp-admin repo discovery -u https://cdn.redhat.com/content/dist/rhel/rhui/server/6/6Server/x86_64/ --cert rcm-debug.pem --type yum
Discovering urls with yum metadata, This could take some time...
Number of Urls Discovered (\): 51

At the very least, we should add this requirement to the help text for repo discovery.

Comment 1 Pradeep Kilambi 2012-02-15 14:43:29 UTC
commit dc7a12d1bdd6b210f10b9566865d68e159e7f4e1

Comment 2 Jeff Ortel 2012-02-17 17:17:49 UTC
build: 0.267

Comment 3 Preethi Thomas 2012-02-20 19:37:09 UTC
both url with ad without '/' works now
[root@preethi ~]# rpm -q pulp
pulp-0.0.267-1.fc15.noarch

[root@preethi ~]# pulp-admin repo discovery --url http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing --type=yum
Discovering urls with yum metadata, This could take some time...
Number of Urls Discovered (\): 8
+------------------------------------------+
 Repository Urls discovered @ [http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing]
+------------------------------------------+
(-)  [1] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-16/x86_64
(-)  [2] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-16/i386
(-)  [3] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-15/x86_64
(-)  [4] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-15/i386
(-)  [5] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/6Server/x86_64
(-)  [6] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/6Server/i386
(-)  [7] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/5Server/x86_64
(-)  [8] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/5Server/i386

Select urls for which candidate repos should be created; use `y` to confirm (h for help):^C[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin repo discovery --url http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/ --type=yum
Discovering urls with yum metadata, This could take some time...
Number of Urls Discovered (/): 8
+------------------------------------------+
 Repository Urls discovered @ [http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/]
+------------------------------------------+
(-)  [1] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-16/x86_64
(-)  [2] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-16/i386
(-)  [3] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-15/x86_64
(-)  [4] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/fedora-15/i386
(-)  [5] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/6Server/x86_64
(-)  [6] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/6Server/i386
(-)  [7] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/5Server/x86_64
(-)  [8] http://repos.fedorapeople.org/repos/pulp/pulp/v1/testing/5Server/i386

Select urls for which candidate repos should be created; use `y` to confirm (h for help):^C[root@preethi ~]#

Comment 4 Preethi Thomas 2012-02-24 20:18:09 UTC
Pulp v1.0 is released
Closed Current Release.

Comment 5 Preethi Thomas 2012-02-24 20:18:57 UTC
Pulp v1.0 is released.


Note You need to log in before you can comment on or make changes to this bug.