require SQLAlchemy >=0.9.9

Bug #1488117 reported by Anna Sortland
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Anna Sortland

Bug Description

The recent commit
https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=f1f332ad6c7fe077131cd47cbeb57a9b1efd4731
requires execution_options parameter that was added in SQLAlchemy 0.9.9. However, ceilometer still requires >=0.9.7:
https://github.com/openstack/ceilometer/blob/master/requirements.txt#L42

This causes this exception on GET /v2/events/ call:
File "/usr/lib/python2.7/site-packages/ceilometer/event/storage/impl_sqlalchemy.py", line 220, in get_events
execution_options={'isolation_level': self.isolation_level})
File "/usr/lib64/python2.7/site-packages/sqlalchemy/orm/session.py", line 851, in connection
bind = self.get_bind(mapper, clause=clause, **kw)
TypeError: get_bind() got an unexpected keyword argument 'execution_options'

Ceilometer should require SQLAlchemy >=0.9.9.

Anna Sortland (annasort)
Changed in ceilometer:
assignee: nobody → Anna Sortland (annasort)
gordon chung (chungg)
Changed in ceilometer:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ceilometer (master)

Change abandoned by Anna Sortland (<email address hidden>) on branch: master
Review: https://review.openstack.org/216304
Reason: Will update global-requirements.txt instead based on Chris' comment.

Revision history for this message
Matthew Edmonds (edmondsw) wrote :

global-requirements change is here: https://review.openstack.org/#/c/216360/

Revision history for this message
Lianhao Lu (lianhao-lu) wrote :
Changed in ceilometer:
status: In Progress → Fix Committed
gordon chung (chungg)
Changed in ceilometer:
milestone: none → liberty-rc1
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: liberty-rc1 → 5.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.