[PATCH manager v2 05/19] ui: form/CRSOptions: allow auto rebalancing only for static and dynamic mode

[Daniel Kral <d.kral@proxmox.com>]

The HA auto rebalancing system can only be used with the static and
dynamic load usage information, so constrain the web interface
accordingly.

Signed-off-by: Daniel Kral <d.kral@proxmox.com>
Reviewed-by: Michael Köppl <m.koeppl@proxmox.com>
Tested-by: Michael Köppl <m.koeppl@proxmox.com>
---
changes v1 -> v2:
- none

I stuck with not removing the contents from the disabled fields as we're
not really consistent whether we do this, but most of the time let it
stay in the fields. Also user would be rather frustrated if their
previously set values would be removed if they changed the scheduling
mode from 'static'/'dynamic' to 'basic'. Another idea is to always
preserve these values, even though this doesn't clue in the users what
configuration is valid in the end. But let's see what users think.

 www/manager6/form/CRSOptions.js | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/www/manager6/form/CRSOptions.js b/www/manager6/form/CRSOptions.js
index 3f3b4fea..3b539fda 100644
--- a/www/manager6/form/CRSOptions.js
+++ b/www/manager6/form/CRSOptions.js
@@ -19,10 +19,14 @@ Ext.define('PVE.form.CRSOptions', {
 
     viewModel: {
         data: {
+            crsMode: '__default__',
             autoRebalancing: 0,
         },
         formulas: {
-            autoRebalancingDisabled: (get) => !get('autoRebalancing'),
+            canUseAutoRebalancing: (get) =>
+                get('crsMode') === 'static' || get('crsMode') === 'dynamic',
+            autoRebalancingDisabled: (get) =>
+                !get('autoRebalancing') || !get('canUseAutoRebalancing'),
         },
     },
 
@@ -42,14 +46,15 @@ Ext.define('PVE.form.CRSOptions', {
                     name: 'ha',
                     fieldLabel: gettext('Scheduling Mode'),
                     deleteEmpty: false,
-                    value: '__default__',
                     comboItems: [
                         ['__default__', Proxmox.Utils.defaultText + ' (basic)'],
                         ['basic', gettext('Basic (Resource Count)')],
                         ['static', gettext('Static Load')],
                         ['dynamic', gettext('Dynamic Load')],
                     ],
-                    defaultValue: '__default__',
+                    bind: {
+                        value: '{crsMode}',
+                    },
                 },
                 {
                     xtype: 'proxmoxcheckbox',
@@ -67,6 +72,7 @@ Ext.define('PVE.form.CRSOptions', {
                     boxLabel: gettext('Automatically rebalance HA resources'),
                     bind: {
                         value: '{autoRebalancing}',
+                        disabled: '{!canUseAutoRebalancing}',
                     },
                 },
                 {
-- 
2.47.3