cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Olivier Lemasle" <olivier.lema...@apalia.net>
Subject Re: Review Request 23150: CLOUDSTACK-7006: Restore template ID in ROOT volume usages
Date Thu, 14 Aug 2014 16:18:40 GMT


> On juin 30, 2014, 3:11 apr├Ęs-midi, Sebastien Goasguen wrote:
> > I applied it to 4.3, I will give Kishan a bit of time to see it and apply to master
and 4.4 (probably 4.4.1 since we are cutting the RC now).

Hi,

Is it now possible to apply this patch on branches 4.4 and master?


- Olivier


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


On juin 30, 2014, 10:53 matin, Olivier Lemasle wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23150/
> -----------------------------------------------------------
> 
> (Updated juin 30, 2014, 10:53 matin)
> 
> 
> Review request for cloudstack, Kishan Kavala and Sebastien Goasguen.
> 
> 
> Bugs: CLOUDSTACK-7006
>     https://issues.apache.org/jira/browse/CLOUDSTACK-7006
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> This patch restore the template ID in usages, for ROOT volume created from a template
(see CLOUDSTACK-7006).
> 
> The regression was on the class com.cloud.storage.VolumeManagerImpl (module "cloud-server")
but it has been renamed org.apache.cloudstack.engine.orchestration.VolumeOrchestrator (module
"cloud-engine-orchestration").
> 
> The patch applies on branchs master and branch 4.4.
> I've attached another patch for branch 4.3 (the only difference is formatting).
> 
> 
> Diffs
> -----
> 
>   engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
9ba3451 
> 
> Diff: https://reviews.apache.org/r/23150/diff/
> 
> 
> Testing
> -------
> 
> Tested in ACS 4.3.
> 1. Deploy a new VM from a template.
> 2. Check if usages for the ROOT volume has the right templateId.
> 3. Deploy a new VM from an ISO, and add a DATA volume.
> 4. Check that there is no regression for these two types of volume usages without templateId.
> 
> 
> File Attachments
> ----------------
> 
> patch version for ACS 4.3
>   https://reviews.apache.org/media/uploaded/files/2014/06/29/eee8a901-4141-46e7-b4ce-d9cbcfd8c85f__cloudstack-7006-for-ACS4.3.patch
> 
> 
> Thanks,
> 
> Olivier Lemasle
> 
>


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