api: To stop its monitor after a sandbox paused

After the sandbox is paused, it's needed to stop its monitor, Otherwise, its monitors will receive timeout errors if it is paused for a long time, thus its monitor will not tell it's a crash caused timeout or just a paused timeout. Fixes: #472 Signed-off-by: fupan <[email protected]>
kata-containers · Aug 23, 2018 · 9868cf2 · 9868cf2
1 parent d10b1d8
commit 9868cf2
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/virtcontainers/sandbox.go b/virtcontainers/sandbox.go
@@ -1203,6 +1203,14 @@ func (s *Sandbox) Pause() error {
 		return err
 	}
 
+	//After the sandbox is paused, it's needed to stop its monitor,
+	//Otherwise, its monitors will receive timeout errors if it is
+	//paused for a long time, thus its monitor will not tell it's a
+	//crash caused timeout or just a paused timeout.
+	if s.monitor != nil {
+		s.monitor.stop()
+	}
+
 	return s.pauseSetStates()
 }