Skip to content

Conversation

@xailong-6969
Copy link

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:

  • Ollama (default: 11434) - configurable via --ollama-port
  • State Service (default: 8000) - configurable via --state-service-port
  • Solution Tester (default: 8008) - configurable via --solution-tester-port
  • Policy Model (default: 8001) - configurable via --policy-model-port
  1. Added port configuration fields to the Config dataclass
  2. Added CLI arguments for each service port with sensible defaults
  3. Updated all container port mappings to use the configured ports instead of hardcoded values
  4. Updated service health checks and API calls to reference the configured ports
  5. Modified training configuration to dynamically set state_service_url based on the configured port

Note: The Zero-style UI remains on fixed port 3003 as indicated in the PR title.

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
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

Successfully merging this pull request may close these issues.

1 participant