check has_feature copy for full clone

copy_vm: re-structure code to allow more checks before fork

fix : signal interrupt don't delete volume on full copy

Currently we  push newvolid to newvollist after qemu-img convert,
so if signal interrupt occur during qemu-img convert, the newvollist is empty and we can't free the volume

Instead, We need to push newvolid to newvollist just after volume creation

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

fix "Use of uninitialized value $target"

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

white-space cleanups

copy_vm: new option to move final VM to other node (option target)

This only works if the VM is on shared storage.

vm_copy : add optional format parameter

can be qcow2,raw,vmdk

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

copy_vm: add name and description option

copy_vm: allow to copy from snapshot

copy_vm: use exclusive lock for running VM

Because we can only run one block job to copy data.

copy_vm : add optional storage parameter

Optionnal target storage for full copy

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

don't send qmp balloon commands if vm is started with a state file

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

parse_drive : return correct format

Currently format is always empty, we need to parse file extension

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

vm_copy : fix typo in description

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

implement copy_vm

This is based on patches from Alexandre Derumier <aderumier@odiso.com>.

It just implements copy/clone for stopped VMs and Templates. Copying
running VMs or copy from snapshots is not implemented.

add qemu_img_convert

also work with snapshot as source for qcow2,rbd,sheepdog.

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

implement shared file locks

and add a first prototype for copy_vm

bump version to 3.0-4

use correct package prefix

fix assertion

move assertion so that we have all checks at the same place.

set template flag earlier

do not allow template creation if there are snapshots.

fix wrong comment

template_create : add template:1 option.

Also allow template create of non base-xxx renamable storage

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

use vm create permissions for templates

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

fix bug 377: make qm rescan work properly

is_template : return 1 if template option == 1

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

add template config option

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

avoid endless loop in QMPClient

do not delete unmatched content from $$input

bump version to 3.0-3

allow sparse restore for sheepdog and rbd

bump version to 3.0-2

parse_net: fix warning about uninitialized value

add better checks to test if backup was successful

virtio disk hot-unplug: drive_del after device_del

we need to remove drive after device has correctly by remove.

With this patch:

Windows : if drive is locked/"mounted", the hot-unplug fail and disk is accessible (100% safe)

Linux: without hotplug module : the hot-unplug fail and disk is accessible (100% safe)

Linux : with hotplug module : the hot-unplug work, but no verification is done is the drive is mounted

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

nic hot-unplug: delete netdev after device removal

Currently we delete netdev before acpi hot-unplug.

if guest don't support hotplug, this break network.

We need to remove netdev after the device had be correctly unplug.(we check that device has been correctly removed)
So, If guest don't support hotplug, It's simply thrown a unplug error message without breaking the network

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

dynamic bridge rate/vlan change on nic update

if model change : we try to unplug like before
if bridge/nat change : we try to unplug like before

else if bridge, but rate/vlan/bridge change, we dynamicly change them without need hotplug

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

pve-bridge : move tap_plug code in PVE::Network::tap_plug

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

pve-bridge : move tap creation code in PVE::Network::tap_create

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

pve-bridge : move rate limit code to PVE::Network:tap_rate_limit

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

start 3.0 development

bump version to 3.0-1

fix backup-cancel timeout

bump version to 2.3-16

qemu-server: vncproxy: qm: wait max 10s for the socket if it does not exist

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

bump version to 2.3-15

Revert "vncproxy: wait max 10s for the socket if it does not exist"

This reverts commit 58504d5edfe531ace87c40258e1b15e1a9f21c06.

The patch only works if the VM is on local host.

vzdump: improve error reporting

And use kill(9) instead of kill(-9), which simply hangs

fix changelog

vncproxy: wait max 10s for the socket if it does not exist

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

fix check if a backing file exist

bump version to 2.3-14

bugfix #340 : don't set cache=none to cdrom

Cdrom devices don't works with cache=none and no reason to force cache mode on cdrom

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

default cpu type is kvm64 now

migration : display qm resume error in task log

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

forbid online resize a virtio windows bootdisk

Currently is give us bsod, it's a windows virtio-blk driver bug.

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 2.3-13

set cache=none if option is not defined

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

usb-tablet : always use uhci controller

- usb-tablet on ehci is buggy
- convert to -device syntax

info usb before
----------------
  Device 0.2, Port 1, Speed 12 Mb/s, Product QEMU USB Tablet

info usb after
--------------
  Device 0.2, Port 1, Speed 12 Mb/s, Product QEMU USB Tablet

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

uhci controller : use new -device syntax

we need an id to allow hotplug

info pci before
---------------
  Bus  0, device   1, function 2:
    USB controller: PCI device 8086:7020
      IRQ 11.
      BAR4: I/O at 0xc040 [0xc05f].
      id ""

info pci after
--------------
  Bus  0, device   1, function 2:
    USB controller: PCI device 8086:7020
      IRQ 11.
      BAR4: I/O at 0xc040 [0xc05f].
      id "uhci"

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

fix changelog

Revert "enable usb2 controller by default"

This reverts commit 42869ac07d4b899c7f6d09c7e0b2a0a6552246bb.

disable hotplug by default

This reverts commit 6c52b6794e10fb5e4bdad2b417b60496afc067a5

Testing revealed that this has too many side effect, so we do not want
to change that for the 2.3 release.

add offline backup for templates

fix backup parameters for pve-qemu-kvm 1.4-4

bump version to 2.3-11

depend on pve-qemu-kvm 1.4-4

fix tablet hotplug

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 2.3-10

hotplug/unplug devices by default

hot-unplug : add some warnings for differents guest os

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

qemu_netdevadd : convert to qmp

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

qemu_netdevdel : convert to qmp

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

qemu_deviceadd : convert to qmp

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

qemu_devicedel : convert to qmp

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

hotplug/unplug tablet on config update

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

enable usb2 controller by default

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

vzdump : check if volid exist with volume_size_info

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

template_create : check has_feature template

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

add missing newline

remove wrong permission check

template_create: check for 'clone' feature

is_template: always return a defined value

use 0 instead of undef

vmstatus : return template if vm is a template

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

move template checks up one level

To avoid multiple/repeated calls to load_config().

forbid snapshot create if current it's a template

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

forbid offline migration of a non shared volume if it's a clone

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

forbid vm_start if current config is a template.

if files (raw,qcow2) are a template, we forbid vm_start.

note : the readonly protection do already the job, but we need a clear message for users

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

forbid rollback if current config is a template.

if a qcow2 current is a template, we can't rollback to a previous snapshot.

(note that file readonly protection do already the job, but we need a clear error message for user)

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

forbid snapshot delete if it's a template

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

add is_template sub

return 1 if vm is a full template (all disks are base image)
return 2 if vm is a semi-tempalte (mix of base and non-base image)

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

add template_create

qm template <vmid> [-disk virtio0]

convert a full vm to a template (or only a disk if specify)

we orignal disk to /base (file) or base- (lvm,rbd,sheepdog,nexenta)
we create a snapshot @base if storage need it for clone
we protect the volume or snapshot

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

do not set cache=none for .raw files

use qemu default instead (writeback)

remove expected_downtime from migration status

bump version to 2.3-9

depend on pve-qemu-kvm >= 1.4-1

qemu 1.4 fix : rename stats-polling-interval to guest-stats-polling-interval

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

fix tar restore: correctly check if VM config already exists

allow to suspend/resume VM during backup

bump version to 2.3-7

cancel backup before stopping the vm

bump version ro 2.3-6

allow to run as qemu-system-x86_64

This is usefull to debug.

fix bug #307: correctly restore disk settings

bump version to 2.3-5