[EAGLE-7214 ]: Add configuration options for health check port in local-runner#957
Merged
patricklundquist merged 2 commits intomasterfrom Feb 21, 2026
Merged
Conversation
- Added `--health-check-port`, `--disable-health-check`, and `--auto-find-health-check-port` flags to `clarifai model local-runner` CLI command. - Updated `ModelServer` and `ModelRunner` to accept and respect the configured health check port. - Fixed a bug in `ModelRunner` where `start_health_server_thread` was called twice, causing "Address already in use" errors. - Implemented logic to automatically find an available port if requested. - Implemented logic to disable the health check server if port is set to -1 or 0. - Passed the health check port configuration to the Docker container when running in container mode. Co-authored-by: patricklundquist <1460278+patricklundquist@users.noreply.github.com>
Contributor
|
@patricklundquist why is this? So we can run multiple local runners on the same machine? |
Contributor
|
or was this port clashing with something |
Contributor
Author
|
Port clashing |
Contributor
Author
|
Also the multiple local runners case would be nice |
Minimum allowed line rate is |
ackizilkale
approved these changes
Feb 20, 2026
patricklundquist
deleted the
local-runner-health-port-10446615120966861997
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
--health-check-port,--disable-health-check, and--auto-find-health-check-portflags toclarifai model local-runnerCLI command.ModelServerandModelRunnerto accept and respect the configured health check port.ModelRunnerwherestart_health_server_threadwas called twice, causing "Address already in use" errors.