Skip to content

Fixes #2543: bug with panic when DOCKER_CLI_EXPERIMENTAL is incorrect#2544

Merged
tiborvass merged 1 commit intodocker:masterfrom
qoo2p5:master
May 28, 2020
Merged

Fixes #2543: bug with panic when DOCKER_CLI_EXPERIMENTAL is incorrect#2544
tiborvass merged 1 commit intodocker:masterfrom
qoo2p5:master

Conversation

@qoo2p5
Copy link
Contributor

@qoo2p5 qoo2p5 commented May 24, 2020

- What I did

I've fixed the bug described in #2543.

- How I did it

  1. I've added a check in cli initialization.
  2. I've added panic(err) in the place which caused panic before. It will help to find similar problems easier in the future.

- How to verify it

The steps are described in the issue. You can see that after my commit everything works as expected.

- Description for the changelog

Fixed bug with panic when DOCKER_CLI_EXPERIMENTAL is set to incorrect value.

…is incorrect

Signed-off-by: Daniil Nikolenko <qoo2p5@gmail.com>
@thaJeztah
Copy link
Member

@tiborvass @silvin-lubecki PTAL (need to check if this affects the 19.03 branch as well)

@tiborvass tiborvass merged commit 263bb53 into docker:master May 28, 2020
@tiborvass
Copy link
Collaborator

This does affect 19.03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants