Feature #3531

Dashboard should display the sources of inherited parameters/groups/classes/nodes

Added by Ken Barber about 2 years ago. Updated about 1 year ago.

Status:Closed Start date:04/11/2010
Priority:Normal Due date:
Assignee:Nick Lewis % Done:

0%
Category:-
Target version:1.1.0
Keywords:inheritance, roadmapped Affected URL:
Branch:http://github.com/nicklewis/puppet-dashboard/tree/feature/next/3531 Affected Dashboard version:
Votes: 0

Description

In the case of nodes and groups, it would be nice to see where an inherited attribute/class was derived from. This would save a user from having to click around to figure it out themselves.

For example:

Node “foo” belongs to group “a”. In group “a” an attribute “fruit” was set to “banana”, so in turn node “foo” gains attribute “fruit=banana”.

In dashboard when viewing node “foo” it would be nice to see that group “a” provided “fruit=banana”.

(and a similar scenario would exist for a group that belonged to anther group).

Related issues

related to Puppet Dashboard - Feature #5187: Display Improvements Closed 11/02/2010
related to Puppet Dashboard - Bug #3438: Node show page should display "inherited classes" correct... Accepted 03/26/2010
duplicated by Puppet Dashboard - Feature #4190: Group show page's node listing should display nodes that ... Duplicate 07/09/2010
blocks Puppet Dashboard - Bug #4676: Parameters on views should be sorted Accepted 09/01/2010
precedes Puppet Dashboard - Bug #4601: Tables should render consistently across browsers and pages Closed 08/24/2010
precedes Puppet Dashboard - Bug #4661: Groups and classes should be sorted by name rather than id Closed 08/30/2010

History

#1
Updated by Jeff McCune almost 2 years ago

This falls under the node classification work and relates to #2933.

#2
Updated by James Turnbull almost 2 years ago

  • Status changed from Unreviewed to Accepted
  • Target version set to 1.2.0

#3
Updated by Nick Lewis almost 2 years ago

  • Target version changed from 1.2.0 to 1.0.5

#4
Updated by Nick Lewis almost 2 years ago

  • Assignee set to Nick Lewis

#5
Updated by Nick Lewis almost 2 years ago

  • Subject changed from Display the group which provided an inherited value for a node/group to Dashboard should display the sources of inherited parameters/groups/classes/nodes

Generalizing this ticket to cover parameters, groups, classes, and nodes, thus obviating #4190.

#6
Updated by Nick Lewis almost 2 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Assignee changed from Nick Lewis to Igal Koshevoy
  • Branch set to http://github.com/nicklewis/puppet-dashboard/commits/feature-3531-show-inheritance-paths

This patch set includes significant changes to the UI with regard to viewing the inheritance of params, etc, and a fairly large refactor of the helper methods on many of the models to accomodate the changes. Those refactors may need to be refactored again in the future to improve their structure and organization.

#7
Updated by Igal Koshevoy almost 2 years ago

Nick: Looks great a glance, thanks for working on this tricky feature. I’ll do a thorough review after v1.0.4 is shipped.

#8
Updated by Nick Lewis over 1 year ago

I’ve pushed a new version of this branch to fix one broken template.

#9
Updated by Igal Koshevoy over 1 year ago

Note to self: It may be a good idea to review this sooner and include this in the v1.0.4 release.

#10
Updated by Jacob Helwig over 1 year ago

  • Branch changed from http://github.com/nicklewis/puppet-dashboard/commits/feature-3531-show-inheritance-paths to http://github.com/jhelwig/puppet-dashboard/commits/feature-3531-show-inheritance-paths

Nick and I have resolved the merge conflicts to merge this branch into the current next, and pushed it out to my GitHub repo (branch updated to reflect this).

#11
Updated by Igal Koshevoy over 1 year ago

  • Status changed from In Topic Branch Pending Review to Accepted
  • Assignee changed from Igal Koshevoy to Nick Lewis
  • Branch changed from http://github.com/jhelwig/puppet-dashboard/commits/feature-3531-show-inheritance-paths to http://github.com/igal/puppet-dashboard/commits/feature-3531-show-inheritance-paths

Nick: I’ve rebuilt this branch using just its related commits.

#12
Updated by Igal Koshevoy over 1 year ago

  • Branch changed from http://github.com/igal/puppet-dashboard/commits/feature-3531-show-inheritance-paths to http://github.com/igal/puppet-dashboard/tree/feature-3531-show-inheritance-paths_v2

Ooops, I had the wrong URL in the branch field. I’ve corrected it.

#13
Updated by Nick Lewis over 1 year ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Branch changed from http://github.com/igal/puppet-dashboard/tree/feature-3531-show-inheritance-paths_v2 to http://github.com/nicklewis/puppet-dashboard/tree/feature/next/3531

Here’s a rebased and improved branch.

#14
Updated by Nigel Kersten over 1 year ago

  • Keywords changed from inheritance to inheritance, roadmapped

#15
Updated by Nick Lewis over 1 year ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

Merged to next in commit:3c3362139d0a815fa86b5d2b14dbb700980d9e89.

#16
Updated by Matt Robinson over 1 year ago

  • Status changed from Merged - Pending Release to Closed

Merged into master at the end of iteration.

#17
Updated by James Turnbull about 1 year ago

  • Target version changed from 1.0.5 to 1.1.0

Also available in: Atom PDF