lp:~cerberus/nova/lp828207

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~hudson-openstack/nova/trunk at revision 1458

Brian Waldon (community): Approve on 2011-08-18

Ed Leafe (community): Approve on 2011-08-18

Diff: 28 lines (+9/-3)

1 file modified

nova/compute/api.py (+9/-3)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #828207: host API extension broken after recent changes

Low

Fix Released

Link a bug report

Related blueprints

1454. By Matt Dietz on 2011-08-18

resolve conflicts after upstream merge

1453. By Matt Dietz on 2011-08-18

Change the call name

1452. By Matt Dietz on 2011-08-18

Syntax error

1451. By Matt Dietz on 2011-08-18

Moved compute calls to their own handler

1450. By Brian Lamar on 2011-08-17

DB object was being casted to dict() in API code. This did not work as intended and logic has been updated to reflect a more accurate way of getting information out of DB objects.

1449. By dan wendlandt on 2011-08-16

I'm taking Thierry at his word that I should merge early and merge often :)

In Diablo-3 we introduced "vif-plugging" to the hypervisor "virt" layer, allowing flexibility in how vNICs are attached to the network switch. This allowed non-linux bridge switch technologies (e.g., Open vSwitch, 802.1qbh) to be used with nova.

This blueprint adds a similar capability to linux_net.py, allowing the L3/DHCP capabilities to be "plugged" into Quantum networks. Like in the virt layer, we created a vif driver that represents the behavior of Nova prior to the change (LinuxBridgeInterfaceDriver) and make it the default. We also add a new driver for Open Vswitch that can be enabled using a flag (LinuxOVSInterfaceDriver). The code is designed to support other drivers as well.

Most of the interesting code is at the bottom of linux_net.py, where the drivers are defined. I had to pull some common code related to setting IPs on devices out of ensure_bridge() so it could be used by either approach. The driver's plug() method is invoked by the VlanManager's _setup_network() method. Currently unplug() is unused, which seems to be inline with how the existing nova code works.

In many places in linux_net.py, I had to tweak functions to accept the name of the linux device to configure, rather than just assuming it was the 'bridge' field in the network object, since with OVS it could be any linux device. The code I am least sure about are the changes to bin/nova-dhcpbridge. I changed to this key off of the network ID, rather than the bridge name.

I've tested this with the linux bridge and with the OVS vif-plugging driver. I was able to confirm that L3 forwarding and DHCP were operating correctly.

1448. By Vish Ishaya on 2011-08-16

Fixes issue with exceptions getting eaten in image/s3.py if there is a failure during register. The variables referenced with locals() were actually out of scope.

1447. By Josh Kearney on 2011-08-16

Allow local_gb size to be 0. libvirt uses local_gb as a secondary drive, but XenServer uses it as the root partition's size. Now we support both.

Include some random PEP8 fixes.

1446. By Jason Kölker on 2011-08-16

Use netaddr's subnet features to calculate subnets.

1445. By Vladimir Popovski on 2011-08-16

* Added search instance by metadata.
* instance_get_all_by_filters should filter deleted