-
Notifications
You must be signed in to change notification settings - Fork 23
K8SPXC-1538 Added Exposing HAproxy section #225
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
K8SPXC-1538 Added Exposing HAproxy section #225
Conversation
…nheritance by replica proxy
docs/haproxy-conf.md
Outdated
@@ -72,6 +72,24 @@ until the upgraded Percona XtraDB Cluster member becomes synced, and then | |||
proceeds to upgrade the next member. When the upgrade is finished for all | |||
the readers, then the writer Percona XtraDB Cluster member is finally upgraded. | |||
|
|||
## Exposing HAProxy | |||
|
|||
You can expose HAProxy, so that clients can connect to your database cluster from the outside. To do so, you need to set the service type `LoadBalancer` for the `haproxy-primary` service. Additionally, list the client IP addresses from which the load balancer should be reachable in the `loadBalancerSourceRanges` option. |
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.
it is optional option
docs/haproxy-conf.md
Outdated
spec: | ||
haproxy: | ||
exposePrimary: | ||
enabled: true |
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.
we are deprecating the enabled field, so i think it should be removed
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.
In what release are we deprecating it? What is used instead?
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.
Added a confif for replicas since exposeReplicas remains
Documented the setup inheritance by replica proxy