Skip to content

Add managed process crate#284

Merged
MOZGIII merged 1 commit intomainfrom
mzg/2026-03-26/managed-process
Mar 26, 2026
Merged

Add managed process crate#284
MOZGIII merged 1 commit intomainfrom
mzg/2026-03-26/managed-process

Conversation

@MOZGIII
Copy link
Copy Markdown
Collaborator

@MOZGIII MOZGIII commented Mar 26, 2026

This PR is a part of the work towards the workers refactor; it introduces a crate for managed processes which is to be used in the worker pools.

This is a concise building block that encapsulates the behaviors of a process that needs to behave in a suitable-fro-the-process-pool manner: it has to terminate when we exit, even if we got SIGKILL-ed and are unable to gracefully terminate the process pool.

This crate is not used just yet, but the plan is to build new set of concise crates on top of it, and eventually replace the waymark-remote crate. For a preview of the full picture check out this PR: #278.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

Coverage Report

Python Coverage

Metric Coverage
Lines 75.8%
Branches 57.8%

Download HTML Report

Rust Coverage

Metric Coverage
Lines 65.4%
Branches N/A

Download HTML Report

Compared to main branch

@MOZGIII MOZGIII requested a review from piercefreeman March 26, 2026 09:57
@MOZGIII MOZGIII force-pushed the mzg/2026-03-26/managed-process branch from a1a5244 to 7370e31 Compare March 26, 2026 14:35
@MOZGIII MOZGIII merged commit 17e99e9 into main Mar 26, 2026
15 checks passed
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.

2 participants