lp:~rackspace-titan/nova/libvirt-no-globals

Branch merges

Propose for merging

No branches dependent on this one.

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

Alex Meade (community): Approve on 2011-05-31

Vish Ishaya (community): Approve on 2011-05-25

Diff: 464 lines (+82/-61)

6 files modified

nova/tests/test_libvirt.py (+35/-29)
nova/utils.py (+9/-0)
nova/virt/libvirt/connection.py (+18/-25)
nova/virt/libvirt/firewall.py (+10/-7)
run_tests.py (+9/-0)
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

1111. By Brian Lamar on 2011-05-25

Added the ability to "skip" tests, as provided by unittest2 (which back-ports python2.7 functionality and can be removed later once python2.7+ is required). The ability to skip tests was needed because the current method of skipping displayed "OK" when skipping tests. This will make things much more evident to the user when tests are being skipped. Along with this change is `utils.optional_import` which allows for removal of all "globals" being used in libvirt code.

1110. By Brian Lamar on 2011-05-25

Created new libvirt directory, moved libvirt_conn.py to libvirt/connection.py, moved libvirt templates, broke out firewall and network utilities.

1109. By Ed Leafe on 2011-05-25

The code for getting an opaque reference to an instance assumed that there was a reference to an instance obj available when raising an exception. I changed this from raising an InstanceNotFound exception to a NotFound, as this is more appropriate for the failure, and doesn't require an instance ID.

1108. By Johannes Erdfelt on 2011-05-24

eventlet.spawn_n() expects the function and arguments, but it expects the arguments unpacked since it uses *args.

1107. By Johannes Erdfelt on 2011-05-24

Using the root-password subcommand of the nova client results in the password being changed for the instance specified, but to a different unknown password. The patch changes nova to use the password specified in the API call.

1106. By Chris Behrens on 2011-05-24

Pretty simple. We call openssl to encrypt the admin password, but the recent changes around this code forgot to strip the newline off the read from stdout.

1105. By termie on 2011-05-24

Fixes a bug related to incorrect reparsing of flags and prevents many extra reparses.

1104. By Dave Walker on 2011-05-24

When adding a keypair with ec2 API that already exists, give a friendly error and no traceback in nova-api

1103. By Anne Gentle on 2011-05-24

Fixes some minor doc issues - misspelled flags in zones doc and also adds zones doc to an index for easier findability

1102. By Renuka Apte on 2011-05-23

Fixes euca-attach-volume for iscsi using Xenserver

Minor changes required to xenapi functions to get correct format for volume-id, iscsi-host, etc.