LazyPluggable doesn't support option groups
Bug #1093043 reported by
aeva black
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Undecided
|
Davanum Srinivas (DIMS) | ||
oslo-incubator |
Won't Fix
|
Undecided
|
Davanum Srinivas (DIMS) |
Bug Description
While converting the baremetal options to use an OptGroup, I tried to convert baremetal_
See comments on this review:
https:/
Changed in nova: | |
status: | Confirmed → In Progress |
Changed in nova: | |
status: | In Progress → Confirmed |
assignee: | Davanum Srinivas (DIMS) (dims-v) → nobody |
Changed in nova: | |
milestone: | none → grizzly-3 |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | grizzly-3 → 2013.1 |
To post a comment you must log in.
Looks like the key is CONF[self.__pivot] needs to work with "baremetal. db_backend" .
if not self.__backend:
backend_ name = CONF[self.__pivot]
raise exception. NovaException( msg)
if backend_name not in self.__backends:
msg = _('Invalid backend: %s') % backend_name
Basically we need to look up the group first and then lookup the key in the group. Easiest way is to enhance cfg._get_opt_info to deal with "baremetal. db_backend"