fix(FR-2384): enable LLM Chat Test button when any route is HEALTHY#6178
fix(FR-2384): enable LLM Chat Test button when any route is HEALTHY#6178nowgnuesLee wants to merge 1 commit intomainfrom
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via 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. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 8.12% (-0% 🔻) |
1403/17286 |
| 🔴 | Branches | 7.31% (+0% 🔼) |
883/12073 |
| 🔴 | Functions | 5.34% | 258/4834 |
| 🔴 | Lines | 7.81% (-0% 🔻) |
1316/16840 |
Test suite run success
773 tests passing in 36 suites.
Report generated by 🧪jest coverage report action from b1d3e63
e58c011 to
780a6b8
Compare
780a6b8 to
1548b84
Compare
1548b84 to
9b328a9
Compare
There was a problem hiding this comment.
Pull request overview
Updates the Endpoint Detail page so the “LLM Chat Test” button is enabled when at least one underlying route is HEALTHY (instead of requiring the overall endpoint status to be HEALTHY), with a fallback path for backends that don’t support route-node.
Changes:
- Adds a
healthyRoutesalias query (filtered toHEALTHY) and deriveshasAnyHealthyRouteto drive the button’s disabled state. - Switches the “LLM Chat Test” button disabling logic to
!hasAnyHealthyRoute. - Removes
desired_session_countfallbacks in a couple places (replica display + destroying-category helper), which affects backward compatibility.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| react/src/pages/EndpointDetailPage.tsx | Adds a healthyRoutes count query and uses it to decide whether to enable “LLM Chat Test”; also changes replica display to use replicas only. |
| react/src/components/EndpointList.tsx | Adjusts isEndpointInDestroyingCategory to consider replicas only (dropping desired_session_count). |
ironAiken2
left a comment
There was a problem hiding this comment.
It appears that some of the comments I left have not been addressed. Please check this.
9b328a9 to
19ae0a7
Compare
19ae0a7 to
b1d3e63
Compare

Resolves #6176 (FR-2384)
Summary
HEALTHYHEALTHYstatus by adding a separatehealthyRoutesquery withfilter: { status: [HEALTHY] }route-nodeChanges
react/src/pages/EndpointDetailPage.tsxhealthyRoutesalias query to fetch count of HEALTHY routeshasAnyHealthyRoutederived stateendpoint?.status !== 'HEALTHY'to!hasAnyHealthyRouteTest plan
route-nodesupport (should fall back to endpoint status)🤖 Generated with Claude Code