Bug 1492397

Summary:	consistent binary prefix units for sizes (MiB, GiB)
Product:	[oVirt] ovirt-engine	Reporter:	Yaniv Kaul <ykaul>
Component:	ovirt-engine-ui-extensions	Assignee:	Scott Dickerson <sdickers>
Status:	CLOSED DEFERRED	QA Contact:	Lucie Leistnerova <lleistne>
Severity:	low	Docs Contact:
Priority:	low
Version:	4.2.0	CC:	bugs, gshereme, michal.skrivanek
Target Milestone:	---	Flags:	sbonazzo: ovirt-4.4-
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	No Doc Update
Doc Text:	undefined	Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-04-01 14:55:17 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	UX	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Yaniv Kaul 2017-09-17 07:44:32 UTC

Description of problem:
For a given VM, the user portal shows 512.0 MiB (why the .0?) defined memory. The Admin Portal shows 512MB. Please align with each other, preferably to the truth and get rid of the .0

Version-Release number of selected component (if applicable):
ovirt-web-ui-1.2.1-1.el7.centos.noarch

Comment 1 Michal Skrivanek 2017-09-18 08:57:05 UTC

https://github.com/oVirt/ovirt-web-ui/issues/341 will remove the .0
Admin portal should use correct units, AFAIK the binary prefix is used in the dashboard but everywhere else it is not (Hosts, VMs, etc.)
Since it covers all places across teams moving to ux for tracking

Comment 2 Oved Ourfali 2017-09-19 04:58:05 UTC

I don't see any need to touch all places at the moment.
If the dashboard has something else, I guess it can be fixed.
Scott - can you take a look?

Comment 3 Yaniv Kaul 2017-09-19 05:17:38 UTC

(In reply to Oved Ourfali from comment #2)
> I don't see any need to touch all places at the moment.
> If the dashboard has something else, I guess it can be fixed.
> Scott - can you take a look?

But it's not the dashboard - I'v seen it in the VM properties - when you drill down into VM details.

Comment 4 Oved Ourfali 2017-09-19 05:19:35 UTC

(In reply to Yaniv Kaul from comment #3)
> (In reply to Oved Ourfali from comment #2)
> > I don't see any need to touch all places at the moment.
> > If the dashboard has something else, I guess it can be fixed.
> > Scott - can you take a look?
> 
> But it's not the dashboard - I'v seen it in the VM properties - when you
> drill down into VM details.

The ".0"?
In the admin portal I see MB.
And Michal mentioned they have it fixed for the VM portal.

Comment 5 Yaniv Kaul 2017-09-19 05:26:16 UTC

(In reply to Oved Ourfali from comment #4)
> (In reply to Yaniv Kaul from comment #3)
> > (In reply to Oved Ourfali from comment #2)
> > > I don't see any need to touch all places at the moment.
> > > If the dashboard has something else, I guess it can be fixed.
> > > Scott - can you take a look?
> > 
> > But it's not the dashboard - I'v seen it in the VM properties - when you
> > drill down into VM details.
> 
> The ".0"?

No, that's in the VM portal.

> In the admin portal I see MB.

I think it should be MiB? I think dashboard have aligned properly to the conventions, not sure the rest of the UI?

> And Michal mentioned they have it fixed for the VM portal.

Comment 6 Oved Ourfali 2017-09-19 05:28:23 UTC

I don't see us prioritizing that at the moment.
Targeting to 4.3.0 for consideration.

Comment 7 Michal Skrivanek 2020-03-19 15:40:40 UTC

We didn't get to this bug for more than 2 years, and it's not being considered for the upcoming 4.4. It's unlikely that it will ever be addressed so I'm suggesting to close it.
If you feel this needs to be addressed and want to work on it please remove cond nack and target accordingly.

Comment 8 Michal Skrivanek 2020-04-01 14:55:17 UTC

ok, closing. Please reopen if still relevant/you want to work on it.