Agent should advertise prefix for instance-ip based on alloc-unit

Bug #1595671 reported by Nischal Sheth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Won't Fix
High
RAVI KIRAN
Trunk
Fix Committed
High
RAVI KIRAN

Bug Description

Support for allocation unit was added via bug 1589588. An alloc-unit
field has been added to IpamSubnetType. The default value is 1.

Agent should advertise an IP prefix based on the alloc-unit value.
For example if an instance-ip has address 10.1.1.17 and alloc-unit
is 16, a route needs to be advertised for 10.1.1.16/28.

This should be supported for both inet and inet6 addresses.
It should also work in conjunction with the flat-subnet feature (see
bug 1592970).

Nischal Sheth (nsheth)
description: updated
description: updated
Nischal Sheth (nsheth)
tags: added: quench2
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/21515
Submitter: RAVI KIRAN (<email address hidden>)

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

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

commit eda15808d531850f2d5901d643e5d486be9d12aa
Author: Ravi BK <email address hidden>
Date: Wed Jun 29 12:21:41 2016 +0530

Advertise Instance-ip with prefix using alloc-unit

Issue: There is a requirement of assigning alloc-block(power of 2)
vm instance-ip. Today we are creating UC routes for instance-ip
with /32 prefix. Now we need to create and advertise routes
for instance-ip with prefix calculated based on alloc-unit present
in IPAM configuration.
Fix: Store alloc-unit configuration in VnIpam structure and use
it while VM Interface is being created. Add UC route accordingly
and RouteExport will automatically exports the route.

Change-Id: I870a52b3bc732b5476bf17f75c46fe5599077d8a
Closes-Bug: #1595671

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

Review in progress for https://review.opencontrail.org/21723
Submitter: Manish Singh (<email address hidden>)

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

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

commit e524ecce8704214d79d6f75a06b3763b9f7c31a1
Author: Manish <email address hidden>
Date: Thu Jul 7 21:45:52 2016 +0530

Fix UT.

Following commit modified installation of vm interface route. It is being
installed when IPAM is present. Test case had a test to check for evpn route and
inet route in absence of Ipam. That fails because of this commit. Fixing the
same.
https://github.com/Juniper/contrail-controller/commit/eda15808d531850f2d5901d643e5d486be9d12aa

Closes-bug: #1595671
Change-Id: Id1936792c0b36e9d1d46520b31330dc590dc7e90

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.