Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.
Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.

Bug 1097647

Summary: Hammer subscription list should show correct required parameter
Product: Red Hat Satellite Reporter: Sachin Ghai <sghai>
Component: HammerAssignee: Adam Price <adprice>
Status: CLOSED CURRENTRELEASE QA Contact: sthirugn <sthirugn>
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: bkearney, jmontleo, sthirugn
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/6168
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-09-11 12:25:11 UTC Type: Bug
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: 1101657    
Bug Blocks:    

Description Sachin Ghai 2014-05-14 08:58:32 UTC
Description of problem:

[root@dhcp201-170 ~]#  hammer subscription list
missing param 'system_id' in parameters


The message should say: Missing param 'system-id' in parameters or may be 'content host-id'

In any case we shoud not use 'underscore' for parameter..should be 'hiphen'
Version-Release number of selected component (if applicable):
sat6 beta snap4:

Satellite-6.0.3-RHEL-6-20140508.1


How reproducible:
always

Steps to Reproduce:
1. 
2.
3.

Actual results:


Expected results:
The message should say: Missing param 'system-id' in parameters or may be 'content host-id'

Additional info:

Comment 1 RHEL Program Management 2014-05-14 09:25:34 UTC
Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.

Comment 3 Bryan Kearney 2014-06-04 13:56:16 UTC
This is fixed by https://bugzilla.redhat.com/show_bug.cgi?id=1101657. I am moving this to ON_QA. Please do not test this until you see that https://bugzilla.redhat.com/show_bug.cgi?id=1101657 has been moved to ON_QA.

Comment 4 Sachin Ghai 2014-06-06 10:25:42 UTC
Getting same result with sat6 beta snap8 compose2 (Satellite-6.0.3-RHEL-6-20140605.0).

And as per comment 3, https://bugzilla.redhat.com/show_bug.cgi?id=1101657 is now on_qa. But not sure how that bug is relevant to this.

[root@dhcp201-193 yum.repos.d]# hammer subscription list
missing param 'system_id' in parameters

Still it throws missing parameter with underscore (_)

[root@dhcp201-193 yum.repos.d]# hammer subscription list --help
Usage:
    hammer subscription list [OPTIONS]

Options:
    --activation-key ACTIVATION_KEY_NAME  
    --activation-key-id ACTIVATION_KEY_ID Activation key ID
    --organization ORGANIZATION_NAME  
    --organization-id ORGANIZATION_ID Organization ID
    --organization-label ORGANIZATION_LABEL  
    --system SYSTEM_NAME           
    --system-id SYSTEM_ID         UUID of the system
    -h, --help                    print help

Since changes  are yet to be made so request to dev please update --system_id to --content-host-id  thanks.

Comment 5 Adam Price 2014-06-10 16:26:08 UTC
@Sachin your request in comment#4 seems like it would be better suited in a separate bug. I completely agree that "system" should be removed from all places, but this bug is about `subscription list` enforcing the correct parameters.

the system -> content-host mapping is currently being worked on for hammer-cli in the following two issues:

http://projects.theforeman.org/issues/6092
http://projects.theforeman.org/issues/6093

Comment 6 Adam Price 2014-06-11 16:16:13 UTC
Created redmine issue http://projects.theforeman.org/issues/6168 from this bug

Comment 7 Adam Price 2014-06-11 16:21:44 UTC
https://github.com/Katello/katello/pull/4247

Comment 9 sthirugn@redhat.com 2014-09-03 18:23:52 UTC
Verified.

hammer> subscription list
Error: Missing options to search organization

hammer> subscription list --organization-id=3
------------------------------------------|----------|---------|--------------|-----------|----------|------------|----------------------------------|-------------------------------------------|-----------|---------
NAME                                      | CONTRACT | ACCOUNT | SUPPORT      | QUANTITY  | CONSUMED | END DATE   | ID                               | PRODUCT                                   | QUANTITY  | ATTACHED
------------------------------------------|----------|---------|--------------|-----------|----------|------------|----------------------------------|-------------------------------------------|-----------|---------
newsat6productaug20                       |          |         |              | Unlimited | 0        | 2044-08-12 | 8a90e0e547f4e3fe0147f4f86d8d000b | newsat6productaug20                       | Unlimited | 0       


Version Tested:
GA Snap 7 - Satellite-6.0.4-RHEL-6-20140829.0

* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.23-1.el6_5.noarch
* candlepin-common-1.0.1-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.23-1.el6_5.noarch
* candlepin-tomcat6-0.9.23-1.el6_5.noarch
* elasticsearch-0.90.10-6.el6sat.noarch
* foreman-1.6.0.42-1.el6sat.noarch
* foreman-compute-1.6.0.42-1.el6sat.noarch
* foreman-gce-1.6.0.42-1.el6sat.noarch
* foreman-libvirt-1.6.0.42-1.el6sat.noarch
* foreman-ovirt-1.6.0.42-1.el6sat.noarch
* foreman-postgresql-1.6.0.42-1.el6sat.noarch
* foreman-proxy-1.6.0.30-1.el6sat.noarch
* foreman-selinux-1.6.0.14-1.el6sat.noarch
* foreman-vmware-1.6.0.42-1.el6sat.noarch
* katello-1.5.0-30.el6sat.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.6-1.el6sat.noarch
* katello-installer-0.0.62-1.el6sat.noarch
* openldap-2.4.23-34.el6_5.1.x86_64
* openldap-devel-2.4.23-34.el6_5.1.x86_64
* pulp-katello-0.3-4.el6sat.noarch
* pulp-nodes-common-2.4.1-0.5.rc1.el6sat.noarch
* pulp-nodes-parent-2.4.1-0.5.rc1.el6sat.noarch
* pulp-puppet-plugins-2.4.1-0.5.rc1.el6sat.noarch
* pulp-puppet-tools-2.4.1-0.5.rc1.el6sat.noarch
* pulp-rpm-plugins-2.4.1-0.5.rc1.el6sat.noarch
* pulp-selinux-2.4.1-0.5.rc1.el6sat.noarch
* pulp-server-2.4.1-0.5.rc1.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 10 Bryan Kearney 2014-09-11 12:25:11 UTC
This was delivered with Satellite 6.0 which was released on 10 September 2014.