honor acl setting with zfs

cleanup OVZ config recovery

$conf->{rootfs} is supposed to be the property string value,
not the parsed property string. since this method is called
only twice (once for retrieving the rootfs information only,
once for retrieving the config only) and the second call
never needs the 'rootfs' part of the configuration, we can
safely not set it instead of introducing ugly workarounds
(e.g. setting a fake volume path or worse).

Debian: correctly setup getty service on systemd based containers

File /etc/inittab does not exist on systemd based containers.

update changelog

code cleanup

push/pull permissions: use octal by default

fix #942: restore ACL and other rootfs options from backup

unless overridden by explicitly setting the rootfs
parameter, restoring from a backup will now copy the rootfs
properties from the backup archive, except for 'volume' and
'ro' (for obvious reasons).

skip some additional config options when restoring

-unusedX in all cases, even when creating
-parent when merging config from backup

don't keep old existing config when restoring

this breaks the expected ranking of configuration options
and does not make much sense in any case.

bump version to 1.0-61

add support for ubuntu xenial

bump version to 1.0-60

vm_status: return more verbose HA state

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

update changelog

cleanup previous patch: remove unused var

Fix #929: delete veths in the post-stop hook

Unfortunately it can still happen that LXC's network link
deletion netlink messages get dropped/ignored. This is the
same issue as initially reported on the forums by sigxcpu in
October, however, it seems that some users hit this problem
more reliably currently.

bump version to 1.0-59

setup: ability to ignore files

Make the ct_* file wrapper functions ignore files for which
a file named .pve-ignore.$name exists.

vzdump: fix redirection to stdout

update changelog

debian: support containers upgraded to use systemd

These otherwise spawn consoles at /dev/pts%I and cause
errors in the logs about the container-getty@ services.
This happens for instance when dist-upgrading from wheezy to
jessie.

symlink pct.conf.5 to ct.conf

fix max disk size in container status

we already get the bytes from parse_ct_rootfs (via JSONParser disk_size),
no need to multiply it again

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

regex cleanup

add missing defined check on validate_ssh_keys

use pve-doc-generator, bump version to 1.0-58

bump version to 1.0-57

Expose ssh_keys feature over API

this adds a new optional parameter to the create_vm API
method, using which public ssh keys can be setup for the
container at creation.

the ssh-public-keys parameter is mapped to a filepath for
pct using the new string->filepath mapping mechanism.

Add ssh key setup to create_rootfs

expose the new post_create_hook parameter in create_rootfs

Add setup tests for authorized keys

include ssh keys in previous test cases, add one extra test
case for a non-default $HOME read from /etc/passwd

Add authorized ssh key setup to post_create_hook

provide helpers to copy public ssh keys to a user's
$HOME/.ssh/authorized_keys file, creating directories and
files when needed.

Use these in post_create_hook to setup provided ssh keys for
the root user.

Add missing '\n' to content in ct_modify_file

if the provided content is not '\n'-terminated and non-empty,
add a trailing '\n', otherwise the END marker breaks.

Add 'perms' option to ct_modify_file

passing this option allows the caller to (re)set the file
permissions of the modified file.

bump version to 1.0-56

use consistent prefix for worker names

update changelog

Fix #925: activate volume before trying to resize

update changelog

enable quotas for passed through /dev devices

style fix

bump version to 1.0-55

Add read_password sub for 'pct create'

Read the container root password from stdin when creating a
container with 'pct create ... -password', instead of
providing it as command line argument. This is consistent
with 'pveum adduser' and pvesh, as described in #737 and #777.

use ascicode compatible markup

s/Note:/NOTE:/

bump version to 1.0-54

Fix #918: add /dev/mapper symlinks for dm-* devices

Mount canonicalizes paths unless the -c option is used. This
is mostly fine but for device-mapper nodes (/dev/dm-*) it'll
fetch the /dev/mapper/* path and pass that to the mount
system call resulting in /proc/mounts showing the
/dev/mapper path. This is neither the one we provided (since
we use /dev/$vg/$lv), nor the one userspace tools will find
in /dev currently.
Since the dm-* paths are rather inconvenient to look at we
decided to keep mount's behavior and compensate by providing
the /dev/mapper symlinks for devices via the autodev hook.

cleanup: avoid an -o '' empty-string option

cleanup: $mounted_dev is never set in this code path

This might avoid some confusion in the future...

autodev: create missing paths

bump version to 1.0-53

spelling/typo correction

Add force parameter for migration with bind/dev mp

Add a new 'force' parameter that allows to force the
migration of a container despite configured bind or device
mountpoints, which will be ignored/skipped.

Refactor content type check into sub

call create_disks after updating rootfs

this allows to set the rootfs to <storage>:<size>,
automatically creating an empty volume of the specified
size on the specified storage, like for non-rootfs mps.

Check content type when adding/updating volumes

the non-'rootdir' storages are filtered out in the web
interface already, but using the API/CLI it was still
possible to add volumes on storages without the 'rootdir'
content type.

this check is only used for mountpoints using our storage
backends, bind/dev mounts still work like before.

update changelog

require -experimental for pct clone and template

bump version to 1.0-52

Prevent race conditions in snapshot mode backup

Instead of dropping the 'backup' lock early on when doing
snapshot backups, drop it temporarily for snapshot
operations that set their own 'snapshot' lock, and protect
the "unlock_vm, snapshot_XX, lock_vm" sequence by holding an
flock for the config file.

Before this change it was possible to interfere with the
backup job by setting a different lock with another
operation inbetween the call to unlock_vm and
snapshot_create (or snapshot_delete).

The final lock_vm is re-introduced in order to be more
consistent with the other backup modes and to prevent
changes to the configuration file before assemble()
reloads the configuration that is included in the backup.

Use set_lock and remove_lock

instead of manually setting and removing the lock config
parameter.

update changelog

fix #912: centos/redhat confusion

With the introduction of unmanaged containers a check was
added to verify that the ostype is the same as the
autodetected one.
Since our CentOS plugin was named 'redhat' but the ostype
setting was 'centos' this error triggered and rendered
centos containers unusable.

Renamed the LXC::Setup::Redhat package LXC::Setup::CentOS
and changed the plugin name to and autodetected type from
'redhat' to 'centos' as well.

bump version to 1.0-51

implement cpulimit hotplugging

config: allow deleting cpuunits and cpulimit

These were simply missing.

update changelog

implement rate limiting

bump version to 1.0-50

Decide whether volume is added as unused in subclass

follow the change in PVE::AbstractConfig

list_active_containers: fix regex

allow \s+ as seperator

bump version to 1.0-49

change update_etc_hosts to use ct_modify_file

Setup::Base: more generic ct_modify_file function

Replaced ct_modify_file_head_portion() with ct_modify_file()
taking a header and and replacing the section in-place
rather than always moving it to the top of the file.

The behavior for when a section is not defined can be
defined via the provided options (replace, prepend, or
append).

Document the difference between stop and shutdown

bump version to 1.0-48

Move JSONFormat code to PVE::LXC::Config

Some of this should move into PVE::AbstractConfig soon.

Refactor mountpoint and general conf methods

Move add_unused_volume into abstract
pve-common/src/PVE/AbstractConfig.pm, because it is
identical for LXC and Qemu.

Move classify_mountpoint, is_volume_in_use, has_dev_console,
mountpoint_names, foreach_mountpoint_XX and get_vm_volumes
to PVE::LXC::Config because they only deal with config
related matters.

(Some of) the latter methods might get moved to or become
implementations of methods in PVE::AbstractConfig in the
future.

Rework snapshot code, has_feature

Move snapshot_create, snapshot_delete and snapshot_rollback
into abstract pve-common/src/PVE/AbstractConfig.pm,
splitting LXC-specific parts into __snapshot_XX methods in
src/PVE/LXC/Config.pm.

check_freeze_needed, snapshot_prepare and snapshot_commit
are downgraded to private __snapshot_XX methods (in
PVE::AbstractConfig and PVE::LXC::Config).

has_feature is made an implementation of the abstract
has_feature, and thus moves into src/PVE/LXC/Config.pm

Refactor config-related methods into AbstractConfig

Move load_config, write_config, lock_config[_xx],
check_lock, has_lock, set_lock, remove_lock, is_template,
check_protection and config_file to pve-common/src/PVE/
AbstractConfig.pm since they are identical for LXC and
Qemu.

Move cfs_config_path and config_file_lock to implementation
of PVE::AbstractConfig in src/PVE/LXC/Config.pm

Drop create_config and replace it with write_config.

pct list: Add a 'Lock' column

This seems more useful than a 'mounted' state as a
mount-locked container can also be running.

Drop used unused volumes when writing config

instead of only when updating the config via pct set or the
API.

This enables us to add unused volumes without calling
is_volume_in_use and mimics the behaviour in QemuServer.pm

Don't apply snapshot config in snapshot_commit

We hold a lock from snapshot_prepare until snapshot_commit,
so there is no need to copy back the snapshot config to the
actual config.

This allows us to move snapshot_commit to the abstract
 common code base.

cleanup: remove duplicate code

run_with_loopdev: reuse existing loopdevs

This will avoid triggering the MMP when mounting a
container locally outside lxc.

bump version to 1.0-47

allow starting and stopping a mounted container

Implement pct mount/unmount

These are for emergency maintenance and put the container
into a locked state ('mounted' lock).

added set_lock and remove_lock

Since set_lock does the implied lock_config() and
load_config() it also returns the loaded config afterwards
as there is no other meaningful return value defined for
this function since failure to apply the lock throws an
exception.

remove_lock() also takes a lock name in order to make sure
only the correct lock is being removed.

Implement pct push/pull

These commands are for running containers only.

To avoid symlink problems these implementations switch
between the host's and container's namespace to open the
corresponding file handles, then copy the data.

With unprivileged containers we also enter the container's
user-namespace with pct-push and switch to the container's
root user before opening the destination for writing in
order to create new files with the mapped root user.

Refactor has_feature

add unused running flag for signature compatibility to
QemuServer.pm
add backup_only flag to replace former feature workaround,
needed to skip backup=no mountpoints if called from VZDump.

Make snapshot_save_vmstate proper sub

Fix #898: typo

Adapt and add testcases for mp support

Add mp support to snapshots

Modelled after QemuServer's way of handling multiple drives.

Check volume usage in snapshots before deleting

pct enter: check if container runs before lxc-attach

else we get:
 >
 > lxc-attach: attach.c: lxc_attach: 710 failed to get the init pid

which an arbitrary user could misunderstand.

Add snapshot rollback testcases

Cleanup test output

Use run_command instead of system to stop CT

Add '\n' to die statements

only root@pam should be able to add bind mounts