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

bump version to 0.9-11

remove <vmid>.fw file after lxc vm destroy

Related to Bugfix 603: vmid.fw file not deleted

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

cleanup directory structure

add file access methods to LXCSetup::Base

These should be used for all container file access as in the
future they'll support user-id mapping.

mountpoint: add support for host /dev/xxx block device passthrough

example:
mp0: volume=/dev/sda,mp=/home/

Fix this request:
http://forum.proxmox.com/threads/23256-LXC-Cannot-assign-a-block-device-to-container

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

Added PVE::LXCSetup::Base::setup_securetty

archlinux template support

mark mp[N] as experimental feature

check quorum inside pre-start hook

move setup_mountpoint into mount hook

I want to keep that code private.

cleanup docs

use standard code inside post stop hook

use standard code inside pre-start hook

no need to use those hacks.

mount hook : setup_mountpoints

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

add lxc.mount.entry for subvol

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

add pve-prestart-hook and move activate volumes inside it

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

add lxc-pve-poststop-hook and move vm_stop_cleanup

It's better to vm_stop_cleanup inside poststop hook,
in case of CT failure (wrong start for example)

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

vm_stop_cleanup : detach loopdevices for non rootfs mountpoint

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

vm_start : attach loop devices for non rootfs mountpoint

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

do not allow hotplug of cmode option

Hotplug does not work because we use dtach.

add find_loopdev sub

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

add blockdevices_list sub

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

add loopdevices_list sub

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

implement new cmode option

So that the user can choose to use /dev/ttyX or /dev/console or /bin/sh.

foreach_mountpoint: loop with defined order

deactivate all mountpoint at ct stop

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