Skip to content

refactor(pagebroker): organize transfer code and tests - #434

Closed
tzachi-runai wants to merge 2 commits into
dyn-4108/model-streamer-transfer-configfrom
dyn-4108/pagebroker-transfer-layout
Closed

tzachi-runai wants to merge 2 commits into
dyn-4108/model-streamer-transfer-configfrom
dyn-4108/pagebroker-transfer-layout

Conversation

@tzachi-runai

@tzachi-runai tzachi-runai commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

This change is absorbed into #264 in the replacement stack: #264 → #258 → #383 → #410 → #416 → #469.

The retained PRs now contain the final layout, shared transfer API, and complete engine selection at their respective introduction points.

Refs #267 and #237.

@copy-pr-bot

copy-pr-bot Bot commented Oct 1, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@tzachi-runai
tzachi-runai added this pull request to stack #436 October 1, 2026 08:44
@tzachi-runai tzachi-runai changed the title refactor(pagebroker): organize transfer engines and filesystem helpers refactor(pagebroker): organize transfer code and tests Oct 1, 2026
@tzachi-runai
tzachi-runai force-pushed the dyn-4108/pagebroker-transfer-layout branch from 0fa0f5e to 3c5ad0f Compare October 1, 2026 11:08
Tzachi Lapidot added 2 commits October 5, 2026 16:35
Signed-off-by: Tzachi Lapidot <tlapdot@nvidia.com>
Signed-off-by: Tzachi Lapidot <tlapdot@nvidia.com>
@tzachi-runai
tzachi-runai force-pushed the dyn-4108/model-streamer-transfer-config branch from f400a5e to f159d3c Compare October 5, 2026 18:13
@tzachi-runai
tzachi-runai force-pushed the dyn-4108/pagebroker-transfer-layout branch from 3c5ad0f to 844d8d8 Compare October 5, 2026 18:13
@tzachi-runai
tzachi-runai removed this pull request from stack #436 October 11, 2026 13:25
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