Merge pull request #1163 from mabes/fix-libetc-dir

templates: remove creation of bogus directory in Debian templates

templates: remove creation of bogus directory

An incorrect quoting introduced in bf39edb caused a /{lib,etc} folder to
appear in Debian templates

The very next line :
    mkdir -p "${rootfs}/etc/systemd/system/getty.target.wants

makes creating ${rootfs}/etc/systemd/system/ unnecessary in the first
place

Signed-off-by: Maxime Besson <maxime.besson@smile.fr>

Merge pull request #1162 from brauner/2016-08-31/remove_halt_symlink

templates: rm halt.target -> sigpwr.target symlink

templates: rm halt.target -> sigpwr.target symlink

Given commit 330ae3d350e060e5702a0e5ef5d0faeeeea8df6e:

    lxccontainer: detect if we should send SIGRTMIN+3

    This is required by systemd to cleanly shutdown. Other init systems should not
    have SIGRTMIN+3 in the blocked signals set.

we should stop symlinking halt.target to sigpwr.target for systemd.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1157 from evgeni/doxygen-no-full-path-names

set FULL_PATH_NAMES=NO in doc/api/Doxyfile

set FULL_PATH_NAMES=NO in doc/api/Doxyfile

otherwise the generated docs have the full build path in them
and nonbody cares that the files were built in
 /build/lxc-_BVY2u/lxc-2.0.4/src/lxc/

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

Merge pull request #1155 from tych0/migration-fixes

Migration fixes

c/r: write status only after trying to parse the pid

Previously, we write a "success" status but tried to parse the pid. This
meant that we wouldn't notice a successful restore but failure to parse the
pid, which was a little strange.

We still don't know the child pid, so we will end up with a restored
process tree and a running container, but at least in this case the API
will return false indicating that something failed.

We could kill(-1, 9) in this case, but since liblxc runs as root sometimes
(e.g. LXD), that would be a Very Bad Thing.

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

remove extra 'ret'

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

Merge pull request #1154 from stgraber/master

c/r: Fix pid_t on some arches

c/r: Fix pid_t on some arches

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

Merge pull request #1152 from tych0/fix-migration-2.0.4

c/r: use /proc/self/tid/children instead of pidfile

c/r: use /proc/self/tid/children instead of pidfile

All we really needed a unique temp file for was passing the pid. Since CRIU
opened this with O_EXCL | O_CREAT, this was "safe" (users could still
overwrite it afterwards, but the monitor would immediately die since the
only valid number in there was the init process).

In any case, we can just read /proc/self/tid/children, which lists the
child process.

Closes #1150

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

Merge pull request #1151 from brauner/2016-08-26/fix_lxc_device

tools: use boolean for ret in lxc_device.c

tools: use boolean for ret in lxc_device.c

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1139 from brauner/2016-08-18/use_exit_in_all_tools

tools: use exit(EXIT_*) everywhere

Merge pull request #1145 from glensc/pld-patch

add PLD Linux support to upstream repo

templates: add PLD Linux support

from pld-linux repo:
https://github.com/pld-linux/lxc/commits/auto/th/lxc-2.0.4-1/lxc-pld.in.sh
https://github.com/pld-linux/lxc/commits/auto/th/lxc-2.0.4-1/lxc-pld.patch

Signed-off-by: Elan Ruusamäe <glen@delfi.ee>

Merge pull request #1148 from glensc/fed-openssl

templates: fedora requires openssl binary

templates: fedora requires openssl binary

/usr/share/lxc/templates/lxc-fedora: line 1078: openssl: command not found

Signed-off-by: Elan Ruusamäe <glen@delfi.ee>

Merge pull request #1146 from glensc/fd-200

use fd 9 instead of 200

templates: use fd 9 instead of 200

to catch up mksh changes from 17abf27

Signed-off-by: Elan Ruusamäe <glen@delfi.ee>

Merge pull request #1144 from brauner/2016-08-19/seccomp_non_functional_changes

seccomp: non functional changes

seccomp: non functional changes

- log more errnos
- adapt coding style

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1141 from brauner/2016-08-19/add_missing_arches

confile: add more archs to lxc_config_parse_arch()

Merge pull request #1142 from brauner/2016-08-19/add_s390x_seccomp_support

seccomp: add support for s390x

seccomp: remove double include and order includes

Signed-off-by: Christian Brauner <cbrauner@suse.de>

seccomp: add support for s390x

Signed-off-by: Christian Brauner <cbrauner@suse.de>

confile: add more archs to lxc_config_parse_arch()

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1133 from jcowgill/mips

Add MIPS support

seccomp: fix strerror()

Signed-off-by: James Cowgill <james410@cowgill.org.uk>

seccomp: Add mips and mips64 entries to lxc_config_parse_arch

Fixes "unsupported personality" warnings when starting containers.

Signed-off-by: James Cowgill <james410@cowgill.org.uk>

templates: Add mips hostarch detection to debian

Signed-off-by: James Cowgill <james410@cowgill.org.uk>

seccomp: Implement MIPS seccomp handling

MIPS processors implement 3 ABIs: o32, n64 and n32 (similar to x32). The kernel
treats each ABI separately so syscalls disallowed on "all" arches should be
added to all three seccomp sets. This is implemented by expanding compat_arch
and compat_ctx to accept two compat architectures.

After this, the MIPS hostarch detection code and config section code is added.

Signed-off-by: James Cowgill <james410@cowgill.org.uk>

utils: Add mips signalfd syscall numbers

Signed-off-by: James Cowgill <james410@cowgill.org.uk>

tools: unify exit() calls outside of main()

This also changes the help exit status of lxc-unshare and lxc-usernsexec. So
far they did exit(EXIT_FAILURE) whereas all other tools do exit(EXIT_SUCCESS).
Let's align them with the rest of the tools. This should be safe to do.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

tools: use exit(EXIT_*) everywhere

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1140 from ysbnim/master

Update korean manuals

doc: Add --rcfile to Japanese common opts

Update for commit 71d74a8

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

doc: Add --rcfile to Korean common opts

Update for commit 71d74a8

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

doc: Update Korean lxc-attach(1)

 * Update for commit 03b0398
 * Fix mistranslation words

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

Merge pull request #1138 from stgraber/master

Use full GPG fingerprint instead of long IDs.

Use full GPG fingerprint instead of long IDs.

With how easy it is to create a collision on a short ID nowadays and
given that the user doesn't actually have to remember or manually enter
the key ID, lets just use the full fingerprint from now on.

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

Merge pull request #1122 from Blub/rcfile-as-common-arg

Rcfile as common arg

cleanup: replace tabs wth spaces in usage strings

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

cleanup: whitespaces in option alignment for lxc-execute

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

cleanup: tools: remove --name from lxc-top usage message

It doesn't have any effect on what lxc-top does and is only
accepted on account of being part of the common option list.

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

doc: add --rcfile to common opts

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

tools: set configfile after load_config

In order to cause c->is_defined() to become true.

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

tools: move --rcfile to the common options list

In almost all commands it's a useful addition to the -n
switch which is a common option, too.

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

Merge pull request #1134 from brauner/2016-08-16/load_correct_ovl_module

bdev: use correct overlay module name

bdev: use correct overlay module name

- Assume that the module name is "overlay" per default and not "overlayfs".
- Assume that the overlay version we are using requires a workdir.
- When we mount an overlay filesystem and we fail with ENODEV retry once with
  the module name we haven't already used.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1132 from stgraber/master

Update gitignore

Add lxc-test-utils to .gitignore

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

Fix .gitignore after /tools/ split

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

Merge pull request #1131 from brauner/2016-08-16/fix_make_dist

autotools: fix tests

autotools: fix tests

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1129 from brauner/2016-08-13/minimal_unit_testing

[RFC] add minimal unit testing for liblxc functions

tests: add unit tests for lxc_string_in_array()

Signed-off-by: Christian Brauner <cbrauner@suse.de>

tests: add unit tests for lxc_string_replace()

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1128 from brauner/2016-08-14/fix_text_file_busy

tools: use fclose() after fdopen() + add newline

tools: use fclose() after fdopen() + add newline

So far we accidently used close() on the original file descriptor. (After
fdopen() the original fd is considered private and should not be used anymore.
The close operations should be performed on the new file handle. We did the
correct thing on error but not on success.) Using close() on the original fd
caused "Text file busy" errors and prevented the cloned tmpfs container from
starting.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1127 from brauner/2016-08-13/add_missing_newline

tools: add missing newline in lxc-create output

tools: add missing newline in lxc-create output

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1125 from brauner/2016-08-12/smarter_btrfs_subvol_detection

bdev: be smarter about btrfs subvolume detection

Merge pull request #1126 from brauner/2016-08-13/fix_runtime_path_for_lxc_ls

tools: use correct runtime path for lxc-ls

tools: use correct runtime path for lxc-ls

So far we've simply been using RUNTIME_PATH for the privileged and unprivileged
case. We should actually use XDG_RUNTIME_DIR for the unprivileged case.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

bdev: be smarter about btrfs subvolume detection

When a container c is on a btrfs filesystem but is directory backed, copying
the container will default to snapshot. This is because of
should_default_to_snapshot() returning true in this case because c is on a
btrfs filesystem. We should make sure that should_default_to_snapshot() only
returns true, when c itself is a btrfs subvolume.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1120 from tenforward/update_man

Update English and Japanese lxc-attach(1)

doc: Update Japanese lxc-attach(1)

* Add undocumented options (-v/--set-var, --keep-var, -f/--rcfile)
* Change order of option in SYNOPSIS (-L that is placed after "command")
* Add long options in SYNOPSIS

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

doc: Update lxc-attach(1)

* Add undocumented options (-v/--set-var, --keep-var, -f/--rcfile)
* Change order of option in SYNOPSIS (-L that is placed after "command")
* Add long options in SYNOPSIS

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

Merge pull request #1119 from brauner/2016-08-11/fix_log_output

log: fix log_append_stderr()

log: fix log_append_stderr()

Log output currently looks like this:

lxc-copy: debbdev/lxcdir.c: dir_clonepaths: 45 directories cannot be snapshotted.  Try aufs or overlayfs.

we rather want it to be:

lxc-copy: deb: bdev/lxcdir.c: dir_clonepaths: 45 directories cannot be snapshotted.  Try aufs or overlayfs.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1118 from brauner/2016-08-09/remove_readdir_r

tree-wide: replace readdir_r() with readdir()

tree-wide: replace readdir_r() with readdir()

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1116 from clandmeter/alpine-aarch64

templates: add support for new arch on Alpine Linux

templates: add support for new arch on Alpine Linux

Signed-off-by: Carlo Landmeter <clandmeter@gmail.com>

Merge pull request #1114 from Blub/lxc-attach-f

lxc-attach: add -f option & cleanup

whitespace cleanup

The rest of this block uses spaces instead of tabs already.

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

add -f option to lxc-attach

Otherwise a container with a non-standard configuration file
can be started but not attached to.
Fixes the following case:
 # lxc-start -n ct -f /different/path/my.config
 # lxc-attach -n ct
 Error: container ct is not defined

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

Merge pull request #1111 from ysbnim/master

doc: Add documentation of lxc.syslog to Korean lxc.container.conf(5)

doc: Add documentation of lxc.syslog to Korean lxc.container.conf(5)

Update for commit 204dfdf

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

Merge pull request #1109 from brauner/2016-07-31/append_to_search_path

partially reorganize repo

Merge pull request #1110 from tenforward/japanese

doc: Add documentation of lxc.syslog to Japanese lxc.container.conf(5)

doc: Add documentation of lxc.syslog to Japanese lxc.container.conf(5)

and add local6 to valid facility

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

tools, tests: reorganize repo

- tools: move lxc commands to common subfolder
- tests: adapt include search path

Signed-off-by: Christian Brauner <cbrauner@suse.de>

cgroups: move cgroup files to common subfolder

Signed-off-by: Christian Brauner <cbrauner@suse.de>

bdev: add subdirectories to search path

This allows us to avoid using relative includes which is cleaner in the long
run when we create subdirectories for other components of liblxc.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1105 from bapt/syslog

Add syslog support

Add documentation for the new syslog logging mechanism

Signed-off-by: Baptiste Daroussin <bapt@gandi.net>

Add a new logger: syslog and Print the VM name in the logs

To activate it specify lxc.syslog = <afacility>
For now the available facilities are: daemon, local[0-7] others will be
rejected

syslog got only activated after the function that checks for inheritance
of fd is passed in order to make sure the syslog fd is not inherited
and prevent the creation of the fd is any log have been issued during
the checks (which would end up in an infinite loop)

Signed-off-by: Fatih ACAR <fatih.acar@gandi.net>
Signed-off-by: Ahmed Amamou <ahmed@gandi.net>
Signed-off-by: Baptiste Daroussin <bapt@gandi.net>
Signed-off-by: William Dauchy <william@gandi.net>

Merge pull request #1108 from brauner/2016-07-29/fix_android_part_n

criu: replace tmpnam() with mkstemp()

criu: replace tmpnam() with mkstemp()

Signed-off-by: Christian Brauner <cbrauner@suse.de>

Merge pull request #1106 from Blub/next

conf: set pty_info to NULL after free

conf: set pty_info to NULL after free

This fixes a double free corruption on container-requested
reboots when lxc_spawn() fails before receiving the ttys, as
lxc_fini() (part of __lxc_start()'s cleanup) calls
lxc_delete_tty().

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

Merge pull request #1102 from chenhaiq/master

attach: use setns instead of unshare

Merge pull request #1104 from brauner/2016-07-27/fix_android_getline

Revert "Merge pull request #1103 from brauner/2016-07-27/fix_android_…

Revert "Merge pull request #1103 from brauner/2016-07-27/fix_android_getline"

This reverts commit 25796416084f3cecf036bd922d6ead094500191e, reversing
changes made to 813d7f1453e6e28adc0fdfac62d9c4597a87e430.

Contrary to what we suspected the android build errors were not caused by
missing wrong relative include paths. Rather, they were caused by autoconf.

Signed-off-by: Christian Brauner <cbrauner@suse.de>

attach: setns instead of unshare in lxc-attach

lxc-checkpoint will fail because process createdy by lxc-attach has
incorrect cgroup ns. It needs to use "setns" instead of "unshare"
to set cgroup ns.

Signed-off-by: Chen Haiquan <oc@yunify.com>

Merge pull request #1103 from brauner/2016-07-27/fix_android_getline

bdev: include own getline function in bdev.h