<div dir="ltr">Roman, thanks a lot for guidelines! I've updated the change and removed configure_db parameter.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 11, 2016 at 4:58 PM, Roman Podoliaka <span dir="ltr"><<a href="mailto:rpodolyaka@mirantis.com" target="_blank">rpodolyaka@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Anna,<br>
<br>
Thank you for working on this in Neutron!<br>
<br>
EngineFacade is initialized lazily internally - you don't have to do<br>
anything for that in Neutron (you *had to* with "old" EngineFacade -<br>
this is the boiler plate your patch removes).<br>
<br>
I believe, you should be able to call configure(...) unconditionally<br>
as soon as you have parsed the config files. Why do you want to<br>
introduce a new conditional?<br>
<br>
Moreover, if you only have connections to one database (unlike Nova,<br>
which also has Cells databases), you don't need to call configure() at<br>
all - EngineFacade will read the values of config options registered<br>
by oslo.db on the first attempt to get a session / connection.<br>
<br>
Thanks,<br>
Roman<br>
<div><div class="h5"><br>
On Wed, May 11, 2016 at 4:41 PM, Anna Kamyshnikova<br>
<<a href="mailto:akamyshnikova@mirantis.com">akamyshnikova@mirantis.com</a>> wrote:<br>
> Hi guys!<br>
><br>
> I'm working on adoption of new engine facade from oslo.db for Neutron [1].<br>
> This work requires us to get rid of lazy init for engine facade. [2] I<br>
> propose change [3] that adds configure_db parameter which is False by<br>
> default, so if work with db will be required configure_db=True should be<br>
> passed manually.<br>
><br>
> NOTE: this will affect all external repos depending on Neutron!<br>
><br>
> I'm considering making this argument mandatory to force every project<br>
> depending on this function explicitly make a decision there.<br>
><br>
> I want to encourage reviewers to take a look at this change and l'm looking<br>
> forward all suggestions.<br>
><br>
> [1] - <a href="https://bugs.launchpad.net/neutron/+bug/1520719" rel="noreferrer" target="_blank">https://bugs.launchpad.net/neutron/+bug/1520719</a><br>
> [2] -<br>
> <a href="http://specs.openstack.org/openstack/oslo-specs/specs/kilo/make-enginefacade-a-facade.html" rel="noreferrer" target="_blank">http://specs.openstack.org/openstack/oslo-specs/specs/kilo/make-enginefacade-a-facade.html</a><br>
> [3] - <a href="https://review.openstack.org/#/c/312393/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/312393/</a><br>
><br>
> --<br>
> Regards,<br>
> Ann Kamyshnikova<br>
> Mirantis, Inc<br>
><br>
</div></div>> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Regards,<div>Ann Kamyshnikova</div><div>Mirantis, Inc</div></div></div>
</div>