-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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 v2.7.2 - Cherry picked the commits from main #3620
Release v2.7.2 - Cherry picked the commits from main #3620
Conversation
…t connection draining (kubernetes-sigs#3577) * Add example annotation for NLB unhealthy target connection draining * Add emtpyline back in
…#3608) due to missing cache
Fixes kubernetes-sigs#3431 Signed-off-by: Alex Berger <alex-berger@gmx.ch>
…sigs#3618) * Add a note to recommend to use compatible chart and image versions * Update golang.org/protobuf version to fix CVE-2024-24786
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release-2.7 #3620 +/- ##
===============================================
- Coverage 55.34% 55.29% -0.05%
===============================================
Files 149 149
Lines 8973 8988 +15
===============================================
+ Hits 4966 4970 +4
- Misses 3668 3679 +11
Partials 339 339 ☔ View full report in Codecov by Sentry. |
Hey @shraddhabang is it possible to include commit ac21af6? |
Hey @ikosenn , We are planning release of your commit for v2.8.0 as its a big change for a patch release and we need to get internal security sign off for this feature addition. The current ETA for next minor release is end of April. |
Thanks for the quick response. Makes it sound serious. Might you have an ETA on when 2.8.0 might be out? |
No worries. This is a standard process for such a large contribution. We are tracking it for end of April or early May. |
Thanks. Will keep an eye on 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
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: M00nF1sh, shraddhabang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
Release v2.7.2 - Cherry picked the commits from main
Ran and verified E2E tests on the test image with these commits.
Checklist
README.md
, or thedocs
directory)BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯