Specify port for healthcheck #132
-
|
I'm working on setting up The main port that everything runs on for the container is Looking at the logs for the container, it's running fine and not reporting errors, it's continually collecting metrics, I just can't get kamal proxy to see it as healthy and start sending it traffic. It's just reporting the regular I also added a port mapping so that I could check the status of the container locally on the machine and it's returning a 200. |
Beta Was this translation helpful? Give feedback.
Replies: 8 comments
-
|
@nickhammond the health check path doesn't accept a port. It's purely a path portion, and will always be fetched using the target's host & port. So in this case, is the problem that there's no suitable path on the target's main port that could signify whether that target is healthy or not? |
Beta Was this translation helpful? Give feedback.
-
|
@kevinmcconnell Yes, because the only endpoints available on the port that needs to be open( |
Beta Was this translation helpful? Give feedback.
-
|
@kevinmcconnell Another potential option would be running a health check command that could check the status but I know that's not currently supported. |
Beta Was this translation helpful? Give feedback.
-
|
@kevinmcconnell What's the environment that the healthcheck runs in? Can I publish another port locally and use that for the healthcheck instead of the primary one? Just trying to think of alternative options. |
Beta Was this translation helpful? Give feedback.
-
|
@kevinmcconnell Any suggested workarounds for this? |
Beta Was this translation helpful? Give feedback.
-
|
Hi @nickhammond & @kevinmcconnell I'm facing similar issue and I also need kamal to allow port customization for health check. I just raised a PR: #152 |
Beta Was this translation helpful? Give feedback.
-
|
@giosakti Nice! Just posted a question but this would be great to get added in. |
Beta Was this translation helpful? Give feedback.
-
|
I've merged the associated PR for this (#152), and we'll get it into the next release. |
Beta Was this translation helpful? Give feedback.
I've merged the associated PR for this (#152), and we'll get it into the next release.