-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Add error details for failed jobs #35691
Add error details for failed jobs #35691
Conversation
1. Add error details for failed jobs in the job list view and job detail view 2. Add error details for failed serve apps in the application detail view 3. Fix color of StatusChip for HTTPPRoxy detail page 4. Add max width for error details for Serve deployments Signed-off-by: Alan Guo <[email protected]>
lgtm! |
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.
cc @scottsun94 for the UI design approval
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.
I'm not familiar with TSX, but the Job code looks reasonable and the screenshots look good and like what I would expect.
Can we reduce the width of status message column a bit for job table? It seems that you have to expand to see the useful info for most of the time? |
Signed-off-by: Alan Guo <[email protected]>
LGTM. Thanks! |
@alanwguo I'm tracking this now, can you please create the pick after merging? |
Add error details for failed jobs in the job list view and job detail view Add error details for failed serve apps in the application detail view Fix color of StatusChip for HTTPPRoxy detail page Add max width for error details for Serve deployments
Add error details for failed jobs in the job list view and job detail view Add error details for failed serve apps in the application detail view Fix color of StatusChip for HTTPPRoxy detail page Add max width for error details for Serve deployments
Add error details for failed jobs in the job list view and job detail view Add error details for failed serve apps in the application detail view Fix color of StatusChip for HTTPPRoxy detail page Add max width for error details for Serve deployments
Add error details for failed jobs in the job list view and job detail view Add error details for failed serve apps in the application detail view Fix color of StatusChip for HTTPPRoxy detail page Add max width for error details for Serve deployments Signed-off-by: e428265 <[email protected]>
Error details for jobs
Http proxy detail page
Serve Application detail page
Why are these changes needed?
Related issue number
partially addresses #35685
Checks
git commit -s
) in this PR.scripts/format.sh
to lint the changes in this PR.method in Tune, I've added it in
doc/source/tune/api/
under thecorresponding
.rst
file.