Bug 1103945

Summary:	hammer repo list --product=<name> returns wrong repos
Product:	Red Hat Satellite	Reporter:	Partha Aji <paji>
Component:	Hammer	Assignee:	Walden Raines <walden>
Status:	CLOSED CURRENTRELEASE	QA Contact:	sthirugn <sthirugn>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.0.3	CC:	jhutar, jmontleo, mmccune, sthirugn
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
URL:	http://projects.theforeman.org/issues/6029
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-09-11 12:21:41 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:	1103944

Description Partha Aji 2014-06-03 00:41:44 UTC

hammer repository list  --product-id=2 --organization-id=1 works well and returns the repositories belonging to the product. 
OTOH

hammer repository list  --product=foo --organization-id=1 where foo is the name of the product, returns repositories belonging to all products in that organization. 

After some debugging it looks like the repositories controller never really gets a product_id sent as parameter when --product=foo is used.

Comment 1 Partha Aji 2014-06-03 00:41:46 UTC

Created from redmine issue http://projects.theforeman.org/issues/6029

Comment 3 Partha Aji 2014-06-03 20:03:14 UTC

I believe this is being addressed in 
http://projects.theforeman.org/issues/5747

Comment 4 Bryan Kearney 2014-06-16 18:04:45 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/6029 has been closed

Comment 6 sthirugn@redhat.com 2014-07-29 18:38:33 UTC

Verified.

Filtering seem to work fine now.

hammer> repository list --organization-id=3 --product-id=95
---|------------|--------------|--------------|----
ID | NAME       | PRODUCT      | CONTENT TYPE | URL
---|------------|--------------|--------------|----
22 | testrepo34 | newproduct23 | yum          |    
---|------------|--------------|--------------|----


hammer> repository list --organization-id=3 --product=newproduct23
---|------------|--------------|--------------|----
ID | NAME       | PRODUCT      | CONTENT TYPE | URL
---|------------|--------------|--------------|----
22 | testrepo34 | newproduct23 | yum          |    
---|------------|--------------|--------------|----


hammer> repository list --organization-id=3
---|------------------------------------------------------------|---------------------------------|--------------|---------------------------------------------------------------------------
ID | NAME                                                       | PRODUCT                         | CONTENT TYPE | URL                                                                       
---|------------------------------------------------------------|---------------------------------|--------------|---------------------------------------------------------------------------
2  | Red Hat Enterprise Linux 6 Server Kickstart x86_64 6Server | Red Hat Enterprise Linux Server | yum          | https://cdn.redhat.com/content/dist/rhel/server/6/6Server/x86_64/kickstart
3  | Red Hat Enterprise Linux 6 Server RPMs i386 6Server        | Red Hat Enterprise Linux Server | yum          | https://cdn.redhat.com/content/dist/rhel/server/6/6Server/i386/os         
4  | Red Hat Enterprise Linux 6 Server RPMs x86_64 6Server      | Red Hat Enterprise Linux Server | yum          | https://cdn.redhat.com/content/dist/rhel/server/6/6Server/x86_64/os       
22 | testrepo34                                                 | newproduct23                    | yum          |                                                                           
---|------------------------------------------------------------|---------------

Version Tested:
Satellite-6.0.4-RHEL-6-20140723.0

* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.19-1.el6_5.noarch
* candlepin-scl-1-5.el6_4.noarch
* candlepin-scl-quartz-2.1.5-5.el6_4.noarch
* candlepin-scl-rhino-1.7R3-1.el6_4.noarch
* candlepin-scl-runtime-1-5.el6_4.noarch
* candlepin-selinux-0.9.19-1.el6_5.noarch
* candlepin-tomcat6-0.9.19-1.el6_5.noarch
* elasticsearch-0.90.10-4.el6sat.noarch
* foreman-1.6.0.29-1.el6sat.noarch
* foreman-compute-1.6.0.29-1.el6sat.noarch
* foreman-gce-1.6.0.29-1.el6sat.noarch
* foreman-libvirt-1.6.0.29-1.el6sat.noarch
* foreman-ovirt-1.6.0.29-1.el6sat.noarch
* foreman-postgresql-1.6.0.29-1.el6sat.noarch
* foreman-proxy-1.6.0.21-1.el6sat.noarch
* foreman-selinux-1.6.0-8.el6sat.noarch
* foreman-vmware-1.6.0.29-1.el6sat.noarch
* katello-1.5.0-27.el6sat.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.6-1.el6sat.noarch
* katello-installer-0.0.56-1.el6sat.noarch
* openldap-2.4.23-32.el6_4.1.x86_64
* pulp-katello-0.3-3.el6sat.noarch
* pulp-nodes-common-2.4.0-0.23.beta.el6sat.noarch
* pulp-nodes-parent-2.4.0-0.23.beta.el6sat.noarch
* pulp-puppet-plugins-2.4.0-0.23.beta.el6sat.noarch
* pulp-puppet-tools-2.4.0-0.23.beta.el6sat.noarch
* pulp-rpm-plugins-2.4.0-0.23.beta.el6sat.noarch
* pulp-selinux-2.4.0-0.23.beta.el6sat.noarch
* pulp-server-2.4.0-0.23.beta.el6sat.noarch
* python-ldap-2.3.10-1.el6.x86_64
* ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
* ruby193-rubygem-runcible-1.1.0-2.el6sat.noarch

Comment 8 Bryan Kearney 2014-09-11 12:21:41 UTC

This was delivered with Satellite 6.0 which was released on 10 September 2014.