rpc-api-review: should be able to use non-global conf
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ceilometer |
Fix Released
|
High
|
Unassigned | ||
| oslo-incubator |
Fix Released
|
High
|
Unassigned | ||
Bug Description
Issue raised by Julien here:
http://
Basically, we should be able to create a new ConfigOpts() object and have the rpc code use that instead of global cfg.CONF
An rpc.get_impl() call may be sufficient:
+def get_impl(conf):
+ conf.register_
+ try:
+ return = importutils.
+ except ImportError:
+ # For backwards compatibility with older nova config.
+ impl = conf.rpc_
+ 'nova.openstack
+ return importutils.
Prototype patch for ceilometer here: https:/
| description: | updated |
| Doug Hellmann (doug-hellmann) wrote | #1 |
| Changed in ceilometer: | |
| status: | New → Confirmed |
| Changed in ceilometer: | |
| importance: | Undecided → High |
| Mark McLoughlin (markmc) wrote | #2 |
| Changed in oslo: | |
| status: | Confirmed → Fix Released |
| gordon chung (chungg) wrote | #3 |
| Changed in ceilometer: | |
| status: | Confirmed → Fix Committed |
| Changed in ceilometer: | |
| milestone: | none → juno-2 |
| status: | Fix Committed → Fix Released |
| Changed in ceilometer: | |
| milestone: | juno-2 → 2014.2 |
In practice the logging package also caused an issue. We solved it in ceilometer's nova notifier plugin with some heavy rewriting and module shadowing. See https:/