Bug 790218

Summary:	Repo discovery url must end in a trailing slash
Product:	[Retired] Pulp	Reporter:	James Slagle <jslagle>
Component:	user-experience	Assignee:	Pradeep Kilambi <pkilambi>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	1.0.0	CC:	tsanders
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

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.