cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sateesh Chodapuneedi" <sateesh.chodapune...@citrix.com>
Subject Re: Review Request: Failed to Prepare Secondary Storage in VMware - CLOUDSTACK-1262
Date Thu, 14 Feb 2013 06:18:18 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9435/#review16563
-----------------------------------------------------------



plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
<https://reviews.apache.org/r/9435/#comment35141>

    How about reading the location from paths.script in environment.properties file?
    I think in that way whenever packaging changes the path need not be modified in manager's
source code.


- Sateesh Chodapuneedi


On Feb. 13, 2013, 9:21 p.m., Rayees Namathponnan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9435/
> -----------------------------------------------------------
> 
> (Updated Feb. 13, 2013, 9:21 p.m.)
> 
> 
> Review request for cloudstack, Venkata Siva Vijayendra Bhamidipati, Sateesh Chodapuneedi,
and Rohit Yadav.
> 
> 
> Description
> -------
> 
> This patch for defect https://issues.apache.org/jira/browse/CLOUDSTACK-1262, in latest
builds  failed to prepare secondary storage VM in VMware, since we are expecting systemvm.iso
is available at /usr/lib64/cloud/common//vms/systemvm.iso instead of /usr/share/cloudstack-common/vms/systemvm.iso
> 
> Branch 4.1
> 
> 
> This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-1262.
> 
> 
> Diffs
> -----
> 
>   plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
88e03f5 
> 
> Diff: https://reviews.apache.org/r/9435/diff/
> 
> 
> Testing
> -------
> 
> Tested with VMware (Branch 4.1)
> 
> 
> Thanks,
> 
> Rayees Namathponnan
> 
>


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