Skip to content

Remove temporarily the engine env report - #2061

Open
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-71from
tom/refactor-miles/op13-72
Open

Remove temporarily the engine env report#2061
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-71from
tom/refactor-miles/op13-72

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Aug 2, 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/op13-71 branch from c646bc5 to 44edc76 Compare August 2, 2026 10:01
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 2b304c8 to 7fe9b44 Compare August 2, 2026 10:01
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 44edc76 to 71f2914 Compare August 2, 2026 12:41
@fzyzcjy
fzyzcjy requested a review from yushengsu-thu as a code owner August 2, 2026 12:41
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 7fe9b44 to 89b87e1 Compare August 2, 2026 12:41
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 71f2914 to d9a6d38 Compare August 3, 2026 23:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch 2 times, most recently from 8fa52b8 to f2ab9f8 Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch 2 times, most recently from c08007c to 96c2166 Compare August 4, 2026 06:55
@fzyzcjy
fzyzcjy requested a review from Rockdu as a code owner August 4, 2026 06:55
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from f2ab9f8 to 638eb61 Compare August 4, 2026 06:55
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 96c2166 to 29a9ca8 Compare August 4, 2026 07:27
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 638eb61 to 6761845 Compare August 4, 2026 07:27
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 29a9ca8 to 0057289 Compare August 4, 2026 07:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch 2 times, most recently from a609eb8 to d4dedaa Compare August 6, 2026 03:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 0ca4551 to bec133f Compare August 6, 2026 07:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch 3 times, most recently from f4e8551 to b50d123 Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 56f9144 to 330b868 Compare August 10, 2026 06:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from b50d123 to 14ef373 Compare August 10, 2026 06:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 330b868 to e2336a1 Compare August 10, 2026 16:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 14ef373 to 0a8ed16 Compare August 10, 2026 16:07
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from e2336a1 to 3874e75 Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 0a8ed16 to 2174d54 Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 3874e75 to ca4af71 Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 2174d54 to 5e84932 Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from ca4af71 to ea3432d Compare August 12, 2026 09:32
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch 2 times, most recently from 4b70217 to 85f188f Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 0336efa to ed13622 Compare August 12, 2026 14:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 85f188f to 4e63224 Compare August 12, 2026 14:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from ed13622 to 26a890e Compare August 12, 2026 23:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch 2 times, most recently from 015d0ba to b04027d Compare August 13, 2026 01:13
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 0f6a496 to bbe51bd Compare August 13, 2026 03:11
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from b04027d to 6eab71e Compare August 13, 2026 03:11
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from bbe51bd to 7b94705 Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 6eab71e to 400247b Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 7b94705 to 9b86ba7 Compare August 13, 2026 06:00
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 400247b to 03e6a02 Compare August 13, 2026 06:00
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from 9b86ba7 to cb3eb64 Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 03e6a02 to 8a526bd Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-71 branch from cb3eb64 to b0cc6f8 Compare August 14, 2026 04:46
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 8a526bd to 089bd14 Compare August 14, 2026 04:46

@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.

Reviewed in stack context through #2176: this intentionally removes the actor-bound rollout env report before #2063 moves engine lifecycle to RayWorkerManager, and the engine-side replacement is implemented post-checkpoint in #2509. LGTM for this staging PR.

@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-72 branch from 089bd14 to 8aa0745 Compare August 29, 2026 12:38
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