volume resize: Fix issue with volume resize on VMWare (deploy as-is templates) by Pearl1594 · Pull Request #4829 · apache/cloudstack · GitHub

Description

This PR fixes the issue pertaining to volume resize on VMWare for deploy as-is templates. VMware deploy as-is templates are those that are deployed as per the specification in the imported OVF. Hence override root disk size will not be adhered to for such templates. Moreover, when we deploy VMs in stopped state and resize the volume, the root disk doesn't get resized but the volume size is merely updated in the DB.
This PR also includes the following (for deploy as-is templates):

Disables overriding root disk size during VM deployment on the UI
Disables selection of compute offerings with root disk size specified, at the time of deployment
Provided users with the option to deploy VM is stopped state via UI (so as to give an option to users to resize the volumes before starting the VM)

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)

Feature/Enhancement Scale or Bug Severity

Bug Severity

BLOCKER
Critical
Major
Minor
Trivial

Screenshots (if appropriate):

How Has This Been Tested?

Prior to the fix:

Deployed a VM in stopped state, resized volume from 3GB to 8GB

While the UI shows that the Volume is resized, the root volume inside the VM doesn't seem to have gotten resized:

Post fix

Deployed a VM in stopped state, resized the volume from 3GB to 8GB

Notice that the volume size as shown in the UI complies with the value inside the VM
Or verify the same on vcenter

…VM for deploy as it

…emplates)

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: ✖️ centos7 ✖️ centos8 ✖️ debian. SL-JID 138

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: ✖️ centos7 ✖️ centos8 ✔️ debian. SL-JID 140

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: ✔️ centos7 ✖️ centos8 ✔️ debian. SL-JID 141

Looks good - perhaps also adding a check on the resizevolume and fail with a message when the volume is created from a deploy-as-is template?

@Pearl1594 is this required for ISO as well?

Thanks @sureshanaparti It's not required for ISO , have made the necessary change.

… as is template

Looks good - perhaps also adding a check on the resizevolume and fail with a message when the volume is created from a deploy-as-is template?

@nvazquez added checks to see if a compute offering with root disk size mentioned is used / overriding root disk size is attempted when deploying a vm with deploy as-is template and will accordingly throw an exception.

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: ✔️ centos7 ✖️ centos8 ✖️ debian. SL-JID 156

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: ✖️ centos7 ✔️ centos8 ✔️ debian. SL-JID 158

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 160

@blueorangutan test matrix

@Pearl1594 a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests [S]

@Pearl1594 a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 208

@blueorangutan test matrix

@Pearl1594 a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

Trillian test result (tid-227)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33443 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4829-t227-xenserver-71.zip
Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
Smoke tests completed. 85 look OK, 1 have error(s)
Only failed tests results shown below:

Test	Result	Time (s)	Test File
test_01_scale_vm	Failure	12.27	test_scale_vm.py

Trillian test result (tid-228)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35299 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4829-t228-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Smoke tests completed. 85 look OK, 1 have error(s)
Only failed tests results shown below:

Test	Result	Time (s)	Test File
test_01_migrate_VM_and_root_volume	Error	74.35	test_vm_life_cycle.py
test_02_migrate_VM_with_two_data_disks	Error	51.19	test_vm_life_cycle.py

Trillian test result (tid-229)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42211 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4829-t229-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Smoke tests completed. 84 look OK, 2 have error(s)
Only failed tests results shown below:

Test	Result	Time (s)	Test File
test_03_deploy_and_upgrade_kubernetes_cluster	Failure	796.21	test_kubernetes_clusters.py
ContextSuite context=TestVAppsVM>:setup	Error	43.39	test_vm_life_cycle.py

code LGTM

Tests LGTM

With @shwstppr to manually test it

Tested on VMware 65u2 env. Works fine.
UI disables Root disk resize option for newly registered templates unlike earlier where it was shown but not honoured.
Earlier

Now

Root volume resize of stopped VM deployed with a newly registered template (deployasis) - Successful
Root volume resize of stopped VM deployed with an ISO - Successfull

While volume resize behaviour works for most cases CKS cluster noderootdisksize is not working as expected. Will need similar changes from f5e866c

…ze-volume-issue

@blueorangutan package

@Pearl1594 a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 267

@blueorangutan test centos7 vmware-67u3

@Pearl1594 a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests

LGTM. Tested CKS node root disk resize with latest changes

Pearl Dsilva added 3 commits March 15, 2021 17:18

resize root disk + Hide override root disk size during deployment of …

26bbc1a

…VM for deploy as it

resize volume: Fix issue with volume resize on VMWare (deploy as-is t…

8b93bb2

…emplates)

Cleanup

5169c0c

Pearl1594 requested a review from nvazquez March 16, 2021 14:52

nvazquez reviewed Mar 16, 2021

View reviewed changes

sureshanaparti reviewed Mar 17, 2021

View reviewed changes

Add exceptions when svc offering with root disk size used with deploy…

f0a3d6e

… as is template

yadvr added this to the 4.15.1.0 milestone Mar 17, 2021

Allow resize of root disk for volume of a VM deployed from an ISO

1c3fc3c

nvazquez reviewed Mar 22, 2021

View reviewed changes

Comment thread ui/public/locales/en.json Resolved

Comment thread ui/public/locales/en.json Resolved

Comment thread ui/src/views/compute/DeployVM.vue Resolved

cleanup

4eda67e

nvazquez approved these changes Mar 24, 2021

View reviewed changes

Pearl1594 marked this pull request as ready for review March 24, 2021 04:25

sureshanaparti approved these changes Mar 24, 2021

View reviewed changes

yadvr assigned shwstppr Mar 24, 2021

shwstppr approved these changes Mar 26, 2021

View reviewed changes

shwstppr requested changes Mar 26, 2021

View reviewed changes

Pearl Dsilva added 2 commits March 26, 2021 17:34

Merge branch '4.15' of https://github.com/apache/cloudstack into resi…

3a05c9b

…ze-volume-issue

Support for volume resize for CKS

08faa04

shwstppr approved these changes Mar 26, 2021

View reviewed changes

yadvr merged commit 9717669 into apache:4.15 Mar 29, 2021

GabrielBrascher mentioned this pull request Jun 9, 2021

Failed to scale between Service Offerings with the same root disk size #5095

Merged

3 of 12 tasks

mib1185 mentioned this pull request Jun 10, 2021

ScaleVMCmd for ingested VMware VMs broken by #4829 #5096

Closed

                           for (DiskTO vol : sortedDisks) {
                               if (vol.getType() == Volume.Type.ISO || deployAsIs && vol.getType() == Volume.Type.ROOT) {
+                                  rootDiskTO = vol;
+                                  resizeRootDiskOnVMStart(vmMo, rootDiskTO, hyperHost, context);

volume resize: Fix issue with volume resize on VMWare (deploy as-is templates) by Pearl1594 · Pull Request #4829 · apache/cloudstack · GitHub

Repository navigation

volume resize: Fix issue with volume resize on VMWare (deploy as-is templates) - #4829

volume resize: Fix issue with volume resize on VMWare (deploy as-is templates)#4829yadvr merged 9 commits intoapache:4.15apache/cloudstack:4.15from shapeblue:resize-volume-issueshapeblue/cloudstack:resize-volume-issueCopy head branch name to clipboard

Conversation

Pearl1594 commented Mar 16, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Types of changes

Feature/Enhancement Scale or Bug Severity

Bug Severity

Screenshots (if appropriate):

How Has This Been Tested?

Prior to the fix:

Post fix

Uh oh!

Pearl1594 commented Mar 16, 2021

Uh oh!

blueorangutan commented Mar 16, 2021

Uh oh!

blueorangutan commented Mar 16, 2021

Uh oh!

Pearl1594 commented Mar 16, 2021

Uh oh!

blueorangutan commented Mar 16, 2021

Uh oh!

blueorangutan commented Mar 16, 2021

Uh oh!

nvazquez commented Mar 16, 2021

Uh oh!

blueorangutan commented Mar 16, 2021

Uh oh!

blueorangutan commented Mar 16, 2021

Uh oh!

nvazquez left a comment

Choose a reason for hiding this comment

Uh oh!

sureshanaparti Mar 17, 2021

Choose a reason for hiding this comment

Uh oh!

Pearl1594 Mar 17, 2021

Choose a reason for hiding this comment

Uh oh!

Pearl1594 commented Mar 17, 2021

Uh oh!

Pearl1594 commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 17, 2021

Uh oh!

Pearl1594 commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 17, 2021

Uh oh!

Pearl1594 commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 17, 2021

Uh oh!

Pearl1594 commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 17, 2021

Uh oh!

blueorangutan commented Mar 22, 2021

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Pearl1594 commented Mar 23, 2021

Uh oh!

blueorangutan commented Mar 23, 2021

Uh oh!

blueorangutan commented Mar 23, 2021

Uh oh!

Pearl1594 commented Mar 23, 2021

Uh oh!

volume resize: Fix issue with volume resize on VMWare (deploy as-is templates)#4829
yadvr merged 9 commits into
apache:4.15from
shapeblue:resize-volume-issue

Pearl1594 commented Mar 16, 2021 •
edited
Loading