public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Shannon Sterz <s.sterz@proxmox.com>, Lukas Wagner <l.wagner@proxmox.com>
Cc: Proxmox Datacenter Manager development discussion
	<pdm-devel@lists.proxmox.com>
Subject: Re: [pdm-devel] [PATCH datacenter-manager 2/3] ui: make NodeStatusPanel available as a widget
Date: Fri, 7 Nov 2025 09:55:07 +0100	[thread overview]
Message-ID: <dda978b0-dbf5-4998-a8fd-e3bb8610c0a0@proxmox.com> (raw)
In-Reply-To: <DE2BR7PG3BE2.3MAYBBJKA7G8Z@proxmox.com>

Am 07.11.25 um 09:37 schrieb Shannon Sterz:
> On Fri Nov 7, 2025 at 8:31 AM CET, Lukas Wagner wrote:
>> - The widget appears a bit too dense/crowded, at least compared to the
>>   other widgets that we already have. I think this mostly due to the
>>   three buttons, as well as the long kernel version string, on my PDM
>>   dev host this is
>>
>> 	  Linux 6.14.8-2-pve #1 SMP PREEMPT_DYNAMIC PMX 6.14.8-2 (2025-07-22T10:04Z)
>>
>>   which appears to be quite a bit longer than what we show in PVE/PBS:
>>
>> 	  Linux 6.14.11-4-pve (2025-10-10T08:04Z)
> 
> just a small note, this is how seem to display this information
> throughout pdm at least, so i'd keep that for now.

Fine for now, but I'd prefer to align this to the new variant, which PVE
adopted two years ago. For most users the pre-emption mode of their kernel
is really useless detail (there are many other kconfig options that have
a similar impact), and frankly I do not really get why it's included in
uname, but not our thing to change - displaying only the most relevant
here should be done in any case (but can be a follow-up to sync the look
over all places).

https://git.proxmox.com/?p=pve-manager.git;a=commitdiff;h=be04f8ee8af61eaa3730ae74e6e3c7d4266474ca

[...]

> alright i'll make the fingerprint opt-in and revert to the tab
> based approach from v1 then.

thx!


_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel


  reply	other threads:[~2025-11-07  8:54 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-06 12:43 [pdm-devel] [PATCH datacenter-manager/proxmox/yew-comp 0/6] add node status panel to proxmox datacenter manager Shannon Sterz
2025-11-06 12:43 ` [pdm-devel] [PATCH proxmox 1/1] node-status: add node status crate Shannon Sterz
2025-11-06 19:41   ` [pdm-devel] applied: " Thomas Lamprecht
2025-11-06 12:43 ` [pdm-devel] [PATCH yew-comp 1/2] node info: extend NodeStatus enum to include NodeStatus from proxmox-rs Shannon Sterz
2025-11-06 20:44   ` [pdm-devel] applied: " Thomas Lamprecht
2025-11-06 12:43 ` [pdm-devel] [PATCH yew-comp 2/2] node status panel: add a panel that show the current status of a node Shannon Sterz
2025-11-06 12:43 ` [pdm-devel] [PATCH datacenter-manager 1/3] api-types/api: add endpoints for querying the node's status Shannon Sterz
2025-11-06 20:57   ` [pdm-devel] applied: " Thomas Lamprecht
2025-11-06 12:43 ` [pdm-devel] [PATCH datacenter-manager 2/3] ui: make NodeStatusPanel available as a widget Shannon Sterz
2025-11-06 21:10   ` Thomas Lamprecht
2025-11-07  7:31     ` Lukas Wagner
2025-11-07  8:37       ` Shannon Sterz
2025-11-07  8:55         ` Thomas Lamprecht [this message]
2025-11-06 12:43 ` [pdm-devel] [PATCH datacenter-manager 3/3] nodes: remove unnecessary rustfmt::skip macro Shannon Sterz
2025-11-06 20:57   ` [pdm-devel] applied: " Thomas Lamprecht
2025-11-06 12:44 ` [pdm-devel] [PATCH datacenter-manager/proxmox/yew-comp 0/6] add node status panel to proxmox datacenter manager Shannon Sterz

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=dda978b0-dbf5-4998-a8fd-e3bb8610c0a0@proxmox.com \
    --to=t.lamprecht@proxmox.com \
    --cc=l.wagner@proxmox.com \
    --cc=pdm-devel@lists.proxmox.com \
    --cc=s.sterz@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
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal