-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Update kubernetes version to 1.7 or newer. #694
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
Jenkins job istio.github.io/presubmit passed |
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.
it doesn’t work with 1.7.2 which is why it says 1.7.4 or newer
@ldemailly what is the reason of 1.7.2 does not work, at least it works for 1.7.3 in my env. |
I forget but there was an issue - going to close this for now |
@ldemailly I think that we need to update kubectl version to 1.7.3 as 1.7.3 also works, I got some question from customers about the version who is using 1.7.3, so better update to 1.7.3 |
All of the 1.7 kubernetes can works well with istio CRD resources, we should update it to avoid confuse.
Jenkins job istio.github.io/presubmit passed |
2 similar comments
Jenkins job istio.github.io/presubmit passed |
Jenkins job istio.github.io/presubmit passed |
Updating to 1.7.3 is fine with me given Guang has tested it and we didn't recall any issue but we picked 1.7.4 given it was the latest. |
I am pretty sure there is a bug with older than 1.7.4 but maybe it was only initializer related ? or was it about validation hooks ? @andraxylia do you remember? |
@ldemailly We have a wechat group for istio in China and I got some questions for the kubernetes version for istio, and some people are using 1.7.3 and they also did some test and found istio works well. I think that we should update the document so as not bring confuse, comments? |
sorry to keep repeating the same thing but I'm sure there was a reason (it's not that we picked the latest at the time) - maybe if you go over the release notes it may show up but I hope @andraxylia or @ayj or someone remembers |
I don't recall the reason. Maybe a GKE specific requirement since they bump up the latest minor version frequently and make previous minor versions unavailable for new clusters? |
I searched a bit and didn't find except that we put 1.7.4 for the first time in 0.2.2 |
Jenkins job istio.github.io/presubmit passed |
Thanks for digging around, I'll merge it. |
All of the 1.7 kubernetes can works well with istio CRD resources,
we should update it to avoid confuse.
/cc @linsun