Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libcontainer: configs: initial support for timens #2963

Closed
wants to merge 1 commit into from

Conversation

metux
Copy link
Contributor

@metux metux commented May 25, 2021

Allow explicit unsharing of time namespace.

Signed-off-by: Enrico Weigelt, metux IT consult info@metux.net

Allow explicit unsharing of time namespace.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
@cyphar
Copy link
Member

cyphar commented May 25, 2021

This isn't really sufficient, in order to really make use of the timens you need to configure the time offsets when the container first starts (and that would require switching nsenter to use clone3, as well as figure out a reasonable way to configure it -- which will probably require runtime-spec changes).

@cyphar cyphar added this to the post-1.0 milestone May 26, 2021
@AkihiroSuda AkihiroSuda marked this pull request as draft June 2, 2021 12:18
@kolyshkin
Copy link
Contributor

@metux if you want to work on that, the first step is working on the proposal in opencontainers/runtime-spec#1062, which is still pending. Once merged, we can look into implementing timens.

Closing for now as much more changes are needed than adding a few constants (for more details, see #2345).

@kolyshkin kolyshkin closed this Jun 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants