Fix display of properties related to backup size

[bernardodemarco]

Description

In the backup details page, the properties Size and Virtual Size are being displayed in bytes, and the property Disk size (in GB) is being displayed in kilobytes.

This PR converts these values into a more readable format, so that they match the displayed units. Thus, a pop-up has been added to each one of the properties to display their values in bytes.

Types of changes

• Breaking change (fix or feature that would cause existing functionality to change)
• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)
• build/CI

Feature/Enhancement Scale or Bug Severity

Bug Severity

• BLOCKER
• Critical
• Major
• Minor
• Trivial

Screenshots (if appropriate):

Before the changes

After the changes with closed pop-ups

After the changes with opened pop-ups

How Has This Been Tested?

1. I created a VM backup with a DummyBackupProvider.
2. I accessed the backup details page and verified that the properties values were converted correctly.

[weizhouapache]

@bernardodemarco
do you have screenshots before/with this PR ?

[bernardodemarco]

@bernardodemarco do you have screenshots before/with this PR ?

Yes, they are in the PR's description.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 4.38%. Comparing base (6c6023b) to head (3426c76).
⚠️ Report is 606 commits behind head on 4.19.

❗ There is a different number of reports uploaded between BASE (6c6023b) and HEAD (3426c76). Click for more details.

HEAD has 16 uploads less than BASE

Flag	BASE (6c6023b)	HEAD (3426c76)
simulator-marvin-tests	15	0
unit-tests	1	0

Additional details and impacted files

@@             Coverage Diff              @@
##               4.19   #8845       +/-   ##
============================================
- Coverage     30.13%   4.38%   -25.76%     
============================================
  Files          5355     361     -4994     
  Lines        376690   28666   -348024     
  Branches      54815    5000    -49815     
============================================
- Hits         113533    1258   -112275     
+ Misses       248185   27269   -220916     
+ Partials      14972     139    -14833     

Flag	Coverage Δ
simulator-marvin-tests	?
uitests	4.38% <ø> (ø)
unit-tests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[DaanHoogland]

@blueorangutan ui

[blueorangutan]

@DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

[blueorangutan]

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/8845 (QA-JID-311)

[winterhazel]

LGTM, manually verified in QA.

Before

After

[DaanHoogland]

clgtm

[JoaoJandre]

@sureshanaparti can we merge this one?