-
Notifications
You must be signed in to change notification settings - Fork 26
Enable debuglogs and clickdeploy, fix llm uservice #60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: cheehook <[email protected]>
Signed-off-by: wwanrif <[email protected]>
Signed-off-by: wwanrif <[email protected]>
Signed-off-by: cheehook <[email protected]>
fix text area not grey out during agent is generating response Signed-off-by: cheehook <[email protected]>
Signed-off-by: wwanrif <[email protected]>
Signed-off-by: wwanarif <[email protected]>
fix llm uservice, added LOGFLAGS, update app-frontend port added building GenAIComps images scripts update registry cmd py to show digest of the images listed updated _e2e test workflow Signed-off-by: wwanarif <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a debug logs view, integrates a one-click "clickdeploy" feature in the FlowListTable, and fixes the LLM uservice along with associated port and URL updates.
- Added a new
PodLogsViewfor viewing debug logs (debuglogs/index.jsxand backend router). - Extended the FlowListTable to include an observability menu and one-click deployment dialog.
- Updated port mappings and fixed default model in the LLM uservice and related templates.
Reviewed Changes
Copilot reviewed 63 out of 63 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
packages/ui/src/views/debuglogs/index.jsx |
New debug logs UI component |
packages/ui/src/views/tracer/index.jsx |
JSX comment fix and header integration |
packages/ui/src/ui-component/table/FlowListTable.jsx |
Observability menu & 1-Click Deployment column |
packages/ui/src/ui-component/dialog/OneClickDeploymentDialog.jsx |
New dialog for one-click deployment |
packages/server/src/routes/debuglog_router.py |
New FastAPI route for pod logs |
app-frontend/react/src/components/Conversation/Conversation.tsx |
Fixed default model value |
Comments suppressed due to low confidence (2)
studio-frontend/packages/ui/src/views/tracer/index.jsx:173
- JSX does not support JavaScript
//comments inside the returned JSX. Wrap commented lines with{/* ... */}or remove the invalid comment to avoid a syntax error.
// <Box sx={{ display: "flex", height: "100vh", p: 4, gap: 2, position: "relative" }}>
Signed-off-by: thee <[email protected]>
Signed-off-by: wwanarif <[email protected]>
Signed-off-by: wwanarif <[email protected]>
Signed-off-by: wwanarif <[email protected]>
siawchen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
enable debuglogs view, clickdeploy feature
fix llm uservice, added LOGFLAGS, update app-frontend port
added building GenAIComps images scripts
update registry cmd py to show digest of the images listed
updated _e2e test workflow