Skip to content

Restructure start_engines into per-cell blocks and assume engines in one cell have same lifecycle - #1877

Open
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op5-12from
tom/refactor-miles/op5-13
Open

Restructure start_engines into per-cell blocks and assume engines in one cell have same lifecycle#1877
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op5-12from
tom/refactor-miles/op5-13

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Part of #1837

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from 028d28b to b5b34d0 Compare July 28, 2026 13:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch 2 times, most recently from 6dcfbd7 to 69a82f8 Compare August 3, 2026 23:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from b5b34d0 to 588c610 Compare August 3, 2026 23:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from 69a82f8 to da8a37d Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from 588c610 to 6daddff Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from da8a37d to 7cecf83 Compare August 4, 2026 06:57
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from 6daddff to 34d1cd5 Compare August 4, 2026 06:57
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from 7cecf83 to eb791e9 Compare August 6, 2026 07:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from 34d1cd5 to 0e997f4 Compare August 6, 2026 07:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from eb791e9 to 89eef17 Compare August 10, 2026 06:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from 0e997f4 to 1c046e4 Compare August 10, 2026 06:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from 89eef17 to f2c73a2 Compare August 10, 2026 16:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch 2 times, most recently from b009b5b to ff8f3fb Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from f2c73a2 to d15cdac Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from ff8f3fb to c87a468 Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from 7f6db1e to eb54bd8 Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch from c87a468 to 034cd28 Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-12 branch from eb54bd8 to c7039dc Compare August 12, 2026 23:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op5-13 branch 4 times, most recently from f893574 to d5fbe26 Compare August 13, 2026 06:01
…one cell have same lifecycle

Each cell now creates its actors, allocates its ports and sets its
addressing as one contiguous block, keeping the flat-order allocation
sequence identical; the init handles are still awaited together.

@guapisolo guapisolo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved for the #2176 safe-merge checkpoint: #1886/#2115 make cell startup atomic with rollback, and #1889 restores cross-cell startup concurrency; both fixes remain present through the checked chain tip.

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