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
Use cos-stable-59-9460-64-0 instead of cos-beta-59-9460-20-0. #47530
Use cos-stable-59-9460-64-0 instead of cos-beta-59-9460-20-0. #47530
Conversation
Hi @mindprince. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
@k8s-bot ok to test |
109ec4a
to
e601514
Compare
/assign @gmarek @pwittrock |
/approve |
cc/ @abgworrall Adam has a pending pr to upgrade cos image to another minor version. Can we collaborate together to include all required fixes? I am leaving @abgworrall to make the final call here. |
@abgworrall I can update this PR to use |
- It contains a fix for ipaliasing. - It contains a fix which decouples GPU driver installation from kernel version. Remove dead code that has now moved to another repo as part of kubernetes#47467
e601514
to
3a86c97
Compare
Updated to include #47670 |
@bowei , this is now the PR that should precede your tests getting re-enabled. |
@abgworrall Can you add milestone 1.7 to this, so that this gets merged. |
I'm not in kubernetes org either, so I can't help :/
…On Fri, Jun 16, 2017 at 4:22 PM Rohit Agarwal ***@***.***> wrote:
@abgworrall <https://github.com/abgworrall> Can you add milestone 1.7 to
this, so that this gets merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#47530 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AJ9WxK7QGLoIPU7LcWyWLHm_DSwH6qpAks5sEw45gaJpZM4N6MUe>
.
|
/cc @dchen1107 |
/lgtm |
/lgtm Ok, I changed my mind. I discussed with @abgworrall earlier to wait for Monday morning to merge this pr since it changes the default COS image which most tests are running on, and there are many pending pr waiting for being merged. But I looked at the testgrid which @abgworrall ran with, cos-stable-59-9460-64-0 looks pretty good. Thanks! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dchen1107, dims, mindprince, vishh Associated issue: 47467 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
@mindprince: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Automatic merge from submit-queue (batch tested with PRs 47530, 47679) |
Remove dead code that has now moved to another repo as part of #47467
Release note:
/sig node