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

Michael Köppl
2026-04-13T16:01:08Z

On Thu Apr 9, 2026 at 1:41 PM CEST, Daniel Kral wrote:

[snip]

> @@ -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}',

not a big deal, but wanted to mention it nonetheless: if a user selects
e.g. static mode and enables auto rebalancing and then later switches to
basic mode, the checkbox remains checked visually, but the value is
still set to false once submitted. It's just a cosmetic issue, but
something I noticed during my testing.

> },
> },
> {