Bug 2037203

Summary: "Running VMs" card is too small in Virtualization Overview
Product: OpenShift Container Platform Reporter: Guohua Ouyang <gouyang>
Component: Console Kubevirt PluginAssignee: Phillip Bailey <phbailey>
Status: CLOSED ERRATA QA Contact: Guohua Ouyang <gouyang>
Severity: medium Docs Contact:
Priority: medium    
Version: 4.10CC: aos-bugs, gouyang, phbailey, tnisan
Target Milestone: ---   
Target Release: 4.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2022-03-10 16:37:09 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Running VMs in overview page none

Description Guohua Ouyang 2022-01-05 09:03:25 UTC
Created attachment 1849001 [details]
Running VMs in overview page

Description of problem:
"Running VMs" card is too small in Virtualization Overview
- the content of the left VM cannot display completely
- need adding a sidebar to scroll the VM list in the bottom

Version-Release number of selected component (if applicable):
OCP 4.10.0-0.nightly-2022-01-04-214702

How reproducible:
100%

Steps to Reproduce:
1. create at least 3 running VMs 
2. visit Overview page
3.

Actual results:


Expected results:


Additional info:

Comment 1 Guohua Ouyang 2022-01-26 08:27:15 UTC
Hi Phillip,
please note the popup text for the VM in the chart is not showing completely, would you create another PR to fix it?

Comment 2 Phillip Bailey 2022-01-26 14:24:08 UTC
Hi, Guohua!

I missed that part in the bug description. That problem is related more to the placement of the card than it is the size of it. The tooltip is displayed, but because the card is placed to the far left there isn't enough room for the tooltip to be seen. This is a shortcoming of the library being used as it should be able to detect whether there is enough room to display the tooltip in that orientation and use a different orientation if there isn't. I'll look into it, but I think that should be a separate bug from the legend as the issues aren't related.

Comment 3 Phillip Bailey 2022-01-26 14:37:22 UTC
It looks as though this is an issue in PatternFly. The underlying chart library (Victory) doesn't have an option for tooltips whereas PatternFly does. I don't see an option to control the direction in which the tooltips are displayed, which would have been a decent workaround until boundary detection could be added. I'll open a bug with them.

Comment 5 Guohua Ouyang 2022-01-27 08:33:05 UTC
(In reply to Phillip Bailey from comment #3)
> It looks as though this is an issue in PatternFly. The underlying chart
> library (Victory) doesn't have an option for tooltips whereas PatternFly
> does. I don't see an option to control the direction in which the tooltips
> are displayed, which would have been a decent workaround until boundary
> detection could be added. I'll open a bug with them.

thanks for the explain, I will move the bug to verified.
For the tooltip issue, I'm fine to live with it. :)

Comment 8 errata-xmlrpc 2022-03-10 16:37:09 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Container Platform 4.10.3 security update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:0056