add current process state to the error message

alakesh · alakesh · commit 16f3d67b5a80 · 2021-09-21T13:29:57.000-07:00
It will aid debugging if a procees delete fails, to figure out whether
it is paused or running.

Signed-off-by: Alakesh Haloi &lt;alakeshh@amazon.com&gt;
diff --git a/process.go b/process.go
@@ -210,7 +210,7 @@ func (p *process) Delete(ctx context.Context, opts ...ProcessDeleteOpts) (*ExitS
 	}
 	switch status.Status {
 	case Running, Paused, Pausing:
-		return nil, errors.Wrapf(errdefs.ErrFailedPrecondition, "process must be stopped before deletion")
+		return nil, errors.Wrapf(errdefs.ErrFailedPrecondition, "current process state: %s, process must be stopped before deletion", status.Status)
 	}
 	r, err := p.task.client.TaskService().DeleteProcess(ctx, &tasks.DeleteProcessRequest{
 		ContainerID: p.task.id,

Original file line number	Diff line number	Diff line change
`@@ -210,7 +210,7 @@ func (p process) Delete(ctx context.Context, opts ...ProcessDeleteOpts) (ExitS`
`210`	`210`	`}`
`211`	`211`	`switch status.Status {`
`212`	`212`	`case Running, Paused, Pausing:`
`213`		`- return nil, errors.Wrapf(errdefs.ErrFailedPrecondition, "process must be stopped before deletion")`
	`213`	`+ return nil, errors.Wrapf(errdefs.ErrFailedPrecondition, "current process state: %s, process must be stopped before deletion", status.Status)`
`214`	`214`	`}`
`215`	`215`	`r, err := p.task.client.TaskService().DeleteProcess(ctx, &tasks.DeleteProcessRequest{`
`216`	`216`	`ContainerID: p.task.id,`