docs(framework): Update cloud deployment guides for HTTP Control API - #8089
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4b87409355
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
🟢 Approval recommended
The updates are doc-only and correctly align all referenced Control API ports/addresses with the HTTP API default (8000) while ensuring the server binds to an externally reachable interface.
Pull request overview
This PR updates Flower cloud deployment documentation to reflect the HTTP Control API defaults (port 8000) and ensure the SuperLink HTTP server is reachable from outside the container/VM by binding to 0.0.0.0.
Changes:
- Update OpenShift and GCP Kubernetes manifests to expose
8000(instead of9093) for the Control API and pass--host 0.0.0.0. - Update configuration examples to point SuperLink Control API addresses to
:8000. - Update Azure guide networking and startup instructions to use port
8000and bind the HTTP Control API to0.0.0.0.
Critical issues
- None found.
Simplicity/readability suggestions
- None required.
Consistency concerns
- None blocking.
Whether the PR should be split
- No; the changes are tightly related and scoped to the same documentation update.
Overall verdict
Documentation-only changes are consistent and align with the SuperLink HTTP API default port (8000) and the need to bind to 0.0.0.0 for external access.
File summaries
| File | Description |
|---|---|
| framework/docs/source/how-to-run-flower-on-red-hat-openshift.rst | Update OpenShift deployment/service and config example to use Control API on 8000 and bind host to 0.0.0.0. |
| framework/docs/source/how-to-run-flower-on-gcp.rst | Update GCP deployment/service and config example to use Control API on 8000 and bind host to 0.0.0.0. |
| framework/docs/source/how-to-run-flower-on-azure.rst | Update Azure networking instructions, SuperLink startup command, and config example for Control API on 8000. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
No description provided.