You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cloudstack.apache.org by "Marcus Sorensen (JIRA)" <ji...@apache.org> on 2013/10/18 01:48:43 UTC

[jira] [Resolved] (CLOUDSTACK-4887) CLVM broken

     [ https://issues.apache.org/jira/browse/CLOUDSTACK-4887?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marcus Sorensen resolved CLOUDSTACK-4887.
-----------------------------------------

    Resolution: Fixed

> 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: Marcus Sorensen
>            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)