<div dir="ltr">Hi,<div><br></div><div>I'm ok with both accepting this patch and reverting the commit, which introduced the regression, but it would be really nice to have these DB optimizations in Nova.</div><div><br></div>
<div>As for your concern of accepting such optimizations. I don't think, it's a problem of such patches themselves, but rather with the lack of comprehensive tests of complex OpenStack installations in our CI (at the same time I personally believe our CI is the best thing ever happened to OpenStack :), CI team you really rock!).</div>
<div><br></div><div>Anyway, TripleO-CI found this regression. Maybe we should consider adding its job to Nova check/gate pipelines?</div><div><div><br></div><div>Thanks,</div><div>Roman</div></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Sep 9, 2013 at 1:59 PM, Nikola Đipanov <span dir="ltr"><<a href="mailto:ndipanov@redhat.com" target="_blank">ndipanov@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="HOEnZb"><div class="h5">On 09/09/13 11:25, Roman Podolyaka wrote:<br>
> Hi,<br>
><br>
> There is a patch on review (<a href="https://review.openstack.org/#/c/45422/" target="_blank">https://review.openstack.org/#/c/45422/</a>)<br>
> fixing <a href="https://bugs.launchpad.net/tripleo/+bug/1221620" target="_blank">https://bugs.launchpad.net/tripleo/+bug/1221620</a> which has<br>
> importance 'Critical' in Nova and TripleO (long story short: currently<br>
> Nova Baremetal deployments with more than one baremetal node won't work).<br>
><br>
> It would be really nice to have this patch reviewed by core developers,<br>
> so we can fix the bug ASAP.<br>
><br>
<br>
</div></div>Hey - thanks for responding quickly - I commented on the patch and tbh I<br>
am starting to be -1 on this due to issues mentioned on the review.<br>
<br>
I will accept that my take on this is too conservative :) and remove a<br>
-1 if needed to get this in, but at this point, I have some doubts<br>
weather this is the right approach.<br>
<br>
Cheers,<br>
<br>
N.<br>
<br>
<br>
> Thanks,<br>
> Roman<br>
><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>
><br>
<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>
</blockquote></div><br></div>