Allow overriding root disk offering & size, and expunge old root disk while restoring a VM by vishesh92 · Pull Request #8800 · apache/cloudstack · GitHub

Description

This PR allows overriding root diskoffering id & size, and expunge old root disk while restoring VM. This is useful when you have to restore a VM with a new template but the newer template's size is smaller than the existing one.

New parameters for restoreVirtualMachine:

diskofferingid - Override root volume's diskoffering
rootdisksize - Override root volume's size (in GB)
expunge - Option to expunge old volumes after restore
details - Option to specify custom parameters for the selected disk offering

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

Major
Minor

Screenshots (if appropriate):

How Has This Been Tested?

Tested Manually

@blueorangutan package

@vishesh92 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 Report

Attention: Patch coverage is 57.14286% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 30.90%. Comparing base (0043540) to head (ec0790c).
Report is 12 commits behind head on 4.19.

Files	Patch %	Lines
.../src/main/java/com/cloud/vm/UserVmManagerImpl.java	44.44%	9 Missing and 6 partials ⚠️
...e/cloudstack/api/command/user/vm/RestoreVMCmd.java	33.33%	0 Missing and 2 partials ⚠️
...n/java/com/cloud/vm/VirtualMachineManagerImpl.java	83.33%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               4.19    #8800      +/-   ##
============================================
- Coverage     30.99%   30.90%   -0.10%     
+ Complexity    34369    34272      -97     
============================================
  Files          5355     5355              
  Lines        376630   376715      +85     
  Branches      54807    54822      +15     
============================================
- Hits         116741   116412     -329     
- Misses       244535   244947     +412     
- Partials      15354    15356       +2

Flag	Coverage Δ
simulator-marvin-tests	24.72% <52.38%> (-0.14%)	⬇️
uitests	4.38% <ø> (-0.01%)	⬇️
unit-tests	16.57% <11.90%> (-0.01%)	⬇️

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.

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

@blueorangutan package

@vishesh92 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.

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

@blueorangutan test

@vishesh92 a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-9532)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42393 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8800-t9532-kvm-centos7.zip
Smoke tests completed. 129 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test	Result	Time (s)	Test File

@blueorangutan package

@vishesh92 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.

Code LGTM

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

@blueorangutan test

@vishesh92 a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-9585)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 47382 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8800-t9585-kvm-centos7.zip
Smoke tests completed. 128 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test	Result	Time (s)	Test File
test_02_trigger_shutdown	Failure	336.76	test_safe_shutdown.py

Hi, @vishesh92, good change!

Could you describe your tests? Have you tried it with a compute offering with Disk offering strictness? What about expunge permissions?

[SF] Trillian test result (tid-9720)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 50313 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8800-t9720-kvm-centos7.zip
Smoke tests completed. 128 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test	Result	Time (s)	Test File
test_01_events_resource	Error	412.26	test_events_resource.py

@blueorangutan test

@vishesh92 a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-9740)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 47010 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8800-t9740-kvm-centos7.zip
Smoke tests completed. 125 look OK, 4 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test	Result	Time (s)	Test File
test_01_events_resource	Error	288.71	test_events_resource.py
test_01_events_resource	Error	288.72	test_events_resource.py
test_04_deploy_vm_for_other_user_and_test_vm_operations	Failure	87.23	test_network_permissions.py
ContextSuite context=TestNetworkPermissions>:teardown	Error	1.36	test_network_permissions.py
test_create_pvlan_network	Error	0.07	test_pvlan.py
test_01_redundant_vpc_site2site_vpn	Failure	397.83	test_vpc_vpn.py

@blueorangutan package

@sureshanaparti 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.

clgtm

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

@blueorangutan package

@vishesh92 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.

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

@blueorangutan test

@vishesh92 a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-9759)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 47374 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8800-t9759-kvm-centos7.zip
Smoke tests completed. 127 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test	Result	Time (s)	Test File
test_01_events_resource	Error	292.22	test_events_resource.py
test_01_events_resource	Error	292.23	test_events_resource.py
test_04_deploy_vm_for_other_user_and_test_vm_operations	Failure	89.92	test_network_permissions.py
ContextSuite context=TestNetworkPermissions>:teardown	Error	1.38	test_network_permissions.py

LGTM, manually tested

… while restoring a VM (apache#8800) * Allow overriding root diskoffering id & size while restoring VM * UI changes * Allow expunging of old disk while restoring a VM * Resolve comments * Address comments * Duplicate volume's details while duplicating volume * Allow setting IOPS for the new volume * minor cleanup * fixup * Add checks for template size * Replace strings for IOPS with constants * Fix saveVolumeDetails method * Fixup * Fixup UI styling

vishesh92 added 2 commits March 19, 2024 14:32

Allow overriding root diskoffering id & size while restoring VM

f8cf8a3

UI changes

2c1bee9

boring-cyborg Bot added component:api component:compute component:orchestration component:UI labels Mar 19, 2024

vishesh92 changed the title Allow overriding root disk offering while restoring a VM Allow overriding root disk offering & size while restoring a VM Mar 19, 2024

yadvr added this to the 4.19.1.0 milestone Mar 19, 2024

Allow expunging of old disk while restoring a VM

76e3990

sureshanaparti changed the title Allow overriding root disk offering & size while restoring a VM Allow overriding root disk offering & size, and expunge old root disk while restoring a VM Mar 26, 2024

sureshanaparti approved these changes Mar 26, 2024

View reviewed changes

vishesh92 marked this pull request as ready for review March 27, 2024 08:27

Resolve comments

ec0790c

Replace strings for IOPS with constants

856b22e

sureshanaparti approved these changes Apr 8, 2024

View reviewed changes

Fix saveVolumeDetails method

37de99c

vishesh92 added 2 commits April 10, 2024 17:39

Fixup

c5cf358

Fixup UI styling

4e59e30

borisstoyanov approved these changes Apr 12, 2024

View reviewed changes

yadvr marked this pull request as ready for review April 12, 2024 12:17

yadvr removed the status:needs-testing label Apr 12, 2024

yadvr merged commit b998e7d into apache:4.19 Apr 12, 2024

yadvr deleted the add-disk-offering-restore-vm branch April 12, 2024 12:17

This was referenced Apr 12, 2024

Fixup 4.19 build issue #8905

Merged

Unstable 4.19 and main branch smoketests #8931

Closed

weizhouapache mentioned this pull request Apr 22, 2024

Fix null pointer exception in restore VM #8930

Merged

3 of 13 tasks

Pearl1594 mentioned this pull request Mar 18, 2025

Fix smoke tests due to change in behavior of restore VM #10583

Merged

1 of 12 tasks

sureshanaparti mentioned this pull request May 8, 2025

VMware 80u2 and 80u3 updates/fixes #10586

Merged

2 of 14 tasks

suryag1201 mentioned this pull request Sep 9, 2026

Custom IOPS not applied to root disk when deploying a VM from a template #14103

Open

Allow overriding root disk offering & size, and expunge old root disk while restoring a VM by vishesh92 · Pull Request #8800 · apache/cloudstack · GitHub

Repository navigation

Allow overriding root disk offering & size, and expunge old root disk while restoring a VM - #8800

Allow overriding root disk offering & size, and expunge old root disk while restoring a VM#8800yadvr merged 14 commits intoapache:4.19apache/cloudstack:4.19from shapeblue:add-disk-offering-restore-vmshapeblue/cloudstack:add-disk-offering-restore-vmCopy head branch name to clipboard

Conversation

vishesh92 commented Mar 19, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Types of changes

Feature/Enhancement Scale

Screenshots (if appropriate):

How Has This Been Tested?

Uh oh!

vishesh92 commented Mar 19, 2024

Uh oh!

blueorangutan commented Mar 19, 2024

Uh oh!

codecov Bot commented Mar 19, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

blueorangutan commented Mar 19, 2024

Uh oh!

vishesh92 commented Mar 19, 2024

Uh oh!

blueorangutan commented Mar 19, 2024

Uh oh!

blueorangutan commented Mar 19, 2024

Uh oh!

vishesh92 commented Mar 20, 2024

Uh oh!

blueorangutan commented Mar 20, 2024

Uh oh!

blueorangutan commented Mar 20, 2024

Uh oh!

vishesh92 commented Mar 26, 2024

Uh oh!

blueorangutan commented Mar 26, 2024

Uh oh!

sureshanaparti left a comment

Choose a reason for hiding this comment

Uh oh!

blueorangutan commented Mar 26, 2024

Uh oh!

vishesh92 commented Mar 26, 2024

Uh oh!

blueorangutan commented Mar 26, 2024

Uh oh!

blueorangutan commented Mar 27, 2024

Uh oh!

gp-santos commented Mar 28, 2024

Uh oh!

blueorangutan commented Apr 5, 2024

Uh oh!

vishesh92 commented Apr 5, 2024

Uh oh!

blueorangutan commented Apr 5, 2024

Uh oh!

blueorangutan commented Apr 6, 2024

Uh oh!

sureshanaparti commented Apr 8, 2024

Uh oh!

blueorangutan commented Apr 8, 2024

Uh oh!

sureshanaparti left a comment

Choose a reason for hiding this comment

Uh oh!

blueorangutan commented Apr 8, 2024

Uh oh!

vishesh92 commented Apr 8, 2024

Uh oh!

blueorangutan commented Apr 8, 2024

Uh oh!

blueorangutan commented Apr 8, 2024

Uh oh!

vishesh92 commented Apr 8, 2024

Uh oh!

blueorangutan commented Apr 8, 2024

Uh oh!

blueorangutan commented Apr 9, 2024

Allow overriding root disk offering & size, and expunge old root disk while restoring a VM#8800
yadvr merged 14 commits into
apache:4.19from
shapeblue:add-disk-offering-restore-vm

vishesh92 commented Mar 19, 2024 •
edited
Loading

codecov Bot commented Mar 19, 2024 •
edited
Loading