Feature #12829

Separate Smart variables and Smart parameters in Host/HostGroup -> Parameters tab.

Added by Jitendra Yejare over 2 years ago. Updated 5 months ago.

Status:Closed
Priority:Low
Assigned To:Shira Maximov
Category:Smart Variables
Target version:Team Ivan backlog
Difficulty: Bugzilla link:1291730
Found in release: Pull request:https://github.com/theforeman/foreman/pull/5125
Story points-
Velocity based estimate-
Release1.17.0Release relationshipAuto

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1291730
Description of problem:
Separate Smart variables and Smart parameters for class/classes in Host/HostGroup -> Parameters tab.
Its very confusing there to identify which is variable and which is parameter.

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

How reproducible:

Steps to Reproduce:
1. Override a parameter value of class 'something' to display in Host/HostGroup -> Parameters tab.
2. Create a new Smart variable in class 'something' to display in Host/HostGroup -> Parameters tab.
3. Go to Host/HostGroup -> Parameters tab.

Actual results:
The smart variable and smart class parameters lists together, which is very difficult to understand which is what. At least when we will have large number of parameters and variables in class.

Expected results:

Additional info:

Associated revisions

Revision cb46abb7
Added by Shira Maximov 5 months ago

Fixes #12829 - add identifier for smart variables/parameters

History

#1 Updated by Dominic Cleal over 2 years ago

  • Category set to Smart Variables
  • Priority changed from Normal to Low

It doesn't really matter in this context. The point is that they are meant to be configured together.

#2 Updated by Ori Rabin 5 months ago

  • Assigned To set to Shira Maximov
  • Status changed from New to Assigned

#3 Updated by Ori Rabin 5 months ago

  • Target version set to Team Ivan backlog

#4 Updated by The Foreman Bot 5 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5125 added

#5 Updated by Tomer Brisker 5 months ago

  • Release set to 1.17.0

#6 Updated by Shira Maximov 5 months ago

  • % Done changed from 0 to 100
  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF