-
Notifications
You must be signed in to change notification settings - Fork 16.8k
[stable/keycloak] use tcp instead of udp for jgroups #9040
Conversation
Hi @dojadop. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with 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. |
36f8bc2
to
c3e5643
Compare
ff6fd5f
to
f617792
Compare
f617792
to
8c45773
Compare
/ok-to-test |
* add note on PROXY_ADDRESS_FORWARDING when using ingress * fixes helm#8355 Signed-off-by: Michael Dop <michael.p.dop@gmail.com>
Signed-off-by: Michael Dop <michael.p.dop@gmail.com>
Signed-off-by: Reinhard Nägele <unguiculus@gmail.com>
8c45773
to
a311f84
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dojadop, unguiculus 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 |
* [stable/keycloak] use tcp instead of udp for jgroups * add note on PROXY_ADDRESS_FORWARDING when using ingress * fixes helm#8355 Signed-off-by: Michael Dop <michael.p.dop@gmail.com> * Remove PROXY_ADDRESS_FORWARDING comment Signed-off-by: Michael Dop <michael.p.dop@gmail.com> * Bump version Signed-off-by: Reinhard Nägele <unguiculus@gmail.com>
Signed-off-by: Michael Dop michael.p.dop@gmail.com
What this PR does / why we need it:
The default jgroups protocol is udp so need to change back to tcp. When using udp the the nodes cannot communicate to each other and start timing out causing ha keycloak to fail when using DNS_PING.
Which issue this PR fixes
Special notes for your reviewer: