Bug 1309226 - 'hidden value' flag is not listed in hammer parameter info output
Summary: 'hidden value' flag is not listed in hammer parameter info output
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Hammer
Version: Nightly
Hardware: Unspecified
OS: Unspecified
medium vote
Target Milestone: Unspecified
Assignee: orabin
QA Contact: Katello QA List
URL: http://projects.theforeman.org/issues...
Depends On:
Blocks: 1278718
TreeView+ depends on / blocked
Reported: 2016-02-17 09:34 UTC by Jitendra Yejare
Modified: 2019-09-26 13:57 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2016-07-27 11:18:17 UTC
Target Upstream Version:

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 13750 0 None None None 2016-04-22 16:10:25 UTC

Description Jitendra Yejare 2016-02-17 09:34:45 UTC
Description of problem:
hidden value flag is not displayed in the output of hammer sc-param info --id <id>.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. From hammer CLI, info some parameter
# hammer sc-param info --id <id>

Actual results:
The 'hidden value' flag for that parameter is not displayed/listed in output.
#hammer sc-param info --id 319
Ignoring ruby-libvirt-0.5.2 because its extensions are not built.  Try: gem pristine ruby-libvirt --version 0.5.2
Id:              319
Parameter:       admin_password
Default Value:   [123]
Override:        true
Puppet class:    nova::api
Description:     sadsa dsadwq 3r q3rq  f fdfs dfdsf dsd   23e 3rrdsf dsdsfdfdf 323e3                        ewewe#@#@#@#@#@!*()*&(&&$%%#@$#
Type:            array
Required:        false
    Rule: kit,sit
Override values: 
    Merge overrides:     no
    Merge default value: no
    Avoid duplicates:    no
    Order:               hostgroup, organization, fqdn, domain, os
     1) Id:    26
        Match: os=rhel76
        Value: ["sit"]
Created at:      2016/01/27 10:18:52
Updated at:      2016/02/17 09:23:59

Expected results:
'hidden value' flag should be added/displayed in info output of sc-param.

Additional info:

Comment 1 Jitendra Yejare 2016-02-17 09:37:50 UTC
Created redmine issue http://projects.theforeman.org/issues/13750 from this bug

Comment 3 Bryan Kearney 2016-02-22 11:06:56 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/13750 has been closed
Ori Rabin
Applied in changeset commit:hammer-cli-foreman|b41686566e7c16cb8ad084e7510bd59b32ef21bf.

Comment 5 Jitendra Yejare 2016-03-23 07:25:59 UTC
The issue is still reproducible in Sat 6.2 snap 4.

The hidden value flag is not displayed in the hammer info output of smart class parameter as well as smart variable.

So moving this to Failed QA.

Comment 6 orabin 2016-03-28 07:38:02 UTC
This fix is available in hammer_cli_foreman version 0.6.2.
Snap 4 is using 0.5.1 so this should be tested again when the hammer version is updated or cherry-picked for the next snap.

Comment 8 Kedar Bidarkar 2016-05-24 12:59:53 UTC
 ~]# hammer sc-param info --id 16
[Foreman] Password for admin: 
Id:              16
Parameter:       ca_file
Default Value:   ${$::foreman_scap_client::params::ca_file}
Override:        false
Puppet class:    foreman_scap_client
Type:            string
Hidden Value?:   false
Required:        false
Override values: 
    Merge overrides:     no
    Merge default value: no
    Avoid duplicates:    no
    Order:               fqdn, hostgroup, os, domain
Created at:      2016/05/19 09:33:41
Updated at:      2016/05/19 09:33:41

We can now see the "hidden value" flag from the hammer cli output.

VERIFIED with sat62-snap12.1

Comment 9 Bryan Kearney 2016-07-27 11:18:17 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


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