Skip to content
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

[WFLY-18415] Support internal EJB client access to deployments on Ope… #534

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tadamski
Copy link
Contributor

@tadamski tadamski commented Sep 5, 2023

…nShift over HTTP

Please don't merge. This analysis is in initial state and the PR was open to take advantage of github in the design discussions.

@tadamski tadamski added the hold label Sep 5, 2023
@rachmatowicz
Copy link

rachmatowicz commented Sep 6, 2023

@tadamski if you could convert this to a draft pull request, it wouldn't be merged until out of draft state.

Just FYI, mod-cluster is not (and is not currently planned to be) suitable for deployment in OpenShift, asked Rado specifically about this. The recommendation is to use a cloud native load balancer, such as a service.

@rachmatowicz
Copy link

Also, you mention stateful sets and persistent IP addresses: stateful sets don't provide stable IP addresses, but only stable hostnames (e.g. web-0, web-1, web-2): https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#stable-network-id

@rachmatowicz
Copy link

My view: if we assume that the deployment is via the Wildfly operator and that the EJB client application makes use of the "http" protocol (and so the wildfly-http-client library), all that is needed is an intermediary service between the client and the deployment which understands cookie based affinity management. If OpenShift provides such a service then that could be used, otherwise a custom cloud-native service could be written and deployed via the operator.

@tadamski tadamski marked this pull request as draft September 6, 2023 21:15
@rachmatowicz
Copy link

This note about internal load balancers may be of interest: https://kubernetes.io/docs/concepts/services-networking/service/#internal-load-balancer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants