Bug 672881

Summary: pulp-admin repo delete --id= deletes all repos
Product: [Retired] Pulp Reporter: Mike McCune <mmccune>
Component: user-experienceAssignee: Sayli Karmarkar <skarmark>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: cperry, skarmark
Target Milestone: ---Keywords: Triaged
Target Release: Sprint 20   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 672888 (view as bug list) Environment:
Last Closed: 2011-08-16 12:06:34 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 Mike McCune 2011-01-26 16:37:17 UTC
If you don't specify a repo id on delete pulp-admin will delete all repos:

# pulp-admin repo list
+------------------------------------------+
       List of Available Repositories
+------------------------------------------+

Id                 	rpm-fusion-free          
Name               	rpm-fusion-free          
FeedURL            	http://download1.rpmfusion.org/free/fedora/releases/14/Everything/x86_64/os/
FeedType           	yum                      
Arch               	noarch                   
Sync Schedule      	None                     
Packages           	411                      
Files              	0                        
Distributions      	None                     
Publish            	True                     
Clones             	[]                       
Groups             	None            


$ pulp-admin repo delete --id=
Successful deleted repository [  ]

$ pulp-admin repo list
No repositories available to list

Comment 1 Sayli Karmarkar 2011-01-28 20:07:21 UTC
Changed get_required_option routine so none of the required options accept empty value:

$ sudo pulp-admin repo delete --id=
Usage: pulp-admin <options> repo delete <options>

pulp-admin: error: Please provide a value for Option --id; see --help

Comment 2 Sayli Karmarkar 2011-01-28 20:15:20 UTC
Slight change in error message:

$ sudo pulp-admin repo delete --id=
Usage: pulp-admin <options> repo delete <options>

pulp-admin: error: --id option requires an argument

Comment 3 Jay Dobies 2011-02-02 20:07:06 UTC
Fixed in 0.134.

Comment 4 Preethi Thomas 2011-02-02 22:07:27 UTC
verified

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



[root@preethi published]# pulp-admin repo delete --id=
Usage: pulp-admin <options> repo delete <options>

pulp-admin: error: --id option requires an argument
[root@preethi published]# pulp-admin repo list
+------------------------------------------+
       List of Available Repositories
+------------------------------------------+

Id                 	foo                      
Name               	bar                      
FeedURL            	None                     
FeedType           	None                     
Arch               	noarch                   
Sync Schedule      	None                     
Packages           	0                        
Files              	0                        
Distributions      	None                     
Publish            	True                     
Clones             	[]                       
Groups             	None                     


Id                 	f14-update               
Name               	f14-updates              
FeedURL            	http://ftp.linux.ncsu.edu/pub/fedora/linux//updates/14/x86_64/
FeedType           	yum                      
Arch               	noarch                   
Sync Schedule      	None                     
Packages           	5932                     
Files              	0                        
Distributions      	None                     
Publish            	True                     
Clones             	[]                       
Groups             	None                     


[root@preethi published]#

Comment 5 Preethi Thomas 2011-08-16 12:06:34 UTC
Closing with Community Release 15

pulp-0.0.223-4.

Comment 6 Preethi Thomas 2011-08-16 12:20:38 UTC
Closing with Community Release 15

pulp-0.0.223-4.