Re: [PATCH datacenter-manager v4 00/10] refactor and partially fix #7371

["Lukas Wagner" <l.wagner@proxmox.com>]

On Wed Sep 23, 2026 at 3:17 PM CEST, Dominik Csapak wrote:
> the first 6 patches are pure refactors and should not change anything
> user visible, could be applied independently.
>
> patch 7,8,10 partially fix #7371 by showing the HA maintenance status
> of PVE nodes in various places.
>
> patch 9 is just a fix that came up  (thanks @lukas)
>
> chagnes from v3:
> * rebase on master
> * add serde skip_serialize_if for option none on the ha maintenance field
> * added new patch for sorting by memory usage correctly
>
> changes from v2:
> * add missing doc comments
> * refine wording a bit and make it more consistent (HA -> ha for example)
> * change from string 'hastate' to proper rust enum 'ha-state' in
>   PveNodeResource struct
>
> changes from v1:
> * fixed some display problems: NaN% when maxmem was set to 0.0,
>   0.0% cpu when the node was offline, fixed by not showing values for
>   offline nodes
> * add a new patch for showing tha badge too in the node card's title
> * various typo fixes
> * added #[serde(default)] to the new hastate field to allow
>   deserialization from older versions
> * some smaller fixes  (left over comments, naming, etc.)
>
>

Thanks for these patches!

One minor thing that I discovered while testing this series again:
The node name is now cut-off by the HA badge if the viewport is on the
smaller side (tested it with 1600px width, which is not *that* narrow).

Off-list, we already discussed this issue, we agreed that one solution
would be to move the guest tags to the same column as the name (same as
in PVE) and to remove the formerly dedicated tag column completely,
giving the name + tag + badge column a lot more space in total.

We agreed that removing the tag column would be a separate patch that
can be independently applied, and then this series can be rebased on top
of that.