fix: support SharedMountPoint volume checks for importVm#12946
fix: support SharedMountPoint volume checks for importVm#12946DaanHoogland merged 1 commit intoapache:4.22from
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md)
|
1b85de6 to
adbf902
Compare
weizhouapache
left a comment
There was a problem hiding this comment.
code lgtm
not tested yet
|
@prashanthr2 |
adbf902 to
fac1eea
Compare
|
@blueorangutan package |
|
@harikrishna-patnala 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]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 17341 |
|
[SF] Trillian test result (tid-15808)
|
|
[LL] Trillian Build Failed (tid-7185) |
|
[LL] Trillian Build Failed (tid-7188) |
rajujith
left a comment
There was a problem hiding this comment.
@prashanthr2
Import VM works.
Import Volume doesn't show any primary storage NFS and sharedmount as well in UI.
API for import volume fails:
(localcloud) 🐱 > import volume storageid=ef5cb028-27cc-4c03-8046-77f03209e29b path=data1.qcow2 diskofferingid=1f10a877-5026-45e5-9554-8e2443c29369 name=data1
{
"account": "admin",
"accountid": "6f603367-33d8-11f1-bb73-1e0090000b68",
"cmd": "org.apache.cloudstack.api.command.admin.volume.ImportVolumeCmd",
"completed": "2026-04-09T06:30:02+0000",
"created": "2026-04-09T06:30:02+0000",
"domainid": "b57f0ddf-33d7-11f1-bb73-1e0090000b68",
"domainpath": "ROOT",
"jobid": "d8cbbee7-2d31-4caa-9023-9d7adfde7d91",
"jobprocstatus": 0,
"jobresult": {
"errorcode": 530,
"errortext": "Importing and unmanaging volume are not supported for pool type SharedMountPoint on hypervisor KVM"
},
"jobresultcode": 530,
"jobresulttype": "object",
"jobstatus": 2,
"userid": "6f611500-33d8-11f1-bb73-1e0090000b68"
}
🙈 Error: async API failed for job d8cbbee7-2d31-4caa-9023-9d7adfde7d91
|
Awesome work, congrats on your first merged pull request! |
Summary
Fixes #12944
Testing