Skip to content

Commit

Permalink
Propagate GetContainer error from event processor
Browse files Browse the repository at this point in the history
Before this change we just accept that any error is "not found" and it
could be something else, but even if it it is just a "not found" kind of
error this should be dealt with from the container store and not the
event processor.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
  • Loading branch information
cpuguy83 committed Jul 11, 2019
1 parent 2fc3480 commit 54e30a6
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions daemon/monitor.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ package daemon // import "github.com/docker/docker/daemon"

import (
"context"
"errors"
"fmt"
"runtime"
"strconv"
"time"
Expand All @@ -12,6 +10,7 @@ import (
"github.com/docker/docker/container"
libcontainerdtypes "github.com/docker/docker/libcontainerd/types"
"github.com/docker/docker/restartmanager"
"github.com/pkg/errors"
"github.com/sirupsen/logrus"
)

Expand All @@ -29,8 +28,8 @@ func (daemon *Daemon) setStateCounter(c *container.Container) {
// ProcessEvent is called by libcontainerd whenever an event occurs
func (daemon *Daemon) ProcessEvent(id string, e libcontainerdtypes.EventType, ei libcontainerdtypes.EventInfo) error {
c, err := daemon.GetContainer(id)
if c == nil || err != nil {
return fmt.Errorf("no such container: %s", id)
if err != nil {
return errors.Wrapf(err, "could not find container %s", id)
}

switch e {
Expand Down

0 comments on commit 54e30a6

Please sign in to comment.