Thanks Gary,<div><br></div><div>Another higher level question here is how do we make sure that OSLO changes like this are better communicated to impacted core projects. </div><div><br></div><div>Adding Mark (and the openstack-dev list) to let him comment more broadly on this.</div>
<div><br></div><div>Dan</div><div><br><br><div class="gmail_quote">On Sun, Mar 17, 2013 at 8:37 AM, Salvatore Orlando <span dir="ltr"><<a href="mailto:sorlando@nicira.com" target="_blank">sorlando@nicira.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p>Thanks Gary,</p>
<p>I think your analysis and your conclusions make sense.<br>
I noticed the DB API change when we were trying to debug this problem, but the change seemed a bit too large for RC1.<br>
It makes sense now doing it in Havana, and perhabs backporting to stable/grizzly.</p>
<p>Thanks again for tackling this issue.<br>
Salvatore</p>
<div class="gmail_quote">Il giorno 17/mar/2013 16:26, "Gary Kotton" <<a href="mailto:gkotton@redhat.com" target="_blank">gkotton@redhat.com</a>> ha scritto:<div><div class="h5"><br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
Over the last few days I have worked on a number of problems when is sql_dbpool_enable is set a True. After a ton of debugging today I referred to the Nova implementation (where it was originally taken from) and discovered the following:<br>
<a href="https://github.com/openstack/oslo-incubator/commit/02c12aade7a0c28c66cb45b54786c90c0ae8fb09" target="_blank">https://github.com/openstack/<u></u>oslo-incubator/commit/<u></u>02c12aade7a0c28c66cb45b54786c9<u></u>0c0ae8fb09</a><br>
The problems that I was looking at were as follows:<br>
1. When starting with a clean database and the various agents are already running, multiple agent entries are created in the database<br>
2. Once the entries are sorted an update of a agent entry causes an exception [1]<br>
I think that either we should drop the Grizzly support or mark it as experimental. Any ideas or comments?<br>
I the coming days I will try and update our H version to make use of the common database interface.<br>
Thanks<br>
Gary<br>
<br>
1.<br>
2013-03-17 11:24:27 ERROR [quantum.openstack.common.rpc.<u></u>amqp] Exception during message handling<br>
Traceback (most recent call last):<br>
File "/opt/stack/quantum/quantum/<u></u>openstack/common/rpc/amqp.py", line 430, in _process_data<br>
rval = self.proxy.dispatch(ctxt, version, method, **args)<br>
File "/opt/stack/quantum/quantum/<u></u>common/rpc.py", line 43, in dispatch<br>
quantum_ctxt, version, method, **kwargs)<br>
File "/opt/stack/quantum/quantum/<u></u>openstack/common/rpc/<u></u>dispatcher.py", line 133, in dispatch<br>
return getattr(proxyobj, method)(ctxt, **kwargs)<br>
File "/opt/stack/quantum/quantum/<u></u>db/agents_db.py", line 167, in report_state<br>
plugin.create_or_update_agent(<u></u>context, agent_state)<br>
File "/opt/stack/quantum/quantum/<u></u>db/agents_db.py", line 156, in create_or_update_agent<br>
context.session.add(agent_db)<br>
File "/usr/local/lib/python2.7/<u></u>dist-packages/sqlalchemy/orm/<u></u>session.py", line 452, in __exit__<br>
raise<br>
TypeError: exceptions must be old-style classes or derived from BaseException, not NoneType<br>
<br>
<br>
-- <br>
Mailing list: <a href="https://launchpad.net/~quantum-core" target="_blank">https://launchpad.net/~<u></u>quantum-core</a><br>
Post to : <a href="mailto:quantum-core@lists.launchpad.net" target="_blank">quantum-core@lists.launchpad.<u></u>net</a><br>
Unsubscribe : <a href="https://launchpad.net/~quantum-core" target="_blank">https://launchpad.net/~<u></u>quantum-core</a><br>
More help : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/<u></u>ListHelp</a><br>
</blockquote></div></div></div>
<br>--<br>
Mailing list: <a href="https://launchpad.net/~quantum-core" target="_blank">https://launchpad.net/~quantum-core</a><br>
Post to : <a href="mailto:quantum-core@lists.launchpad.net">quantum-core@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/~quantum-core" target="_blank">https://launchpad.net/~quantum-core</a><br>
More help : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>Dan Wendlandt <div>Nicira, Inc: <a href="http://www.nicira.com" target="_blank">www.nicira.com</a><br><div>twitter: danwendlandt<br>
~~~~~~~~~~~~~~~~~~~~~~~~~~~<br></div></div>
</div>