Marcus Sorensen created CLOUDSTACK-4887:
-------------------------------------------
Summary: CLVM broken
Key: CLOUDSTACK-4887
URL: https://issues.apache.org/jira/browse/CLOUDSTACK-4887
Project: CloudStack
Issue Type: Bug
Security Level: Public (Anyone can view this level - this is the default.)
Components: KVM
Affects Versions: Future
Reporter: Marcus Sorensen
Assignee: Chris Suich
Priority: Blocker
Fix For: Future
Chris,
I think commit 180cfa19 broke CLVM primary storage for KVM. I'm failing VM
deploy from template. I've been building a 'sanity check' test that focuses on
the KVM specific suff (tests storage types and supported host OS for now), and
this bubbled up.
I reverted this one part in my local code and am testing (seems to fix it), but
since I'm not clear on the refactor efforts I'm not sure what it should really
be changed to in order to meet your requirements and keep CLVM working.
@@ -65,7 +68,7 @@ public class StorageSubsystemCommandHandlerBase implements
StorageSubsystemComma
DataStoreTO srcDataStore = srcData.getDataStore();
DataStoreTO destDataStore = destData.getDataStore();
- if ((srcData.getObjectType() == DataObjectType.TEMPLATE) &&
(srcDataStore instanceof NfsTO) && (destData.getDataStore().getRole() ==
DataStoreRole.Primary)) {
+ if ((srcData.getObjectType() == DataObjectType.TEMPLATE) &&
(destData.getObjectType() == DataObjectType.TEMPLATE &&
destData.getDataStore().getRole() == DataStoreRole.Primary)) {
//copy template to primary storage
return processor.copyTemplateToPrimaryStorage(cmd);
} else if (srcData.getObjectType() == DataObjectType.TEMPLATE &&
srcDataStore.getRole() == DataStoreRole.Primary && destDataStore.getRole() ==
DataStoreRole.Primary) {
4.2 command:
{ Cmd , MgmtId: 52241639751, via: 1, Ver: v1, Flags: 100111,
[{"org.apache.cloudstack.storage.command.CopyCommand":{"srcTO":{"org.apache.cloudstack.storage.to.TemplateObjectTO":{"path":"template/tmpl/1/201/fe75caa3-78fd-38ba-b34c-101d0502df2e.qcow2","origUrl":"http://marcus.mlsorensen.com/cloudstack-extras/tiny-centos-63.qcow2","uuid":"4795671b-dd72-4e62-9aac-c0e1d6732003","id":201,"format":"QCOW2","accountId":1,"checksum":"44cd0e6330a59f031460bc18a40c95a2","hvm":true,"displayText":"tiny","imageDataStore":{"com.cloud.agent.api.to.NfsTO":{"_url":"nfs://172.17.10.10:/nfs/secondary","_role":"Image"}},"name":"201-1-2b35186d-79a6-33dc-8b33-83eb650e5e1d","hypervisorType":"KVM"}},"destTO":{"org.apache.cloudstack.storage.to.VolumeObjectTO":{"uuid":"73f86d88-ccff-4dfb-ac86-2d76b7891117","volumeType":"ROOT","dataStore":{"org.apache.cloudstack.storage.to.PrimaryDataStoreTO":{"uuid":"08e8d399-b238-48f9-a497-1f7f5285d655","id":2,"poolType":"CLVM","host":"localhost","path":"vg0","port":0}},"name":"ROOT-6","size":1073741824,"volumeId":6,"vmName":"i-1-6-VM","accountId":1,"format":"QCOW2","id":6,"hypervisorType":"KVM"}},"executeInSequence":true,"wait":10800}}]
}
4.2 response:
{ Ans: , MgmtId: 52241639751, via: 1, Ver: v1, Flags: 110,
[{"org.apache.cloudstack.storage.command.CopyCmdAnswer":{"newData":{"org.apache.cloudstack.storage.to.VolumeObjectTO":{"path":"f69879c0-ae3b-433a-841f-f1f5afc04fc7","accountId":0,"format":"RAW","id":0}},"result":true,"wait":0}}]
}
master command:
{ Cmd , MgmtId: 52241639751, via: 1(devcloud-kvm-u), Ver: v1, Flags: 100111,
[{"org.apache.cloudstack.storage.command.CopyCommand":{"srcTO":{"org.apache.cloudstack.storage.to.TemplateObjectTO":{"path":"template/tmpl/1/4/02170718-210a-3d8c-91a2-2793ed52f1d8.qcow2","origUrl":"http://download.cloud.com/releases/2.2.0/eec2209b-9875-3c8d-92be-c001bd8a0faf.qcow2.bz2","uuid":"07088e98-1fda-11e3-a1ff-000c29d82947","id":4,"format":"QCOW2","accountId":1,"checksum":"ed0e788280ff2912ea40f7f91ca7a249","hvm":false,"displayText":"CentOS
5.5(64-bit) no GUI
(KVM)","imageDataStore":{"com.cloud.agent.api.to.NfsTO":{"_url":"nfs://172.17.10.10:/nfs/secondary","_role":"Image"}},"name":"centos55-x86_64","hypervisorType":"KVM"}},"destTO":{"org.apache.cloudstack.storage.to.VolumeObjectTO":{"uuid":"e08f2a84-0d8b-4c6e-9593-a7554fd57b78","volumeType":"ROOT","dataStore":{"org.apache.cloudstack.storage.to.PrimaryDataStoreTO":{"uuid":"0f77072d-c76f-449c-bc6d-e2504644c0ca","id":2,"poolType":"CLVM","host":"localhost","path":"vg0","port":0}},"name":"ROOT-7","size":8589934592,"volumeId":7,"vmName":"i-1-7-VM","accountId":1,"format":"QCOW2","id":7,"hypervisorType":"KVM"}},"executeInSequence":true,"wait":10800}}]
}
master response:
{ Ans: , MgmtId: 52241639751, via: 1, Ver: v1, Flags: 110,
[{"com.cloud.agent.api.Answer":{"result":false,"details":"not implemented
yet","wait":0}}] }
--
This message was sent by Atlassian JIRA
(v6.1#6144)