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
cherry pick of knp 0.0.27 #107037
cherry pick of knp 0.0.27 #107037
Conversation
@jdnurme: This cherry pick PR is for a release branch and has not yet been approved by Release Managers. To merge this cherry pick, it must first be approved ( AFTER it has been approved by code owners, please ping the kubernetes/release-managers team in a comment to request a cherry pick review. (For details on the patch release process and schedule, see the Patch Releases page.) Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Hi @jdnurme. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
please update the description and release note referencing the memory leak being fixed (and the |
undo of utils update
/triage accepted |
/retest |
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 - Thanks for the backport @jdnurme ! :)
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jdnurme, liggitt, rata 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 |
/remove-triage accepted |
/triage accepted |
Who can/should approve the cherry-pick ? |
ping @kubernetes/release-managers |
Is only 1.23 affected or should we cherry-pick to other branches as well? |
@xmudrii other branches are affected as well. This should also be backported to 1.21 and 1.22 too @jkh52 did some changes to allow this to be backported as lower as 1.21 (see kubernetes-sigs/apiserver-network-proxy@084136e) |
@rata Thanks for the update! Can you please also create other cherry-picks and get them approved? We tend to prefer to merge cherry-picks fixing the same issue across release branches at the same time. |
@xmudrii thanks, I've just created the others :) |
@jdnurme please take a look if you can you too :) |
EDIT: please updates the server and agent images to match which is proper per https://github.com/kubernetes-sigs/apiserver-network-proxy/blob/master/RELEASE.md /lgtm |
@jkh52 who are you talking to? I don't have permissions to do anything in that doc. So, I assume you are talking to @jdnurme ? Also, what images are missing update? This PR does update the manifests, and the tags exist in the anp repo (https://github.com/kubernetes-sigs/apiserver-network-proxy/tags). What is missing? |
Sorry for causing confusion, too many tabs open. I mean that feeback for #107069. This PR looks correct. |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Updates konnectivity-network-proxy to v0.0.27. This includes a memory leak fix for the network proxy
see: #106922
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: