Bug 1166158

Summary:

[RFE] In host's edit page, show the source for the value of puppet class parameters

Product:

Red Hat Satellite

Reporter:

Bryan Kearney <bkearney>

Component:

Configuration Management

Assignee:

Katello Bug Bin <katello-bugs>

Status:

CLOSED ERRATA

QA Contact:

Sachin Ghai <sghai>

Severity:

low

Docs Contact:

Priority:

unspecified

Version:

6.0.4

CC:

bkearney, dcleal, sghai, sthirugn

Target Milestone:

Unspecified

Keywords:

FutureFeature, Triaged

Target Release:

Unused

Hardware:

Unspecified

OS:

Unspecified

URL:

http://projects.theforeman.org/issues/7163

Whiteboard:

Fixed In Version:

Doc Type:

Enhancement

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2015-08-12 05:19:32 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:

Attachments:

Description	Flags
Assuming additional_info box will show the source with hostgroup name, as from where the value is coming	none

Description Bryan Kearney 2014-11-20 13:41:34 UTC

Opened as a PR by breakfastattiffs
Currently, on the edit hosts page, under the parameters tab, if a parameter was overridden in the host's hostgroup (or that hostgroup's hostgroup), the additional information popup for that parameter says "Matcher: hostgroup."
This is not very useful because the parameter's value could be coming from the hostgroup or a grandparent hostgroup. It would be more useful to display the actual hostgroup's name so that we know precisely from which parent the parameter's overridden value is coming from.

Comment 1 Bryan Kearney 2014-11-20 13:41:35 UTC

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

Comment 2 Bryan Kearney 2014-11-20 13:42:01 UTC

I am pulling down important features from the 1.7 release. You can read more abotu this feature at 

http://theforeman.org/manuals/1.7/index.html#Releasenotesfor1.7

I believe that QE will be done by the community on this feature.

Comment 3 Bryan Kearney 2014-11-20 13:52:03 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/7163 has been closed
-------------
Anonymous
Applied in changeset commit:04c6b0197e93f6c8a616c8695260bcd01563d044.

Comment 7 Sachin Ghai 2015-03-11 11:23:58 UTC

Can someone please help to share the steps to test this out ?

Comment 8 Sachin Ghai 2015-03-11 11:26:54 UTC

Created attachment 1000350 [details]
Assuming additional_info box will show the source with hostgroup name, as from where the value is coming

Comment 9 Sachin Ghai 2015-03-11 11:28:28 UTC

As per comment8, the info box shows the activation key's source name is sgroup to which the key associated first.  Is there anything else QE needs to validate ?

Comment 10 Sachin Ghai 2015-03-11 12:43:30 UTC

Thanks Bryan, I got the info from this PR: https://github.com/theforeman/foreman/pull/1667

So looks like its working as expected. As per comment 8, I can see the hostgroup name.

Verified with:
sat6.1 beta snap5
Satellite-6.1.0-RHEL-7-20150303.0

Comment 11 Bryan Kearney 2015-08-11 13:22:13 UTC

This bug is slated to be released with Satellite 6.1.

Comment 12 errata-xmlrpc 2015-08-12 05:19:32 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.

https://access.redhat.com/errata/RHSA-2015:1592