incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Edison Su <Edison...@citrix.com>
Subject RE: Review Request: cloud-vmware.jar needs to be in the system vm for the PremiumSecondaryStorageResource to work
Date Wed, 25 Jul 2012 00:18:17 GMT
I added a temporary fix in commit, 27e9cdbece932bae9a68d5ec28bf2941f4fc7545, to use com.cloud.storage.resource.NfsSecondaryStorageResource,
instead of PremiumSecondaryStorageResource.
Unless we figure it out how to build/package VMware, we can leave it there.

From: Hugo Trippaers [mailto:HTrippaers@schubergphilis.com]
Sent: Tuesday, July 24, 2012 4:24 PM
To: Edison Su
Cc: cloudstack; Edison Su
Subject: Re: Review Request: cloud-vmware.jar needs to be in the system vm for the PremiumSecondaryStorageResource
to work

Hey disheng,

Good point, however current it means that the secondary storage vm will not work as the default
(premium) configuration will tell the secstorage vm to use the premium storage class which
depends on VMware.

A work around could be to attempt to load the VMware class by code and on class load exception
load the regular Nfs secstorage class.


Cheers,

Hugo

Sent from my iPhone

On 24 jul. 2012, at 23:05, "disheng su" <edison.su@citrix.com<mailto:edison.su@citrix.com>>
wrote:
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5804/




the agent package depends on vmware, if vmware hypervisor is used. We can't just add PremiumSecondaryStorageResource
back into core, as it depends on vmware library, thus may break the apache license.

As we already build vmware separately, need a way to add vmware related jar files into agent.tgz,
such as "ant package-agent-with-vmware"


- disheng


On July 6th, 2012, 11:50 a.m., Hugo Trippaers wrote:
Review request for cloudstack.
By Hugo Trippaers.

Updated July 6, 2012, 11:50 a.m.

Description

PremiumSecondaryStorageResource depends on VmwareSecondaryStorageContextFactory and VmwareSecondaryStorageResourceHandler,
which were move to cloud-vmware.jar


Testing

Create new zone

Download template after ssvm has booted


Diffs

 *   build/build-cloud.xml (b8ad09e)
 *   build/package.xml (2b5a74b)

View Diff<https://reviews.apache.org/r/5804/diff/>



Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message