cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Girish Shilamkar" <gir...@clogeny.com>
Subject Re: Review Request 13806: CLOUDSTACK-4407: Use extractTemplate API to get hypervisor specific template information
Date Thu, 29 Aug 2013 08:41:30 GMT


> On Aug. 29, 2013, 7:09 a.m., Prasanna Santhanam wrote:
> > Do you want to fix the other test suites listed in the bug report with different
patches? The problem with the hypervisor-specific template exists in other suites as in the
bug report.

I almost forgot about other tests to be updated. I will upload a different patch.


- Girish


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


On Aug. 28, 2013, 8:02 a.m., Girish Shilamkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13806/
> -----------------------------------------------------------
> 
> (Updated Aug. 28, 2013, 8:02 a.m.)
> 
> 
> Review request for cloudstack and Prasanna Santhanam.
> 
> 
> Bugs: CLOUDSTACK-4407
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Template url, hypervisor and format were defined in Service class to be Xenserver specific
>     and therefore registering a new template failed on Vmware and KVM.
>     Fixed this to get hypervisor specific info for registering new template.
> 
> TODO: Update other tests to use the new function get_builtin_template_info
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_templates.py e4599d4 
>   tools/marvin/marvin/integration/lib/base.py b5d086b 
>   tools/marvin/marvin/integration/lib/common.py 4f5acef 
> 
> Diff: https://reviews.apache.org/r/13806/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Girish Shilamkar
> 
>


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