Bug 631805

Summary: inconsistent cli option names for"package name" for packagegroup add_package and delete_package
Product: [Retired] Pulp Reporter: Preethi Thomas <pthomas>
Component: z_otherAssignee: Pradeep Kilambi <pkilambi>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: low Docs Contact:
Priority: low    
Version: unspecifiedCC: whayutin
Target Milestone: ---Keywords: Triaged
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-02-17 16:52:14 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 Preethi Thomas 2010-09-08 12:38:29 UTC
Description of problem:


In packagegroup add_package the option for package names are

 -n PNAMES, --name=PNAMES
                        Packages to be added.
                        To specify multiple packages use multiple -n


While in packagegroup delete_package the option is


--pkgname=PKGNAME     Package name




[root@preethi ~]# pulp-admin -uadmin -padmin packagegroup delete_package --helpUsage: pulp-admin -u <username> -p <password> packagegroup delete_package [OPTIONS]


Options:
  -h, --help            show this help message and exit
  --repoid=REPOID       Repository Label
  --id=GROUPID          Group id
  --pkgname=PKGNAME     Package name
  --type=GROUPTYPE      Type of list to delete package from, example
                        'mandatory', 'optional', 'default'
  -u USERNAME, --username=USERNAME
                        username for access to Pulp.  Default user admin is
                        included with base install.
  -p PASSWORD, --password=PASSWORD
                        password for access to Pulp.
[root@preethi ~]# pulp-admin -uadmin -padmin packagegroup add_package --helpUsage: pulp-admin -u <username> -p <password> packagegroup add_package [OPTIONS]


Options:
  -h, --help            show this help message and exit
  --repoid=REPOID       Repository Label
  --id=GROUPID          Group id
  -n PNAMES, --name=PNAMES
                        Packages to be added.
                        To specify multiple packages use multiple -n
  --type=GROUPTYPE      Type of list to add package to, example 'mandatory',
                        'optional', 'default'
  -u USERNAME, --username=USERNAME
                        username for access to Pulp.  Default user admin is
                        included with base install.
  -p PASSWORD, --password=PASSWORD
                        password for access to Pulp.

Comment 1 Pradeep Kilambi 2010-10-27 19:50:33 UTC
both should be -n now..

[pkilambi@prad ~]$ sudo pulp-admin -uadmin -padmin packagegroup add_package --help
Usage: pulp-admin <options> packagegroup add_package <options>

Options:
  -h, --help            show this help message and exit
  --id=ID               package group id (required)
  -r REPOID, --repoid=REPOID
                        repository label (required)
  -n NAME, --name=NAME  packages to be added; to specify multiple packages use
                        multiple -n (required)
  -t GROUPTYPE, --type=GROUPTYPE
                        type of list to add package to, example 'mandatory',
                        'optional', 'default', 'conditional'
  --requires=REQUIRES   required package name, only used by 'conditional'
                        package group type

[pkilambi@prad ~]$ sudo pulp-admin -uadmin -padmin packagegroup delete_package --help
Usage: pulp-admin <options> packagegroup delete_package <options>

Options:
  -h, --help            show this help message and exit
  --id=ID               package group id (required)
  -r REPOID, --repoid=REPOID
                        repository label (required)
  -n NAME, --name=NAME  package name (required)
  -t GROUPTYPE, --type=GROUPTYPE
                        type of list to delete package from, example
                        'mandatory', 'optional', 'default'

Comment 2 Jay Dobies 2010-10-29 16:58:30 UTC
Fixed in 0.77.

Comment 3 Jay Dobies 2010-11-03 19:35:33 UTC
Fixed in build 0.78.

Comment 4 Preethi Thomas 2010-11-10 13:27:37 UTC
verified

[root@preethi ~]# pulp-admin -u admin -p admin packagegroup add_package --help
Usage: pulp-admin <options> packagegroup add_package <options>

Options:
  -h, --help            show this help message and exit
  --id=ID               package group id (required)
  -r REPOID, --repoid=REPOID
                        repository label (required)
  -n NAME, --name=NAME  packages to be added; to specify multiple packages use
                        multiple -n (required)
  -t GROUPTYPE, --type=GROUPTYPE
                        type of list to add package to, example 'mandatory',
                        'optional', 'default', 'conditional'
  --requires=REQUIRES   required package name, only used by 'conditional'
                        package group type
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin -u admin -p admin packagegroup delete_package --help
Usage: pulp-admin <options> packagegroup delete_package <options>

Options:
  -h, --help            show this help message and exit
  --id=ID               package group id (required)
  -r REPOID, --repoid=REPOID
                        repository label (required)
  -n NAME, --name=NAME  package name (required)
  -t GROUPTYPE, --type=GROUPTYPE
                        type of list to delete package from, example
                        'mandatory', 'optional', 'default'

Comment 5 Preethi Thomas 2011-02-17 16:52:14 UTC
Closing with community release 

pulp-0.0.139-1.fc14.noarch