[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#902350: systemd-container: /proc/1/environ doesn't contain



On Fri, 2018-06-29 at 06:16 +0900, Hideki Yamane wrote:
> Hi,
> 
> On Mon, 25 Jun 2018 16:55:40 +0200 Cyril Brulebois <kibi@debian.org>
> wrote:
> > Please anchor the grep pattern to avoid accidental matches against
> > variables whose name ends with container (e.g. foo_container), and
> > values which start with lxc-libvirt (e.g. lxc-libvirt-something-else);
> > adding ^ on the left and $ on the right would do that.
> 
>  Thanks, indeed. And Ben suggested just use $container and it's
>  better.

It would be better than running 'env' to test a single variable value,
but it is still not right to assume that the container environment
variable is inherited.  In fact, I checked with 'systemd-nspawn -B' and
it was not inherited by a login shell.

As I commented on the merge request, reading /run/systemd/container (if
present) and then pid 1's environment seems like it will be the most
reliable approach.

Ben.

> 
> > It seems there are quite a number of other places with such issues.
> 
>  I'll look into it, later.
> 
> 
-- 
Ben Hutchings
This sentence contradicts itself - no actually it doesn't.

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: