glance store initialization should be improved

Bug #1383722 reported by Stuart McLaren
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
glance_store
Fix Released
Medium
Unassigned

Bug Description

Currently the glance store initialization is explicitly called in the api start script:

 cat glance/cmd/api.py
 .
 .
 .

        glance_store.register_opts(config.CONF)
        glance_store.create_stores(config.CONF)
        glance_store.verify_default_store()

Other parameter's are picked up for free if we perform a configuration reload on sighup (eg sqlalchemy settings).
(An exception is logging where a log.setup() needs to be called).

It would be great, in order to fit cleanly with configuration reload, to have the glance_store initialization be called automatically, triggered by the eventlet.wsgi.server() call in glance.common.wsgi.

Changed in glance:
assignee: nobody → Flavio Percoco (flaper87)
importance: Undecided → Medium
Revision history for this message
Stuart McLaren (stuart-mclaren) wrote :

FWIW somewhat equivalent bug for osprofiler: https://bugs.launchpad.net/osprofiler/+bug/1407649

Revision history for this message
Ian Cordasco (icordasc) wrote :

Flavio are you working on this?

Changed in glance:
status: New → Confirmed
Revision history for this message
Flavio Percoco (flaper87) wrote :

Cindy (cpallares) is working on this as part of the glance_store refactor.

Changed in glance:
assignee: Flavio Percoco (flaper87) → Cindy Pallares (cindy-pallaresq)
affects: glance → glance-store
Changed in glance-store:
assignee: Cindy Pallares (cpallares) → nobody
Revision history for this message
Cyril Roelandt (cyril-roelandt) wrote :

I believe this was fixed with ea2252b4fd4eb5b2888b5dc57dd9b9cfe4388bdf

Changed in glance-store:
status: Confirmed → 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.