Re: [RFC PATCH 02/27] containers: Implement containers as kernel objects

David Howells
2019-02-19T23:03:44Z

Trond Myklebust <trondmy@hammerspace.com> wrote:

> Do we really need a new system call to set up containers? That would
> force changes to all existing orchestration software.

No, it wouldn't. Nothing in my patches forces existing orchestration software
to change, unless it wants to use the new facilities - then it would have to
be changed anyway, right? I will grant, though, that the extent of the change
might vary.

> Given that the main thing we want to achieve is to direct messages from
> the kernel to an appropriate handler, why not focus on adding
> functionality to do just that?

Because it's *not* just that that is added here. There are a number of things
this patchset (and one it depends on) provides:

(1) The ability to intercept request_key() upcalls that happen inside a
container, filtered by operative namespace.

(2) The ability to provide a per-container keyring that can hold keys that
can be used inside the container without any action on behalf of the
denizens of the container.

(3) The ability to grant permissions to a *container* as a subject, allowing
it and its denizens to use, but not necessarily read, modify, link or
invalidate a key.

(4) The ability to create superblocks inside a container with a separate
mount namespace from outside, such that they can use the container keys,
thereby allowing the root of a container to be on an authenticated
filesystem.

> Is there any reason why a syscall to allow an appropriately privileged
> process to add a keyring-specific message queue to its own
> user_namespace and obtain a file descriptor to that message queue might
> not work?

Yes. That forces the use of a new user_namespace for every container in which
you want to use any of the above features. The user_namespace is already way
too big and intrusive a hammer as it is.

> With such an implementation, the fallback mechanism could be to walk
> back up the hierarchy of user_namespaces until a message queue is
> found, and to invoke the existing request_key mechanism if not.

That's definitely wrong. /sbin/request-key should *not* be spawned if the key
to be instantiated is not in all the init namespaces.

I went with a container object with namespaces for a reason: initially, it was
so that the upcall could take place inside of the container's namespaces, but
now it's do that any request that doesn't match the namespaces on the
container gets rejected at the boundary - so that some daemon up the chain
doesn't try servicing a request for which it can't access the config data or
would end up talking out of the wrong NIC.

I can drop the container object part of it for the moment.

I could instead create 1-3 new namespaces:

(1) A namespace with an upcall-interception point.

(2) A namespace with a container keyring.

(3) A namespace with a subject ID for use in key ACLs.

I think I should also consider adding:

(4) A namespace with keyring names in it. I'm leaning towards this not being
part of user_namespace because these probably should not be visible
between containers.

David