diff options
| author | Devananda van der Veen <devananda.vdv@gmail.com> | 2012-11-08 11:47:25 -0800 |
|---|---|---|
| committer | Devananda van der Veen <devananda.vdv@gmail.com> | 2012-11-09 09:42:33 -0800 |
| commit | f8e3ee704eec252cfb774319252831637632b0a8 (patch) | |
| tree | 5c944013f249c880e54583ec18a089c8ffe1df99 /nova/db | |
| parent | 7cf93105ba670a298e3cabcd1f5c6d4211cab145 (diff) | |
improve session handling of dnsdomain_list
improve session handling of dnsdomain_list
and add unit tests for this method
Change-Id: I45e0e9df19ddfbafbcfde425255587d5ba530092
Diffstat (limited to 'nova/db')
| -rw-r--r-- | nova/db/sqlalchemy/api.py | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/nova/db/sqlalchemy/api.py b/nova/db/sqlalchemy/api.py index 63ef26a0e..74e6d7b3b 100644 --- a/nova/db/sqlalchemy/api.py +++ b/nova/db/sqlalchemy/api.py @@ -962,15 +962,8 @@ def dnsdomain_unregister(context, fqdomain): @require_context def dnsdomain_list(context): - session = get_session() - records = model_query(context, models.DNSDomain, - session=session, read_deleted="no").\ - all() - domains = [] - for record in records: - domains.append(record.domain) - - return domains + query = model_query(context, models.DNSDomain, read_deleted="no") + return [row.domain for row in query.all()] ################### |