cleanup mp size parameter (add units, same code as QemuServer)

bump version to 0.9-17

mount_all: make sure rootfs exists

avoid warning about undefined value

bump version to 0.9-16

Don't die in vm_shutdown when we have no errors

This fixes bugzilla entry: https://bugzilla.proxmox.com/show_bug.cgi?id=707

CT protection mode added

bump version to 0.9-15

resize: add bash completion support

support resizing of owned container disks

fix block device access

-) The condition was apparently to ignore /dev/ paths while
   that's actually what it was supposed to handle... (other
   paths aren't devices...).
-) Get rid of the blockdevices_list heuristics, it doesn't
   work reliably for all types of devices.
-) Check whether a device is a block device via S_ISBLK on
   the device file.
-) Don't try to follow symlinks as the name we provide in
   the mp config is the name we want in the container.
-) Blacklisted loop devices as they pose a security risk.

get rid of most of the loop-devices code

-) loop devices are now attached in mountpoint_mount, and
immediately detached in order to set the auto-clear flag

Keeping track of loop-devices is otherweise next to
impossible and a security concern.
We mount the filesystems for the container. We do not
support full loop device access for containers for a simple
reason: once a container detached a loop device, the
startup of another container might reuse it, exposing its
devices to the first container, generating unwatned cross
container access permissions.

Loop devices are also set to auto-clear, so that we do not
need to worry about detaching them when stopping the
container.

fix unlock handling after migration.

this will handle if the migration was successful or not.
this is important, because this determines where the config is.

try to generate files atomic (all or nothing)

use complete_migration_target from PVE::Cluster

add the capability to unlock a Container if it is locked

check is mounted before try to unmount disk.

this is neccessay because otherwise when a storage is not mounted, like when it is offline, the migration crash.

create_disk: correctly pass nomp parameter to  print_ct_mountpoint

pct exec: fix bash completion

move create_disks to PVE::LXC to avoid cyclic dependency

add support to add mountpoints with pct set

made create_disk non-private
made print_ct_mountpoint include 'mp=' unless 'nomp' is set
added an mkdirs flag to mount_all
create mount directories for now mountpoints simply by
  calling mount_all followed by umount_all

create_rootfs: revert previous change and always call umount_all

Because we also want to unmount mounts which do not use loop devices,
for example bind mounts, subvols, /dev/xyz, ...

cleanup coding style

format_disk: cleanups, never try to format subvolumes

fix coding style

disk formatting and mounting changed

-) format disks after vdisk_alloc instead of in mount_all
-) make mount_all return the loop device list so it can be
   passed to umount_all optionally
-) umount_all takes an optional loopdevs list to avoid
   listing loopdevs when not necessary

use run_cli instead of run (name changed in pve-common)

use PVE::Cluster::complete_next_vmid

auto-generate bash completion config

pct: use separate class for CLIHandler (src/PVE/CLI/pct.pm)

do not generate pod files for lxc hooks

added pct exec

Using the new 'extra-args' options for command arguments.

Examples:
 # pct exec 100 ls /
 # pct exec 100 -- ls -l /

Convert to disksize to GB after reading the config file

This fixes a bug when calling pct restore a vzdump backup:
we store the rootfs size in KB, but create_disks expects that a GB parameter.
So the restore was failing trying to create something of a Petabyte image.

Formatting '/var/lib/vz/images/110/vm-110-disk-1.raw', fmt=raw size=4503599627370496
unable to create image: qemu-img: /var/lib/vz/images/110/vm-110-disk-1.raw: The image size is too large for file format 'raw'

Enhance pct man page with DESCRIPTION, EXAMPLES, FILES, SEE ALSO sections

typo fixup: dettach -> detach

add missing archlinux ostype confdesc enum entry

fix some warnings shown during 'make dinstall'

destroy owned mountpoints on pct destroy

lxc-stop fails when 'kill' and 'timeout' options are used together

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

add bash completion support

use multiple mount protection (mmp)

normalize mount path

mount_all: unly call umount on error

fixed call for cfg file and error when CT is running and on local storage

When trying to migrate only throw an error when the CT is on an non
shared storage AND running.
Also fixed an remaining PVE::OpenVZ call to LXC.

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

Removing dead code from CT stop call and fixing typo

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

create_rootfs: correctly deactivate volumes (fix previous commit)

umount_all: fix previous commit

add offline migration code

skip lxc hooks on non-PVE containers

Our containers all only have a number as name. If any other
character appears in a container name, skip our hooks in
order to allow lxc to be used manually without interference
from PVE.

use loop device for all 'raw' formated disks on directory based storage

lxc detect error message changed

underscore is a valid lxc config key character

implement mount_all/umount_all

parse_ct_mountpoint: allow to parse "--rootfs 0"

cleanup error message

HA resources check before destroying lxc

bump version to 0.9-14

run spiceterm and return spiceproxy config only if CT is running

This mirrors the behavior of qemu VMs, if the container is not
running, return an error when trying to launch the spiceproxy
API call.

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

vzdump: correctly implement snapshot backup

use normal snapshot code to generate snapshots.

merge volid_path into mountpount_mount

It is easier to handle special cases like zfs subvol mounts this way.
Also pass snapshot names to enable mounting of snapshots (required
by vzdump)

lock_aquire: fix reference counting and lock name

vzdump: use correct pid to access container root

has_feature: skip further tests if we already detected an error

has_feature: check all mountpoints

cleanup: remove useless PVE::LXC prefix

create_rootfs: force rootfs mp to '/'

restore: do not restore mpX, arch and ostype options from previous config

create_disks: removed default_storage parameter

create: do not pass mointpoints to update_pct_config, simplify restore logic

create_disks: correctly protect code with eval

The previous code simply ignores errors.

vm_create : add support for mountpoints

rename alloc_roofs with create_disks,
to handle multiple disks.

$size param is removed, and must be provide like for qemuserver:

example:
pct create vmid template -rootfs local:5 -mp0 local:10

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

Correct package name in sub update_ipconfig()

allow to load configs from CTs located on other nodes

This allows load_config to load also configs from LXC container
which arent't located on the same node.

This is needed to fix the bug that doesn't let view the noVNC
console from an CT located on another node, whereas viewing it on
the same node as the container is located works fine.

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

remove tests for /dev/xxx and use get_vm_volumes

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

add get_vm_volumes sub

similar to qemu-server, return only volids with storeid

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

update_lxc_config : remove foreach_mountpoint

- we only define rootfs.
- also add support for /dev/.. and directory as rootfs

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

white space cleanups

volid_path : test device and local directory

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

vzdump: always store config inside $task->{tmpdir}

Which may be different than $task->{snapdir}. We avoid
to modify the original filesystem (which can be a
readonly snapshot).

fix bug #691 part1: fix vzdump stop mode backup on zfs

mountpoint_mount: die if something fails

code cleanup

vzdump: always check if we have a valid rootfs volid

do not pass 'noerr' flags to parse_volume_id

vzdump: fail if we have mountpoints

code cleanup

bump version to 0.9-13

detect ostype when restoring OpenVZ containers

Fixes #692

Typo in create_rootfs() subroutine

bump version to 0.9-12

mountpoint_mount: remove unused parameter $ms

make create_rootfs sub generic

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

volid_path: don't format $path with loop:/

This is only used in lxc config and not a real path

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

add mountpoint_mount sub

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

add dettach_loops sub

an move code from vm_stop_cleanup inside it

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

add attach_loops sub

and move code from prestart hook inside it

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

mount hook : code cleanup

- use sub path
- manage bind mount for non devices
- split mount and device_cgroup  (we'll need mount later for backup for example)

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

update_lxc_config : code_clenaup

- use volid_path sub
- we now manage only rootfs, subvol mount.entry will be manage in mount hook

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

add volid_path

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

implement pct listsnapshot cli command

split PVE::API2::LXC into smaller parts