public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Alexander Zeidler <a.zeidler@proxmox.com>
To: Aaron Lauterer <a.lauterer@proxmox.com>,
	Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH manager 6/7] report: add info about (un)used memory slots
Date: Thu, 18 Apr 2024 15:32:52 +0200	[thread overview]
Message-ID: <571770cca597a266fb2ae76b1f5cfff41f03e4ed.camel@proxmox.com> (raw)
In-Reply-To: <4e5145ba-abdc-499c-a748-027a0306d773@proxmox.com>

On Thu, 2024-04-18 at 12:16 +0200, Aaron Lauterer wrote:
> I am not sure how often we actually need that information as it can add 
> quite a bit of additional lines in the report in larger machines with 
> many memory slots.

Good point. So in this case we could simply compact the output by counting, without loosing any information:

# dmidecode -t 16,17 | grep -P "^\t(Max[^:]*city|Size|Part)" | sed -Ez "s/\n(\tP[^\n]*)/\1/g" | sort | uniq -c
      1 	Maximum Capacity: 2 TB
      4 	Size: 16 GB	Part Number: 18ASF2G72PZ-2G6D1
      4 	Size: No Module Installed


> It might be better to keep that command in a cheatsheet to ask for it if 
> actually needed instead of polluting the report :)
> 
> On  2024-04-18  11:16, Alexander Zeidler wrote:
> > * to see if a RAM upgrade is slot/capacity-wise possible
> > * to spot added/replaced RAM that may now be causing issues
> > 
> > 	Maximum Capacity: 2 TB
> > 	Size: 16 GB	Part Number: 18ASF2G72PZ-2G6D1
> > 	Size: 16 GB	Part Number: 18ASF2G72PZ-2G6D1
> > 	Size: 16 GB	Part Number: 18ASF2G72PZ-2G6D1
> > 	Size: 16 GB	Part Number: 18ASF2G72PZ-2G6D1
> > 	Size: No Module Installed
> > 	Size: No Module Installed
> > 	Size: No Module Installed
> > 	Size: No Module Installed



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


  reply	other threads:[~2024-04-18 13:33 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-18  9:16 [pve-devel] [PATCH manager 1/7] report: add kernel command line from current boot Alexander Zeidler
2024-04-18  9:16 ` [pve-devel] [PATCH manager 2/7] report: create "jobs" section, add `jobs.cfg` Alexander Zeidler
2024-04-18 20:25   ` [pve-devel] applied: " Thomas Lamprecht
2024-04-18  9:16 ` [pve-devel] [PATCH manager 3/7] report: add `apt-cache policy` to list recognized APT sources Alexander Zeidler
2024-04-18 20:28   ` [pve-devel] applied: " Thomas Lamprecht
2024-04-18  9:16 ` [pve-devel] [PATCH manager 4/7] report: list held back packages Alexander Zeidler
2024-04-18 20:28   ` [pve-devel] applied: " Thomas Lamprecht
2024-04-18  9:16 ` [pve-devel] [PATCH manager 5/7] report: overhaul `dmidecode` related output Alexander Zeidler
2024-04-18 10:24   ` Mira Limbeck
2024-04-18 12:30     ` Alexander Zeidler
2024-04-18  9:16 ` [pve-devel] [PATCH manager 6/7] report: add info about (un)used memory slots Alexander Zeidler
2024-04-18 10:16   ` Aaron Lauterer
2024-04-18 13:32     ` Alexander Zeidler [this message]
2024-04-18 10:20   ` Mira Limbeck
2024-04-18 13:38     ` Alexander Zeidler
2024-04-18  9:16 ` [pve-devel] [PATCH manager 7/7] report: add recent boot timestamps which may show fencing/crash events Alexander Zeidler
2024-04-18 10:43   ` Mira Limbeck
2024-04-18 15:45     ` Alexander Zeidler
2024-04-19  8:56       ` Mira Limbeck
2024-04-18  9:19 ` [pve-devel] [PATCH manager 1/7] report: add kernel command line from current boot Alexander Zeidler
2024-04-18 10:16 ` Aaron Lauterer
2024-04-18 11:05 ` Mira Limbeck
2024-04-18 15:46   ` Alexander Zeidler
2024-04-18 20:25 ` [pve-devel] applied: " Thomas Lamprecht

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=571770cca597a266fb2ae76b1f5cfff41f03e4ed.camel@proxmox.com \
    --to=a.zeidler@proxmox.com \
    --cc=a.lauterer@proxmox.com \
    --cc=pve-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
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal