rewrite db.service_get_all() to really get all services
Bug #723171 reported by
Christian Berendt
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Wishlist
|
Josh Kleinpeter |
Bug Description
At the moment the method service_get_all can only return all disabled or all enabled services, but not _all_ services. This should be fixed.
def service_
if not session:
session = get_session()
result = session.
return result
Related branches
lp://staging/~jkleinpeter/nova/lp723171
- Josh Kearney (community): Approve
- Jesse Andrews (community): Approve
- Jay Pipes (community): Approve
-
Diff: 85 lines (+13/-10)5 files modifiedbin/nova-manage (+1/-1)
nova/api/ec2/admin.py (+1/-1)
nova/api/ec2/cloud.py (+2/-2)
nova/db/api.py (+1/-1)
nova/db/sqlalchemy/api.py (+8/-5)
Changed in nova: | |
assignee: | nobody → Josh Kleinpeter (jkleinpeter) |
status: | Confirmed → In Progress |
Changed in nova: | |
status: | In Progress → Fix Committed |
Changed in nova: | |
milestone: | none → 2011.2 |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
This would change the semantics of service_get_all, for example as used by _describe_ availability_ zones in nova/api/ ec2/cloud. py... Why do you think it needs fixing ? What is broken with current situation ?