-
Notifications
You must be signed in to change notification settings - Fork 351
Open
Labels
type: docsImprovement to the documentation for an API.Improvement to the documentation for an API.
Description
Description
I think there is a mistake in the example here.
It feels very unlikely that the /startup and /liveness routes are on different ports, but in the example one is referring to 9090 and the other 9801.
Indeed, just below there is a comment which suggests they should be on the same port:
# Enable HTTP healthchecks on port 9801. This enables /liveness,
# /readiness and /startup health check endpoints. Allow connections
# listen for connections on any interface (0.0.0.0) so that the
# k8s management components can reach these endpoints.
- name: CSQL_PROXY_HEALTH_CHECK
value: "true"
- name: CSQL_PROXY_HTTP_PORT
value: "9801"
- name: CSQL_PROXY_HTTP_ADDRESS
value: 0.0.0.0
Potential Solution
I expect this is probably a typo and the ports should be the same.
Additional Details
No response
Metadata
Metadata
Assignees
Labels
type: docsImprovement to the documentation for an API.Improvement to the documentation for an API.