connection_trace does not work with DB2 backend

Bug #1274523 reported by Matt Riedemann
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Invalid
Undecided
Jay Bryant
OpenStack Compute (nova)
Fix Released
Low
Eugeniya Kudryashova
oslo-incubator
Fix Released
Medium
Brant Knudson
oslo.db
Fix Released
Undecided
Unassigned

Bug Description

When setting connection_trace=True, the stack trace does not get printed for DB2 (ibm_db).

I have a patch that we've been using internally for this fix that I plan to upstream soon, and with that we can get output like this:

2013-09-11 13:07:51.985 28151 INFO sqlalchemy.engine.base.Engine [-] SELECT services.created_at AS services_created_at, services.updated_at AS services_updated_at, services.deleted_at AS services_deleted_at, services.deleted AS services_deleted, services.id AS services_id, services.host AS services_host, services."binary" AS services_binary, services.topic AS services_topic, services.report_count AS services_report_count, services.disabled AS services_disabled, services.disabled_reason AS services_disabled_reason
FROM services WHERE services.deleted = ? AND services.id = ? FETCH FIRST 1 ROWS ONLY
2013-09-11 13:07:51,985 INFO sqlalchemy.engine.base.Engine (0, 3)
2013-09-11 13:07:51.985 28151 INFO sqlalchemy.engine.base.Engine [-] (0, 3)
        File /usr/lib/python2.6/site-packages/nova/servicegroup/drivers/db.py:92 _report_state() service.service_ref, state_catalog)
        File /usr/lib/python2.6/site-packages/nova/conductor/api.py:270 service_update() return self._manager.service_update(context, service, values)
        File /usr/lib/python2.6/site-packages/nova/openstack/common/rpc/common.py:420 catch_client_exception() return func(*args, **kwargs)
        File /usr/lib/python2.6/site-packages/nova/conductor/manager.py:461 service_update() svc = self.db.service_update(context, service['id'], values)
        File /usr/lib/python2.6/site-packages/nova/db/sqlalchemy/api.py:505 service_update() with_compute_node=False, session=session)
        File /usr/lib/python2.6/site-packages/nova/db/sqlalchemy/api.py:388 _service_get() result = query.first()

Tags: db
Matt Riedemann (mriedem)
Changed in oslo:
assignee: nobody → Matt Riedemann (mriedem)
Matt Riedemann (mriedem)
Changed in oslo:
assignee: Matt Riedemann (mriedem) → nobody
Changed in nova:
assignee: nobody → Matt Riedemann (mriedem)
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

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

Changed in oslo:
assignee: nobody → Brant Knudson (blk-u)
status: New → In Progress
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
Jay Bryant (jsbryant)
Changed in cinder:
status: New → Confirmed
assignee: nobody → Jay Bryant (jsbryant)
Brant Knudson (blk-u)
Changed in keystone:
assignee: nobody → Brant Knudson (blk-u)
Feilong Wang (flwang)
Changed in glance:
status: New → Triaged
importance: Undecided → Low
assignee: nobody → Fei Long Wang (flwang)
Xu Han Peng (xuhanp)
Changed in neutron:
assignee: nobody → Xu Han Peng (xuhanp)
status: New → Confirmed
lvdongbing (dbcocle)
Changed in heat:
assignee: nobody → lvdongbing (dbcocle)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to oslo.db (master)

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

Changed in oslo:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on oslo.db (master)

Change abandoned by Brant Knudson (<email address hidden>) on branch: master
Review: https://review.openstack.org/92182
Reason: Taken care of by https://review.openstack.org/#/c/106406/

Revision history for this message
Brant Knudson (blk-u) wrote :

This was fixed in oslo.db by Michael Bayer with https://review.openstack.org/#/c/106406/

Changed in oslo:
status: In Progress → Fix Committed
no longer affects: keystone
Revision history for this message
Brant Knudson (blk-u) wrote :

Keystone isn't affected since all we have to do is pick up an oslo.db that includes https://review.openstack.org/#/c/106406/

Revision history for this message
Viktor Serhieiev (vsergeyev) wrote :

Is it still affects Glace?

Revision history for this message
Matt Riedemann (mriedem) wrote :

This should be fixed in nova when nova starts using oslo.db:

https://review.openstack.org/#/c/101901/

Changed in nova:
importance: Undecided → Low
status: Confirmed → In Progress
Changed in nova:
assignee: Matt Riedemann (mriedem) → Eugeniya Kudryashova (ekudryashova)
Thierry Carrez (ttx)
Changed in oslo-incubator:
status: Fix Committed → Fix Released
milestone: none → juno-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/101901
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=1b83b2f11054ddd3f72fe75c5cef496060afcb3a
Submitter: Jenkins
Branch: master

commit 1b83b2f11054ddd3f72fe75c5cef496060afcb3a
Author: Andrey Kurilin <email address hidden>
Date: Sat Jun 21 13:56:42 2014 +0300

    Move to oslo.db

    Replace common oslo code nova.openstack.common.db by usage
    of oslo.db library and remove common code.

    Replaced catching of raw sqlalchemy exceptions by catches
    of oslo.db exceptions(such as DBError, DBDuplicateEntry, etc).

    Co-Authored-By: Eugeniya Kudryashova <email address hidden>

    Closes-Bug: #1364986
    Closes-Bug: #1353131
    Closes-Bug: #1283987
    Closes-Bug: #1274523
    Change-Id: I0649539e071b2318ec85ed5d70259c949408e64b

Changed in nova:
status: In Progress → Fix Committed
Revision history for this message
Viktor Serhieiev (vsergeyev) wrote :

This was fixed in oslo.db by Michael Bayer with https://review.openstack.org/#/c/106406/

Changed in oslo.db:
status: New → Fix Committed
Changed in oslo.db:
milestone: none → 1.0.0
Thierry Carrez (ttx)
Changed in oslo.db:
status: Fix Committed → Fix Released
Zane Bitter (zaneb)
Changed in heat:
importance: Undecided → Low
status: New → Triaged
Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-rc1 → 2014.2
Revision history for this message
Sean McGinnis (sean-mcginnis) wrote : Bug Cleanup

Closing stale bug. If this is still an issue please reopen.

Changed in cinder:
status: Confirmed → Invalid
Thomas Herve (therve)
no longer affects: heat
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Closing stale bug. If this is still an issue please reopen.

no longer affects: neutron
Revision history for this message
Markus Zoeller (markus_z) (mzoeller) wrote :

FWIW, DB2 support got removed from Nova in Newton, see release notes of "14.0.0.0b1" http://docs.openstack.org/releasenotes/nova/index.html

Matt Riedemann (mriedem)
no longer affects: glance
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.