Bug 1064289

Summary: No help for cluster policy parameters
Product: [Retired] oVirt Reporter: Eli Mesika <emesika>
Component: ovirt-engine-webadminAssignee: Jiri Moskovcak <jmoskovc>
Status: CLOSED NOTABUG QA Contact: bugs <bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.4CC: acathrow, dfediuck, ecohen, emesika, gklein, iheim, jmoskovc, mgoldboi, yeylon
Target Milestone: ---   
Target Release: 3.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: sla
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-02-24 08:37:07 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:
Attachments:
Description Flags
Screenshot none

Description Eli Mesika 2014-02-12 11:10:41 UTC 
Description of problem:
No help for cluster policy parameters 
When you play with the Cluster Policy numbers you see that they are dependent on each other but you got no help how to set correct values  

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


How reproducible:


Steps to Reproduce:
1.Have a Cluster and edit its policy to be VM_Evenly_Distributed 
2.You will see by default properties 
  HighVmCount 10
  MigrationThreshold 5
  SPMVMGrace 5
3.Try to change values to

  HighVmCount 3
  MigrationThreshold 1
  SPMVMGrace 1

4. Press OK
5. You will get MigrationThreshold and SPMVMGrace colored as is it has wrong value without any other explanation (see attached screen-shot)



Actual results:
No help on how to set the value or readable error message 

Expected results:
Help for the properties and the valid values or at least a detailed error message 

Additional info:
Comment 1 Eli Mesika 2014-02-12 11:13:05 UTC 
Created attachment 862204 [details]
Screenshot
Comment 2 Itamar Heim 2014-02-16 08:23:32 UTC 
Setting target release to current version for consideration and review. please
do not push non-RFE bugs to an undefined target release to make sure bugs are
reviewed for relevancy, fix, closure, etc.
Comment 3 Jiri Moskovcak 2014-02-19 13:35:51 UTC 
There is a popup if you leave the mouse cursor on the highlighted entry box. It's the same as for other policies. Is that enough or do we want something fancier?
Comment 4 Eli Mesika 2014-02-20 13:09:23 UTC 
(In reply to Jiri Moskovcak from comment #3)
> There is a popup if you leave the mouse cursor on the highlighted entry box.
> It's the same as for other policies. Is that enough or do we want something
> fancier?

This popup says nothing about the parameters (just displaying the same parameter name ...) and their relations to other parameters , I had found that not only that I don't know what the parameters means but also I do not know how other parameters are affected by changing its value ....
Comment 5 Jiri Moskovcak 2014-02-21 08:22:06 UTC 
The parameters are described on the feature page http://www.ovirt.org/Features/Even_VM_Count_Distribution. If there is a request to add it to gui, we'll need to involve UI designer.
Comment 6 Eli Mesika 2014-02-22 22:18:01 UTC 
Please see the BZ description , main problem here is that the values are dependent and if you put 'wrong' values you got no clue how to fix that apart of a red rectangle on the values fields ...
Comment 7 Jiri Moskovcak 2014-02-24 08:36:39 UTC 
(In reply to Eli Mesika from comment #6)
> Please see the BZ description , main problem here is that the values are
> dependent and if you put 'wrong' values you got no clue how to fix that
> apart of a red rectangle on the values fields ...

No, the allowed values for the field doesn't depend on values in other fields. In your case the problem is, that the values for MigrationThreshold and SpmVmGrace can't be lower than 5 which is clear from the regexp shown as the help text:

  "HighVmCount" : "^([0-9][0-9]*)$",
  "MigrationThreshold" : "^([5-9][0-9]*)$",
  "SpmVmGrace":"^([5-9][0-9]*)$"