Bug 670307 - pulp-admin errata list allows both --repoid and --consumerid to be specified
Summary: pulp-admin errata list allows both --repoid and --consumerid to be specified
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
: Sprint 20
Assignee: Jay Dobies
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks: verified-to-close
TreeView+ depends on / blocked
 
Reported: 2011-01-17 20:35 UTC by Jay Dobies
Modified: 2011-08-16 12:20 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-08-16 12:07:15 UTC
Embargoed:


Attachments (Terms of Use)

Description Jay Dobies 2011-01-17 20:35:24 UTC
$ pulp-admin errata list --repoid foo --consumerid bar
No errata available to list

<jdob_> what happens if I pass both --consumerid and --repoid to errata list?
<prad> jdob_, should not be able to do that.. if we're its a bug

I'm guessing we should check in the CLI to make sure that we don't allow both to be specified, otherwise the user might think the behavior is just listing errata for that consumer from that repo.

Comment 1 Jay Dobies 2011-01-19 16:55:10 UTC
Example:
$ pulp-admin errata list --repoid foo --consumerid bar
Please select either a consumer or a repository, not both


commit	939150ddfb09e6deb1a01627aa3bb6b400dc602c
tree	87a4dbcb1bd4569803a4711ee298f0ae0860e505


670307 - Added a check to ensure either consumer or repo is specified.


src/pulp/client/core/errata.py

Comment 2 Jay Dobies 2011-01-21 19:18:45 UTC
Fixed in 0.130.

Comment 3 Preethi Thomas 2011-01-25 20:06:33 UTC
fails-qa

[root@preethi ~]# rpm -q pulp
pulp-0.0.131-1.fc14.noarch
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin errata list
A consumer or a repository is required to lookup errata


[root@preethi ~]# pulp-admin errata list --repoid=f14-update --consumerid=preethi.usersys.redhat.com
+------------------------------------------+
    Available Errata in Repo [f14-update]
+------------------------------------------+
FEDORA-2010-17333 , FEDORA-2010-18758 , FEDORA-2010-18974 , FEDORA-2010-18378 , FEDORA-2010-18372 , FEDORA-2010-18377 , FEDORA-2010-19153 , FEDORA-2010-19151 , FEDORA-2010-16513 , FEDORA-2010-17102 , FEDORA-2010-17101 , FEDORA-2010-17032 , FEDORA-2010-17033 , FEDORA-2010-17036 , FEDORA-2011-0080 , FEDORA-2010-17326 , FEDORA-2011-0171 , FEDORA-2010-18168 , FEDORA-2010-17489 , FEDORA-2010-17792 , FEDORA-2010-18760 , FEDORA-2010-17864 , FEDORA-2010-18943 , FEDORA-2010-18942 , FEDORA-2010-18412 , FEDORA-2010-18945 , FEDORA-2010-19143 , FEDORA-2010-19145 , FEDORA-2010-17005 , FEDORA-2010-17314 , FEDORA-2010-16306 , FEDORA-2010-18950 , FEDORA-2010-18404 , FEDORA-2010-19136 , FEDORA-2010-17655 , FEDORA-2010-17653 , FEDORA-2010-17495 , FEDORA-2010-17789 , FEDORA-2010-17783 , FEDORA-2010-17784 , FEDORA-2010-17786 , FEDORA-2010-17878 , FEDORA-2010-19223 , FEDORA-2010-19224 , FEDORA-2010-18400 , FEDORA-2010-19042 , FEDORA-2010-17120 , FEDORA-2010-17215 , FEDORA-2010-16224 , FEDORA-2010-16227 , FEDORA-2010-17011 , FEDORA-2010-17017 , FEDORA-2010-18216 , FEDORA-2010-16316 , FEDORA-2010-16318 , FEDORA-2011-0192 , FEDORA-2011-0191 , FEDORA-2010-18612 , FEDORA-2010-19303 , FEDORA-2010-16822 ,

Comment 4 Jay Dobies 2011-02-03 21:05:34 UTC
commit	a141440403f8a50aae5a6a9cf4555f86878b77fc
tree	345b4e172b4886eae11b5edff65bc72616f6ab64


670307 - Added separation of argument validation between consumer and
admin CLIs.


bin/pulp-client
src/pulp/client/core/errata.py

Comment 5 Jay Dobies 2011-02-04 19:24:02 UTC
Fixed in 0.135.

Comment 6 Preethi Thomas 2011-02-07 19:32:34 UTC
[root@preethi ~]# rpm -q pulp
pulp-0.0.135-1.fc14.noarch

[root@preethi ~]# pulp-admin errata list --repoid=f14-update --consumerid=preethi.usersys.redhat.com
Please select either a consumer or a repository, not both

Comment 7 Preethi Thomas 2011-08-16 12:07:15 UTC
Closing with Community Release 15

pulp-0.0.223-4.

Comment 8 Preethi Thomas 2011-08-16 12:20:56 UTC
Closing with Community Release 15

pulp-0.0.223-4.


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