Bug 1667828 - Add measurement units for disk size in VM disk page
Summary: Add measurement units for disk size in VM disk page
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: User Experience
Version: 1.4
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 2.0
Assignee: Yaacov Zamir
QA Contact: Guohua Ouyang
URL:
Whiteboard:
: 1674473 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-01-21 09:08 UTC by Guohua Ouyang
Modified: 2019-07-24 20:15 UTC (History)
8 users (show)

Fixed In Version: 2.0.0-9.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-07-24 20:15:50 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
miss_unit (42.24 KB, image/png)
2019-01-22 12:00 UTC, Guohua Ouyang
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2019:1850 0 None None None 2019-07-24 20:15:59 UTC

Description Guohua Ouyang 2019-01-21 09:08:14 UTC
Description of problem:
There is no measurement units for disk size in VM disk page.

Version-Release number of selected component (if applicable):
1.4.0-10

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Tomas Jelinek 2019-01-22 11:09:15 UTC
Where do you miss the units? In the VM details you have the units and in create VM dialog you have "size (GB)". Can you please provide a screenshot to point the place out?

Comment 2 Guohua Ouyang 2019-01-22 12:00:05 UTC
Created attachment 1522385 [details]
miss_unit

Comment 3 Tomas Jelinek 2019-02-11 12:44:29 UTC
*** Bug 1674473 has been marked as a duplicate of this bug. ***

Comment 4 Filip Krepinsky 2019-02-21 10:45:01 UTC
https://github.com/kubevirt/web-ui-components/pull/232

Comment 5 Guohua Ouyang 2019-04-22 13:12:52 UTC
How about make it look consistently with the wizard storage page, Gi -> SIZE (GI)?

Comment 6 Filip Krepinsky 2019-04-23 13:14:56 UTC
This cannot be done because there might be sizes in different units per each row (Mi,Gi, etc.). We should rather add unit choosing to CreateVmDialog.

Comment 7 Guohua Ouyang 2019-04-25 02:03:13 UTC
(In reply to Filip Krepinsky from comment #6)
> This cannot be done because there might be sizes in different units per each
> row (Mi,Gi, etc.). We should rather add unit choosing to CreateVmDialog.

There is no other units except Gi on disks page currently. If it's going to have different units for them, please implement it and make it looks consistently in both places.

Comment 8 Tomas Jelinek 2019-05-09 11:37:20 UTC
> There is no other units except Gi on disks page currently. If it's going to have different units for them, please implement it and make it looks consistently in both places.

The thing is that we do not support to pick other units than Gi when creating the disks in UI. Hence, it is only hardcoded as Gi.
But, kubevirt does support also other ones and it is present in the yaml file. So, every disk can have different unit, if created in command line. In order to represent that correctly in the UI, we show the unit per disk in VM details, since they can be different.

We will implement the possibility to pick the unit per disk when creating the disk in the UI, but not yet there and than it will become consistent. Until than, I believe the current implementation is ok (even if not completely visually consistent, but correct). Moving back to on_qa.

Comment 9 Guohua Ouyang 2019-05-09 12:19:39 UTC
so move it to verified as the units is added.

Comment 11 errata-xmlrpc 2019-07-24 20:15:50 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, 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/RHEA-2019:1850


Note You need to log in before you can comment on or make changes to this bug.