From 6eb5720233ddbf00a614e82524ade35a0e3cfc5c Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Mon, 22 Mar 2021 13:20:40 +0100 Subject: [PATCH] Fix daemon.getExecConfig(): not using typed errNotRunning() error This makes daemon.getExecConfig return a errdefs.Conflict() error if the container is not running. This was originally the case, but a refactor of this code changed the typed error (`derr.ErrorCodeContainerNotRunning`) to a non-typed error; a793564b2591035aec5412fbcbcccf220c773a4c Signed-off-by: Sebastiaan van Stijn --- daemon/exec.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/daemon/exec.go b/daemon/exec.go index 78a593a6e2..302842196b 100644 --- a/daemon/exec.go +++ b/daemon/exec.go @@ -59,7 +59,7 @@ func (daemon *Daemon) getExecConfig(name string) (*exec.Config, error) { return nil, containerNotFound(name) } if !ctr.IsRunning() { - return nil, fmt.Errorf("Container %s is not running: %s", ctr.ID, ctr.State.String()) + return nil, errNotRunning(ctr.ID) } if ctr.IsPaused() { return nil, errExecPaused(ctr.ID)