update_vm_async: new asynchronous API

snapshot rollback: use pc-i440fx-1.4 as default

require VM.Config.HWType priviledges to modify 'machine' configuration

bump version to 3.0-19

migrate: pass --machine parameter to remote 'qm start' command

snapshot: save/restore 'machine' configuration

implement get_current_qemu_machine

API extension: add 'machine' parameter to vm_start

This allows us to overwrite the machine type at start - useful for
migration to make sure we use the same machine at the target node.

add new 'machine' configuration

fix typo

qemu_drive_mirror: correctly call block-job-cancel if block-job-complete fails

bump version to 3.0-18

move_disk: implement delete flag

rename move to move_disk

Because that API call can only move a single disk (not the whole VM).

bump version to 3.0-17

clone_disk: test if requested format is supported - else use default

move_vm: add original disk as unusedX

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