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
Automated cherry pick of #87933 upstream release 1.17 1582743755 #88586
Automated cherry pick of #87933 upstream release 1.17 1582743755 #88586
Conversation
/assign @yujuhong |
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
To ensure I'm understanding, with this change, the 1.17 branch will now pass the conformance tests which were previously failing? That seems like a worthy reason for a cherry-pick, especially given the isolated scope of this change.
Out of curiosity, should we also cherry-pick to 1.16 and 1.15? Tbh... I'm not positive about the actual instances in which folks would be running the specific conformance tests you saw breaking.
Yes, this change allows conformance tests to pass w/ legacy docker runtime, as described above. |
On Thu, Feb 27, 2020 at 09:01:12AM -0800, James DeFelice wrote:
> /lgtm
>
> To ensure I'm understanding, with this change, the 1.17 branch will now pass the conformance tests which were previously failing? That seems like a worthy reason for a cherry-pick, especially given the isolated scope of this change.
>
> Out of curiosity, should we also cherry-pick to 1.16 and 1.15? Tbh... I'm not positive about the actual instances in which folks would be running the specific conformance tests you saw breaking.
Yes, this change allows conformance tests to pass w/ legacy docker runtime, as described above.
I haven't tested this patch against 1.15 yet, but the problem probably exists there too.
--
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub:
#88586 (comment)
Cool! While we have the cherry-pick PR for 1.17, we might as well also do for
1.15 and 1.16. If you tag me in the cherry-picks, I'm happy to mark as lgtm.
|
@yujuhong PTAL |
@tallclair PTAL; would like to land this for the next 1.17.x release |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: derekwaynecarr, jdef The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@kubernetes/patch-release-team PTAL this has been approved |
Cherry-pick of #87933 on release-1.17
#87933: Fix docker/journald logging conformance
For details on the cherry pick process, see the cherry pick requests page.
/kind bug