From b2ca930a79013a6cd195c3a234f80aba64b83415 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Fabian=20Gr=C3=BCnbichler?= Date: Fri, 26 Feb 2016 09:17:48 +0100 Subject: [PATCH] Refactor has_feature backup_only is currently not used as snapshot backups are handled by qemu, but this makes the signatures identical to LXC.pm's has_feature. --- PVE/QemuServer.pm | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 0f7c485b..504294c6 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -5964,7 +5964,8 @@ sub snapshot_prepare { if defined($conf->{snapshots}->{$snapname}); my $storecfg = PVE::Storage::config(); - die "snapshot feature is not available\n" if !has_feature('snapshot', $conf, $storecfg); + die "snapshot feature is not available\n" + if !has_feature('snapshot', $conf, $storecfg, undef, undef, $snapname eq 'vzdump'); $snap = $conf->{snapshots}->{$snapname} = {}; @@ -6367,13 +6368,14 @@ sub snapshot_delete { } sub has_feature { - my ($feature, $conf, $storecfg, $snapname, $running) = @_; + my ($feature, $conf, $storecfg, $snapname, $running, $backup_only) = @_; my $err; foreach_drive($conf, sub { my ($ds, $drive) = @_; return if drive_is_cdrom($drive); + return if $backup_only && !$drive->{backup}; my $volid = $drive->{file}; $err = 1 if !PVE::Storage::volume_has_feature($storecfg, $feature, $volid, $snapname, $running); }); -- 2.39.5