Clean up warnings about enginefacade

Bug #1554226 reported by Jay Pipes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Danil Akhmetov

Bug Description

You see a bunch of the following in Nova's test runs:

Captured stderr:
~~~~~~~~~~~~~~~~
    /home/jaypipes/repos/nova/.tox/py27/local/lib/python2.7/site-packages/oslo_db/sqlalchemy/enginefacade.py:254: OsloDBDeprecationWarning: EngineFacade is deprecated; please use oslo_db.sqlalchemy.enginefacade
      self._legacy_facade = LegacyEngineFacade(None, _factory=self)

We should use oslo_db.sqlalchemy.enginefacade now in all cases.

Mohan (mmuppidi)
Changed in nova:
assignee: nobody → Mohan (mmuppidi)
Mohan (mmuppidi)
Changed in nova:
status: Confirmed → In Progress
Mohan (mmuppidi)
Changed in nova:
assignee: Mohan (mmuppidi) → nobody
Revision history for this message
Jeff Augustine (ja224e) wrote :

Changing status back to confirmed. Will see what I can do!

Changed in nova:
assignee: nobody → Jeffrey Augustine (ja224e)
status: In Progress → Confirmed
Changed in nova:
status: Confirmed → In Progress
Revision history for this message
Sivasathurappan Radhakrishnan (siva-radhakrishnan) wrote :

Jeffrey Augustine Can you please let me know if you are still working in this bug ?

Revision history for this message
Jeff Augustine (ja224e) wrote :

Sorry for the delay! Just returning from the summit and should now have time. If no update by the end of week I will release it to you.

Revision history for this message
Sivasathurappan Radhakrishnan (siva-radhakrishnan) wrote :

Thanks Jeffrey!

Jeff Augustine (ja224e)
Changed in nova:
assignee: Jeffrey Augustine (ja224e) → nobody
Revision history for this message
Sivasathurappan Radhakrishnan (siva-radhakrishnan) wrote :

changing it to confirmed to avoid any confusion on the status of the bug

Changed in nova:
assignee: nobody → Sivasathurappan Radhakrishnan (siva-radhakrishnan)
Revision history for this message
Sivasathurappan Radhakrishnan (siva-radhakrishnan) wrote :

Please ignore my previous comment.

Revision history for this message
Sivasathurappan Radhakrishnan (siva-radhakrishnan) wrote :

I don't see in happening in py27 anymore but I could see single occurrence of it in tox -e py34.

Changed in nova:
assignee: Sivasathurappan Radhakrishnan (siva-radhakrishnan) → nobody
Robin Naundorf (senk)
Changed in nova:
status: In Progress → Confirmed
Changed in nova:
assignee: nobody → Tanvir Talukder (tanvirt16)
Changed in nova:
assignee: Tanvir Talukder (tanvirt16) → nobody
Danil Akhmetov (dinobot)
Changed in nova:
assignee: nobody → Danil Akhmetov (dinobot)
Revision history for this message
Danil Akhmetov (dinobot) wrote :

I can't reproduce the problem neither with py27 nor py34: it seems like oslo_db.sqlalchemy.enginefacade is being used everywhere already.

Changed in nova:
status: Confirmed → Fix Committed
Revision history for this message
Markus Zoeller (markus_z) (mzoeller) wrote :

As we use the "direct-release" model in Nova we don't use the
"Fix Comitted" status for merged bug fixes anymore. I'm setting
this manually to "Fix Released" to be consistent.

[1] "[openstack-dev] [release][all] bugs will now close automatically
    when patches merge"; Doug Hellmann; 2015-12-07;
    http://lists.openstack.org/pipermail/openstack-dev/2015-December/081612.html

Changed in nova:
status: Fix Committed → Fix Released
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.