Skip to content

refactor(engine): manage child process lifecycles#2160

Open
jrusso1020 wants to merge 1 commit into
07-10-refactor_engine_add_browser_leasesfrom
07-10-refactor_engine_manage_child_processes
Open

refactor(engine): manage child process lifecycles#2160
jrusso1020 wants to merge 1 commit into
07-10-refactor_engine_add_browser_leasesfrom
07-10-refactor_engine_manage_child_processes

Conversation

@jrusso1020

@jrusso1020 jrusso1020 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

What

Give FFmpeg and related subprocesses one managed lifecycle.

Why

Timeout, cancellation, stderr bounds, and SIGTERM/SIGKILL behavior were duplicated and inconsistent across encoding phases.

How

Add ManagedChildProcess with composed signals, grace escalation, guaranteed reap, and typed termination reasons, then migrate encode/probe/extract paths.

Test plan

  • managed process, encoder, ffprobe, extraction, and assembly tests
  • Stack-wide lint, format, build, typecheck, and relevant integration gates

jrusso1020 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from f5fa1ad to 64e3a64 Compare July 11, 2026 18:21
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch 2 times, most recently from 13a4f11 to bad4059 Compare July 12, 2026 04:01
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from 64e3a64 to 33fae28 Compare July 12, 2026 04:01
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from bad4059 to 40abcdc Compare July 12, 2026 04:32
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from 33fae28 to 4885de6 Compare July 12, 2026 04:32
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from 40abcdc to d93f968 Compare July 12, 2026 04:58
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from 4885de6 to 8f6797a Compare July 12, 2026 04:58
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from d93f968 to d069da3 Compare July 12, 2026 08:54
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch 2 times, most recently from 2dcb4ec to fc3a9cb Compare July 12, 2026 09:11
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from 0243086 to d3228f6 Compare July 13, 2026 15:52
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from fc3a9cb to 881be1e Compare July 13, 2026 15:52
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from d3228f6 to bd95455 Compare July 13, 2026 17:29
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch 2 times, most recently from 46b689d to 9295542 Compare July 13, 2026 18:09
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch 2 times, most recently from 7a244a3 to 246c941 Compare July 13, 2026 18:35
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from 9295542 to 1a9af9b Compare July 13, 2026 18:35
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from 246c941 to fa435fb Compare July 13, 2026 20:27
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch 2 times, most recently from 131f444 to f906eee Compare July 13, 2026 20:33
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from fa435fb to e9734f0 Compare July 13, 2026 20:33
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from f906eee to bfe60e6 Compare July 13, 2026 22:09
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from 1dd7199 to 82cdfe7 Compare July 13, 2026 22:18
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch 2 times, most recently from 5e82fec to fc61671 Compare July 13, 2026 23:08
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_add_browser_leases branch from 82cdfe7 to 5f2aacb Compare July 13, 2026 23:08
@jrusso1020 jrusso1020 force-pushed the 07-10-refactor_engine_manage_child_processes branch from fc61671 to b683ddb Compare July 14, 2026 00:03
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