Skip to content

feat(FR-2677): add DeploymentReplicasTab with replica detail Drawer#6920

Closed
agatha197 wants to merge 1 commit into
graphite-base/6920from
04-22-feat_fr-2677_add_deploymentreplicastab_with_replica_detail_drawer
Closed

feat(FR-2677): add DeploymentReplicasTab with replica detail Drawer#6920
agatha197 wants to merge 1 commit into
graphite-base/6920from
04-22-feat_fr-2677_add_deploymentreplicastab_with_replica_detail_drawer

Conversation

@agatha197
Copy link
Copy Markdown
Contributor

@agatha197 agatha197 commented Apr 22, 2026

Resolves FR-2677.

Summary

Adds the Flow 3 Replicas tab for the Deployment detail page. Renders a paginated BAITable of ModelReplica rows (ID, health via ReplicaStatusTag, Active Pool indicator from activenessStatus, traffic ratio placeholder, session link, revision, createdAt). Clicking a row opens an antd Drawer with the full replica detail — readiness / liveness / activeness statuses, session ID, revision name, endpoint URL, createdAt.

Scope

  • New: react/src/components/DeploymentReplicasTab.tsx
  • Props: { deploymentFrgmt: DeploymentReplicasTab_deployment$key }
  • Uses useRefetchableFragment on ModelDeployment with replicas(orderBy, limit, offset) pagination; tab-local state (current, pageSize, order, selected) persisted via nuqs under rCurrent / rPageSize / rOrder / rSelected.
  • Drawer is inline (keeps PR scope small, easy to extract later).
  • Drawer-only fields (readinessStatus, activenessStatus, revision.name, networkAccess.endpointUrl) are projected from the same fragment for cache reuse.

Dependencies (separate PRs)

  • FR-2666 i18n keys (deployment.*, replicaStatus.*) — sibling PR; diff below includes them so CI can pass standalone.
  • FR-2667 ReplicaStatusTag + tsconfig.json stories exclude — sibling PR; diff below includes them so CI can pass standalone.

Once FR-2666 and FR-2667 merge to main, this branch can be rebased and those commits will drop out cleanly.

TODO / needs-backend

  • TODO(needs-backend): FR-2677 on the trafficRatio column — ModelReplica does not expose trafficRatio today; that value lives on the sibling Route object. Wire it when the backend adds a pass-through field or when this tab adopts a parallel routes(deploymentId: …) query.

Verify

bash scripts/verify.sh locally — all checks pass (Relay, Lint, Format, TypeScript).

@github-actions github-actions Bot added the size:L 100~500 LoC label Apr 22, 2026
Copy link
Copy Markdown
Contributor Author

agatha197 commented Apr 22, 2026


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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

@agatha197 agatha197 changed the base branch from worktree-agent-a1202d5b to main April 22, 2026 10:55
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026

Coverage report for ./react

St.
Category Percentage Covered / Total
🔴 Statements
8.46% (-0.02% 🔻)
1757/20780
🔴 Branches
7.74% (-0.03% 🔻)
1131/14610
🔴 Functions
5.06% (-0.02% 🔻)
285/5631
🔴 Lines
8.18% (-0.02% 🔻)
1649/20163
Show new covered files 🐣
St.
File Statements Branches Functions Lines
🔴
... / DeploymentReplicasTab.tsx
0% 0% 0% 0%

Test suite run success

856 tests passing in 39 suites.

Report generated by 🧪jest coverage report action from 982ffdc

@agatha197 agatha197 changed the base branch from main to graphite-base/6920 April 22, 2026 14:01
@agatha197 agatha197 force-pushed the 04-22-feat_fr-2677_add_deploymentreplicastab_with_replica_detail_drawer branch from 5e477e9 to 982ffdc Compare April 22, 2026 14:02
This was referenced Apr 22, 2026
@agatha197
Copy link
Copy Markdown
Contributor Author

Consolidated into grouped PR stack (Foundation/Components/Launcher/List/Detail). Closing this individual PR.

@agatha197 agatha197 closed this Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100~500 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant