Bug 691752

Summary: 'cds unassociate_repo' shows --repo_id instead of --repoid in error message
Product: [Retired] Pulp Reporter: Sachin Ghai <sghai>
Component: nodesAssignee: Jay Dobies <jason.dobies>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: low Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: jconnor, skarmark
Target Milestone: ---Keywords: Triaged
Target Release: Sprint 27   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-24 20:16:32 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Sachin Ghai 2011-03-29 12:01:03 UTC 
Description of problem: when you unsassociate a repo from CDS, --help option will show the usage like:

pulp-admin: error: Option --repo_id is required; please see --help

Version-Release number of selected component (if applicable): Build 157 and 158 on RHEL6 x86_64


How reproducible:
- Regsiter a CDS with pulp server
- Associate a repo
- then unassociate a repo


Steps to Reproduce:
1. Register CDS 
[root@dhcp193-127 ~]# pulp-admin -u admin -p admin cds register --hostname dhcp193-96.pnq.redhat.com
Successfully registered CDS [dhcp193-96.pnq.redhat.com]


2. Associate repo
root@dhcp193-127 ~]# pulp-admin -u admin -p admin cds associate_repo --repoid rhel6 --hostname dhcp193-96.pnq.redhat.com
Successfully associated CDS [dhcp193-96.pnq.redhat.com] with repo [rhel6]

[root@dhcp193-127 ~]# pulp-admin -u admin -p admin cds unassociate_repo --hostname dhcp193-96.pnq.redhat.com
Usage: pulp-admin <options> cds unassociate_repo <options>

pulp-admin: error: Option --repo_id is required; please see --help
			===================	

  
Actual results:
pulp-admin: error: Option --repo_id is required; please see --help
			===================	

Expected results:
The error option should be --repoid instead of --repo_id

Additional info: This error message appears only when you pass all required options except the repo id with 'cds unassociate_repo'
Comment 1 Jay Dobies 2011-08-10 14:32:18 UTC 
commit 961f028a1a077f13768eaa621d996975fb46e1cd
Author: Jay Dobies <jason.dobies>
Date:   Wed Aug 10 10:30:24 2011 -0400

    691752 - Corrected the argument name in the error text

src/pulp/client/admin/plugins/cds.py
Comment 2 Jeff Ortel 2011-08-12 21:48:29 UTC 
build: 0.224
Comment 3 Preethi Thomas 2011-09-02 14:24:41 UTC 

verified
[root@preethi ~]# rpm -q pulp
pulp-0.0.229-1.fc15.noarch

[root@preethi ~]# pulp-admin cds unassociate_repo --hostname test 
Usage: pulp-admin <options> cds unassociate_repo <options>

pulp-admin: error: Option --repoid is required; please see --help
[root@preethi ~]#
Comment 4 Preethi Thomas 2012-02-24 20:16:32 UTC 
Pulp v1.0 is released
Closed Current Release.
Comment 5 Preethi Thomas 2012-02-24 20:17:51 UTC 
Pulp v1.0 is released.