Bug 975874

Summary:	error messages for a null --repo-id are not consistent in rpm repo
Product:	[Retired] Pulp	Reporter:	Jeremy Cline <jcline>
Component:	rpm-support	Assignee:	pulp-bugs
Status:	CLOSED UPSTREAM	QA Contact:	Preethi Thomas <pthomas>
Severity:	unspecified	Docs Contact:
Priority:	low
Version:	2.2 Beta	CC:	mhrivnak, nobody, rbarlow
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-28 21:57:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jeremy Cline 2013-06-19 13:59:12 UTC

Description of problem: when a null repo id is given (--repo-id=) some commands present a validation error to the users, but other commands print the command options again with the message that --repo-id is required and was not specified.


Version-Release number of selected component (if applicable):
[jeremy@jcline ~]$ rpm -q pulp-rpm-admin-extensions 
pulp-rpm-admin-extensions-2.2.0-0.4.beta.fc18.noarch


How reproducible: always


Steps to Reproduce:
1. use '--repo-id=' on any command in the rpm repo content, copy, and remove sections (in the case of copy it's --from-repo-id and --to-repo-id).
2. compare to using '--repo-id=' in the rpm repo update, delete, export, sync, and uploads sections.

Actual results:
[jeremy@jcline ~]$ pulp-admin rpm repo remove rpm --repo-id=
Command: rpm
Description: remove RPMs from a repository

Available Arguments:

*snipped*

The following options are required but were not specified:
  --repo-id

[jeremy@jcline ~]$ pulp-admin rpm repo uploads rpm --repo-id=
Validation failed for argument [--repo-id]: value must contain only letters,
numbers, underscores, periods and hyphens



Expected results: A consistent error throughout the commands.


Additional info:

Comment 2 Brian Bouterse 2015-02-28 21:57:15 UTC

Moved to https://pulp.plan.io/issues/359