[pve-devel] perhaps a bug in QemuConfig.pm?
datanom.net
mir at datanom.net
Tue Jun 6 13:52:00 CEST 2017
Hi all,
When creating a live snapshot this function is called:
sub __snapshot_save_vmstate {
my ($class, $vmid, $conf, $snapname, $storecfg) = @_;
my $snap = $conf->{snapshots}->{$snapname};
my $target;
# search shared storage first
PVE::QemuServer::foreach_writable_storage($conf, sub {
my ($sid) = @_;
my $scfg = PVE::Storage::storage_config($storecfg, $sid);
return if !$scfg->{shared};
$target = $sid if !$target || $scfg->{path}; # prefer file based
storage
});
if (!$target) {
# now search local storage
PVE::QemuServer::foreach_writable_storage($conf, sub {
my ($sid) = @_;
my $scfg = PVE::Storage::storage_config($storecfg, $sid);
return if $scfg->{shared};
$target = $sid if !$target || $scfg->{path}; # prefer file based
storage;
});
}
$target = 'local' if !$target;
[....]
It doesn't seem obvious to me why we prefer shared storage over local
storage?
Above assumes that we are able to store two snapshots for each VM/CT we
are making a live snapshot for on the share storage.
--
Hilsen/Regards
Michael Rasmussen
Get my public GnuPG keys:
michael <at> rasmussen <dot> cc
http://pgp.mit.edu:11371/pks/lookup?op=get&search=0xD3C9A00E
mir <at> datanom <dot> net
http://pgp.mit.edu:11371/pks/lookup?op=get&search=0xE501F51C
mir <at> miras <dot> org
http://pgp.mit.edu:11371/pks/lookup?op=get&search=0xE3E80917
--------------------------------------------------------------
----
This mail was virus scanned and spam checked before delivery.
This mail is also DKIM signed. See header dkim-signature.
More information about the pve-devel
mailing list