Merge lp://qastaging/~james-page/charms/quantal/nova-cloud-controller/quantum-support into lp://qastaging/~charmers/charms/precise/nova-cloud-controller/trunk

Proposed by James Page
Status: Merged
Merge reported by: Adam Gandelman
Merged at revision: not available
Proposed branch: lp://qastaging/~james-page/charms/quantal/nova-cloud-controller/quantum-support
Merge into: lp://qastaging/~charmers/charms/precise/nova-cloud-controller/trunk
Diff against target: 793 lines (+564/-16)
10 files modified
config.yaml (+51/-1)
files/create_ext_net.py (+122/-0)
files/create_tenant_net.py (+143/-0)
hooks/lib/nova/folsom (+31/-7)
hooks/lib/nova/nova-common (+18/-1)
hooks/lib/openstack-common (+22/-0)
hooks/nova-cloud-controller-common (+40/-0)
hooks/nova-cloud-controller-relations (+134/-6)
metadata.yaml (+2/-0)
revision (+1/-1)
To merge this branch: bzr merge lp://qastaging/~james-page/charms/quantal/nova-cloud-controller/quantum-support
Reviewer Review Type Date Requested Status
Adam Gandelman Pending
charmers Pending
Review via email: mp+137595@code.qastaging.launchpad.net

Commit message

Add support for Quantum API Server

Description of the change

This merge adds support for Quantum to the nova-cloud-controller charm.

This is provides as an extra option for network-manager; The Quantum API server is setup and configured on the cloud controller node alongside the Nova API servers.

At the moment only the OpenvSwitch plugin for quantum is supported; however the integration is written in such a way that its possible to add support for other options later on.

These changes rely on lp:~james-page/charms/precise/mysql/shared-db-multi which is currently proposed for merge.

To post a comment you must log in.
52. By James Page

Notify compute and quantum relations when keystone identity changes

53. By James Page

Minor update to ensure that quantum plugin configuration is passed
to nova-compute and quantum-gateway services.

54. By James Page

Ensure IP addresses are used instead of hostnames in the appropriate places

55. By James Page

Add missing call to get_ip

56. By James Page

Minor tweak to flow control in get_ip

57. By James Page

Bind quantum to 0.0.0.0

Revision history for this message
Adam Gandelman (gandelman-a) wrote :

This looks good for the most part. There are some issues I found elsewhere outlined in the quantum-gateway bug. I'll merge this when the changes are ready for merging into nova-compute.

58. By James Page

Make get_ip backwards compatible, validate return prior to use

Revision history for this message
Adam Gandelman (gandelman-a) wrote :

James-

Getting install hook errors with the newest change.

[[ -n $local_ip ]] || juju-log "Unable to resolve local IP address" \
&& exit 1

needs to be:

[[ -n $local_ip ]] || (juju-log "Unable to resolve local IP address" \
&& exit 1)

59. By James Page

Fixed problem with get_ip validation

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: