Bug 1223926

Summary: content-host errata list failed
Product: Red Hat Satellite Reporter: sthirugn <sthirugn>
Component: Content ManagementAssignee: Adam Price <adprice>
Status: CLOSED CURRENTRELEASE QA Contact: sthirugn <sthirugn>
Severity: high Docs Contact:
Priority: unspecified    
Version: 6.1.0CC: bbuckingham, bkearney, mmccune
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/10598
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-08-12 13:55:34 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:

Description sthirugn@redhat.com 2015-05-21 17:51:50 UTC
Description of problem:
content-host errata list failed

Version-Release number of selected component (if applicable):
Sat 6.1 GA Snap 5

How reproducible:
Always

Steps to Reproduce:
hammer> content-host errata list --id=5f9651b3-5cdd-4b24-902b-3889861e821a
missing param 'system_id' in parameters


Actual results:
Error as shown above

Expected results:
No error

Additional info:

Comment 1 sthirugn@redhat.com 2015-05-21 17:56:53 UTC
None of the command options work
hammer> content-host errata list --id=5f9651b3-5cdd-4b24-902b-3889861e821a --content-view-id=4
missing param 'system_id' in parameters


hammer> content-host errata list --id=5f9651b3-5cdd-4b24-902b-3889861e821a --environment-id=2
missing param 'system_id' in parameters

Comment 2 sthirugn@redhat.com 2015-05-21 17:59:50 UTC
This bug was found while verifying BZ 1219593

Comment 4 Adam Price 2015-05-22 19:07:58 UTC
Created redmine issue http://projects.theforeman.org/issues/10598 from this bug

Comment 5 Adam Price 2015-05-26 19:16:00 UTC
Hammer output when the bug was filed:

$ hammer content-host errata list --help
Usage:
    hammer content-host errata list [OPTIONS]
 
Options:
 --by BY                           Field to sort the results on
 --content-view CONTENT_VIEW_NAME  Content view name to search by
 --content-view-id CONTENT_VIEW_ID content view numeric identifier
 --environment ENVIRONMENT_NAME    Name to search by
 --environment-id ENVIRONMENT_ID    
 --full-results FULL_RESULTS       Whether or not to show all results
                                   One of true/false, yes/no, 1/0.
 --id ID                           UUID of the content host
 --name NAME                       Name to search by
 --order ORDER                     Sort field and order, eg. ‘name DESC’
 --page PAGE                       Page number, starting at 1
 --per-page PER_PAGE               Number of results per page to return
 --search SEARCH                   Search string
 -h, --help                        print help

Notice --id and --name. These two options are inconsistent with other content-host errata commands. The correct fix for this bug is to replace --id and --name with --content-host-id and --content-host in Apipie to be consistent with other content-host errata subcommands.

After the Apipie changes, the options list will look like the following:

$ hammer content-host errata list --help
Usage:
    hammer content-host errata list [OPTIONS]

Options:
 --by BY                                 Field to sort the results on
 --content-host CONTENT_HOST_NAME        Name to search by
 --content-host-id CONTENT_HOST_ID       UUID of the content host
 --content-view CONTENT_VIEW_NAME        Content view name to search by
 --content-view-id CONTENT_VIEW_ID       content view numeric identifier
 --environment ENVIRONMENT_NAME          Name to search by
 --environment-id ENVIRONMENT_ID          
 --full-results FULL_RESULTS             Whether or not to show all results
                                         One of true/false, yes/no, 1/0.
 --order ORDER                           Sort field and order, eg. ‘name DESC’
 --organization ORGANIZATION_NAME        Organization name to search by
 --organization-id ORGANIZATION_ID       organization ID
 --organization-label ORGANIZATION_LABEL Organization label to search by
 --page PAGE                             Page number, starting at 1
 --per-page PER_PAGE                     Number of results per page to return
 --search SEARCH                         Search string
 -h, --help                              print help

Comment 6 Adam Price 2015-05-26 19:17:09 UTC
https://github.com/Katello/katello/pull/5250

Comment 8 sthirugn@redhat.com 2015-06-04 15:57:17 UTC
Verified in Sat 6.1 GA Snap 7

hammer> content-host errata list --content-host-id=4bab4d39-d574-4903-a789-67b7e9ff11d2 --organization-id=4 --content-view='rhel7partialcv'  --lifecycle-environment='QE'
-------------------------------------|----------------|--------|----------------------|------------
ID                                   | ERRATUM ID     | TYPE   | TITLE                | INSTALLABLE
-------------------------------------|----------------|--------|----------------------|------------
000f740b-ce6c-429d-a052-7ab3ee641329 | RHBA-2014:0946 | bugfix | unzip bug fix update |            
-------------------------------------|----------------|--------|--------

But one issue is still open in https://bugzilla.redhat.com/show_bug.cgi?id=1228316#c3

Comment 9 Bryan Kearney 2015-08-11 13:18:51 UTC
This bug is slated to be released with Satellite 6.1.

Comment 10 Bryan Kearney 2015-08-12 13:55:34 UTC
This bug was fixed in version 6.1.1 of Satellite which was released on 12 August, 2015.