X Tutup
Skip to content

info: print errors to stderr#3044

Merged
thaJeztah merged 1 commit intodocker:masterfrom
cpuguy83:fix_info_output
Jul 13, 2021
Merged

info: print errors to stderr#3044
thaJeztah merged 1 commit intodocker:masterfrom
cpuguy83:fix_info_output

Conversation

@cpuguy83
Copy link
Collaborator

@cpuguy83 cpuguy83 commented Apr 8, 2021

Errors always need to go to stderr.
This also fixes a test in moby/moby's integration-cli which is checking
to see if errors connecting to the daemon are output on stderr.

Errors always need to go to stderr.
This also fixes a test in moby/moby's integration-cli which is checking
to see if errors connecting to the daemon are output on stderr.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member

Tests were failing because the .golden files needed to be updated; I pushed the test-updates to this branch

@codecov-commenter
Copy link

Codecov Report

Merging #3044 (4ce521c) into master (5f07d7d) will increase coverage by 1.48%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3044      +/-   ##
==========================================
+ Coverage   57.08%   58.57%   +1.48%     
==========================================
  Files         299      299              
  Lines       18756    21501    +2745     
==========================================
+ Hits        10707    12594    +1887     
- Misses       7178     7984     +806     
- Partials      871      923      +52     

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah
Copy link
Member

@silvin-lubecki @StefanScherer PTAL

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah thaJeztah merged commit 9262fad into docker:master Jul 13, 2021
@thaJeztah thaJeztah added this to the 21.xx milestone Jul 13, 2021
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.

4 participants

X Tutup