# Note about locking: we use flock on the config file protect
@@ -3777,7 +3779,7 @@ sub qemu_volume_snapshot {
my $running = check_running($vmid);
-return if !PVE::Storage::volume_snapshot($storecfg, $volid, $snap,
$running);
+PVE::Storage::volume_snapshot($storecfg,
It is better to check if a VM is running in QemuServer then in Storage.
for the Storage there is no difference if it is running or not.
Signed-off-by: Wolfgang Link w.l...@proxmox.com
---
PVE/Storage.pm | 4 ++--
PVE/Storage/ISCSIDirectPlugin.pm | 2 +-
PVE/Storage/LVMPlugin.pm
Hi all@list,
I'm think we should check if we make snapshots not at the Storage.
IMHO it should be done at the QemuServer.
Is there any matter to kept it?
___
pve-devel mailing list
pve-devel@pve.proxmox.com
It is better to check if a VM is running in QemuServer then in Storage.
for the Storage there is no difference if it is running or not.
Signed-off-by: Wolfgang Link w.l...@proxmox.com
---
PVE/QemuServer.pm | 21 -
1 file changed, 20 insertions(+), 1 deletion(-)
diff --git
---
www/manager5/Readme.md | 22 ++
1 file changed, 22 insertions(+)
create mode 100644 www/manager5/Readme.md
diff --git a/www/manager5/Readme.md b/www/manager5/Readme.md
new file mode 100644
index 000..87e00c3
--- /dev/null
+++ b/www/manager5/Readme.md
@@ -0,0 +1,22 @@
+ PVE::Storage::volume_snapshot($storecfg, $volid, $snap) if
storage_support_snapshop($volid, $storecfg);
This seem to be wrong.
we can't do a qcow2 snapshot with qemu-img if the vm is running, we need to
use qmp in this case.
Form what I see this case is already handled inside
Form what I see this case is already handled inside
storage_support_snapshop()?
PVE::Storage::volume_snapshot($storecfg, $volid, $snap) if
storage_support_snapshop($volid, $storecfg);
return if !$running;
vm_mon_cmd($vmid, snapshot-drive, device = $deviceid, name = $snap);
and
Couldn't we reuse volume_has_feature from storage plugins ?
We can, but this is misleading. It is not a storage feature, it
is a feature of qemu.
___
pve-devel mailing list
pve-devel@pve.proxmox.com
So, with this code, we always call qemu-img snasphot for qcow2, which is wrong
if the vm is running.
(for rbd,zfs,.. no problem).
I think that's why we passed the running flag to pve-storage.
Ah, yes - the logic is wrong, but fixable.
___
We need to use qemu snapshot-drive if the VM is running?
For qcow2 file yes.
found some reference here:
https://www.suse.com/documentation/sles11/book_kvm/data/cha_qemu_guest_inst_qemu-img.html
WARNING: Do not create or delete virtual machine snapshots with the qemu-img
snapshot command while
Also,
+ PVE::Storage::volume_snapshot($storecfg, $volid, $snap) if
storage_support_snapshop($volid, $storecfg);
This seem to be wrong.
we can't do a qcow2 snapshot with qemu-img if the vm is running, we need to use
qmp in this case.
- Mail original -
De: Wolfgang Link
So, with this code, we always call qemu-img snasphot for qcow2, which is
wrong
if the vm is running.
(for rbd,zfs,.. no problem).
I think that's why we passed the running flag to pve-storage.
Ah, yes - the logic is wrong, but fixable.
Or is it the other way around? We need to use
dpkg -i lxc-pve-dev_1.1.1-1_amd64.deb
Setting up lxc-pve (1.1.1-1) ...
usermod: invalid option -- 'v'
It's hanging in lxc-pve.postinst
# create subuid/subgui map for root
# (to run unprivileged containers as root)
usermod -v 10-165535 -w 10-165535 root
Hi,
does it make sense to define
+my $snap_storage = {zfspool = 1, rbd = 1, zfs = 1, sheepdog = 1};
..
sub storage_support_snapshot()
in qemuserver ?
Couldn't we reuse volume_has_feature from storage plugins ?
- Mail original -
De: Wolfgang Link w.l...@proxmox.com
À: pve-devel
I am currently testing with the latest release.
you talk about pvetest for wheezy?
___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
(about upgrade, we need also to install
apt-get install systemd-sysv
to get systemd working after reboot)
Ah, OK.
___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
hi,
I am currently testing with the latest release.
2 points:
- At the end of the update with pvetest, it propose to replace it with
pvetest pve - enterprise.
- A problem of installation dependencies ceph hammer :
The following packages have unmet dependencies:
ceph : Depends:
pvetest for jessie
___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
do you plan to update jessie pvetest repo soon ?
I would like to test upgrade from wheezy with new packages version and lxc
Yes, I can update if you want.
___
pve-devel mailing list
pve-devel@pve.proxmox.com
do you plan to update jessie pvetest repo soon ?
I would like to test upgrade from wheezy with new packages version and lxc
Yes, I can update if you want.
OK, the repository is up to date now - please test.
___
pve-devel mailing list
On April 30, 2015 at 8:09 PM Moula BADJI moul...@hotmail.com wrote:
- A problem of installation dependencies ceph hammer :
There are still no ceph packages for jessie, sorry.
One question: to make a new HA-PVE, a small howto ?
Sorry, there is currently no docu.
Old system? This package is targeted for debian jessie.
Yes, (it was the upgraded wheezy, seem that passwd was not upgraded correctly)
(about upgrade, we need also to install
apt-get install systemd-sysv
to get systemd working after reboot)
- Mail original -
De: dietmar
Seem that passwd was not updated to jessie.
Works fine now
- Mail original -
De: aderumier aderum...@odiso.com
À: pve-devel pve-devel@pve.proxmox.com
Envoyé: Jeudi 30 Avril 2015 15:01:28
Objet: [pve-devel] trying to install lxc-pve, usermod error in
lxc-pve.postinst
dpkg -i
#usermod -v 10-165535 -w 10-165535 root
usermod: invalid option -- 'v'
Old system? This package is targeted for debian jessie.
___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
BTW,
do you plan to update jessie pvetest repo soon ?
I would like to test upgrade from wheezy with new packages version and lxc
- Mail original -
De: aderumier aderum...@odiso.com
À: dietmar diet...@proxmox.com
Cc: pve-devel pve-devel@pve.proxmox.com
Envoyé: Jeudi 30 Avril 2015
25 matches
Mail list logo