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.we...@proxmox.com>
> > > > > > > > Signed-off-by: Alexander Zeidler <a.zeid...@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') },
> > > >
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
