proxmox-mirrors/pve-manager
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-manager synced 2025-08-14 03:57:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

pve7to8: avoid confusing warning about required setting 'storage' for vzdump

Browse Source 
It's required in the schema for notes-template and protected, but when
parsing vzdump.conf, it shouldn't matter whether the storage parameter
is set or not.

The warning is ugly and users might interpret it as something that
needs to be acted upon for the upgrade:
parse error in '/etc/vzdump.conf' - 'storage': missing property - 'notes-template' requires this property\nmissing property - 'protected' requires this property

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
(cherry picked from commit 517abd0cd2)
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Fiona Ebner 2023-06-21 17:02:01 +02:00 committed by Thomas Lamprecht
parent e63cea8930
commit daf74a20fd
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
PVE/CLI/pve7to8.pm
View File

@ -614,6 +614,8 @@ sub check_backup_retention_settings {
eval {
my $confdesc = PVE::VZDump::Common::get_confdesc();
# vzdump.conf by itself doesn't need to honor any 'requires'
delete $confdesc->{$_}->{requires} for keys $confdesc->%*;
my $fn = "/etc/vzdump.conf";
my $raw = PVE::Tools::file_get_contents($fn);