Re: Review Request 19779: Fixed[CLOUDSTACK-5150]:-Creating template from a VM in Simulator results in incorrect size

[daan Hoogland <da...@gmail.com>]

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



plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorStorageProcessor.java
<https://reviews.apache.org/r/19779/#comment75883>

    I would rather you get the size from the source. Can you have a look?


- daan Hoogland


On March 28, 2014, 7:10 a.m., Girish Chaudhari wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19779/
> -----------------------------------------------------------
> 
> (Updated March 28, 2014, 7:10 a.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-5150
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5150
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Problem Statement:- In Simulator, When provisioning the VM using the personal template, results into the NullPointerException. 
> Root Cause:- The template creation command is setting up the size as NULL in the tempalte_store_ref table and which is something causing this issue. 
> Solution: To set up the size to 100 ( any Non-Null value) in the template create mocking process, which ultimately setting up the size to 100 in the tempalte_store_ref table and hence solving the issue.
> 
> 
> Diffs
> -----
> 
>   plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorStorageProcessor.java 8a78b87 
> 
> Diff: https://reviews.apache.org/r/19779/diff/
> 
> 
> Testing
> -------
> 
> Tested it on the 4.3, the size in tempalte_store_ref table is getting populated as 100 and could instantiate the VM using the personal template.
> 
> 
> Thanks,
> 
> Girish Chaudhari
> 
>