From: "Shannon Sterz" <s.sterz@proxmox.com>
To: "Lukas Wagner" <l.wagner@proxmox.com>
Cc: Proxmox Datacenter Manager development discussion
<pdm-devel@lists.proxmox.com>
Subject: Re: [pdm-devel] [PATCH datacenter-manager/yew-comp v3 0/3] add node status panel to proxmox datacenter manager
Date: Wed, 12 Nov 2025 15:34:19 +0100 [thread overview]
Message-ID: <DE6SH3XQ8DCT.2EM67A8BAQ2AF@proxmox.com> (raw)
In-Reply-To: <DE6SAQOIM6IK.2QRLKSZV6W8GV@proxmox.com>
On Wed Nov 12, 2025 at 3:26 PM CET, Lukas Wagner wrote:
> On Mon Nov 10, 2025 at 9:58 AM CET, Shannon Sterz wrote:
>> this series slightly adapts the previously applied node status panel and
>> integrates it into pdm. the first commit changes the way we display
>> kernel version, dropping the preempt mode and similar information to get
>> this more aligned with pve/pbs. the second patches removes and adapts
>> the tools of the NodeStatusPanel to make it less crowded. the final
>> commit adds a new tab to the "Administration" menu that shows the
>> current node status and allows for shutting down or rebotting the node.
>>
>
> Gave this a quick test on the latest master branches.
>
> Noticed that where was a missing dependency for `proxmox-node-status` in
> ui/Cargo.toml.
>
> We discussed off-list how this new node status view should look like
> once there are resource graphs. I raised the concern that the "Reboot"
> and "Shutdown" buttons look a bit out-of-place in the current iteration.
> We agreed to try out a 'card-like' view, similar to the new firewall
> view or the global update overview, with the Shutdown and Reboot buttons
> on the top right of the card.
> The graphs should then later be on the same card; this makes it visually
> consistent with the PVE/PBS status view cards. Whether this single card
> should have borders or not is up for debate. I think it looks quite
> nice, but I understand that it also wastes a bit of space.
thanks, i'll send a v4 with the new layout in a minute.
_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel
prev parent reply other threads:[~2025-11-12 14:34 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-10 8:58 Shannon Sterz
2025-11-10 8:58 ` [pdm-devel] [PATCH yew-comp v3 1/2] node_info: only use build date from kernel version Shannon Sterz
2025-11-10 8:58 ` [pdm-devel] [PATCH yew-comp v3 2/2] node status panel: remove power management and make fingerprint opt-in Shannon Sterz
2025-11-10 8:58 ` [pdm-devel] [PATCH datacenter-manager v3 1/1] ui: add a Node Status tab to the administration panel Shannon Sterz
2025-11-12 14:26 ` [pdm-devel] [PATCH datacenter-manager/yew-comp v3 0/3] add node status panel to proxmox datacenter manager Lukas Wagner
2025-11-12 14:34 ` Shannon Sterz [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DE6SH3XQ8DCT.2EM67A8BAQ2AF@proxmox.com \
--to=s.sterz@proxmox.com \
--cc=l.wagner@proxmox.com \
--cc=pdm-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox