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

Feature Request - Add go-sockaddr for health check templating #887

Open
kalenarndt opened this issue May 19, 2022 · 1 comment
Open

Feature Request - Add go-sockaddr for health check templating #887

kalenarndt opened this issue May 19, 2022 · 1 comment

Comments

@kalenarndt
Copy link

Description

Currently in 0.60 beta there is an option to specify the url for a health check of the CTS worker. Depending the deployment a Consul agent might not be running locally which can cause issues with the health check. In environments with ephemeral IPs it would be beneficial to allow for the use of the go-sockaddr library to enumerate the IP address of the worker for the health check.

Use Cases

Health checks for the CTS service without having to use a Consul agent locally.

Alternative Solutions

Solution 1 - If you are using Kubernetes you could create a custom service (ClusterIP) and then use that service via kubedns for the health check of the worker. Down side with this is you would need 1 service per worker

Solution 2 - If you are on Kubernetes use the metadata from the pod status to pass in the IP as an environment variable and use additional templating to specify this IP as the health check URL (requires an container port to be defined)

Solution 3 - Run the consul agent where the CTS worker currently resides (standard for most deployments)

Solution 4 - I haven't tried this yet but you could configure liveness probes and configure the CTS worker in connect so consul incorporates those default checks in Kubernetes. https://www.consul.io/docs/k8s/connect/health

Additional context

This issue

@devarshishah3
Copy link
Contributor

@kalenarndt - Does populating the service_address variable using an Env variable suffice for the non-local Consul agents?

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

No branches or pull requests

2 participants