public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH manager] fix #4335: report: add datacenter.cfg to output
@ 2023-02-07 15:34 Max Carrara
  2023-02-08 14:21 ` Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Max Carrara @ 2023-02-07 15:34 UTC (permalink / raw)
  To: pve-devel, m.carrara

Simply includes the contents of /etc/pve/datacenter.cfg
in the general system info section.

Signed-off-by: Max Carrara <m.carrara@proxmox.com>
---
 PVE/Report.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/PVE/Report.pm b/PVE/Report.pm
index 90b7cb1c..f40e84f3 100644
--- a/PVE/Report.pm
+++ b/PVE/Report.pm
@@ -37,6 +37,7 @@ my $init_report_cmds = sub {
 		sub { dir2text('/etc/apt/sources.list.d/', '.*sources') },
 		'lscpu',
 		'pvesh get /cluster/resources --type node --output-format=yaml',
+		'cat /etc/pve/datacenter.cfg',
 	    ],
 	},
 	'system-load' => {
-- 
2.30.2





^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [pve-devel] [PATCH manager] fix #4335: report: add datacenter.cfg to output
  2023-02-07 15:34 [pve-devel] [PATCH manager] fix #4335: report: add datacenter.cfg to output Max Carrara
@ 2023-02-08 14:21 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2023-02-08 14:21 UTC (permalink / raw)
  To: Proxmox VE development discussion, Max Carrara

Am 07/02/2023 um 16:34 schrieb Max Carrara:
> Simply includes the contents of /etc/pve/datacenter.cfg
> in the general system info section.

Looks ok, albeit it could be slightly more fitting in the cluster section, but
no hard feelings from my side - maybe get the opinion of devs currently manning
the enterprise support team if it fits general and is often relevant so should be
near the top, or if it might crowd other info more there and cluster would be
a better fit after all.




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-02-08 14:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-07 15:34 [pve-devel] [PATCH manager] fix #4335: report: add datacenter.cfg to output Max Carrara
2023-02-08 14:21 ` Thomas Lamprecht

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