Bug 691752 - 'cds unassociate_repo' shows --repo_id instead of --repoid in error message
Summary: 'cds unassociate_repo' shows --repo_id instead of --repoid in error message
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: nodes
Version: unspecified
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ---
: Sprint 27
Assignee: Jay Dobies
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-03-29 12:01 UTC by Sachin Ghai
Modified: 2012-02-24 20:17 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-24 20:16:32 UTC


Attachments (Terms of Use)

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.


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