Skip to content

docs(framework): Update cloud deployment guides for HTTP Control API - #8089

Merged
jafermarq merged 5 commits into
mainfrom
http-control-api-port
Sep 5, 2026
Merged

docs(framework): Update cloud deployment guides for HTTP Control API#8089
jafermarq merged 5 commits into
mainfrom
http-control-api-port

Conversation

@jafermarq

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 4, 2026 14:57
tanertopal
tanertopal previously approved these changes Sep 4, 2026
@tanertopal
tanertopal enabled auto-merge (squash) September 4, 2026 14:58

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

Comment thread framework/docs/source/how-to-run-flower-on-gcp.rst

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 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 of 9093) 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 8000 and bind the HTTP Control API to 0.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.

@jafermarq
jafermarq disabled auto-merge September 4, 2026 15:00
@github-actions github-actions Bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Sep 4, 2026
@jafermarq
jafermarq enabled auto-merge (squash) September 4, 2026 15:18
@jafermarq
jafermarq merged commit d2d0daf into main Sep 5, 2026
70 checks passed
@jafermarq
jafermarq deleted the http-control-api-port branch September 5, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainer Used to determine what PRs (mainly) come from Flower maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants