checkpoint: don't print error if --pre-dump is set

Obviously, either --pre-dump or --leave-running implies
that we don't want to remove the container.

Fixes: 87712d288
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
This commit is contained in:
Kir Kolyshkin
2020-04-18 18:20:34 -07:00
parent bf0a8e1747
commit 3006db2bfe
+2 -6
View File
@@ -63,12 +63,8 @@ checkpointed.`,
fatalf("Container cannot be checkpointed in %s state", status.String()) fatalf("Container cannot be checkpointed in %s state", status.String())
} }
options := criuOptions(context) options := criuOptions(context)
if !options.LeaveRunning || !options.PreDump { if !(options.LeaveRunning || options.PreDump) {
// destroy prints out an error if we tell CRIU to // destroy container unless we tell CRIU to keep it
// leave the container running:
// ERRO[0000] container is not destroyed
// The message is correct, but we actually do not want
// to destroy the container in this case.
defer destroy(container) defer destroy(container)
} }
// these are the mandatory criu options for a container // these are the mandatory criu options for a container