qemu_machine_pxe : return $machine if no pxe

Currently migration is broken, because qemu_machine_pxe return nothing if no pxe rom exist.

That mean that we don't pass -machine flag to migration, and migration is broken between qemu 2.4->2.5

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

Make snapshot_prepare and snapshot_commit proper subs

bump version to 4.0-58

deactivate volumes if vm start command fails

also for current master
cleanup of patch from Dhaussy Alexandre from 02/15/2016

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 4.0-57

rename $ost to $ostype

disable vnc server and add -nographic is no vga card is present

For passthrough, we need to be sure that card is primary,
so we need to disable vnc server

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

passthrough : reenable hyperv and add hv_vendor_id for windows

qemu 2.5 support a new hyper-v feature: hv_vendor_id
This allow nvidia drivers to install on windows with hyper-v feature on.

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

delete ram state files when restoring a backup

currently we leave orphaned vmstate files when we restore a
backup over a vm, which has snapshots with saved ram state.

this patch deletes those files on a restore.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

Refactor snapshot code

Split vmstate saving into its own sub in preparation of
LXC/QemuServer codebase refactoring.

bump version to 4.0-56

If we freeze the fs with the Qemu-Guest-Agent test if QGA is running.

update changelog

Drop skiplock from write_config

Since write_config was always called with skiplock=1 except
once, it makes sense to drop this parameter like in
PVE::LXC::write_config . If needed in the future, the
caller can use check_lock before write_config anyway.

Refactor update_config_nolock -> write_config

The method update_config wrapped update_config_nolock
using lock_config, but to prevent update races the whole
"read config", "do something", "write config" flow was
always protected by lock_config anyway, and update_config
was never called.

Thus, we can safely drop update_config and rename
update_config_nolock to write_config like in PVE::LXC .

bump version to 4.0-55

use better usb config parsing

since we want the usb3 option to be really boolean and not only
'usb3=yes', we have to change the usb json format a little

to not break existing configs for 'usbX: spice', we set the 'host'
option as non-optional and default_key and allow 'spice' as its
content (this also makes the option less ambiguous)

another side effect is that previously accepted multiple 'host='
entries are now forbidden

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 4.0-54

remove a superfluous condition

This is only reached if the $line from which $virtdev
originates matches, and the part in $virtdev can never be
false then.

restore: deal with new backup=0 property string

add usb3 option for usb-devices

adding a flag for usb devices (usb3), if this is set to yes,
add a xhci controller and attach the specified devices to it

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 4.0-53

pass $skiplock all the way through to destroy_vm

The API passes $skiplock to vm_destroy() which performed a
check conditionally depending on the $skiplock parameter and
then simply calls destroy_vm() inside lock_config() which
did yet another check_lock() without any way to avoid that.

Added the $skiplock parameter to destroy_vm() and removed
the conditional check in vm_destroy() as both happened after
locking the config.

disk hotplug : allow hotplug of physical host /dev/ disks

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

bump version to 4.0-52

close tunnel after migration is finish.

if we do not close it, there is a change that the tunnel stays open and the next migration will not work.

detect_zeroes is a boolean

bump version to 4.0-51

Fix #878: disk-size format

disk-size is a format, not a type

Fix #879: exclusion of disk for backup

With the change to a property string the backup and iothread
properties were changed from type string to type boolean and
need to be treated as such.

Create firewall dir on VM restore

update changelog

fix PVE::HA use clause so HA resources get registered

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

bump version to 4.0-50

add hidden option to cpu type

This passes kvm=off to qemu's -cpu switch to disable KVM
identification via the cpuid instruction.

update version

Use format raw if a disk is passthrough to VM.

To prevent error at VM starting, when we passthrough a harddrive from host to vm.

clone: correctly activate volumes (consider snapshots)

bump version to 4.0-48

use safe_string_ne for trunks

It's a list of numbers, not just one.

bump version to 4.0-47

add support for network trunks

This add support for net trunks vlan filtering
for ovs and linux vlan-aware bridge

Can be mixed with current "tag" option

examples:
----------

allow only 802.1Q packets with vlanid 2,3,4 :

netx: .....,trunks=2,3,4

allow only 802.1Q packets with vlanid 2,3,4 and tag non-802.1Q packets to vlanid 5 :

netx: tag=5,trunks=2,3,4

tag non-802.1Q packets to vlanid 5

netx: tag=5
Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

use property strings for drive options

use property string for smbios1

bump version to 4.0-46

Closes #787: add Haswell-noTSX and Broadwell-noTSX cpu types

bump version to 4.0-45

io throttle: pass pool parameters (*_max)

These have been passed but haven't been used.

bump version to 4.0-44

ovmf : don't pass x-vga to vfio-pci

x-vga vfio-pci flag is to enable seabios quirks only.

This patch keep using x-vga=on from proxmox config, to disable hyperv,kvm=off,vga=none by default
but don't pass x-vga to vfio-pci when ovmf is enabled.

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

check for qorum when starting a VM

bump version to 4.0-43

disable hyper-v enlightment when xvga is enabled

Geforce drivers (> 344.11) crash with code43 error if they detect hyper-v enlighment with pci passtthrough
http://awilliam.github.io/presentations/KVM-Forum-2014/#/5/3

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

cleanup: remove all generated .bash-completion files

bump version to 4.0-42

add detect_zeroes option

On some storages BLKZEROOUT commands do not work properly
and return without error while having no effect whatsoever.
This can produce various filesystem errors and thus needs
to be made optional.

A drive can now have 'detect_zeroes=off' to disable this
behavior. By default the behavior is the same as before:
always-on (and set to 'unmap' if discard is enabled).

fix bug #841: replace get_used_paths with is_volume_in_use

get_used_paths returned a hash of used paths for all the
volumes in a VM's config, which is not enough to figure out
whether there are snapshots, as snapshots often have
different paths.  Eg. on ZFS it is not enough to check for
/dev/zvol/tank/vm-123-disk-1 because the snapshot's path is
/dev/zvol/tank/vm-123-disk-1@snap1 and thus we allowed
deleting the drive. Then when trying to delete the snapshot
later you get:
  zfs error: cannot open 'tank/vm-751-disk-1': dataset does not exist
and it refuses to delete the snapshot.

Since its only use was to check whether or not a drive is
still in use it is now renamed to is_volume_in_use and
beside checking paths now also checks volume-ids as those
should stay the same.

bump version to 4.0-41

always copy OVMF_VARS-pure-efi.fd file

To avoid the impression that we can store BIOS settings. They also
gets lost when we migrate the VM.

introcude new 'bios' option (replace ovmf option)

add ovmf uefi roms support V2

changelog:

use OVMF-pure-efi.fd instead OVMF_CODE-pure-efi.fd to have virtio drivers support

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

bump version to 4.0-40

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.