Re: [PATCH datacenter-manager v2 7/9] ui: pve: show ha maintenance mode for nodes

[Dominik Csapak <d.csapak@proxmox.com>]

On 8/20/26 1:07 PM, Lukas Wagner wrote:
> On Tue Aug 18, 2026 at 3:25 PM CEST, Dominik Csapak wrote:
>> By rendering a small `Badge` after the nodename in the pve tree and the
>> resource tree, when the node is in ha maintenance mode.
>>
>> I opted for showing the ha state here differently than in PVE, because
>> seeing the online/offline state separately from the hastate can make
>> sense (so there is no ambiguity if the node is online or offline).
>>
>> To do this we have to add the hastate to the `PveNodeResource` and wire
>> that through from the /cluster/resources call.
>>
>> We also have to update the pwt-assets submodule to get the necessary CSS
>> classes for the badge.
>>
>> This partially fixes #7371.
>>
>> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
>> ---
>>   lib/pdm-api-types/src/resource.rs |  3 +++
>>   server/src/api/resources.rs       |  1 +
>>   ui/pwt-assets                     |  2 +-
>>   ui/src/pve/tree.rs                | 11 +++++++++--
>>   ui/src/renderer.rs                | 14 +++++++++++++-
>>   ui/src/widget/resource_tree.rs    |  3 ++-
>>   6 files changed, 29 insertions(+), 5 deletions(-)
>>
>> diff --git a/lib/pdm-api-types/src/resource.rs b/lib/pdm-api-types/src/resource.rs
>> index c433a284..288d8543 100644
>> --- a/lib/pdm-api-types/src/resource.rs
>> +++ b/lib/pdm-api-types/src/resource.rs
>> @@ -490,6 +490,9 @@ pub struct PveNodeResource {
>>       pub status: String,
>>       /// Subscription level
>>       pub level: String,
>> +    /// HA State
>> +    #[serde(default)]
>> +    pub hastate: String,
> 
> I wonder, could we use a proper type here? E.g. something like
> 
> enum HaState {
>      Maintenance,
>      ...,
>      Unknown
> }

sure, I'd like that too. I just adapted to the code around this^^
(e.g. the level and status field are similar strings)

> 
> I'd also avoid carrying over these 'word amalgamations' and rather call
> it at least 'ha_state' (serialized to 'ha-state'), but not a biggie for
> me.
> 
>>   }
> 

sure, i guess i was too fixated on keeping the fields the same
as the pve ones