-
Notifications
You must be signed in to change notification settings - Fork 690
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 compatibility docs and release cutting notes #3796
Update compatibility docs and release cutting notes #3796
Conversation
- Adds release notes markdown template - Calls out in release process doc about what to include in release notes - Modifies notes in compatibility matrix to remove ambiguity - Minor updates in Philosophy document to bring it up to date Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
Codecov Report
@@ Coverage Diff @@
## main #3796 +/- ##
=======================================
Coverage 75.83% 75.83%
=======================================
Files 107 107
Lines 7338 7338
=======================================
Hits 5565 5565
Misses 1652 1652
Partials 121 121 |
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.
I think this looks good, just a couple nits. I'd suggest also updating https://github.com/projectcontour/contour#prerequisites to have more general language along the lines of "refer to the compatibility matrix for supported versions" so we don't have to remember to update it.
Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
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
Might need to also call out dropped support for Ingress v1beta1. I know it's removed from newer k8s versions, but could see users asking about this. |
- Clean up compatibility matrix text - Add a note to ingress doc about removal Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
notes