Improvements on information about disks size and offering in VM deployment wizard's InfoCard

[winterhazel]

Description

The VM deployment wizard only shows one of the disk offerings that will be used to create the VM's volumes, which corresponds to the data disk if using a template, and the root disk if using a ISO. The disk offering that will be used to create the root disk is not shown when deploying through a template. This confuses users, since the wizard does not explicitly show what offering will be used for each disk. Furthermore, the instance summary often presents an incorrect value for the size of the disks.

This PR makes some changes in the deployment wizard in order to explicitly show what offerings will be used for the root and data disks, and to display their correct size.

Types of changes

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

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

• [ ] Major
• [X] Minor

Bug Severity

• [ ] BLOCKER
• [ ] Critical
• [ ] Major
• [X] Minor
• [ ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

I created one disk offering:

1. Name and description: Associated. Custom disk size: false. Disk size (in GB): 10.

And two compute offerings:

1. Name and description: Associated. Compute only disk offering: false: Disk offerings: Associated.
2. Name and description: Custom. Compute only disk offering: false: Disk offerings: Custom Disk.

In every test described below, I deployed the VM and verified that the VM's volumes were created using the disk offerings and the sizes displayed in the UI.

Tests deploying from a template:

• I selected the compute offering Small Instance;
• I selected the compute offering Small Instance, enabled root disk offering override and selected Medium;
• I selected the compute offering Small Instance, enabled root disk offering override, selected Custom and set the size to 10 GB;
• I selected the compute offering Custom and set the size to 11 GB;
• I selected the compute offering Custom, set the size to 12 GB, enabled override and selected Medium;
• I selected the compute offering Associated;
• I selected the compute offering Associated, enabled override and selected Medium;
• I selected the compute offering Associated, enabled override, selected Custom and set the size to 13 GB;
• I selected the compute offering Small Instance and a Small data disk;
• I selected the compute offering Small Instance, enabled root disk override, selected Medium and a Small data disk;
• I selected the compute offering Small Instance, enabled root disk override, selected the Custom root disk, set the size to 11 GB, selected the Custom Disk data disk and set the size to 12 GB.

Tests deploying from an ISO:

• I selected the compute offering Small Instance and Medium disk size;
• I selected the compute offering Custom and Medium disk size;
• I selected the compute offering Associated and Medium disk size;
• I selected the compute offering Associated, Custom disk size and set the size to 13 GB.

[winterhazel]

@blueorangutan package

[blueorangutan]

@winterhazel a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 4.19%. Comparing base (cb48202) to head (d0f9dcd).

Additional details and impacted files

@@           Coverage Diff            @@
##            main   #9249      +/-   ##
========================================
- Coverage   4.19%   4.19%   -0.01%     
========================================
  Files        369     369              
  Lines      30233   30246      +13     
  Branches    5343    5349       +6     
========================================
  Hits        1269    1269              
- Misses     28820   28833      +13     
  Partials     144     144              

Flag	Coverage Δ
uitests	4.19% <ø> (-0.01%)	:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[blueorangutan]

Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9934

[rohityadavcloud]

@blueorangutan ui

[blueorangutan]

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

[rohityadavcloud]

@winterhazel for pure UI changes, pl use BO ui commands; the packaging commands add 3x load on backend CI systems and may be skipped for UI only PRs. Thanks.

[blueorangutan]

UI build: :heavy_check_mark: Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9249 (QA-JID-380)