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]: Custom serving runtimes can have arbitrarily long names #1394

Closed
1 task done
lucferbux opened this issue Jun 15, 2023 · 4 comments · Fixed by #1718
Closed
1 task done

[Bug]: Custom serving runtimes can have arbitrarily long names #1394

lucferbux opened this issue Jun 15, 2023 · 4 comments · Fixed by #1718
Assignees
Labels
feature/model-serving Model Serving Feature good first issue Good for newcomers kind/bug Something isn't working priority/normal An issue with the product; fix when possible rhods-1.33

Comments

@lucferbux
Copy link
Contributor

lucferbux commented Jun 15, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

When creating a custom runtime, users can give it an arbitrarily long display name. This can cause the "serving runtimes" section of the dashboard to break (i.e. buttons get pushed out of reach) if the name is long enough.

We need to address:

  1. Truncate values when there are no spaces in the text
Screenshot 2023-06-15 at 11 53 37

Expected Behavior

Table doesn't expand as large as the name.

Look at using PF Truncate

Steps To Reproduce

  1. Go to serving runtimes page
  2. Create a new serving runtime with a long display name
@lucferbux lucferbux added kind/bug Something isn't working untriaged Indicates the newly create issue has not been triaged yet labels Jun 15, 2023
@lucferbux lucferbux added good first issue Good for newcomers priority/normal An issue with the product; fix when possible feature/model-serving Model Serving Feature and removed untriaged Indicates the newly create issue has not been triaged yet labels Jun 19, 2023
@ppadti ppadti self-assigned this Aug 28, 2023
@lugi0
Copy link
Contributor

lugi0 commented Sep 25, 2023

This issue is still somewhat present in the Serving runtime dropdown in the Add model server modal.

The text does not wrap (nor is it truncated), and while it does not break the UI it is clear it is extending the name out of bounds of the modal.

I can get the text to wrap by removing the current class from the button element (pf-c-select__menu-item) and instead giving it the same class given to the table element in Serving runtimes (pf-u-text-break-word pf-m-width-70), but it loses the styling of the modal, and using both classes at the same time does not wrap the text, so this might not be a viable solution here.

Attaching a few screenshots for clarity
3
2
1

@lugi0
Copy link
Contributor

lugi0 commented Sep 25, 2023

Actually, the UI does break once a model server is deployed using the serving runtime with the long unbreakable word
image

@andrewballantyne
Copy link
Member

Ah -- we didn't solve it all three spots:

  1. [Solved] Settings => Custom Serving Runtimes
  2. DS Projects => [project] => Models and model servers section
  3. DS Projects => [project] => Create Server => select Runtime dropdown

Seems we need another issue to make sure it's covered in the other two spots. I'll log something in a moment.

@lucferbux lucferbux reopened this Sep 25, 2023
@lucferbux
Copy link
Contributor Author

Ah -- we didn't solve it all three spots:

  1. [Solved] Settings => Custom Serving Runtimes
  2. DS Projects => [project] => Models and model servers section
  3. DS Projects => [project] => Create Server => select Runtime dropdown

Seems we need another issue to make sure it's covered in the other two spots. I'll log something in a moment.

woopsie, once. i reopened it your message has popped up in the timeline, I'm gonna close it again.

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 good first issue Good for newcomers kind/bug Something isn't working priority/normal An issue with the product; fix when possible rhods-1.33
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants