Re: [Devel] [PATCH v5 4/6] modify tar extraction to account for user namespace

2013-05-19 Thread Kir Kolyshkin
On 05/17/2013 09:26 AM, Glauber Costa wrote: From: Glauber Costa glom...@parallels.com If we are running upstream with user namespaces, we need to create the container filesystem not with the ownership preserved, but reflecting the mapping we need to apply. Note that according to our

Re: [Devel] [PATCH v5 5/6] automatically add bridge venet0 when needed

2013-05-19 Thread Kir Kolyshkin
Committed with fixes, see inline. On 05/17/2013 09:26 AM, Glauber Costa wrote: From: Glauber Costa glom...@parallels.com The chosen architecture to deal with --ipadd with upstream containers is to create a veth pair and add the host side information to a bridge called venet0. This way, all the

Re: [Devel] [PATCH v5 0/6] User namespace support for upstream containers

2013-05-19 Thread Kir Kolyshkin
On 05/17/2013 09:26 AM, Glauber Costa wrote: Kir, In this patchset, I hope to be addressing all your concerns. I don't use cmd_p any longer, as you requested. I have also tried to merge most of your comments at the main userns patch. It is a bit massive code, so if is there still anything

Re: [Devel] [PATCH v5 6/6] allow for distro-specific fix ups at creation time.

2013-05-19 Thread Kir Kolyshkin
On 05/17/2013 11:54 PM, Glauber Costa wrote: +{ +char buf[STR_SIZE]; + +/* Distributions that don't need the fixup will can stop right here */ +if (!actions || !actions-ct_fixup) +return 0; + +if (snprintf(buf, sizeof(buf), %s/%s, root, /etc/rc3.d/S00vz-fixups.sh) 0)

Re: [Devel] [CRIU] [PATCH 1/6] vzctl: split ct_env_create

2013-05-19 Thread Andrew Vagin
On Fri, May 17, 2013 at 10:24:55AM -0700, Kir Kolyshkin wrote: On 05/16/2013 09:47 AM, Andrey Wagin wrote: 2013/5/16 Glauber Costa glom...@parallels.com: On 05/16/2013 04:14 PM, Andrey Vagin wrote: + ret = ct_env_create_real(arg); + if (ret 0) return

Re: [Devel] [CRIU] [PATCH 1/6] vzctl: split ct_env_create

2013-05-19 Thread Kir Kolyshkin
On 05/19/2013 11:59 AM, Andrew Vagin wrote: On Fri, May 17, 2013 at 10:24:55AM -0700, Kir Kolyshkin wrote: On 05/16/2013 09:47 AM, Andrey Wagin wrote: 2013/5/16 Glauber Costa glom...@parallels.com: On 05/16/2013 04:14 PM, Andrey Vagin wrote: + ret = ct_env_create_real(arg); + if (ret

Re: [Devel] [PATCH v5 4/6] modify tar extraction to account for user namespace

2013-05-19 Thread Glauber Costa
On 05/19/2013 09:41 PM, Kir Kolyshkin wrote: + */ +#define VZ_DEFAULT_UID10 +#define VZ_DEFAULT_GID10 I assume these are no longer used, right? right ___ Devel mailing list Devel@openvz.org

Re: [Devel] [CRIU] [PATCH 1/6] vzctl: split ct_env_create

2013-05-19 Thread Andrew Vagin
On Sun, May 19, 2013 at 12:12:47PM -0700, Kir Kolyshkin wrote: On 05/19/2013 11:59 AM, Andrew Vagin wrote: On Fri, May 17, 2013 at 10:24:55AM -0700, Kir Kolyshkin wrote: On 05/16/2013 09:47 AM, Andrey Wagin wrote: 2013/5/16 Glauber Costa glom...@parallels.com: On 05/16/2013 04:14 PM, Andrey

[Devel] [PATCH] allow for distro-specific fix ups at creation time.

2013-05-19 Thread Glauber Costa
From: Glauber Costa glom...@parallels.com We will need that infrastucture when running with Linux upstream, since some support is very unlikely to ever land in the Kernel. We need to do things like account for the fact that udev may kick in and destroy all the setup we have done for /dev. Since