bump version to 1.0-15

improve inline comment

Start a worker in lxc resize.

It is necessary because if we resize a disk it can take longer. so to prevent long waiting time fork a worker process.

move resize to have it available in the pveshell and at the rest api

allow to mount iso images

bump version to 1.0-14

redhat: fix unused values

Move ip_is_in_cidr checks on $d->{gw} into the
defined($d->{gw}) guarded if block to avoid warnings and
useless route files being created when using dhcp.

bump version to 1.0-13

restore: delete config from container after restore

We don't need to leave /etc/vzdump/pct.conf or vps.conf in
the container's directory structure after using it, it only
causes the next backup to have the file twice in the
archive.

restore: make sure only the first pct.conf is extracted

When making a stop/snapshot mode backup of a container that
was already restored from a backup, its /etc/vzdump/pct.conf
file was replacing our newly created one in the archive. We
need to prevent the duplicate file from overwriting our new
one.

redhat: don't use aliases for dual stack networking

A static IPv6 as alias interface for ipv4 doesn't work (RH
has "secondaries" for that), DHCP on aliases doesn't work
either.
The only drawback of putting both in the same file is that
static addresses take longer to be configured if the DHCP
server is slow.

redhat: use the fully qualified hostname

bump version to 1.0-12

hotplug: deal with gateways outside the subnet

systemd: deal with gateways outside the subnet

centos: deal with gateways outside the subnet

debian: deal with gateways in external subnets

allow /32 CIDRs and remove duplicated mask array

bump version to 1.0-11

setup/debian: remove superfluous parameter

The $new parameter only guards the output of the 'auto' line
which is now being tracked in $done_auto, so it's not
needed anymore.

cleanup: reduce hash access

whitespace cleanup

setup/debian: guard ipv4 output

When only an ipv6 address was specified we still tried to
print an ipv4 address which warned and caused additional
newlines to be appended to the file on each start.

setup/debian: avoid extra newlines

At the beginning of a file or between sections, if there
have been two newlines already we don't need to add another.

setup/debian: avoid writing multiple auto lines

bump version to 1.0-10

improve regex to match redhat/centos OS version

disk-size is a format, not a type

code cleanup - remove dead code

volume property is not optional, so this is already verified
by PVE::JSONSchema::parse_property_string()

parsing: throw by default unless $noerr is passed

parse_ct_mountpoint and parse_lxc_mountpoint are now not usd
in schema verification anymore, so instead of returning
undef on error it can now die.

parse_ct_mountpoint now also gets a $noerr parameter as it
is used in foreach_mountpoint, and to be safe we'll just
skip invalid mountpoints there to avoid unexpected
inconsistent states.

update changelog

revert coding style changes, skip ID 7 (reserved for X11)

update inittab instead of replacing it

add note about fsck to changelog

fsck: simplify code

minor cleanups

Add new pct fsck command to check the mountpoints of a container

* the filesystem specific command will be called automatically by fsck
* the -a flag ensures that the filesystem can be fixed without any questions
* the -f flag forces a filesystem check even if the fs seems clean
(flags similar to what the fsck systemd unit uses)

bump version to 1.0-9

remove unused functions

change function parse_size to PVE::JSONSchema::parse_size

fix bug #750: deactivate volumes to be sure there are no volumes active on the source node

fix bug #752: correct size of mount point after resize

fix test if storage allows containers

bump version to 1.0-8

make bridge parameter option - required for hotplug code

bump version to 1.0-7

always create /fastboot to skip run fsck

avoid warning about uninitialized value

Typo in resize_vm subroutine

bump version to 1.0-6

replace disk-size calculation in pct resize

fixes a fixme

remove old comment about looopdevices_list

bump version to 1.0-5

lxc hooks: use run_cli_handler(), remove stale docs

bump version to 1.0-4

remove 'noload' option from zfspool mounts

This should have only been added to raw files in the first
place.

Fixes #742.

pct create: add bash completion for storage parameter

don't add binds and block devices as unused volume

bump version to 1.0-3

code cleanup

fix hotplug error in update_pct_config

Non-hotpluggable values still updated the $conf hash and
so providing a hotpluggable value after a non-hotpluggable
one updated them both in the file.

added lxc.hook.stop to valid_lxc_conf_keys

move nohotplug error to after disk-creation

Added a reminder comment...

add mountpoint deletion support and unused volumes

Like with qemu, deleting a mountpoint now adds an unused
disk, which when deleted will now also delete the underlying
volume.

factor out a delete_mountpoint function for reuse

Use the new JSONSchema::print_property_string

use the disk-size property type

set minimum mtu to 64

Only get PID info in fin_lxc_pid sub

with the -p switch lxc-info only ouputs the PID line,
this makes the call quite a bit faster.

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

fix hardcoded CT uptime in vmstatus

Implement the container uptime by susbtracting the ctime from the
container pid file from the actual time.
This mirrors the behaviour of lxcfs, see get_pid1_time() in lxcfs.c.
This hass some limitations, like frozen or live migrated containers
falsify the real uptime. But as it shows everytime the uptime like
a uptime command in the container would this is forgivable, for now.

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

bump version to 1.0-2

set TERM=linux for containers

some init scripts expects a linux terminals (turnkey).

bump version to 1.0-1

mount snapshots with the noload option

When using block device based snapshots we cannot mount the
filesystem as it's not clean, and we also can't replay the
journal without write access (as even `-o ro` writes to
devices when replaying a journal (see the linux docs under
Documentation/filesystems/ext4.txt section 3 option 'ro')).
So we need to use the "noload" option to avoid replaying the
journal.

vzdump:lxc: sync and skip journal in snapshot mode

We now perform a 'sync' after 'lxc-freeze' and before
creating the snapshot, since we now mount snapshots with
'-o noload' which skips the journal entirely.

vzdump:lxc: activate the right volumes

bump version to 0.9-23

use json schema format for network config

To correctly generate docs for -net[n].

Replace parsing code for -net* and -mp*

Use the new JSONSchema provided parser which also verifies
the parameters with their schemas.

Use format hashes in schemas

Use full schemas and generate_typetext for -net* and -mp*
properties.

When destroying a container, remove bind mounts and block devices from the list of volumes to delete.

CT protection mode

[PATCH 2] changes:

- added common check_protection subroutine
- disk removal or update operation
- restore over protected CT prevention
- changed man page message

vzdump: tar: disable warnings about ignored files

bump version to 0.9-22

allow to parse configuration value with spaces

Required for namenerser and searchdomain lists.

specify data format for hostname, searchdomain and nameserver

add check for newlines before writing configuration

vzdump:lxc: unshare mount namespace

Enter a new mount namespace for stop and snapshot backups
where we need to mount the container to a temporary
location.

mountpoint_mount: disallow symlinks in bind mounts

symlinks in mount paths can cause security issues
assume the following setup:
mp1: local:X,mp=/disk2
mp2: /mnt/shared,mp=/shared
Now the container boots and executes this sequence:
ct:# ln -s /var/lib/lxc/$VMID/etc /disk2/shared
ct:# umount /disk2
ct:# ln -s /mnt /disk2
ct:# umount /shared
ct:# rmdir /shared
ct:# ln -s /etc /shared
ct:# poweroff
Now the owner waits for a stop-mode backup of the container
to be created:
mp1 will be mounted to the host's /mnt because the
container's /disk2 is a symlink to /mnt.
mp2 will now access the replaced /mnt/shared, which is a
symlink to the container's /etc, and mount that over the
container's /shared, which is a symlink to the host's /etc.
Now until the backup is finished the container's owner could
log into the host via ssh using his container's user
credentials.

We'll also unshare the mount namespace when performing such
backups, but it's still a bad idea to allow symlinks
modifying mount container paths.

use sanitize_mountpoint in foreach_mountpoint

This is for consistency as well as it's a minor security
improvement.

If the source is a path we also want to sanitize it in order
to not get broken behavior when using paths like
/dev/../something etc.

lxc: use new disk option names in permission check

made check_ct_modify_config_perm check VM.Config.Disk for
'rootfs' and 'mp\d+' instead of the old 'disk' option.

fix previous commit (s/snapshot/suspend/)

rsync cleanup

It's only one command now, renamed $base to $rsync.

vzdump: suspend mode fixes

-) '-x' is '--one-file-system' (the longer version is easier
to spot.)
-) Use --relative's special handling of `/./` in paths in
order to make --one-file-system and --exclude options work
together the way they should.

Here's the issue:
Say you have thse files in your container:
  /the-file
  /mp0/the-file
And assume /mp0 is a mountpoint.
Naturally you want `-exclude-path /the-file` to only exclude
the first of the two files. This is hard when rsyncing each
mountpoint separately, as the rsync command for mp0 would
see files relative to /mp0, and thus both files would be
excluded unless we modify exclude paths accordingly - which
we can't as they can be arbitrary glob patterns.
Now with rsync's --relative option - assume the container is
mounted at /temp (iow: /temp/ and /temp/mp0). Passing
/temp/mp0/ to rsync would copy the contents of the mp0
mountpoint into the root directory of the destination
(essentially doing the equivalent of `mv mp0/* /` in the
container's backup.). However, rsync's special treatment of
/./ with the --relative option allows us to pass
/temp/./mp0/ which tells rsync that `/mp0` is supposed to be
included in the path, iow. we're actually copying from
/temp/, but we want only its mp0/ directory.
See rsync(1)'s section about --relative for a detailed
description.

cleanup

vzdump: don't unmount filesystems with -mode suspend

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'.