Skip to content

Conversation

@kamran-rapidfireAI
Copy link
Contributor

@kamran-rapidfireAI kamran-rapidfireAI commented Sep 29, 2025

  • Add --port (-p), --mlflow-port, and --api-port CLI flags
  • Apply only to start and restart; validate ports (1–65535)
  • Map flags to env vars RF_FRONTEND_PORT, RF_MLFLOW_PORT, RF_API_PORT
  • Update script runner to support env overrides
  • Print a warning if port flags are passed to non-applicable commands (e.g., init, status)
  • Defaults remain unchanged (frontend 3000, MLflow 5002, API 8080)

Examples:

  • rapidfireai start --port 4000
  • rapidfireai start --mlflow-port 6000 --api-port 9090
  • rapidfireai restart -p 4000

Changes

  • Brief description of the changes in this PR. Include breaking changes or any unusual behavior.

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this change manually
  • I have tested this change in the following environments:
    • Local development
    • Docker environment
    • Other: _______________

Screenshots (if applicable)

Add screenshots to help explain your changes.

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Performance Impact

If this PR affects performance, describe the impact and any optimizations made.

Related Issues

Fixes #(issue number)
Closes #(issue number)
Related to #(issue number)

- Add `--port` (`-p`), `--mlflow-port`, and `--api-port` CLI flags
- Apply only to `start` and `restart`; validate ports (1–65535)
- Map flags to env vars `RF_FRONTEND_PORT`, `RF_MLFLOW_PORT`, `RF_API_PORT`
- Update script runner to support env overrides
- Print a warning if port flags are passed to non-applicable commands (e.g., `init`, `status`)
- Defaults remain unchanged (frontend 3000, MLflow 5002, API 8080)

Examples:
- `rapidfireai start --port 4000`
- `rapidfireai start --mlflow-port 6000 --api-port 9090`
- `rapidfireai restart -p 4000`
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.

3 participants