Bug 975874 - error messages for a null --repo-id are not consistent in rpm repo
Summary: error messages for a null --repo-id are not consistent in rpm repo
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Pulp
Classification: Retired
Component: rpm-support
Version: 2.2 Beta
Hardware: Unspecified
OS: Unspecified
low
unspecified
Target Milestone: ---
: ---
Assignee: pulp-bugs
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-06-19 13:59 UTC by Jeremy Cline
Modified: 2015-02-28 21:57 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-02-28 21:57:15 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Pulp Redmine 359 0 None None None Never

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


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