DM: configure virtual-gateway-address under IRB interface

Bug #1465070 reported by Nischal Sheth
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
High
Suresh Balineni
Trunk
Fix Committed
High
Suresh Balineni

Bug Description

The IP addresses under IRB interface should be configured as follows:

Need to allocate a unique IP address from the subnet for each MX's IRB
address i.e. don't simply configure the GW IP as the IRB IP address. The
IRB IPs thus allocated need to be persisted so that we can re-use the
same addresses when DM is restarted. They should be released if the
MX no longer needs the VN. Note that each MX needs to use a unique
address as the IRB IP address.

Further, the GW IP of the subnet should be configured on the MX as the
virtual-gateway-address under the IRB IP address. Note that all MXs must
use the same virtual-gateway-address for a given VN.

Nischal Sheth (nsheth)
no longer affects: juniperopenstack/r2.0
Revision history for this message
dhoutz (dhoutz) wrote :

Current code to configure IRB also seems to use a hard coded mask of /24:

etree.SubElement(addr, "name").text = gateway + "/24"

Should fix this as well when implementing this.

tags: added: quench
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/12740
Submitter: Suresh Balineni (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/12740
Committed: http://github.org/Juniper/contrail-controller/commit/c9062ca59a9f1f12aa9a221d39b3b9009301b713
Submitter: Zuul
Branch: master

commit c9062ca59a9f1f12aa9a221d39b3b9009301b713
Author: sbalineni <email address hidden>
Date: Wed Jul 29 17:53:23 2015 -0700

DM: Schema changes for allocating irb interface address

Change-Id: I15bce9474bdd00e486945671fb515983d08128d3
Partial-Bug:1465070

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/12968
Submitter: Suresh Balineni (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/12968
Committed: http://github.org/Juniper/contrail-controller/commit/ccfce2be4466e944992dfbf085c17055de614944
Submitter: Zuul
Branch: R2.20

commit ccfce2be4466e944992dfbf085c17055de614944
Author: sbalineni <email address hidden>
Date: Mon Aug 10 15:43:04 2015 -0700

DM: Allocate and persist irb address from each subnet of a VN for each PR

- Allocate one ip from each subnet of a VN for each PR.
- Persist the ip and the association with VN, Subnet, PR in cassandra DB
- On DM restart, read the DB state and maintain allocated ip map in local memory
- On update/restart, find differences between allocated
ip map vs VNC DB state of VN, and take necessary actions
- GW IP of the subnet is configured on the MX as the
virtual-gateway-address under the IRB IP address

Change-Id: Ifcfdf96069a353cf2be1e03a83922796d4dd3471
Closes-Bug: #1465070

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/13299
Submitter: Suresh Balineni (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/13299
Committed: http://github.org/Juniper/contrail-controller/commit/6bc451579d872307f5beeaeb00262382968eda04
Submitter: Zuul
Branch: master

commit 6bc451579d872307f5beeaeb00262382968eda04
Author: sbalineni <email address hidden>
Date: Tue Aug 25 11:22:45 2015 -0700

DM: Allocate and persist irb address from each subnet of a VN for each PR

- Allocate one ip from each subnet of a VN for each PR.
- Persist the ip and the association with VN, Subnet, PR in cassandra DB
- On DM restart, read the DB state and maintain allocated ip map in local memory
- On update/restart, find differences between allocated
ip map vs VNC DB state of VN, and take necessary actions
- GW IP of the subnet is configured on the MX as the
virtual-gateway-address under the IRB IP address

Change-Id: Id0bcb4abeae9fb5eadf93ef9091be0b4da50655f
Closes-Bug: #1465070

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/13393
Submitter: Suresh Balineni (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/13393
Committed: http://github.org/Juniper/contrail-controller/commit/01e4b2c6b6204361208332af0ed8622bb6da9691
Submitter: Zuul
Branch: master

commit 01e4b2c6b6204361208332af0ed8622bb6da9691
Author: sbalineni <email address hidden>
Date: Thu Aug 27 12:35:46 2015 -0700

DM: Fix for double-commit merge error.

While porting this fix from R2.20 to master, this problem
got introduced. Fixed.

Change-Id: I99d8a87dc4557b2f0fea48e4a1f20fcd816cf7ec
Closes-Bug: #1465070

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.22-dev

Review in progress for https://review.opencontrail.org/13927
Submitter: Vinay Vithal Mahuli (<email address hidden>)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.