<div dir="ltr">HI Matthias Runge,<div><br><div>Which feature in trove are you talking about? </div><div>And even which capabilities are missing which will make the patch fail?</div></div><div>I believe the patch has nothing to do with  <a href="https://review.openstack.org/#/c/83503/">https://review.openstack.org/#/c/83503/</a></div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 4, 2014 at 3:45 PM, Matthias Runge <span dir="ltr"><<a href="mailto:mrunge@redhat.com" target="_blank">mrunge@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Wed, Jun 04, 2014 at 02:18:19PM +0530, Harshada Kakad wrote:<br>
> HI All,<br>
> Could someone please, review the bug<br>
> <a href="https://bugs.launchpad.net/horizon/+bug/1301359" target="_blank">https://bugs.launchpad.net/horizon/+bug/1301359</a><br>
><br>
><br>
> Made Size parameter optional while creating DB Instance.<br>
><br>
> While creating Database Instance size parameter depends on<br>
> whether trove_volume_support is set. So size paramater is<br>
> set to mandatory if trove_volume_support is set else its kept optional.<br>
><br>
> Here is the link for review :  <a href="https://review.openstack.org/#/c/86295/" target="_blank">https://review.openstack.org/#/c/86295/</a><br>
><br>
</div>Thank you for your patch. It relies on a (currently not merged) feature in trove.<br>
<br>
In an ideal world (or OpenStack test deployment), your patch would fail,<br>
because capabilities to execute your patch are missing (yet), thus<br>
making it more clear, a cooperation with trove is required.<br>
<span class="HOEnZb"><font color="#888888">--<br>
Matthias Runge <<a href="mailto:mrunge@redhat.com">mrunge@redhat.com</a>><br>
<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>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div><b>Regards,</b></div><div><b>Harshada Kakad</b></div><div><b>--------------------------------------------------------------------</b></div>
<div><b>Sr. Software Engineer</b></div><div><b>C3/101, Saudamini Complex, Right Bhusari Colony, Paud Road, Pune – 411013, India</b></div><div><b>Mobile-9689187388</b></div><div><b>Email-Id : <a href="mailto:harshada.kakad@izeltech.com" target="_blank">harshada.kakad@izeltech.com</a></b></div>
<div><b>website : <a href="http://www.izeltech.com" target="_blank">www.izeltech.com</a></b></div></div>
</div>

<br>
<div><font size="1">*****Disclaimer*****</font></div><div><font size="1">The information contained in this e-mail and any attachment(s) to this message are intended for the exclusive use of the addressee(s) and may contain proprietary, confidential or privileged information of Izel Technologies Pvt. Ltd. If you are not the intended recipient, you are notified that any review, use, any form of reproduction, dissemination, copying, disclosure, modification, distribution and/or publication of this e-mail message, contents or its attachment(s) is strictly prohibited and you are requested to notify us the same immediately by e-mail and delete this mail immediately. Izel Technologies Pvt. Ltd accepts no liability for virus infected e-mail or errors or omissions or consequences which may arise as a result of this e-mail transmission.</font></div><div><font size="1">*****End of Disclaimer*****</font></div>