Skip to content

feat: Supervise the Flow server when Frappe ships it - #653

Closed
ShrihariMahabal wants to merge 1 commit into
frappe:developfrom
ShrihariMahabal:feat/supervise-flow-in-frappe
Closed

ShrihariMahabal wants to merge 1 commit into
frappe:developfrom
ShrihariMahabal:feat/supervise-flow-in-frappe

Conversation

@ShrihariMahabal

@ShrihariMahabal ShrihariMahabal commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

Flow now ships inside Frappe, so frappe/press#7703, which checks for the separate frappe/flow app, no longer turns on the supervised Flow server.

  • When Frappe's pyproject declares [tool.pilot.processes.flow_server], the builder sets flow_server_manager: supervisor in the image's common_site_config.json, so feat(bench): Run the Flow server under supervisor #619 adds the supervisor program (a bench group can still set local).

Needs #619 on app servers first.

@mergify

mergify Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@ShrihariMahabal ShrihariMahabal changed the title feat(builder): Supervise the Flow server when Frappe ships it feat: Supervise the Flow server when Frappe ships it Oct 25, 2026
@greptile-apps

greptile-apps Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The changes appear safe to merge.

Reviews (1) · Last reviewed commit: "feat(builder): Supervise the Flow server..." · Reviewed by Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant