Project

Profile

Help

Story #1665

pulp-consumer repo bind input not validated (pitfall!)

Added by kfiresmith almost 5 years ago. Updated almost 2 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

Running the command 'pulp-consumer rpm bind --repo-id=foo --repo-id=bar --repo-id=zot' would only bind the 'zot' repo. This might be a troublesome pitfall as no error message is generated.

It should be made clear (perhaps via input validation of the command) that only one '--repo-id' entry will be used and all others are discarded.

History

#1 Updated by kfiresmith almost 5 years ago

Oddly enough it appears that some commands will accept multiple arguments, at least in the pulp-admin client - hopefully the back-end logic is similar so the above pitfall can be easily fixed to be more like the below example:

  1. pulp-admin repo group members add --group-id=rhel7-college-baserepos --repo-id=rhel-7-server-pulp27 --repo-id=rhel-7-server-supplementary-rpms
    Successfully added members to repository group [rhel7-college-baserepos]
  1. pulp-admin repo group list
    --------------------------------------------------------------------
    Repository Groups
    --------------------------------------------------------------------

Id: rhel7-college-baserepos
Display Name: None
Description: This is the basic set of repos any college managed EL7 host should
have
Repo Ids: EPEL7, rhel-7-server-supplementary-rpms, rhel-7-server-pulp27
Notes:

#2 Updated by bmbouter almost 2 years ago

  • Status changed from NEW to CLOSED - WONTFIX

#3 Updated by bmbouter almost 2 years ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

#4 Updated by bmbouter almost 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF