move_vm: make it work with PVE::QemuServer:::clone_disk

- use POST instead of PUT (this is not idempotent).
- return task ID
- use declarative permission check

add qm move (storage migration)

qm move <vmid> <disk> <storage> --format [raw|qcow2|vmdk]

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

allow to clone VM when name is not set

factor out clone_disk

As suggested by Alexandre. Also cleanup  qemu_drive_mirror (fix white space).

bump version to 3.0-16

qm rescan: do not add aliases as unused disks.

fix unused disk handling

Show unused disks even if disk is used inside snapshots. But do not allow
to remove those disks.

rescan: add check if config has really changed

fix bug 395: correctly handle unused disk with storage alias

bump version to 3.0-15

qemu_img_format : use raw for as default for other storage (nexenta, sheepdog, ...)

Seem that something have change in qemu.

iscsi qemu block driver need to use raw, or we get a segfault. (like rbd).

sheepdog works also with .raw, so we don't need anymore specific storage format, we can use raw.

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

bump version to 3.0-14

fix bug #389: avoid error if balloon is undefined

bump version to 3.0-13

fix bug #391 - restore: test if requested format is supported

bump version to 3.0-12

use add_vm_to_pool/remove_vm_from_pool from PVE::AccessControl

bump version to 3.0-11

clone disk : keep source volume params

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

clone: check is we can clone to target storage

bump version to 3.0-10

fix bug #381: use PVE::Tools::next_migrate_port()

bump version to 3.0-9

restore: do not restore template flag

A restored VM is never a template. Else we would need to create
base images after the restore, and not all storages supports base
images. Anyways, the user can easily convert the restored VM into a
template later.

bump version to 3.0-8

qemu_img_format: use 'raw' for lvm

With host_device we get the following error:

qemu-img: error while writing sector 0: Bad file descriptor

drive-mirror : die if stats are empty.

If drive have bad sectors, the block job die.
we need to die if stats are empty to avoid this:

transferred: 21440086016 bytes remaining: 34668544 bytes total: 21474754560 bytes progression: 99.84 %
Use of uninitialized value $transferred in subtraction (-) at /usr/share/perl5/PVE/QemuServer.pm line 4611.
Use of uninitialized value $total in subtraction (-) at /usr/share/perl5/PVE/QemuServer.pm line 4611.
Use of uninitialized value $transferred in multiplication (*) at /usr/share/perl5/PVE/QemuServer.pm line 4612.
Use of uninitialized value $total in division (/) at /usr/share/perl5/PVE/QemuServer.pm line 4612.
clone failed: mirroring error: Illegal division by zero at /usr/share/perl5/PVE/QemuServer.pm line 4612.

Maybe it should be improved by catching qmp events, but doesn't seem to work for now

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

set long timeout for query-block-jobs

Like we do for other block job query commands. Seems those commands
can hangs for some time.

bump version to 3.0-7

has_features: also return a list of allowed nodes

This is useful if a feature is restricted to a limited set of nodes.

has_feature: add 'copy' to enum

create template: fix permission check

This function only works if the VM already exists. But pool permissions
are automatically inherited on existing VMs, so there is no need
to explicitly check pool permissions.

clone: correctly add VM to pool

fix typo, whitespace cleanups

vm_clone : add running vm drive mirror

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

add qemu_drive_mirror

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

bump version to 3.0-6

restore: allow to overwrite existing VMs if user has VM.Backup permissions

The requirement to have VM.Allocate it non-obvious and confusing
if the VM already exists. If the user can backup, he should also be able
to restore that backup.

remane copy to clone

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>