LegacyEngineFacade migration issue (Trove)

Bug #1882877 reported by Hirotaka Wakabayashi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo.db
Incomplete
Undecided
Hirotaka Wakabayashi

Bug Description

Hello folks,

I'm trying to use oslo_db.sqlalchemy.enginefacade module in Trove project. Trove currently uses oslo_db.sqlalchemy.session.EngineFacade, which is the alias to LegacyEngineFacade.

I try to replace LegacyEngineFacade with enginefacade but that causes DuplicateOptError because Trove's library initially calls CONF.register_opts function with 'database' group and enginefacade also internally calls CONF.register_opts with 'database' group.

One of my suggestion to solve this problem is to register cfg options to the "database" group only if "database" group is not be registered. I will submit a trivial patch soon. Your advice is greatly appreciated.

Reference: My proposal ticket to use enginefacade instead of EngineFacade
https://storyboard.openstack.org/#!/story/2007498#comment-155528

Thanks in advance,
Hirotaka Wakabayashi

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.db (master)

Fix proposed to branch: master
Review: https://review.opendev.org/734744

Changed in oslo.db:
assignee: nobody → Hirotaka Wakabayashi (hiwkby)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on oslo.db (master)

Change abandoned by "Hervé Beraud <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/oslo.db/+/734744
Reason: There has been no activity on this patch for some time. Apparently this is not really a problem with oslo.db, feel free to reopen it if needed.

Thanks for your understanding

Changed in oslo.db:
status: In Progress → Incomplete
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.