ipv6 flow export support in sandesh and collector

Bug #1519544 reported by Raj Reddy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Medium
Sundaresan Rajangam

Bug Description

for export ipv6 overlay flows we need additional support in sandesh and collector

vrouter-agent changes to export ipv6 flows to collector is tracked with https://bugs.launchpad.net/juniperopenstack/+bug/1526626

Tags: analytics
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/15519
Submitter: Sundaresan Rajangam (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/15519
Committed: http://github.org/Juniper/contrail-sandesh/commit/a11ce36edf3792336c419e3009fbb50f75033f53
Submitter: Zuul
Branch: master

commit a11ce36edf3792336c419e3009fbb50f75033f53
Author: Sundaresan Rajangam <email address hidden>
Date: Tue Dec 1 01:41:10 2015 -0800

Added new datatype "ipaddr" in sandesh

This patch implements a new datatype "ipaddr" in c++ sandesh library.
ipaddr can store either ipv4 or ipv6 ip address. In c++, ipaddr is same
as boost::asio::ip::address.

Change-Id: I7c70a0c84183cd1b3ec40bccde9f564755f04fe5
Partial-Bug: #1519544

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

Review in progress for https://review.opencontrail.org/15545
Submitter: Sundaresan Rajangam (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/15545
Committed: http://github.org/Juniper/contrail-sandesh/commit/562e3115269f9dba9a9ae982607118897b228002
Submitter: Zuul
Branch: master

commit 562e3115269f9dba9a9ae982607118897b228002
Author: Sundaresan Rajangam <email address hidden>
Date: Wed Dec 2 04:51:17 2015 -0800

Fix compilation error due to address::from_string

Use No Exception version of boost::asio::ip::address::from_string()

Change-Id: Ifa75e62dcaa42bf8961654a003f7e489701e3603
Related-Bug: #1519544

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

Review in progress for https://review.opencontrail.org/15614
Submitter: Sundaresan Rajangam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15669
Submitter: Sundaresan Rajangam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15707
Submitter: Sundaresan Rajangam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/15669
Submitter: Sundaresan Rajangam (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/15669
Committed: http://github.org/Juniper/contrail-sandesh/commit/33b0e18fda4ae516b6b10217e88c7924fbf3c59a
Submitter: Zuul
Branch: master

commit 33b0e18fda4ae516b6b10217e88c7924fbf3c59a
Author: Sundaresan Rajangam <email address hidden>
Date: Mon Dec 7 22:21:28 2015 -0800

Implement datatype "ipaddr" in sandesh c/python.

Change-Id: I4517af1949dbba4333bf234d418c5225922ead5b
Partial-Bug: #1519544

description: updated
Revision history for this message
Megh Bhatt (meghb) wrote :

Listing out preliminary support for inet/IPv6 in CQL driver/contrail-collector/contrail-query-engine committed as part of CQL driver changes - https://bugs.launchpad.net/juniperopenstack/+bug/1522930

CQL driver has added some support for inet type which is the type to be used for both IPv4 and IPv6 in CQL/cassandra - cql_if.cc. Currently it is disabled via INET_SUPPORT #ifdef. It is missing IPv6 decoding and encoding. Currently the CQL tables are also created using int32_t as the data type for IP addresses, it needs to change to type inet. This is again controlled via INET_SUPPORT in vizd_table_desc.cc. When implementing support in contrail-query-engine, this #define needs to be enabled - DbDataValueType variant needs to add in inet_addr support too.

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

Review in progress for https://review.opencontrail.org/16139
Submitter: Sundaresan Rajangam (<email address hidden>)

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

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

commit 306d52522ac668c57006df99c69b6432eb10e493
Author: Sundaresan Rajangam <email address hidden>
Date: Thu Jan 7 06:01:03 2016 -0800

CQL inet support in collector and query-engine

- Encoding/Decoding of inet datatype.
- sourceip/destip in Flow tables modified to inet datatype to support
both ipv4/ipv6 address.
- Handle sourceip and destip sent as i32 for backward compatibility.

Change-Id: Iaf5d23eb2bcd44419abcf3915d2da5669f21bfca
Partial-Bug: #1519544

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

Review in progress for https://review.opencontrail.org/16212
Submitter: Sundaresan Rajangam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/16342
Submitter: Sundaresan Rajangam (<email address hidden>)

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

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

commit 6e5b5be95897501887d7c2f6080a60ccd47e854f
Author: Sundaresan Rajangam <email address hidden>
Date: Sun Jan 17 00:59:04 2016 -0800

ipv6 support for overlay flows

- Changed datatype of sourceip and destip from "i32" to "ipaddr" in flow
sandesh.
- Renamed the flow sandesh sturture name from "FlowDataIpv4Object" to
"FlowLogDataObject".
- Test code in analytics for ipv6 flows.
- Agent changes to send sourceip and destip as "ipaddr" type. [Ashok]

Change-Id: I2c49c71091e0f35929ed1445ed45d476db06db4c
Closes-Bug: #1519544
Closes-Bug: #1526626

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.