From 21e5b0d6d04c1a08584769e1f9de02f6ea64f92d Mon Sep 17 00:00:00 2001 From: Laurence Jones Date: Tue, 27 Sep 2022 15:20:30 +0100 Subject: [PATCH] Improvement: Docker one shot error message (#1666) * In one shot, user would only specify one container? --- pkg/acquisition/modules/docker/docker.go | 2 +- pkg/acquisition/modules/docker/docker_test.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/acquisition/modules/docker/docker.go b/pkg/acquisition/modules/docker/docker.go index 0d3404567..998860d7b 100644 --- a/pkg/acquisition/modules/docker/docker.go +++ b/pkg/acquisition/modules/docker/docker.go @@ -321,7 +321,7 @@ func (d *DockerSource) OneShotAcquisition(out chan types.Event, t *tomb.Tomb) er t.Kill(nil) if !foundOne { - return fmt.Errorf("no docker found, can't run one shot acquisition") + return fmt.Errorf("no container found named: %s, can't run one shot acquisition", d.Config.ContainerName[0]) } return nil diff --git a/pkg/acquisition/modules/docker/docker_test.go b/pkg/acquisition/modules/docker/docker_test.go index 2be558620..8928c1ae2 100644 --- a/pkg/acquisition/modules/docker/docker_test.go +++ b/pkg/acquisition/modules/docker/docker_test.go @@ -267,7 +267,7 @@ func TestOneShot(t *testing.T) { }{ { dsn: "docker://non_exist_docker", - expectedErr: "no docker found, can't run one shot acquisition", + expectedErr: "no container found named: non_exist_docker, can't run one shot acquisition", expectedOutput: "", expectedLines: 0, logType: "test",