Skip to content

switch shim log Windows client/server direction#2536

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
jterry75:fix_log_pipe_windows
Aug 8, 2018
Merged

switch shim log Windows client/server direction#2536
crosbymichael merged 1 commit intocontainerd:masterfrom
jterry75:fix_log_pipe_windows

Conversation

@jterry75
Copy link
Copy Markdown
Contributor

@jterry75 jterry75 commented Aug 8, 2018

Switches the client/server direction of the shim-log pipe on Windows so
that the shim is the listener. This allows the containerd client to
reconnect as needed to the log streams.

Signed-off-by: Justin Terry (VM) [email protected]

@jterry75
Copy link
Copy Markdown
Contributor Author

jterry75 commented Aug 8, 2018

@crosbymichael - PTAL

Switches the client/server direction of the shim-log pipe on Windows so
that the shim is the listener. This allows the containerd client to
reconnect as needed to the log streams.

Signed-off-by: Justin Terry (VM) <[email protected]>
@jterry75 jterry75 force-pushed the fix_log_pipe_windows branch from 97878dc to 8362d9a Compare August 8, 2018 19:58
@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 8, 2018

Codecov Report

Merging #2536 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2536   +/-   ##
=======================================
  Coverage   45.05%   45.05%           
=======================================
  Files          94       94           
  Lines        9796     9796           
=======================================
  Hits         4414     4414           
  Misses       4662     4662           
  Partials      720      720
Flag Coverage Δ
#linux 49.08% <ø> (ø) ⬆️
#windows 41.59% <ø> (+0.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c85ae2...8362d9a. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Member

@ehazlett ehazlett left a comment

Choose a reason for hiding this comment

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

LGTM

@crosbymichael crosbymichael merged commit b9eeaa1 into containerd:master Aug 8, 2018
@jterry75 jterry75 deleted the fix_log_pipe_windows branch August 8, 2018 21:10
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.

4 participants