Multiple CD-ROM / ISO Support Per VM by Damans227 · Pull Request #13101 · apache/cloudstack

Damans227 · 2026-05-04T20:53:49Z

Description

Adds multiple CD-ROM support per VM on KVM. Today CloudStack allows only one ISO per VM, forcing a manual ISO-swap workflow to install Windows on KVM (Windows installer + VirtIO drivers). After this PR each VM can hold up to vm.cdrom.max.count ISOs simultaneously, each appearing as its own CD-ROM inside the guest.

What changes:

detachIso takes an optional id (required when multiple ISOs are attached).
listVirtualMachines returns isos[] and cdrommaxcount; legacy isoid still reflects the bootable ISO.
New cluster-scoped config vm.iso.max.count (default 1). Values above the host-advertised cap fail loudly instead of silent clamping.
KVM agent advertises its supported CD-ROM count via the host.cdrom.max.count host detail; management reads from host_details, no hardcoded hypervisor numbers in management code.
New vm_iso_map table holds extra slots; user_vm.iso_id is kept as the bootable pointer. ISO deletion is gated by both tables.

Scope: KVM only.

Design doc: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=421957739

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
Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

Major
Minor

Screenshots (if appropriate):

Screen.Recording.2026-05-04.at.5.20.18.PM.mov

How Has This Been Tested?

End-to-end on a KVM lab across cap=1/2/5, plus the UI manually.
Unit tests added in TemplateManagerImplTest for slot allocation, the duplicate-attach check, and the detach-id resolution helper.

How did you try to break this feature and the system with this change?

Set vm.cdrom.max.count above the host's advertised cap to confirm the validate-and-throw fires on attach/deploy
Tried to re-attach an already-attached ISO
Detached the bootable and non-bootable ISO in turn to confirm user_vm.iso_id and vm_iso_map stay in sync
Stopped and restarted a multi-ISO VM to confirm boot-time persistence
Confirmed legacy single-ISO API consumers (isoid in listVirtualMachines) see no functional change

…alidation

… attached ISOs

…y fetching

… duplicates

codecov · 2026-05-04T21:02:18Z

Codecov Report

❌ Patch coverage is 42.38806% with 193 lines in your changes missing coverage. Please review.
✅ Project coverage is 18.78%. Comparing base (6bc83a3) to head (b9304dd).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...n/java/com/cloud/template/TemplateManagerImpl.java	39.41%	88 Missing and 15 partials ⚠️
...rc/main/java/com/cloud/vm/dao/VmIsoMapDaoImpl.java	0.00%	44 Missing ⚠️
...ava/com/cloud/api/query/dao/UserVmJoinDaoImpl.java	48.78%	15 Missing and 6 partials ⚠️
...apache/cloudstack/api/response/UserVmResponse.java	33.33%	8 Missing ⚠️
...ud/hypervisor/kvm/storage/KVMStorageProcessor.java	0.00%	7 Missing ⚠️
.../schema/src/main/java/com/cloud/vm/VmIsoMapVO.java	83.33%	3 Missing and 1 partial ⚠️
...ervisor/kvm/resource/LibvirtComputingResource.java	60.00%	2 Missing and 2 partials ⚠️
.../cloudstack/api/command/user/iso/DetachIsoCmd.java	0.00%	1 Missing ⚠️
...e/cloudstack/api/response/AttachedIsoResponse.java	96.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #13101      +/-   ##
============================================
+ Coverage     18.75%   18.78%   +0.02%     
- Complexity    17966    18022      +56     
============================================
  Files          6160     6164       +4     
  Lines        552578   552927     +349     
  Branches      67348    67411      +63     
============================================
+ Hits         103660   103874     +214     
- Misses       437512   437619     +107     
- Partials      11406    11434      +28

Flag	Coverage Δ
uitests	`3.53% <ø> (+<0.01%)`	⬆️
unittests	`19.98% <42.38%> (+0.03%)`	⬆️

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

☔ View full report in Codecov by Harness.
📢 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.

Damans227 · 2026-05-04T21:31:09Z

@blueorangutan package

blueorangutan · 2026-05-04T21:32:04Z

@Damans227 a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

blueorangutan · 2026-05-04T21:45:40Z

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

…avior for listByVmId

Damans227 · 2026-05-04T22:20:54Z

@blueorangutan package

blueorangutan · 2026-05-04T22:22:05Z

@Damans227 a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

blueorangutan · 2026-05-04T23:06:40Z

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

…ndling

blueorangutan · 2026-05-14T20:42:55Z

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

Damans227 · 2026-05-14T20:49:48Z

@blueorangutan package

blueorangutan · 2026-05-14T20:52:04Z

@Damans227 a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

blueorangutan · 2026-05-14T21:36:43Z

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

github-actions · 2026-06-08T07:10:17Z

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

kiranchavala · 2026-06-08T07:19:53Z

@Damans227 can you resolve the merge conflicts

kiranchavala · 2026-06-08T07:20:08Z

@blueorangutan test

blueorangutan · 2026-06-08T07:22:03Z

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

Damans227 · 2026-06-08T14:25:27Z

@Damans227 can you resolve the merge conflicts

Lemme take a look.

blueorangutan · 2026-06-08T22:01:54Z

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

Test	Result	Time (s)	Test File

…isplay it

…CD-ROM capacity

Damans227 · 2026-06-10T14:20:36Z

@blueorangutan package

blueorangutan · 2026-06-10T14:22:05Z

@Damans227 a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

blueorangutan · 2026-06-10T14:29:06Z

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

…tructor tests

Damans227 · 2026-06-10T14:41:46Z

@blueorangutan package

blueorangutan · 2026-06-10T14:44:05Z

@Damans227 a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

blueorangutan · 2026-06-10T15:30:00Z

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

Damans227 · 2026-06-10T17:13:06Z

@blueorangutan package

blueorangutan · 2026-06-10T17:14:04Z

@Damans227 a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

blueorangutan · 2026-06-10T17:58:51Z

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

Damans227 added 16 commits May 4, 2026 11:29

pre-allocate a second empty cdrom slot at boot (hardcoded)

d3668e2

drive cdrom slot count via vm.cdrom.max.count ConfigKey

90b415f

add vm_iso_map table + VO/DAO

0516f82

persist multi-ISO state via vm_iso_map

945f2bd

carry target cdrom slot through AttachCommand to KVM agent

9570369

enforce per-VM cdrom cap, clamp to hypervisor max

5fd487a

make detachIso accepts an ISO id

3a13ac7

expose attached ISOs as isos[] in listVirtualMachines response

b05b9f9

extract CDROM_PRIMARY_DEVICE_SEQ constant

6d2ae5a

unit tests for cdrom slot allocation logic

a44b4fb

implement multi-ISO attachment and detachment for VMs with enhanced v…

a6afd26

…alidation

implement multi-ISO display in InstanceTab with computed property for…

71fe082

… attached ISOs

add warning alert for max CDROM selections and enhance global capacit…

2ff3ce6

…y fetching

enhance ISO attachment validation to handle multiple ISOs and prevent…

e394f1c

… duplicates

refactor ISO attachment logic for detachment and validation

e7185ab

add unit tests for ISO detachment resolution and validation logic

1e5bd63

boring-cyborg Bot added component:api component:kvm component:UI labels May 4, 2026

add mock for VmIsoMapDao in UserVmJoinDaoImplTest and set lenient beh…

79e8edf

…avior for listByVmId

Damans227 added 3 commits May 4, 2026 19:32

refactor ISO attachment logic and enhance UI for multi-CDROM management

bac8122

refactor ISO attachment methods to use VM ID and improve parameter ha…

0570cf2

…ndling

remove unnecessary mock for VM ISO mapping in TemplateManagerImplTest

61396a1

Merge remote-tracking branch 'upstream/main' into fr283-multi-cdrom

68ac686

github-actions Bot removed the status:has-conflicts label May 15, 2026

github-actions Bot added the status:has-conflicts label Jun 8, 2026

Merge branch 'main' into fr283-multi-cdrom

817796d

github-actions Bot removed the status:has-conflicts label Jun 8, 2026

github-actions Bot mentioned this pull request Jun 8, 2026

[repo-status] Daily Status Report – June 8, 2026 #13381

Closed

Damans227 added 2 commits June 10, 2026 09:41

feat: add bootable ISO flag to AttachedIsoResponse and update UI to d…

489bf98

…isplay it

refactor: simplify effectiveMaxCdroms method and improve logging for …

e8ece3b

…CD-ROM capacity

test: update AttachedIsoResponseTest to include bootable flag in cons…

6f7fcf4

…tructor tests

Damans227 added 2 commits June 10, 2026 12:52

feat: include bootable flag in AttachedIsoResponse for user VMs

6b399b4

feat: enhance CD-ROM management by defining empty slots for user VMs

b9304dd

Conversation

Damans227 commented May 4, 2026 • 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

Feature/Enhancement Scale

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

Uh oh!

codecov Bot commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Damans227 commented May 4, 2026

Uh oh!

blueorangutan commented May 4, 2026

Uh oh!

blueorangutan commented May 4, 2026

Uh oh!

Damans227 commented May 4, 2026

Uh oh!

blueorangutan commented May 4, 2026

Uh oh!

blueorangutan commented May 4, 2026

Uh oh!

blueorangutan commented May 14, 2026

Uh oh!

Damans227 commented May 14, 2026

Uh oh!

blueorangutan commented May 14, 2026

Uh oh!

blueorangutan commented May 14, 2026

Uh oh!

github-actions Bot commented Jun 8, 2026

Uh oh!

kiranchavala commented Jun 8, 2026

Uh oh!

kiranchavala commented Jun 8, 2026

Uh oh!

blueorangutan commented Jun 8, 2026

Uh oh!

Damans227 commented Jun 8, 2026

Uh oh!

blueorangutan commented Jun 8, 2026

Uh oh!

Damans227 commented Jun 10, 2026

Uh oh!

blueorangutan commented Jun 10, 2026

Uh oh!

blueorangutan commented Jun 10, 2026

Uh oh!

Damans227 commented Jun 10, 2026

Uh oh!

blueorangutan commented Jun 10, 2026

Uh oh!

blueorangutan commented Jun 10, 2026

Uh oh!

Damans227 commented Jun 10, 2026

Uh oh!

blueorangutan commented Jun 10, 2026

Uh oh!

blueorangutan commented Jun 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Damans227 commented May 4, 2026 •

edited

Loading

codecov Bot commented May 4, 2026 •

edited

Loading