-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Version and validation updates for romana networking. #3892
Conversation
@cgilmour Is updating an exsting cluster from romana preview.2 to GA also supported? (Given it is on k8s 1.7/1.8?) |
@Globegitter yes. |
/ok-to-test /lgtm Thanks for the update @cgilmour ! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justinsb 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 |
travis is taking the day off. Closing and re-openning |
Grumble ... Travis what is going on :( |
@justinsb PTAL in the AM see if travis gets a clue |
Aha! Travis says "abuse detected": https://travis-ci.org/kubernetes/kops/requests Will see if I can figure out what that actually means, and then force-merge after local testing |
This is a small update for Romana to update the version being installed, and make sure the version validation is correct - it now requires v1.7.x or higher.
It has been tested on AWS only, with a variety of versions:
Hoping this can be squeezed into the upcoming 1.8 release.