create: open templates as real root

When creating an unprivileged container previously couldn't
open template files the unprivileged namespace did not have
access to. This fixes that.

Note that we cannot pass the file via stdin since tar will
then refuse to extract compressed archives automatically.

remove unused next_free_nbd_dev function

bump version to 2.0-13

migrate: pass the with_snapshots parameter

bump version to 2.0-12

PVE::LXC::Migrate - use replication job, transfer replication state

PVE::LXC::Migrate - add missing "use PVE::LXC::Config;"

use full module path for storage_config

because it is not in the local scope

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

get_replicatable_volumes: add unused volumes

get_replicatable_volumes: add additional tests

- skip volumes on shared storage
- die if $mptype ne 'volume'
- skip volumes if we do not 'own' them

get_replicatable_volumes: pass $vmid parameter

src/PVE/API2/LXC.pm: aquire guest_migration_lock inside worker

Add a migration lock to avoid a replication on rollback-time.

PVE::LXC::Config - implement $cleanup flag for get_replicatable_volumes

migrate: implement insecure storage migration

bump version to 2.0-11

allow width and height parameter for vncproxy

so that the novnc console can request a different screen size

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

bump version to 2.0-10

migrate: aquire guest_migration_lock during

To block replication jobs.

do not allow destroy if there are replication jobs

remove 'replicate' configuration - we will store that in another file

use Syscall module instead of raw syscall numbers

Raw syscall numbers were not platform independent, so replace them
with the new PVE::Syscall module which gets the syscall numbers from
syscall.ph, which provides the platform local values.

we cannot directly use syscall.ph as it is no normal perl module and
so it's usage requires special handling

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

migrate: pass ssh_info to storage_migrate

build: run tests when building

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

tests: exit with -1 in case of failures

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

tests: fix broken snapshot delete tests

these broke when the additonal "snapshot-delete" lock was
introduced.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

tests: use all target to run tests

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

check replicate feature on any config update

get_replicatable_volumes: replacemnet for PVE::ReplicationTools::get_syncable_guestdisks()

correctly scan for volumes inside snapshots.

This will also solve the cyclic package dependency propblem.

use a single replicate property (format pve-replicate)

and remove that replication job status update code. This should be
handled by the replication code itself.

Revert "Integrate storage replica in lxc migration."

This reverts commit 05309cdf3f460733ec00184c0f45350f854d3737.

The commit changes the configuration before the VM is actually
migrated, so it is possible to have a wrong configuration when
migration fails for some reason. Also, I am quite unsure if
this automatic target change is really wanted. The patch also
contains wrong refereces to $self->{opts}->{node}.

Revert "migrate: cleanup replica volume skip condition"

This reverts commit e8b22494e9f2b7d5afd663f91b1574a097c66bf0.

bump version to 2.0-9

Revert "fix 1153: let crypt() handle unicode CT passwords"

This reverts commit cfa3086d95c3a39b584b2d60aa53938e1f30b3dc.

We now corectly encode all parameter as utf8, so the previous
code is now correct.

bump version to 2.0-8

add support for Ubuntu 17.04 (zesty)

bump version to 2.0-7

cleanup replication config update

migrate: cleanup replica volume skip condition

Destroy all remote and local replication datasets when a CT will destroyed.

Integrate storage replica in lxc migration.

Now it is possible to migrate a CT when replica is enabled.
It will reduce replication to an minimal amount.

Insert new properties in the LXC config for the PVE Storage Replica.

buildsys: clean: remove *.buildinfo

config: whitelist lxc.limit.*

bump version to 2.0-6

fix suse version 13 check being inverted

bump version to 2.0-5

allow opensuse >= 42

pct: improve error message output of die

bump version to 2.0-4

add console fix for debian containers using systemd

bump version to 2.0-3

don't create deprecated rsa1 ssh keys

bump version to 2.0-2

poststop: use a reboot trigger file

Since the STOPPED state is set before cgroup cleanup we need
a better way to restart a container: We now install an
ExecStopPost snippet into the lxc@.service which when
encountering a reboot trigger file
(/var/lib/lxc/$vmid/reboot) performs a systemctl restart.

Revert "poststop: reboot: wait for lxc to exit before rebooting"

This reverts commit ff867097e8a5ea1ab6ff1d5cbc381a4212c37254.

bump version to 2.0-1

buildsys: update make upload target for stretch

bump version to 1.0-95

buildsys: update make upload target

poststop: reboot: wait for lxc to exit before rebooting

otherwise it'll leak cgroup directories...

Note that we need to escape the lxc@.service context (by
entering a new scope) as well as close our ties to the lxc
monitor (the stdout pipe), otherwise this never finishes
properly.

add keeplocale parameter to vncproxy run_command

to not overwrite the LC_* environment variables

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

Create: fix architecture detection in restore_archive

For detecting a CT templates architecture we used the `file -b -L`
output from the PVE host side.
If the container has a link:
/bin/sh -> /bin/bash
(Alpine Linux does that, for example) the '-L' flag from file
resolves the $rootfs/bin/sh to /bin/bash and thus checks the
architecture of bash on the PVE system, which is always 64 bit.

Add a helper which chroots in the rootfs to avoid problems with
absolute symlinks and use 'open' to avoid relative symlink problems
read the first 5 bytes from /bin/sh, 4 bytes for the ELF magic number
and the fifth for the ELF class, which tells us if we have a 32
(class 1) or 64 (class 2) bit ELF binary.

Return this information as an exit code to the parent.

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

pct: remove leftover outdated POD content

gets generated by pve-docs

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

also expect ENOTDIR when opening dirs with O_NOFOLLOW

open with O_DIRECTORY on newer kernel uses ENOTDIR instead
of ELOOP for symlinks

don't pull HA modules for doc-generation

run tests with lxc-usernsexec; build-depend on lxc or lxc-pve

buildsys: make job safety

bump version to 1.0-94

use the lxc@ service file provded by lxc

Until now the lxc-start process was sort of a danglign
process in the pvedaemon.service cgroup when a container was
started from the web UI causing long stalls when trying to
restart pvedaemon and potential container kills. (Mostly
problematic when issuing package upgrades).
We now start containers via the lxc@ service file giving
them their own service cgroup.

The downside is that we'll have to patch the lxc@ service
file in the lxc package to Type=forking without the -F
option otherwise all of the the containers' console outputs
will end up in the logs...

fix 1153: let crypt() handle unicode CT passwords

according to "perldoc -f crypt", crypt() should downgrade
unicode strings anyway:

If using crypt() on a Unicode string (which potentially has
characters with codepoints above 255), Perl tries to make
sense of the situation by trying to downgrade (a copy of)
the string back to an eight-bit byte string before calling
crypt() (on that copy).  If that works, good.  If not,
crypt() dies with "Wide character in crypt".

login via Spice and ssh works now at least, the noVNC /
vncterm combo seems to be broken because of an unrelated
unicode issue..

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump version to 1.0-93

add dependency on libpve-guest-common-perl

bump version to 1.0-92

use new PVE::Storage::check_volume_access()

fix #1253: display SSH fingerprints on CT setup

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

bump version to 1.0-91

add setup_environment hook to CLIHandler class

bump version to 1.0-90

Revert "Do not skip unprivileged config parameter when restoring a container"

This reverts commit ca187102dfca383a1125106fef744d3ba525023c.

We extract the backup archive before we read the config, so above patch
generates containers with wrong permissions.

bump version to 1.0-89

vzdump: use --make-rslave instead of --make-rprivate

In a stop mode backup of a running container the container
is started from this namespace, so making it private can
cause mount point leaks.

supress warning when we try to delete a non-existent option

such warnings are just confusing ...

fix #1226: show the right amount of memory usage

we moved the lxc cgroup settings in a sub namespace, but did not use the
total_cache value for subtracting so we showed a wrong used amount

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

fix #1225: restore without config changes for VMUser

this makes the behaviour similar to VM restore operations

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

document bind/device mp permissions better

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

fix typo

bump version to 1.0-88

setup: allow fedora 22-25

This should allow vanilla lxc templates to work without the
double-console issue by removing their getty@.service
replacement. (Since we instead fixup
container-getty@.service)

setup: more general approach to tty paths

Unprivileged containers always use an empty lxc.devttydir
option (iow. don't use the /dev/lxc/ subdirectory).
Alpine and Gentoo don't support it in general.

Define a devttydir() sub in Setup::Base which by default
returns "lxc/" or an empty string depending on whether it is
an unprivileged container. Gentoo and Alpine override it
with one which always returns an empty string.

bump version to 1.0-87

add restart migration to lxc api

this simply adds the restart flag and the optional timeout to the
lxc api required for the restart mode migration

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

implement lxc restart migration

this checks for the 'restart' parameter and if given, shuts down the
container with a optionally defined timeout (default 180s), and
continues the migration like an offline one.

after finishing, we start the container on the target node

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

VZDump: implement stopwait

This was missing in pve-container, qemu-server does it already.

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

bump version to 1.0-86

Do not skip unprivileged config parameter when restoring a container

This allows to keep the parameter when restoring from a backup.

bump version to 1.0-85

start: use root cpuset cgroup if the lxc one fails

This happens when no container has been started yet and the
lxc/ directory has not yet been created in the cpuset
cgroup.

bump version to 1.0-84

start: use a random cpuset at startup

we'll leave the rebalancing to pvestatd's next iteration