github/containerd
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

fix monitor restart

Browse Source 
Signed-off-by: Ye Sijun <junnplus@gmail.com>
This commit is contained in:
Ye Sijun 2022-04-20 18:03:43 +08:00
parent d85ac56050
commit 35994a475c
No known key found for this signature in database
GPG Key ID: 0582626C83FA9CD0
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
runtime/restart/monitor/monitor.go
View File

@ -232,14 +232,17 @@ func (m *monitor) monitor(ctx context.Context) ([]change, error) {
} }
} }
// Task or Status return error, only desired to stop // Task or Status return error, only desired to running
if err != nil { if err != nil {
logrus.WithError(err).Error("monitor") logrus.WithError(err).Error("monitor")
if desiredStatus != containerd.Stopped { if desiredStatus == containerd.Stopped {
continue continue
} }
} }
// Known issue:
// The status may be empty when task failed but was deleted,
// which will result in an `on-failure` restart policy reconcile error.
switch desiredStatus { switch desiredStatus {
case containerd.Running: case containerd.Running:
if !restart.Reconcile(status, labels) { if !restart.Reconcile(status, labels) {