-
Notifications
You must be signed in to change notification settings - Fork 742
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
Update Calico version to v3.8.1 #554
Conversation
I notice now that some of changes I've made are also in #504 |
Hey @lmm, thanks a lot. I'll take a look at this before making a v1.6.0 release |
@mogren WDYT about making these changes for 1.5.2 also (which I see an RC is up for)? |
@tmjd Hey, sorry, already cut the v1.5.2 release. Will try to get v1.6.0 out before the end of this month. |
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.
Tested it and the changes seem fine. Thanks for updating this config!
This is still listed as being in the 1.5.3 release, but that doesn't seem to be the case for the upgrade to v3.8.1 which looks to have been pushed to 1.6.0. |
@nickdgriffin Hey, thanks for catching this, I'll update the configs. |
@mogren No problem, was just reviewing what I needed for an upgrade and noticed the discrepancy - thanks for sorting! |
@mogren I think it's needed on the 1.5.4 branch too, as that's missing it. This also means that anyone that uses the tag (like I did) to get the appropriate version will still get the old one unless they are remade. |
@nickdgriffin I had it locally, forgot to push. Thanks for the reminder! |
Description of changes:
v3.3.6
tov3.8.1
v3.8.1
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.