Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

ct: initialize StdFds for restored containers#485

Closed
avagin wants to merge 6 commits into
docker-archive:criufrom
avagin:cr
Closed

ct: initialize StdFds for restored containers#485
avagin wants to merge 6 commits into
docker-archive:criufrom
avagin:cr

Conversation

@avagin

@avagin avagin commented Mar 26, 2015

Copy link
Copy Markdown
Contributor

avagin added 2 commits March 26, 2015 14:53
In this case CRIU will exit after restoring processes. Here is
no reason to wait the init process.

Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Andrey Vagin <avagin@openvz.org>
@avagin

avagin commented Mar 26, 2015

Copy link
Copy Markdown
Contributor Author

@crosbymichael

Copy link
Copy Markdown
Contributor

Shouldn't stdfds be part of Process and not config?

avagin referenced this pull request Mar 26, 2015
This is work in progress, integrating C/R support from
https://github.com/SaiedKazemi/docker/tree/cr into the
new libcontainer and nsinit.

Signed-off-by: Saied Kazemi <saied@google.com>
Removed debug code, renamed StdFds json property, and added comments
for additional code to be written later.

Signed-off-by: Saied Kazemi <saied@google.com>
avagin added 3 commits March 28, 2015 09:19
Minor clean ups and edits in C/R code
Signed-off-by: Andrey Vagin <avagin@openvz.org>
Signed-off-by: Andrey Vagin <avagin@openvz.org>
@avagin

avagin commented Apr 2, 2015

Copy link
Copy Markdown
Contributor Author

We are going to use another version of this code #486

@avagin avagin closed this Apr 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants