Re: [pve-devel] [PATCH manager v2 2/2] ha/groups: allow editing node priorities through UI

2016-10-06 Thread Thomas Lamprecht
On 10/06/2016 09:39 AM, Dominik Csapak wrote: both patches look good to me Ok, thanks, will resend v3 with a few jslint warnings addressed. On 10/05/2016 11:09 AM, Thomas Lamprecht wrote: This allows the setting and editing node priorities in HA groups. Also fixes the bug where the priorit

Re: [pve-devel] [PATCH manager v2 2/2] ha/groups: allow editing node priorities through UI

2016-10-06 Thread Dominik Csapak
both patches look good to me On 10/05/2016 11:09 AM, Thomas Lamprecht wrote: This allows the setting and editing node priorities in HA groups. Also fixes the bug where the priorities of an existing group (e.g. set through the CLI) where deleted when editing said group through the GUI. Signed-o

[pve-devel] [PATCH manager v2 2/2] ha/groups: allow editing node priorities through UI

2016-10-05 Thread Thomas Lamprecht
This allows the setting and editing node priorities in HA groups. Also fixes the bug where the priorities of an existing group (e.g. set through the CLI) where deleted when editing said group through the GUI. Signed-off-by: Thomas Lamprecht --- changes since v1: * use getWidgetRecord() instead