-
Notifications
You must be signed in to change notification settings - Fork 320
chore: update kube-rbac-proxy args to disable http2 #614
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
chore: update kube-rbac-proxy args to disable http2 #614
Conversation
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: varshab1210 <[email protected]>
Signed-off-by: Siddhesh Ghadi <[email protected]>
This reverts commit 6cff7f8. Previous changes work as expected. Signed-off-by: Siddhesh Ghadi <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
Signed-off-by: Jaideep Rao <[email protected]>
/lgtm |
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: iam-veeramalla 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 |
…#614) * update to go 1.20; disable http2 for servers; upgrade k8s packages Signed-off-by: Jaideep Rao <[email protected]> * remove secure serving option for metrics Signed-off-by: Jaideep Rao <[email protected]> * Changing go version for CI Signed-off-by: varshab1210 <[email protected]> * Disable http/2 on webhook server Signed-off-by: Siddhesh Ghadi <[email protected]> * Revert "Disable http/2 on webhook server" This reverts commit 6cff7f8. Previous changes work as expected. Signed-off-by: Siddhesh Ghadi <[email protected]> * consume keycloak segmentation fault fix Signed-off-by: Jaideep Rao <[email protected]> * update kube-rbac-proxy image Signed-off-by: Jaideep Rao <[email protected]> * undo makefile changes Signed-off-by: Jaideep Rao <[email protected]> * disable http2 for kube-rbac-proxy Signed-off-by: Jaideep Rao <[email protected]> * switch to floating tag for kube-rbac-proxy image Signed-off-by: Jaideep Rao <[email protected]> * consume argocd-operator commit Signed-off-by: Jaideep Rao <[email protected]> * remove http2 disable command line arg Signed-off-by: Jaideep Rao <[email protected]> * disable http2 in kube-rbac-proxy container Signed-off-by: Jaideep Rao <[email protected]> --------- Signed-off-by: Jaideep Rao <[email protected]> Signed-off-by: varshab1210 <[email protected]> Signed-off-by: Siddhesh Ghadi <[email protected]> Co-authored-by: varshab1210 <[email protected]> Co-authored-by: Siddhesh Ghadi <[email protected]>
What type of PR is this?
What does this PR do / why we need it:
disables http2 for kube-rbac-proxy container after picking up patched version of the downstream kube-rbac-proxy image
Have you updated the necessary documentation?
Which issue(s) this PR fixes:
Fixes #?
Test acceptance criteria:
How to test changes / Special notes to the reviewer: