Skip to content

docker info: fix condition for printing debug information#4392

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:debug_relax
Jun 30, 2023
Merged

docker info: fix condition for printing debug information#4392
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:debug_relax

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

relates to:

The daemon collects this information regardless if "debug" is enabled. Print the debugging information if either the daemon, or the client has debug enabled.

We should probably improve this logic and print any of these if set (but some special rules are needed for file-descriptors, which may use "-1".

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

The daemon collects this information regardless if "debug" is
enabled. Print the debugging information if either the daemon,
or the client has debug enabled.

We should probably improve this logic and print any of these if
set (but some special rules are needed for file-descriptors, which
may use "-1".

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #4392 (92d7a23) into master (cb1def7) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4392   +/-   ##
=======================================
  Coverage   59.37%   59.38%           
=======================================
  Files         288      288           
  Lines       24746    24751    +5     
=======================================
+ Hits        14693    14698    +5     
  Misses       9169     9169           
  Partials      884      884           

@thaJeztah thaJeztah merged commit e413dae into docker:master Jun 30, 2023
@thaJeztah thaJeztah deleted the debug_relax branch June 30, 2023 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants