Bug 681239

Summary: user update asks to enter new password even when you give the -P/--password argument
Product: [Retired] Pulp Reporter: Preethi Thomas <pthomas>
Component: z_otherAssignee: Sayli Karmarkar <skarmark>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: low Docs Contact:
Priority: low    
Version: unspecifiedCC: cperry, skarmark
Target Milestone: ---Keywords: Triaged
Target Release: Sprint 24   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-08-16 12:08:49 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, 673053    

Description Preethi Thomas 2011-03-01 14:17:15 UTC
Description of problem:
[root@preethi ~]# rpm -q pulp
pulp-0.0.143-1.fc14.noarch

[root@preethi ~]# pulp-admin user update
Usage: pulp-admin <options> user update <options>

pulp-admin: error: Option --username is required; please see --help
[root@preethi ~]# pulp-admin user update --help
Usage: pulp-admin <options> user update <options>

Options:
  -h, --help           show this help message and exit
  --username=USERNAME  username of user you wish to edit. Not editable
                       (required)
  -P, --password       change user password
  --name=NAME          updated name of user for display purposes
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin -u admin -p test user update --username admin --password=test1 --name "pulptest"
Usage: pulp-admin <options> user update <options>

pulp-admin: error: --password option does not take a value
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin -u admin -p test user update --username admin -P test1 --name "pulptest"
Enter new password for user admin: 


[root@preethi ~]# pulp-admin -u admin -p admin user update --username admin --password test --name "pulptest"
Enter new password for user admin:

Comment 1 Sayli Karmarkar 2011-06-07 05:39:53 UTC
$ sudo pulp-admin user update --help
Usage: pulp-admin <options> user update <options>

Options:
  -h, --help           show this help message and exit
  --username=USERNAME  username of user you wish to edit. Not editable
                       (required)
  --password=PASSWORD  updated password for user; use -P if you do not wish to
                       specify password on command line
  -P                   prompt for updated user password
  --name=NAME          updated name of user for display purposes


$ sudo pulp-admin user update --username=sayli -P
Enter new password for user sayli: 
Re-enter new password for user sayli: 
Successfully updated [ sayli ]

$ sudo pulp-admin user update --username=sayli --password=redhat
Successfully updated [ sayli ]

$ sudo pulp-admin user update --username=sayli --password
Usage: pulp-admin <options> user update <options>

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


Added similar changes for create as well. Moving to modified.

Comment 2 Jeff Ortel 2011-06-08 23:58:29 UTC
build: 0.188

Comment 3 Preethi Thomas 2011-07-06 18:12:30 UTC
verified

[root@preethi ~]# rpm -q pulp
pulp-0.0.200-1.fc14.noarch
[root@preethi ~]# pulp-admin user update
Usage: pulp-admin <options> user update <options>

pulp-admin: error: Option --username is required; please see --help
[root@preethi ~]# pulp-admin user update --username admin --password=redhat --name "test1"
Successfully updated [ admin ]

[root@preethi ~]# pulp-admin user update --username admin -P
Enter new password for user admin: 
Re-enter new password for user admin: 
Successfully updated [ admin ]

[root@preethi ~]# pulp-admin user update --username admin --password=admin --name "test1"
Successfully updated [ admin ]


[root@preethi ~]# pulp-admin user update --username admin --password 
Usage: pulp-admin <options> user update <options>

pulp-admin: error: --password option requires an argument
[root@preethi ~]#

Comment 4 Preethi Thomas 2011-08-16 12:08:49 UTC
Closing with Community Release 15

pulp-0.0.223-4.

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

pulp-0.0.223-4.