drop mkdirs flag on mount_all

It was only used in update_pct_config and isn't required
as mountpoint_mount already creates the directory anyway.

bump version to 0.9-21

vzdump: use the new run_command variant

Use the array-of-array version of run_command to build the
pipe, this should deal with most quoting issues.
Note that tar handles glob patterns in --exclude itself, so
quoting patterns instead of letting the shell resolve them
is also actually more correct.

vzdump: lxc find/mp update 2: archive

To void at least some weird quoting issues, and since tar
has a --one-file-system option, always skips sockets and
also supports exclusion by pattern we now simply use tar
directly instead of passing files listed by 'find'.

vzdump: lxc find/mp update 1: rsync

change rsync_vm to use the new exclude pattern method and
deal with mountpoints

vzdump: replace diskinfo with disks array

replace the diskinfo data coming only from the rootfs with
a disks array and sanitize mountpoints

cleanup

-) whitespace fixup
-) remember storecfg hash value in a variable

rename some $mountpoint vars to $rootdir

disambiguate the two terms

bump version to 0.9-20

fix bug #712: fix stop mode backups when container is running

add support for LVM

add changelog

mountpoint_mount: return info if mount uses loop device

This simplifies code inside update_lxc_config().

vzdump: remove debug statement

bump version to 4.0-19

allow older debian versions (>= 4 (etch))

update changelog

format_disk: call activate_volume()

bump version to 0.9-18

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