lp:~blamar/nova/libvirt-cleanup-branch

Branch merges

Propose for merging

No branches dependent on this one.

On hold for merging into lp:~hudson-openstack/nova/trunk

Monty Taylor (community): Needs Fixing on 2011-07-19

Devin Carlen (community): Approve on 2011-06-30

Ed Leafe (community): Approve on 2011-06-29

William Wolf (community): Approve on 2011-06-29

Alex Meade (community): Approve on 2011-06-28

Diff: 433 lines (+123/-139)

6 files modified

nova/tests/test_libvirt.py (+8/-52)
nova/utils.py (+15/-0)
nova/virt/libvirt/__init__.py (+85/-0)
nova/virt/libvirt/connection.py (+13/-82)
nova/virt/libvirt/firewall.py (+1/-5)
tools/pip-requires (+1/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

1221. By Brian Lamar on 2011-07-08

Merged trunk

1220. By Brian Lamar on 2011-07-01

Merged trunk.

1219. By Brian Lamar on 2011-06-29

review feedback from bcwaldon and ed leafe

1218. By Brian Lamar on 2011-06-28

Two small cleanups in libvirt:

1) Moved flag defines to __init__.py
2) Replaced lazy-loading of imports with utils.optional_import (new)

One unrelated change is the addition of unittest2 to pip-requires. I'm
using the unittest.skipIf method in test_libvirt.py and unittest2 has
backported all Python 2.7 unittest improvements.

1217. By Vish Ishaya on 2011-06-27

This speeds up multiple runs of tests to start up much faster because it only runs db migrations if the test db doesn't exist. It also adds the -r/--recreate-db option to run_tests.sh to delete the tests db so it will be recreated.

1216. By Vish Ishaya on 2011-06-27

Proposing this because it is a critical fix before milestone. Suggestions on testing it are welcome.

1215. By Yuriy Taraday on 2011-06-25

Fix for bug #788265.
Remove created_at, updated_at and deleted_at from instance_type dict returned by methods in sqlalchemy API.

1214. By Todd Willey on 2011-06-25

Add api methods to delete provider firewall rules.

1213. By Salvatore Orlando on 2011-06-25

This small change restores single quotes and double quotes as they were before in the filter expression for retrieving the PIF (physical interface) xenapi should use for creating VLAN interfaces.

I'm not sure why they were altered, possibly because of coding style conventions, but unfortunately xenapi does not accept single quotes for separating filter expression tokens.

1212. By Ewan Mellor on 2011-06-25

Remove the unnecessary insertion of whitespace. This happens to be enough
to make this patch apply on recent versions of XenServer / Xen Cloud Platform.