Re: [pve-devel] [PATCH manager] ui: NodeSelector: check status field for offline nodes

2019-05-10 Thread Dominik Csapak
On 5/9/19 6:32 PM, Thomas Lamprecht wrote: On 5/8/19 11:12 AM, Dominik Csapak wrote: the memory stat is not available if the user has no audit privilges on /nodes/NODENAME but the status is always available but couldn't it be argued that this is desired behaviour? Or is it out of consistency,

Re: [pve-devel] [PATCH manager] ui: NodeSelector: check status field for offline nodes

2019-05-09 Thread Thomas Lamprecht
On 5/8/19 11:12 AM, Dominik Csapak wrote: > the memory stat is not available if the user has no audit privilges on > /nodes/NODENAME but the status is always available but couldn't it be argued that this is desired behaviour? Or is it out of consistency, as the node shows up all elsewhere or? >

[pve-devel] [PATCH manager] ui: NodeSelector: check status field for offline nodes

2019-05-08 Thread Dominik Csapak
the memory stat is not available if the user has no audit privilges on /nodes/NODENAME but the status is always available also it makes simple more sense to check if it is online not only if the memory value is there Signed-off-by: Dominik Csapak --- www/manager6/form/NodeSelector.js | 2 +- 1