Merge lp://qastaging/~springfield-team/charms/precise/quantum-gateway/trunk into lp://qastaging/charms/quantum-gateway

Proposed by ChingWei Chang
Status: Superseded
Proposed branch: lp://qastaging/~springfield-team/charms/precise/quantum-gateway/trunk
Merge into: lp://qastaging/charms/quantum-gateway
Diff against target: 306 lines (+134/-5)
7 files modified
config.yaml (+7/-0)
hooks/quantum_contexts.py (+8/-1)
hooks/quantum_hooks.py (+14/-0)
hooks/quantum_utils.py (+39/-2)
templates/havana/cisco_plugins.ini (+44/-0)
templates/havana/dhcp_agent.ini (+12/-1)
templates/havana/l3_agent.ini (+10/-1)
To merge this branch: bzr merge lp://qastaging/~springfield-team/charms/precise/quantum-gateway/trunk
Reviewer Review Type Date Requested Status
Jorge Niedbalski (community) Approve
James Page Needs Information
Review via email: mp+218169@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2014-04-29.

This proposal has been superseded by a proposal from 2014-07-22.

Description of the change

config.yaml
plugin: new value 'n1kv' to facilitate n1kv specific logic in the charm code
l3-agent: use Linux router for L3 routing service on gateway for N1KV deployment networking.
hooks/quantum_contexts.py
- N1KV python plugin code location and corresponding look-up strings

hooks/quantum_hooks.py:
- N1KV related logic for start/stop neutron-l3-agent
- Error handling like the rest of L2 switch does

hooks/quantum_utils.py
- N1KV context in par with NVP and OVS
- software packages for Quantum gateway with N1KV plugin

templates/havana/cisco_plugins.ini
- Cisco N1KV plugin related parameters. We use generic one across nova-compute and quantum-gateway

templates/havana/dhcp_agent.ini
- Cisco N1KV plugin dnsmasq specific parameters

templates/havana/l3_agent.ini
- Cisco N1KV router specific parameters

To post a comment you must log in.
Revision history for this message
ChingWei Chang (cwchang) wrote : Posted in a previous version of this proposal

Correction on comment #1

The source branch is at :
lp:~springfield-team/charms/precise/quantum-gateway/trunk

Revision history for this message
James Page (james-page) wrote : Posted in a previous version of this proposal

Re-targetted to the correct branch; needs some re-basing and updates based on the features/changes that landed a few weeks ago to support icehouse.

Generally looks ok - please rebase.

One fix required:

+ l3-agent:
26 + type: string
27 + default: enable
28 + description: |
29 + Optional configuration to support use of linux router
30 + Note that this is used only for Cisco n1kv plugin.

This is a boolean - please make it one! you can then use "config('l3-agent') is True". A better name would be 'enable-l3-agent'.

review: Needs Fixing
Revision history for this message
ChingWei Chang (cwchang) wrote : Posted in a previous version of this proposal

James,
Our development model is like the following:

charm trunk/... quantuam gateway
   +
   |-> springfield-team --> quantum gateway

Springfield-team branch is our development branch.

The question is:
How do I rebase the development branch springfield-team to later charm store revion ? I don't see any command or Launchpad knob to provide that function

ChingWei

Revision history for this message
ChingWei Chang (cwchang) wrote :

James,
We have made the l3-agent naming change also merged with the latest Charm Store tips.
Please re-review it, Thanks !
ChingWei

Revision history for this message
James Page (james-page) wrote :

Apologies for not asking this question earlier, but which openstack versions does this plugin support? if its only havana and above, we should not add updates for QUANTUM_* as its not supported; but leaving this out if someone tries to use the features with < havana, the charm will error out rather than doing something undefined and untested.

review: Needs Information
52. By Shiv Prasad Rao

Removing < havana changes for N1kV

53. By Shiv Prasad Rao

Upstream revision 50

Revision history for this message
Shiv Prasad Rao (shivrao) wrote :

1) Removed < havana changes
2) Charmhelper changes have been synced with lp:charm-helpers. So charmhelper changes with this merge can be ignored.

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

This change LGTM +1.

review: Approve
54. By Dulanjalie Ganegedara

adding charmhelper support to use --keyserver option for public PPA

Unmerged revisions

54. By Dulanjalie Ganegedara

adding charmhelper support to use --keyserver option for public PPA

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