Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[20.10] Bump hcsshim for error details fix #42292

Merged
merged 2 commits into from
May 6, 2021

Conversation

cpuguy83
Copy link
Member

This pulls in fixes from the moby branch for hcsshim that 20.10 uses.
Not bumping up to the latest release because the diff is huge.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
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

@cpuguy83
Copy link
Member Author

Error is due to error string matching no longer working since the returned error no longer contains the container command, tracking this in #42270

Whether or not the command path is in the error message is a an
implementation detail.
For example, on Windows the only reason this ever matched was because it
dumped the entire container config into the error message, but this had
nothing to do with the actual error.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
(cherry picked from commit 225e046)
Signed-off-by: Brian Goff <cpuguy83@gmail.com>
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

@tonistiigi @tianon ptal

@thaJeztah thaJeztah merged commit e3cb5ad into moby:20.10 May 6, 2021
@moby moby deleted a comment from joelavoie7881 Jun 30, 2021
@cpuguy83 cpuguy83 deleted the 20.10_hcsshim_no_error_details branch August 17, 2021 20:30
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.

None yet

3 participants