-
Notifications
You must be signed in to change notification settings - Fork 75
Upgrade kubernetes to v1.10.0 #68
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
Conversation
Signed-off-by: pgayvallet <pierre.gayvallet@gmail.com>
Signed-off-by: pgayvallet <pierre.gayvallet@gmail.com>
CI here lacks the capability to actually boot a VM with Kube in a reasonable time so I've taken to running the tests from the Unfortunately the result here is not good. There are 4 tests covering the cross-product of Docker vs CRI for runtime and bridge vs weave for networking. I'm seeing:
The cri ones both failed with:
While the docker ones both failed with:
Not sure what's going on here -- did you manage to boot/initialise a node with this? I'm going to repeat the tests in case I did something wrong with my builds. |
Repeated and have got the first two failures. Looking at the in-progress logs for the third I fully expect it is going to fail after 1900s too. |
FWIW master (018027c) passes, starting to dig a bit deeper now. |
I don't think this PR is complete. e.g.
I don't think they are the cause of the failure but seem like something which should be addressed at this point. |
I wonder if this is the root issue:
|
In https://kubernetes.io/docs/imported/release/notes/#before-upgrading I notice:
So it may be that there is a dependency on a newer cri (which I think might mean the one embedded in containerd v1.1) |
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.
We need to sort out the rtf test failures before merging.
I think I've found a fix (well, more of a workaround really) for the docker parts, still need to dig into the cri half. |
Lots more changes are needed to make this work so I'm going to carry this one. |
No description provided.