Bug 717975

Summary:	Repo discovery to parse if url itself is repository
Product:	[Retired] Pulp	Reporter:	Garik Khachikyan <gkhachik>
Component:	user-experience	Assignee:	Pradeep Kilambi <pkilambi>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	unspecified	CC:	mkoci, pkilambi, skarmark, tsanders
Target Milestone:	---	Keywords:	Triaged
Target Release:	Sprint 27
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-02-24 20:16:27 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	647488

Description Garik Khachikyan 2011-06-30 15:07:18 UTC

Description of problem:
In recent implementation of katello we have an integration of both: create&discovery calls ibnto one: create procedure (I tried through: cli shell).
Thus: bringing a url to the repository creation (/me plans to create a single one: not discover) fails and no repo is being prepared.
If you could handle in discovery call possibility of being the url itself a repo that would be great. thanks.

Version-Release number of selected component (if applicable):
pulp-0.0.191-1.fc14.noarch
katello-0.1.49-1.git.67.631dcc7.fc14.noarch

How reproducible:
always with using recent katello cli command(s)

Steps to Reproduce:
1.provide to discovery code a url being itself a repo.
2.
3.
  
Actual results:
Repo creation fails

Expected results:
The repo should be created.

Additional info:

Comment 1 Todd Sanders 2011-07-03 23:23:00 UTC

Please provide the URL that you were passing to repo discovery.

-Todd

Comment 2 Pradeep Kilambi 2011-08-12 16:58:20 UTC

commit 487ae673b9bf158a4a1fa9752dd6a389f7c9cef5

$ sudo pulp-admin repo discovery --url=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/ --type=yum
Discovering urls with yum metadata, This could take some time...
Number of Urls Discovered (-): 1
+------------------------------------------+
 Repository Urls discovered @ [http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/]
+------------------------------------------+
(-)  [1] http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo

Select urls for which candidate repos should be created; use `y` to confirm (h for help):

Comment 3 Jeff Ortel 2011-08-12 21:48:28 UTC

build: 0.224

Comment 4 Preethi Thomas 2011-08-17 12:15:16 UTC

verified
[root@preethi ~]# rpm -q pulp
pulp-0.0.224-1.fc14.noarch

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

Select urls for which candidate repos should be created; use `y` to confirm (h for help):q
Operation aborted upon user request.
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin repo discovery --url=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/
Usage: pulp-admin <options> repo discovery <options>

pulp-admin: error: Option --type is required; please see --help
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin repo discovery --url=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/ --type=yum
Discovering urls with yum metadata, This could take some time...
Number of Urls Discovered (|): 1
+------------------------------------------+
 Repository Urls discovered @ [http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/]
+------------------------------------------+
(-)  [1] http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo

Select urls for which candidate repos should be created; use `y` to confirm (h for help):

Comment 5 Preethi Thomas 2012-02-24 20:16:27 UTC

Pulp v1.0 is released
Closed Current Release.

Comment 6 Preethi Thomas 2012-02-24 20:17:49 UTC

Pulp v1.0 is released.