Skip to content

Updated the go-winio library to release 0.4.8 that has the fix for Windows Container#1157

Merged
thaJeztah merged 1 commit intodocker:masterfrom
AzureCR:master
Jun 29, 2018
Merged

Updated the go-winio library to release 0.4.8 that has the fix for Windows Container#1157
thaJeztah merged 1 commit intodocker:masterfrom
AzureCR:master

Conversation

@mnltejaswini
Copy link
Copy Markdown
Contributor

@mnltejaswini mnltejaswini commented Jun 29, 2018

- What I did
I updated the Microsoft/go-winio vendor package to 0.4.8 that has the bug fix release that adds support for pipe message read mode, allowing the pipe functionality to work inside Hyper-V-isolated Windows containers.

- How I did it
Updated the vendor.conf
- How to verify it
This should fix the moby/moby#36562 (comment)
- Description for the changelog

Updated the Microsoft/go-winio vendor package to 0.4.8 that has the bug fix release that adds support for pipe message read mode, allowing the pipe functionality to work inside Hyper-V-isolated Windows containers.

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

@mnltejaswini
Copy link
Copy Markdown
Contributor Author

@GordonTheTurtle How do we investigate as to why the validate failed?

@mat007
Copy link
Copy Markdown
Member

mat007 commented Jun 29, 2018

@mnltejaswini I believe you need to run vndr to update the vendored files and commit them along vendor.conf.

@mnltejaswini
Copy link
Copy Markdown
Contributor Author

Thank you @mat007

@thaJeztah
Copy link
Copy Markdown
Member

all green now; looks like it needs a squash though 😅 - let me know if you need a hand with that

…ndows containers

Signed-off-by: Tejaswini Duggaraju <naduggar@microsoft.com>
Copy link
Copy Markdown
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.

all green and squashed, thank you so much!

LGTM

@thaJeztah thaJeztah merged commit 34ba66b into docker:master Jun 29, 2018
@GordonTheTurtle GordonTheTurtle added this to the 18.07.0 milestone Jun 29, 2018
@mnltejaswini
Copy link
Copy Markdown
Contributor Author

@thaJeztah Thank you for merging. when can we expect the next release of the CLI, so we can use it

@thaJeztah
Copy link
Copy Markdown
Member

@mnltejaswini I opened cherry-pick PR's for 18.06, so will likely be in 18.06 RC2. For Docker EE, I expect it to be in the next 18.03 EE patch release. Don't have date for both 😅

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.

5 participants