Skip to content
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

[Bug] Error message on enabling TrustyAI via UI are not descriptive #1996

Closed
1 task done
nehachopra27 opened this issue Oct 20, 2023 · 4 comments · Fixed by #2036
Closed
1 task done

[Bug] Error message on enabling TrustyAI via UI are not descriptive #1996

nehachopra27 opened this issue Oct 20, 2023 · 4 comments · Fixed by #2036
Assignees
Labels
feature/model-serving Model Serving Feature kind/bug Something isn't working priority/normal An issue with the product; fix when possible

Comments

@nehachopra27
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Deploy type

OpenDataHub core version (eg. v1.6.0)

Version

v2

Current Behavior

When Enabling a TrustyAI service via UI, when the Operator is not installed, The error message is 404 Page not Found.
Screenshot 2023-10-20 at 13 34 40

Expected Behavior

Clear Error Message should be available to user so that he can do the configuration.

Steps To Reproduce

  1. Create a Data Science Project
  2. Navigate to DSP page
  3. Click Settings
  4. Check the Enable TrustyAI checkbox

Workaround (if any)

No response

What browsers are you seeing the problem on?

No response

Anything else

No response

@nehachopra27 nehachopra27 added kind/bug Something isn't working priority/normal An issue with the product; fix when possible untriaged Indicates the newly create issue has not been triaged yet labels Oct 20, 2023
@andrewballantyne
Copy link
Member

Ah yeah... we had this with model serving... it had the same 404 error when the API is not there.

@nehachopra27 nehachopra27 changed the title Error message on enabling TrustyAI via UI are not descriptive [Bug] Error message on enabling TrustyAI via UI are not descriptive Oct 20, 2023
@lucferbux lucferbux added feature/model-serving Model Serving Feature and removed untriaged Indicates the newly create issue has not been triaged yet labels Oct 23, 2023
@alexcreasy
Copy link
Contributor

alexcreasy commented Oct 26, 2023

@ppadti use this error message for now, we'll need QE signoff before we merge the ticket.

"The TrustyAI operator is not installed on this cluster."

We'll need to be more descriptive than that, but as a placeholder it's more descriptive than 404 :)

@ppadti
Copy link
Contributor

ppadti commented Oct 26, 2023

@ppadti use this error message for now, we'll need QE signoff before we merge the ticket.

"The TrustyAI operator is not installed on this cluster."

sure, Thanks @alexcreasy

@alexcreasy
Copy link
Contributor

@nehachopra27 we've merged the mitigation we agreed for this to our feature branch. The complete solution for this (i.e. hide the tab altogether if TrustyAI is not installed), will need this PR to merge first: #2032.

@ppadti ppadti closed this as completed Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/model-serving Model Serving Feature kind/bug Something isn't working priority/normal An issue with the product; fix when possible
Projects
Status: Done
Status: No status
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants