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