Re: [pve-devel] [PATCH manager 2/9] report: add `jobs.cfg` to debug related network/load/backup/etc issues

[Alexander Zeidler <a.zeidler@proxmox.com>]

On Mon, 2024-03-25 at 09:08 +0100, Thomas Lamprecht wrote:
> > > > On 22/03/2024 14:59, Alexander Zeidler wrote:
> > > > > > > > Suggested-by: Friedrich Weber <f.weber@proxmox.com>
> > > > > > > > Signed-off-by: Alexander Zeidler <a.zeidler@proxmox.com>
> > > > > > > > ---
> > > > > > > >  PVE/Report.pm | 1 +
> > > > > > > >  1 file changed, 1 insertion(+)
> > > > > > > > 
> > > > > > > > diff --git a/PVE/Report.pm b/PVE/Report.pm
> > > > > > > > index d4191769..6014f13e 100644
> > > > > > > > --- a/PVE/Report.pm
> > > > > > > > +++ b/PVE/Report.pm
> > > > > > > > @@ -35,6 +35,7 @@ my $init_report_cmds = sub {
> > > > > > > >  		'pveversion --verbose',
> > > > > > > >  		'cat /etc/hosts',
> > > > > > > >  		'pvesubscription get',
> > > > > > > > +		'cat /etc/pve/jobs.cfg',
> > > > 
> > > > This does not really fits the general section, and makes it harder to
> > > > see the core subscription and repository state. I'd either add it to the
> > > > cluster one, as cluster-wide jobs config, or to a new "Jobs" section,
> > > > with a slight preference to the latter. 

A new "jobs" section would indeed be more appropriate.

> > > > 
> > > > > > > >  		'cat /etc/apt/sources.list',
> > > > > > > >  		sub { dir2text('/etc/apt/sources.list.d/', '.+\.list') },
> > > > > > > >  		sub { dir2text('/etc/apt/sources.list.d/', '.+\.sources') },
> > > >