-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Release v1.8.0 #2516
Release v1.8.0 #2516
Conversation
maciaszczykm
commented
Oct 24, 2017
•
edited by floreks
Loading
edited by floreks
- update API versions in YAML files using versions from https://kubernetes.io/docs/api-reference/v1.8/ as the release is compatible with 1.8.x only
- update versions in the code
- Release v1.8.0 #2516 (comment)
- tests with HEAD image
- fix User panel bug #2597
- Update screenshot in README
Codecov Report
@@ Coverage Diff @@
## master #2516 +/- ##
==========================================
+ Coverage 53.63% 54.15% +0.52%
==========================================
Files 562 563 +1
Lines 11951 12124 +173
==========================================
+ Hits 6410 6566 +156
+ Misses 5319 5301 -18
- Partials 222 257 +35
Continue to review full report at Codecov.
|
TODO: Update official yamls similar to head yamls (use auto generate certificate feature). |
Remove watch access rule
Is this ready for testing? |
Yes. We have already tested it. |
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.
LGTM, tested @floreks please do final review and merge
LGTM |
Ok, deferring this to Monday. |
#2617 is not a proper fix. I have updated already the yaml in this PR. It is ready to merge. |