Merge pull request #964 from brauner/2016-04-11/lxc_attach_manpage

lxc-attach: add logging option to manpage

Merge pull request #965 from brauner/2016-04-12/lxc_destroy_avoid_double_print

lxc-destroy: avoid double print

lxc-destroy: avoid double print

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #962 from staticfox/gitignore

Ignore temporary files generated by doxygen

lxc-attach: add logging option to manpage

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #963 from brauner/2016-04-11/lxc_create_best_storage

lxc-create: fix -B best option

Merge pull request #961 from staticfox/iterations

Use size_t as an iteration point over int

lxc-create: fix -B best option

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Use size_t as an iteration point over int

This cleans up some sign-compare warnings as well as avoids any
possibilities of unintended signed offsets for indices during
iteration.

Signed-off-by: Matt Ullman <staticfox@staticfox.net>

Ignore temporary files generated by doxygen

Signed-off-by: Matt Ullman <staticfox@staticfox.net>

Merge pull request #958 from brauner/2016-04-08/lxc_attach_simplify_and_log

adapt lxc-attach tests

adapt lxc-attach tests & add test for pty logging

The current tests for lxc-attach pty allocation and I/O redirection rely on the
standard file descriptors of the test script to refer to a pty. If they do not
the tests are effectively useless with respect to pty allocation. We need a test
environment with the standard file descriptors refering to a pty as well. One
solution is to run this test under the script command.

This commit also adds a test whether pty logging works. This test is only
executed when all standard file descriptors refer to a pty.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #956 from hallyn/2016-04-08/cgnfs-defer

cgfsng: defer to cgfs if needed subsystems are not available

cgfsng: defer to cgfs if needed subsystems are not available

This requires us to check that at cgfsng_ops_init, rather than
cgfs_init.  Cache the hierarchy and cgroup.use info globally
rather than putting it into the per-container info, as cgmanager
does.  This is ok as both cgroup.use and the list of usable
hierarchies are in fact global to a lxc run.

Closes #952

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

Merge pull request #954 from brauner/2016-04-08/lxc_attach_simplify_and_log

use lxc_read_nointr() and lxc_write_nointr()

fix lxc_console_cb_tty_*()

Clean exit when read() == -1 && errno != EINTR or read() == 0.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

use lxc_read_nointr() and lxc_write_nointr()

Using EPOLLHUP to determine when to exit the loop is unreliable. Let's exit
clean when read() returns -1 && errno != EINTR or 0.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #953 from brauner/2016-04-08/lxc_attach_simplify_and_log

simplify pty allocation and implement pty logging

simplify pty allocation and implement pty logging

lxc-attach allocated a pty in a manner that relied on ts->stdinfd and
ts->stdoutfd to be set. We had to resort to a trick to get it working when
output is redirected. The new implementation gets rid of the black magic and
also simplifies the code.

This commit also implements pty logging for lxc-attach. Users will now be able
to log commands and corresponding output to a file given that lxc-attach
allocates a pty.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #951 from brauner/2016-04-07/lxc_attach_sigwinch_handler

lxc-attach: attach even without sigwinch handler

lxc-attach: attach even without sigwinch handler

lxc_console_create() calls lxc_console_peer_default() which in turn calls
lxc_console_sigwinch_init() which sets up the lxc_tty_state struct for the
current pty. Prior to this commit lxc_console_sigwinch_init() would consider
failures to install a SIGWINCH handler fatal and and return NULL. This commit
makes failures to install a SIGWINCH handler non-fatal. In such cases the
lxc_tty_state struct will still be set up but the sigfd member, which contains
the fd which receives SIGWINCH events, will be set to -1. (This also entails
that the sigset_t oldmaks field is meaningless.) Callers of
lxc_console_sigwinch_init() and lxc_console_sigwinch_fini() should thus make
sure that sigfd >= 0 or sigfd != -1 before trying to register a SIGWINCH handler
in e.g. an lxc_mainloop or resetting the sigmask (cf. lxc_attach.c).

These changes also imply that lxc_console_sigwinch_init() only fails with
ENOMEM. Thus, all cases where lxc_console_sigwinch_init() returns NULL are to be
considered fatal. This wasn't the case before this commit.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

change version to 2.0.0 in configure.ac

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>

Merge pull request #950 from brauner/2016-04-06/fix_memory_leak

lxc_mount_auto_mounts(): free memory on failure

Merge pull request #948 from brauner/bugfix_branch

lxc-create: bugfixes and small improvements

lxc_mount_auto_mounts(): free memory on failure

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #949 from Blub/netlink_open_error

netlink_open: close socket on error

netlink_open: close socket on error

All uses of netlink_open() assume that on error the
nl_handler doesn't need to be closed, but some error cases
happen after the socket was opened successfully and used to
simply return -errno.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

lxc-create: bugfixes and small improvements

- call lxc_container_put() before we exit()
- reorder includes
- use EXIT_FAILURE and EXIT_SUCCESS macros

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #947 from brauner/bugfix_branch

add more error handling + cleanup copy_storage()

add more error handling + cleanup copy_storage()

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #945 from brauner/bugfix_branch

set lxc.rootfs.backend on container creation

set lxc.rootfs.backend on copy/snapshot as well

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

set lxc.rootfs.backend on container creation

Since specifying lxc.rootfs.backend can lead to performance improvements we
always set it during container creation.

This also fixes a bug. do_bdev_create() tried to be smart and retrieve the lower
directory when bdev->type == overlayfs or aufs thereby cutting the path.
However, this operation is done in ovl_mount() and aufs_mount() and both
functions need the full src path for this. The bug didn't show before because
when creating a overlayfs container with e.g.:

lxc-create -n c -t busybox -B overlayfs

still left bdev->type set to dir and so the code for retrieving the lower dir in
do_bdev_create() was skipped. But by setting lxc.rootfs.backend on container
creation we force bdev->type to be set to e.g. overlayfs and the code gets
executed thereby exposing the bug.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #944 from brauner/bugfix_branch

use correct "overlayfs" instead of "ovl"

use correct "overlayfs" instead of "ovl"

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #943 from tenforward/japanese

doc: Add lxc.rootfs.backend in Japanese lxc.container.conf(5)

doc: Add lxc.rootfs.backend in Japanese lxc.container.conf(5)

Update for commit f903986

Signed-off-by: KATOH Yasufumi <karma@jazz.email.ne.jp>

Merge pull request #942 from ysbnim/master

Add a document of lxc.rootfs.backend in Korean manpage

Add a document of lxc.rootfs.backend in Korean manpage

Update for commit f903986

Signed-off-by: Sungbae Yoo <sungbae.yoo@samsung.com>

Merge pull request #941 from evgeni/spelling

fix spelling mistakes spotted by Debian's lintian

fix spelling mistakes spotted by Debian's lintian

Signed-off-by: Evgeni Golov <evgeni@debian.org>

Merge pull request #936 from overlayfs/btrfs_recursive_destroy

fix btrfs_recursive_destroy

store errno immediately after ioctl

Signed-off-by: Oleg Freedholm <overlayfs@gmail.com>

Merge pull request #940 from hallyn/2016-04-02/bdev.4

switch user facing name to lxc.rootfs.backend

document lxc.rootfs.backend

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

switch user facing name to lxc.rootfs.backend

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

Merge pull request #939 from hallyn/2016-04-02/bdev.3

rootfs.bdev_type: implement get and clear

rootfs.bdev_type: implement get and clear

clear is just set(x, ""), but before this patch that didn't work bc
"" is nota valid bdev type.

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

Merge pull request #938 from hallyn/2016-04-02/bdev.2

use more accurate bdev_type name for rootfs.bdev

use more accurate bdev_type name for rootfs.bdev

It'll make future code reading easier.

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

Merge pull request #937 from hallyn/2016-04-02/bdev

2016 04 02/bdev

honor lxc.rootfs.bdev

If that is specified, then we only use the specified backing store type.

This can be useful if you know that lxc.rootfs is a directory type and
you do not want lxc to waste time searching for zfs, btrfs, etc.

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

add a lxc.rootfs.bdev option (not yet honored)

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

fix btrfs_recursive_destroy

A change in kernel 4.2 caused btrfs_recursive_destroy to
fail to delete unprivileged containers.  This patch restores
the pre-kernel-4.2 behaviour.  Ref: Issue 935.

Signed-off-by: Oleg Freedhom <overlayfs@gmail.com>

change version to 2.0.0.rc15 in configure.ac

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>

Merge pull request #931 from n-eiling/pageserver-support

c/r: support for the criu pageserver

fixed indentation and comments

Signed-off-by: Niklas Eiling <niklas.eiling@rwth-aachen.de>

Merge pull request #934 from GreatFruitOmsk/master

Fix buffer overflow in do_start()

Fix buffer overflow in do_start()

Signed-off-by: Aleksandr Mezin <mezin.alexander@gmail.com>

c/r: support for the criu pageserver
this enables lxc to perform "disk-less migrations" where memory pages are sent directly to the destination machine instead of being written to the sources filesystem first.
For this, the migrate_opts struct has been added the strings "pageserver_address" and "pageserver_port" so that criu can be told where to look for a pageserver.

Signed-off-by: Niklas Eiling <niklas.eiling@rwth-aachen.de>

Merge pull request #932 from n-eiling/criu-log-fix

fix possible buffer overflow

use snprintf instead of strncat

Signed-off-by: Niklas Eiling <niklas.eiling@rwth-aachen.de>

Merge pull request #933 from evgeni/update-debian-releases

update Debian release names

update Debian release names

Squeeze is EOL now, Stretch is the upcoming stable

Signed-off-by: Evgeni Golov <evgeni@golov.de>

fix possible buffer overflow
strncat only returns its first argument and not the end of the written string.
Thus "buf-pos" is always 0 and consquently no range check is performed.

Signed-off-by: Niklas Eiling <niklas.eiling@rwth-aachen.de>

Merge pull request #929 from brauner/2016-03-27/mmap_file_to_str

use smarter error handling for lxc_strmmap()

use common lxc ordering for included headers

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #930 from brauner/2016-03-30/ignore_all_so

also ingore *.so.*

use smarter error handling for lxc_strmmap()

Additionally, fix the comment for lxc_strmmap() and cleanup #includes in
utils.{c,h}.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

also ingore *.so.*

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #928 from stgraber/master

lxc-ubuntu: Fix building on secondary architectures

lxc-ubuntu: Fix building on secondary architectures

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>

change version to 2.0.0.rc14 in configure.ac

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>

Merge pull request #926 from brauner/2016-03-27/mmap_file_to_str

add funs to mmap() files to \0-terminated strings

better naming for mmap helpers

Rename lxc_mmap()/lxc_munmap() to lxc_strmmap()/lxc_strmunmap().

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

use lxc_mmap() and lxc_munmap()

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

add funs to mmap() files to \0-terminated strings

In order to do this we make use of the MAP_FIXED flag of mmap(). MAP_FIXED
should be safe to use when it replaces an already existing mapping. To this
end, we establish an anonymous mapping that is one byte larger than the
underlying file. The pages handed to us are zero filled.  Now we establish a
fixed-address mapping starting at the address we received from our anonymous
mapping and replace all bytes excluding the additional \0-byte with the file.
This allows us to use normal string-handling function. The idea implemented
here is similar to how shared libraries are mapped.

Signed-off-by: Christian Brauner <christian.brauner@mailbox.org>

Merge pull request #927 from tych0/only-use-host-devnull-no-autodev

start: only use host's /dev/null when absolutely necessary

start: only use host's /dev/null when absolutely necessary

See comments for details, but basically, only use the host's /dev/null when
absolutely necessary (i.e. there is no reasonable /dev/null in the
container).

Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>

Merge pull request #924 from evgeni/default-ubuntu-mirror

always provide a default mirror for debootstraping Ubuntu

Merge pull request #925 from evgeni/debian-main-only

only enable Debian's main repository by default

only enable Debian's main repository by default

This inverts the logic done in c2a85d0 to default to "main only" and allow enabling contrib and non-free on user request.

Closes: #625
Signed-off-by: Evgeni Golov <evgeni@debian.org>

always provide a default mirror for debootstraping Ubuntu

debootstrap sometimes selects the wrong mirror due to [1]

[1] https://bugs.debian.org/819300

Signed-off-by: Evgeni Golov <evgeni@debian.org>

Merge pull request #922 from evgeni/httpredir.debian.org

use httpredir.debian.org as the default Debian mirror

use httpredir.debian.org as the default Debian mirror

http.debian.net is an alias anyways and httpredir.debian.org is the official name of the service

Signed-off-by: Evgeni Golov <evgeni@debian.org>

Merge pull request #920 from thtanaka/timezone-fix

Timezone inside the container is not the same as the host

Merge pull request #919 from amezin/master

Fix installation of out-of-tree (VPATH) builds

Timezone inside the container is not the same as the host

In OL6, it is missing /etc/sysconfig/clock

Tested-by: Deepak Patel <deepak.patel@oracle.com>
Signed-off-by: Thomas Tanaka <thomas.tanaka@oracle.com>

Fix installation of out-of-tree (VPATH) builds

Signed-off-by: Aleksandr Mezin <mezin.alexander@gmail.com>

Merge pull request #917 from amezin/master

Fix daemonized containers without autodev and/or /dev mount

start: open /dev/null from "host" /dev

Sometimes, a container doesn't have /dev/null.
For example, I have this problem with Android container.

This fixes #910 (github) for me.

Signed-off-by: Aleksandr Mezin <mezin.alexander@gmail.com>

utils: split null_stdfds() to open_devnull() and set_stdfds()

Signed-off-by: Aleksandr Mezin <mezin.alexander@gmail.com>

lxc-busybox: Remove warning for dynamically linked Busybox

The warning has been present since commit 32b37181ea (with no purpose stated).
Support for dynamically linked Busybox has been added since commit bf6cc73696.
Haven't encountered any issues with dynamically linked Busybox in my last
2 years' testing.

Signed-off-by: Bogdan Purcareata <bogdan.purcareata@nxp.com>
Acked-by: Stéphane Graber <stgraber@ubuntu.com>

Merge pull request #916 from ojkastl/20160324_Fix_path_to_lxc-net_config

lxc.spec.in: fixed hardcoded path to lxc-net config file

start: use LXC_SYNC_ERROR to report errors.

This gives more meaningful error message than "invalid sequence".

Signed-off-by: Aleksandr Mezin <mezin.alexander@gmail.com>

sync: add LXC_SYNC_ERROR to report errors from another process.

Signed-off-by: Aleksandr Mezin <mezin.alexander@gmail.com>

lxc.spec.in: fixed hardcoded path to lxc-net config file

Signed-off-by: Johannes Kastl <git@ojkastl.de>

Merge pull request #914 from brauner/2016-03-23/Purcarea_Bogdan

lxc-busybox: Touch /etc/fstab in the container rootfs

lxc-busybox: Touch /etc/fstab in the container rootfs

Signed-off-by: Bogdan Purcareata <bogdan.purcareata@nxp.com>

Merge pull request #911 from brauner/2016-03-23/Purcarea_Bogdan

open_without_symlink: Don't SYSERROR on something else than ELOOP