qm: Add VMID auto completion to some commands

Add the respective completion function for the:
*) start
*) stop
*) shutdown
*) suspend
*) resume
*) sendkey
qm commands. The destroy command was left out on purpose for now.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

fix bug #828: activate disks before hotplugging them

update changelog

restore firewall config from backup

firewall config will now include in the backup and write to vma file.

update changelog

reenable steal time

we have fixed it in kernel last month
https://www.mail-archive.com/pve-devel@pve.proxmox.com/msg14072.html

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

bump version to 4.0-39

fix bug #783: set KillMode=none, so that systemd don't kill them at shutdown

pve-bridge-hotplug code deduplication

bump version to 4.0-38

Don't treat serial devices as a local resource if they point to a socket.

Close: https://bugzilla.proxmox.com/show_bug.cgi?id=470

qemu_img_convert: activate source volume

For example, this is required if we copy from lvmthin snapshots.

improve complete_storage, only show storages which have the capability to run qemu images.

destroy VM: test for running VM before starting background task

bump version to 4.0.37

add pve-bridge-hotplug script

use it for nic hotplug, because pve-bridge script will
not work after a live migration, because of the PVE_MIGRATED_FROM env var.

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

nic hotplug : use pxe roms for older machinetype

we want to avoid mix some nics with pxe rom and other with efi rom.

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

add qemu_use_old_bios_files sub

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

add qemu_machine_pxe

return machinename with .pxe suffix if a nic with pxe romfile exist

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

bump version to 4.0-36

use qom-get to check if pxe file are used V2

fix qemu 2.4 pxe -> qemu 2.4 efi

Changelog : forget to add a check on qom-get result

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

migration: improve ipv6 case

Qemu parses hostnames in brackets correctly but sets an ipv6
flag for them as if they were ipv6 addresses, only insert
brackets for ipv6 addresses.

bump version to 4.0-35

clone: use a fullclone hash instead of $drive->{full}

'full' is not a valid drive property which are now being
checked in print_property_string. Since it's only a
temporary value and passed to clone_disk as parameter it
can be stored separately.

bump version to 4.0-34

Revert "fix bug 790: encode description before writing to syslog"

This reverts commit 9ac5db408db2dc2380add3090bdd9e04f97e576c.

This is now fixed by a change in PVE::Cluster::log_msg().

update changelog

fix bug 790: encode description before writing to syslog

bump version to 4.0-33

simplify code: consistently use old bios files for older machine types

Don't create unused empty /etc/qemu-server directory

bump version to 4.0-32

use old netdevice bios files for older machine types

bump version to 4.0-31

migrate : add nocheck for resume

Users have reported resume bug when HA is used.

They seem to have a little race (bench show >0s < 1s) between the vm conf file move on source node and replication to,
and resume on target node.

I don't known why this is only with HA, maybe this occur will standard migration too.

Anyway, we don't need to read the vm config file to resume the vm on target host,
as we are sure that the vm is migrated, and config file move action is correct in the cluster.

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

bump version to 4.0-30

convert qmrestore into a PVE::CLI class

and install bash completion helpers.

create: add better check for unused IDs

bump version to 4.0-29

support serial numbers and models for disks

Note that the model is only supported for IDE harddisks via
the -device option, whereas the serial is part of the -drive
option.

Closes #153

bump version to 4.0-28

disable kvm_steal_time

It's currently buggy with live migration

https://bugs.launchpad.net/qemu/+bug/1494350
Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 4.0-27

boot order permissions change

to change to boot order it is needed to assign a user the
VM.Config.Disk privilege set with the add/modify/delete disk
permissions -> now the patch allows the boot order to be (re)defined
with the VM.Config.Options set - the modify any other VM configuration

bump version to 4.0-26

migration: disable compress

it's already disable by default,
but we want to be sure if it's change in later release

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

enable xbzrle

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

fix kvm version parser for CVE stable releases, bump version to 4.0-25

bump version to 4.0-24

VM protection mode

[PATCH v2] changes:

- changed call also for remove VM

pci passthough : make vfio default

and remove old legacy code.
(deprecated and disabled by default since kernel 4.2)

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

code cleanups

VM protection mode

changes:

- added common check_protection subroutine
- disk removal prevention
- restore over protected VM prevention
- changed man page message

add qm.bash-completion to .gitignore

fix error message: allocate to much v-CPUs.

bump version to 4.0-23

avoid warning about undefined value

bump version to 4.0-22

add possibility to restore backup on rbd in krbd mode

add krbd support to online snapshot

bump version to 4.0-21

fix install path

remove generated qm.bash-completion on make clean

improve bash completion

implement a few bash completion helpers

covert qm into a PVE::CLI class, enable bash completions

VM protection mode added

used to prevent an unintended virtual machine remove operation

v3 changes:

- changed man page message
- removed protection parameter (where not needed)

fix move_disk on RBD

activating the disk when use RBD is necessary to have the given path.

cleanup previous patch

HA resources check before destroying vm

bump version to 4.0-20

fix: start kvm with os type other

this check is necessary, because we do not set ostype in qemu config when type is other

Seem that x2apic is now enabled by default when kvm is used

since this commit
http://git.qemu.org/?p=qemu.git;a=commit;h=ef02ef5f4536dba090b12360a6c862ef0e57e3bc

So we just need to disable it for solaris

fix 'uninitialized value in concat' due to unnamed VM

This is an minor fix which let's you start the spice console on an
unnamed VM without getting the 'Use of uninitialized value in
concatenation' error.
Also changes the trailing comma from the $conf definition lineto an
semicolon.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 4.0-19

clone vm firewall config file

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>

bump version to 4.0-18

fix bug #688: if vm is not owner of this disk remove from config

fix bug #688: prevent copy unused disks in the config.

bump version to 4.0-17

fix bug #603: vmid.fw file not deleted

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>

fix bug #517: improve error message

Get a understandble message,
if someone try to increase a qcow2 image where one or more snapshots reside

adapt /config and /pending API calls to force-delete

bump version to 4.0-16

remove vm access permissions after destroy

removed duplicated code which was used in
PVE::AccessControl -> now in PVE::API2::Qemu

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>

merge delete_drive into try_deallocate_drive

It used to be private and is only used once.

pending-delete: remember force-deletes

The -force flag didn't have any effect since the pending
changes didn't carry over the the flag.
Now forced deletes have an exclamation mark prepended to the
option name.

correctly handle empty description in pending section

So that we can delete descriptions with

 qm set <vmid> --descr ''

bump version to 4.0-15

Fixed wrong UUID in Qemu VZDump backup

As the format of the uuid which the qemu monitor returned changed,
it is here adapted to fix errors with not matching uuids.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add memory_unplug support V2

qemu 2.4 feature

changelog: rebase on last git

Note that currently linux guest don't support unplug of dimm when it'sused by kernel memory.
They are some tunning to do with memory zone movable.
http://events.linuxfoundation.org/sites/events/files/lcjp13_chen.pdf

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

bump version to 4.0-14

fixed bug 662, wrong subroutine for parsing startup order

Changed from old, now missing, subroutine parse_startup() to new
pve_parse_startup_order() in qemu-server and pve-manager

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

cpuflags : don't enforce with tcg mode

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

cpuflags : remove enforce for cpumodel=host

Currently enforce with cpumodel=host on amd cpus don't work,
because amd cpus have unsupported flags in qemu.

This is a protection, and this is good.

but cpumodel host should be never use by users for production (only for testing).
For production and stability, users need to choose a true cpu model which filter
the supported cpuflags by qemu.

So I think we can remove the enforce for host model as for testing it's ok.

warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 0]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 1]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 2]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 3]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 4]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 5]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 6]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 7]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 8]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 9]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 12]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 13]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 14]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 15]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 16]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 17]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 23]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 24]
Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

cpuflags : remove -rdtscp for Opteron cpu models

rdtscp is not supported by qemu and with enforce it's not starting
warning: host doesn't support requested feature: CPUID.80000001H:EDX.rdtscp [bit 27]

from to qemu wiki
http://wiki.qemu.org/Features/CPUModels#Disabling_features_that_were_always_disabled_on_KVM
"Fact: currently libvirt runs CPU models having rdtscp without the "enforce" flag, and rdtscp is silently disabled
Consequence: libvirt SHOULD use something like "-cpu Opteron_G5,-rdtscp",
especially when it starts using (or emulating) enforce mode
This will require a solution on libvirt side. QEMU will just provide the mechanisms to report CPU model information
and check what the host and QEMU supports, but the decision to disable rdtscp to be able
to run Opteron_G[2345] needs to be taken by libvirt."

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

vzdump : abord backup if disk have iothread enabled.

Currently backup don't work with iothread feature, and crash qemu

For now, abord the backup if one of the vm drives have iothread enabled until backup code is fixed.

Upstream qemu backup cde already support iothread.
http://git.qemu.org/?p=qemu.git;a=commit;h=761731b1805f6ef64eb615e5b82a0801db3cde78

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

bump version to 4.0-13

qemu-server : drive-mirror : allow to interrupts at the scanning bitmap phase

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

qemuserver : vm_status : add extended stats (disks, nics, memory) V3

Add extended stats results for each nics,disks and memory on full stats mode only.

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