Skip to content

Commit

Permalink
feat: Add notice for users without permission to docker daemon (#535)
Browse files Browse the repository at this point in the history
Signed-off-by: Qi Chen <aseaday@hotmail.com>
  • Loading branch information
aseaday committed Jul 1, 2022
1 parent 07cca9d commit 956fd73
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions pkg/docker/docker.go
Original file line number Diff line number Diff line change
Expand Up @@ -82,11 +82,19 @@ type generalClient struct {
}

func NewClient(ctx context.Context) (Client, error) {
cli, err := client.NewClientWithOpts(client.FromEnv)
cli, err := client.NewClientWithOpts(client.FromEnv, client.WithAPIVersionNegotiation())
if err != nil {
return nil, err
}
cli.NegotiateAPIVersion(ctx)
_, err = cli.Ping(ctx)
if err != nil {
// Special note needed to give users
if strings.Contains(err.Error(), "permission denied") {
err = errors.New(`It seems that current user have no access to docker daemon,
please visit https://docs.docker.com/engine/install/linux-postinstall/ for more info.`)
}
return nil, err
}
return generalClient{cli}, nil
}

Expand Down

0 comments on commit 956fd73

Please sign in to comment.