session flushing error

[gordon chung]

seems like we have overzealous flushing...

2013-05-24 16:27:39 ERROR [ceilometer.openstack.common.db.sqlalchemy.session] DB exception wrapped.
Traceback (most recent call last):
  File "/opt/stack/ceilometer/ceilometer/openstack/common/db/sqlalchemy/session.py", line 417, in _wrap
    return f(*args, **kwargs)
  File "/opt/stack/ceilometer/ceilometer/openstack/common/db/sqlalchemy/session.py", line 589, in flush
    return super(Session, self).flush(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1712, in flush
    raise sa_exc.InvalidRequestError("Session is already flushing")
InvalidRequestError: Session is already flushing
2013-05-24 16:27:39 ERROR [ceilometer.collector.service] Failed to record metering data: Session is already flushing
2013-05-24 16:27:39 ERROR [ceilometer.collector.service] Session is already flushing
Traceback (most recent call last):
  File "/opt/stack/ceilometer/ceilometer/collector/service.py", line 152, in record_metering_data
    self.storage_conn.record_metering_data(meter)
  File "/opt/stack/ceilometer/ceilometer/storage/impl_sqlalchemy.py", line 189, in record_metering_data
    self.session.flush()
  File "/opt/stack/ceilometer/ceilometer/openstack/common/db/sqlalchemy/session.py", line 438, in _wrap
    raise exception.DBError(e)
DBError: Session is already flushing

[gordon chung (chungg)]

seems like we're sharing a single session instance... fixing so we let sqlalchemy sessionmaker in oslo common handle that.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/31291

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/31291
Committed: http://github.com/openstack/ceilometer/commit/c85147c45ea174ce883d71f58c9ddf993c13ac66
Submitter: Jenkins
Branch: master

commit c85147c45ea174ce883d71f58c9ddf993c13ac66
Author: Gordon Chung <email address hidden>
Date: Fri May 31 15:45:53 2013 -0400

    session flushing error

    stop sharing a single session for everything. use common.db.sqlalchemy code
    to handle session management

    re-enable use of CEILOMETER_TEST_SQL_URL

    Change-Id: I9df407704a8db9e37d05b2a80047e5539f2d15d8
    Fixes: bug 1183996