Add configurable ports for CodeAssist services (except zero-style UI)#24
Open
xailong-6969 wants to merge 2 commits intogensyn-ai:mainfrom
Open
Add configurable ports for CodeAssist services (except zero-style UI)#24xailong-6969 wants to merge 2 commits intogensyn-ai:mainfrom
xailong-6969 wants to merge 2 commits intogensyn-ai:mainfrom
Conversation
Add configurable ports for codeassist services and update port references - Add port configuration parameters to Config dataclass for Ollama, State Service, Solution Tester, and Policy Model - Add CLI arguments --ollama-port, --state-service-port, --solution-tester-port, --policy-model-port with default values - Update container port mappings to use configurable ports instead of hardcoded values - Update service health checks and API calls to use configured ports - Pass config object to await_testing_queue_completion and upload_to_huggingface functions - Update training configuration to dynamically set state_service_url based on configured port - Zero-style UI remains on fixed port 3003
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
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.
This change addresses the need for flexible port configuration in CodeAssist services. Previously, most service ports were hardcoded, which could cause conflicts when multiple instances are running or when specific ports are already in use on a system.
This PR adds configurable port parameters for the following CodeAssist services:
Note: The Zero-style UI remains on fixed port 3003 as indicated in the PR title.