Bug 670307

Summary:	pulp-admin errata list allows both --repoid and --consumerid to be specified
Product:	[Retired] Pulp	Reporter:	Jay Dobies <jason.dobies>
Component:	z_other	Assignee:	Jay Dobies <jason.dobies>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	medium	Docs Contact:
Priority:	low
Version:	unspecified	CC:	pkilambi, skarmark, tsanders
Target Milestone:	---	Keywords:	Triaged
Target Release:	Sprint 20
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:07:15 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

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.