<div dir="ltr"><div><div><div><div><span name="Shalini Khandelwal" class="">Goodday, Shalini Khandelwal. <br><br><br></span></div><span name="Shalini Khandelwal" class="">You brought really good topic. I'd like to make few things clear:<br>
</span></div>1. <a id="yui_3_10_3_1_1389692101822_727" rel="nofollow" href="https://bugs.launchpad.net/trove/+bug/1267945">https://bugs.launchpad.net/trove/+bug/1267945</a> there is already registred bug, even before your BP.<br>
</div>2. I see problem in proposed test coverage, since BP is telling us, that the main idea is to improve test coverage (int and units).<br></div><div>3. Patch for trove should be dependent on <a href="https://review.openstack.org/#/c/63636/">https://review.openstack.org/#/c/63636/</a> and <a href="https://review.openstack.org/#/c/63102/">https://review.openstack.org/#/c/63102/</a> because this patches are changing codebase (relared to heat).<br>
<br></div><div>Thanks for making Trove better. <br><br>Best regards, Denis Makogon.<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014/1/14 Shalini Khandelwal <span dir="ltr"><<a href="mailto:shalini.khandelwal@globallogic.com" target="_blank">shalini.khandelwal@globallogic.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Hi,<br><br><br></div>I am writing this mail to shed some light on the implementation of tests for heat based instance creation.<br>
</div>I have implemented unit tests and int-tests for the purpose. These tests are needed in trove because there is no criteria for testing the creation of instance using heat or any template change, in present trove code.<br>

<br>For the purpose a new group GROUP_HEAT is added in 
trove-integration and corresponding tests are written in trove, which 
creates instance using heat.<br></div><b><br>Following is the blueprint implemented:<br></b><a href="https://blueprints.launchpad.net/trove/+spec/test-heat-based-instance-workflow" target="_blank">https://blueprints.launchpad.net/trove/+spec/test-heat-based-instance-workflow</a><br>

<br></div></div><b>Two check-in are as follows:<br># </b><a href="https://review.openstack.org/#/c/66499/" target="_blank">https://review.openstack.org/#/c/66499/</a><br><b># </b><a href="https://review.openstack.org/#/c/66500/" target="_blank">https://review.openstack.org/#/c/66500/</a><br>

<div><div><div><div><br></div><div><b>Both check-in are inter-dependent:</b><br><b>1. </b>As included group (GROUP_HEAT in trove-integration) is not there in present trove code, reddwarf giving -1 (int-tests failed with error-No such module)<br>

</div><div><b>2. </b>Trove-integration check-in needs to be merged so that int-tests written in trove get executed.<br><br><br></div><div>These changes need to be merged for proper testing. Hoping for a positive response and cooperation.<br>

<br clear="all"></div><div><div><div><div><div><div><div dir="ltr"><div><span style="color:rgb(0,0,255)"><span style><font><br>Thanks and Regards,<br></font></span></span></div><font><span style="color:rgb(0,0,255)"><span style><font>Shalini Khandelwal</font></span></span><br>

</font></div></div>
</div></div></div></div></div></div></div></div></div>
<br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>