-
Notifications
You must be signed in to change notification settings - Fork 596
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
Restoring of docker container failed after restarting docker daemon #26
Comments
By CRIU -- yes :) It looks like some in-Docker problem with states management. @boucher, @SaiedKazemi , @avagin . |
Ok. Here is the fork of docker that I'm using.
|
I think this may be fixed in my branch if you try the latest version. |
@boucher Tried your branch but now restore failed (without restarting docker daemon). Here is the restore log.
|
Can you tell me the exact commands you ran? |
I think it's due to the shell-job option. We should not set this option to dump a container. |
Yes, I think so too, just wanted to make sure. I misunderstood how the shell job flag works. I'll revert that particular change now. |
@kimh If you pull again I think it will work. |
@boucher Pull which project? CRIU or docker? I pulled master branch of CRIU and https://github.com/boucher/docker/tree/cr-combined but I don't see your revert. (FYI I recompiled criu with latest master but the issue was not fixed). |
Oh, btw, I run these commands:
|
I think that what both guys said is, that you should run with the --allow-shell option
|
There was no revert commit, I force pushed the change. I reverted the behavioral change though, there's no longer a --no-shell option, and its back to being --allow-shell, which you don't want here. Are you sure you're running commit 792fc89ab? Can you check the log to see that it isn't a new restore error? |
Ah, I didn't think about this, so haven't pulled docker. Yes, using current 792fc89ab worked for me. Now I can restore a container even after restarting docker daemon.
Is this because I run the container without shell? |
Yeah, it's a flag saying that the container is a shell. |
I see. Thanks, @boucher I'll close the issue. |
I experimented if I could restore a container after rebooting my docker daemon but I got
Error response from daemon: Container f6b3120b3fce is not checkpointed
error.Is this supported by CRIU?
The text was updated successfully, but these errors were encountered: