Spec update for DHCP Neutron Port Allocation

+ Updated the pipeline/yang changes
+ DHCP request handling added .
+ Introduce controller-dhcp-mode flag in DHCP Service.

Change-Id: I633571012d6f3ff8a07e0c8e50da9ab05c106c55
Signed-off-by: Achuth Maniyedath <achuth.m@altencalsoftlabs.com>

Bug 8441, 8630 - LB groups not deleted after extra route deletion

Issue :

When an extra route is completely removed from all the DPNs, the
loadbalancing groups are not updated properly.

Analysis:

There was a "NullpointerException" occuring in the
setupLoadbalancingGroup() method.

Fix:

Changes have been made to avoid the NPE. Also, changes were
made in resolveadjacency() method to populate the adjacencylist
correctly.

This patch also fixes the bug 8630
(CSIT Sporadic failures - arp learning suite
 - MIP not in /config/odl-fib:fibEntries/)
as well which resulted due to the exceptions
caused as mentioned above.

Also it reverts some part of the review that landed here:
https://git.opendaylight.org/gerrit/#/c/59081/

Change-Id: Iaffd3916603f33810cfb7fdba2a86ba8826fc9e3
Signed-off-by: Gobinath <gobinath@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Fix null Optional instantiations

Optional type variables must never be null.

Change-Id: I86af0a9c11dca31b00d72818210d0afb0bcbcff9
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Restore VRF imported routes case

In createLocalFibEntry, localNextHopInfo is always null. It used to be
initialised in what became the forEach() loop; this patch restores a
standard foreach loop and tracks whether local next hop info has
actually been seen.

Change-Id: I9dc7a041a1010f7f85003bd225d75f2d03a7d076
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 8706: Restore null-check in createLocalFibEntry

checkCreateLocalFibEntry handles null gracefully, but dereferencing
localNextHopInfoLocal in the method call renders that useless.

Note that just following the forEach(), localNextHopInfo is always
null which suggests something got lost in the transformation...

Change-Id: If15cc4a881c2a1468c84a4e0cebb2a940465f33b
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit b53e6bdf8ab17e4f96d49d607c3eae0a7332b73c)

Minor code clean-up

Replace vrfEntry.getDestPrefix with localNextHopIP in createLocalFibEntry

Change-Id: I7f891ce9a692723829c749736e7f5d89011822d9
Signed-off-by: Janki Chhatbar <jchhatba@redhat.com>

Bug 8657: Fix for failure seen with IPv6 extra-route

Currently when programming an IPv6 extra-route, we see an
exception in Netvirt and the corresponding table-21 flow
is not programmed. This patch addreses this issue.

Change-Id: I5d8192dd4d095ecbb8f0c733a795953a34c48487
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Bug 8552 - [l3vpn]Stale flow entries not getting removed even when all the
VM config are deleted

FIB handling was being interrupted by and NPE, which led to stale remote
table-21 flows. This is because the GroupEntityBuilder has been modified
with respect to the way it adds BucketListInfo.
It tries to addALL the elements from the input List of buckets now, and
hence a null param causes NPE. This has been fixed.

Change-Id: Iafe20e405101c294ce56babbaf5ff0057be55ae7
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Use optional for dataBroker

Change-Id: Ifa83b22b5868ad024557b8a9c6ca9297c24a636a
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 0e1d866844a1876cab7d47db8f5d66ba1b5e7816)

Fix SingleFeatureTest breakage on fibmanager-shell bundle

Looks like the build breakage happened via landing:
https://git.opendaylight.org/gerrit/#/c/58554/

Change-Id: Iddc3ac721e130ab46271624de1c0550aade087cf
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug 8520 - NPE at org.opendaylight.netvirt.vpnmanager.arp.responder.ArpResponderUtil.getActions

Donot setup Arp Responder flows to External Netwrok if GW router MacAddress is Null.

Change-Id: I0a013c3d43199a56786fd7f4a788521617f3df7c
Signed-off-by: Janki Chhatbar <jchhatba@redhat.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Bug 8374: NPE observed on fib-show command

 + Havent been able to reproduce the NPE, but did a bit of
   refactoring with special care about possible NPEs

 + First of all, it's not VrfEntryListener's responsibility to
   print anything on screen. This logic's been moved to the
   fib-show command implementation.

 + I took the change to remove dependency from 'shell' to 'impl'

 + L3VPNTransportTypes defines types that are used in 'shell', so I
   moved it from 'impl' to 'api'.

 + Also moved the dependency to 'itm-api' from 'impl' to 'api'

Change-Id: Ie792c40cf61af0bcefb83be9b0d6c5fcb6fb755c
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Fixes:
 - BgpAlarms: avoid un-necessary reading of neighbor list

Change-Id: I4cded3511710cdeabd2fa8abb75cb35bf47b7bf7
Signed-off-by: SivaKumar Perumalla <sivakumar.perumalla@ericsson.com>

Bug 8646: QoS log level changed to debug

Some QoS logs should not be showing up by default.
Changed these logs to debug.

Change-Id: I2c15f202e32b9adf433275d01dc24cf880bddafb
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

Use TestInterfaceManager that's now in testutils

following https://git.opendaylight.org/gerrit/#/c/58093/

Change-Id: I6bd0c4e445a0a8b58ea54d2a5553412a89d6ff2c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 8615 - Stale flows in table 46, 47 and 44 when SNAT is disabled
(Conntrack Based)

Change-Id: I45f2d84caf691bb12b35a708fe66ec5f8d4cd8d6
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Support ping6 for Neutron internal router interfaces

This patch programs the necessary OVS flows that respond to
ping6 requests addressed to the IPv6 subnet gateway_ips on
the tenant network (i.e., Neutron internal router interfaces).
It is similar to the IPv4 ping auto-responder support we
currently have in Netvirt.

Change-Id: Iec686be7fca6743ce2fff666a29e8912645e8c47
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Bug 8302 fix for ELAN TS table flow leftovers

Since different flowref (regression introduced by vni enforcement
implementation for the datapath) is passed, there are some leftovers in table 36
which causes stale flow comes into effect after ovs restart. This makes
ELAN datapath to fail.

Change-Id: I9785b641c7f05aaed268371653bf3fd2b3e36dc3
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Revert "Bug 8588 - NPE at at org.opendaylight.netvirt.fibmanager.VrfEntryListener.createLocalFibEntry"

This reverts commit 2ceab866932eda11217bbc4be6e809275a1d2d99.

Change-Id: Iff8d954b1b62ecda1903cf0562198e0cb5d7fe98
Signed-off-by: Sam Hague <shague@redhat.com>

EVPN RT2 : UT Bug fixes

1. NPE at evpnMacVrfUtils.
2. EvpnElanInstanceListener is NOT getting hit.
3. Failed to get the dpn tep ip for dpn
4. getEVpnRd : evpnName is NUL
5. IllegalArgumentException for Ipv4address during addprefix
6. Withdraw of routes failed on evpn detach from network
7. On Detach Network from EVPN installed DMAC flows are not getting uninstalled
8. DMAC Flow is getting installed with action=DROP
9. IndexOutOfBoundsException at EvpnMacVrfUtils.addEvpnDmacFlow
10. MacVrfEntryListener add Error : elanName is null for iid
11. MacVrfEntryListener is not hitting
12. ElanMacEntryListener evpnName is NULL for elanInstance
13. On Detach Network from EVPN installed DMAC flows are not getting uninstalled
14. DMAC install on attaching network from evpn, which was detached earlier
15. On creating new port after multiple attach and detach  network to
evpn. Advertise route is failing
16. ElanName was NUll while adding externalTep in BgpUtil.

Change-Id: Ic5a97985ae8ab071bdc26ec93f280234af2bbe75
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>

Bug 8610 - In NAPT HA(Conntrack based) NAT rules are not programmed when
a switch comes back after a failure.

1)The conntrack implementation now responds to RouterDpn changes.
2)An error in populating dpn-vpn list is now fixed.

Change-Id: Ife3c28f29fdb3de9f0108ca1bdb86d2fc0951a21
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Bug 8440: Traffic from remote OVS to LB Group

Traffic to extra-route from a remote switch was not going to the LB
group from the tunnel table.

This review makes sure that the tunnel table points to the LB group and
also makes sure that if the extra route could be reached through another
VM which is present in different switch then the tunnel table directly
points to only one of the VMs in order to avoid looping.

Change-Id: I0964991eef519610487e5454e39a643a4c04536b
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>

Fix NPE in DhcpAllocationPoolManager close()

This NPE happens during shutdown, because dhcpAllocationPoolListener is
null unless isDhcpDynamicAllocationPoolEnabled, so it has to be null
checked before closing it.

2017-06-08 14:15:34,997 | ERROR |  Bundle Shutdown | BeanRecipe
| 15 - org.apache.aries.blueprint.core - 1.6.1 | The blueprint bean
dhcpAllocationPoolManager in bundle
org.opendaylight.netvirt.dhcpservice-impl/0.5.0.SNAPSHOT incorrectly
threw an exception from its destroy method.
java.lang.NullPointerException
        at
org.opendaylight.netvirt.dhcpservice.DhcpAllocationPoolManager.close(DhcpAllocationPoolManager.java:88)

Change-Id: I5d7482a0bf6ffbaf34dea0ebca1566ff641fea7c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 8588 - NPE at at org.opendaylight.netvirt.fibmanager.VrfEntryListener.createLocalFibEntry

Check for null.
Minor code clean

Change-Id: I3a0bd4b4e38ddd0830012bc24a718a39f24ceeaf
Signed-off-by: Janki Chhatbar <jchhatba@redhat.com>

Bug 8609 - CSIT Sporadic failures - vpnservice suite - l3 rule leftover in
table=19

A router interface reference in a VPN will have to be removed before the
host interface references for that subnet in the VPN are removed. This is
to ensure that the FIB Entry of the router interface is not the last entry
to be removed for that subnet in the VPN.
If router interface FIB entry is the last to be removed for a subnet in a
VPN , then all the host interface references in the vpn will already have
been cleared, which will cause failures in cleanup of router interface
flows.

Change-Id: I53d0c99a344e81ecefe2e9235215f64ecbdf020e
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Simplify some streaming constructs

Change-Id: If48c31a56e1667597dc540ac5068fe731058a6e1
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bug 8549 - Inter OVS traffic over EVPN does not work with
openstack-vni-semantics-enforce set to true

Change-Id: I401107b046695ec9af9feebbc47ecce0461b0b07
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>

Bug7830:Conflict modification exception for NAT

Problem Statement:
===================
Conflicting modification for path exceptions was throwing from NAT module
for the flow creation of table21->26 and table47->21

Solution:
============
Table21->26 flow creation was happening in 2 more classes. To avoid
race-condition have introduced DJC to install and remove the flow.

Change-Id: Ib6c47667c9245d50eb51a7baa3d7969aa05b325b
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>

Bug 8614: NPE observed in cloud-sc's VrfListener

Change-Id: Ief4b9b90c96e1e426b9399389538cd14b39fc942
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Bug 8595 - DNAT traffic from DC gateway to FIP fails

Description :

In PDNAT_TABLE, we have a FIP mac match for performing FIP to internal IP
translation. But in L3_LFIB_TABLE, the destination mac has not been
updated and as a result, the reverse traffic for FIP were dropped.

Changes done to update packet with destination FIP mac as a result,
PDNAT_TABLE has specifc match and FIP to internal IP translation been done
properly.

Change-Id: I3fec733c7a8c049808a09828b8090daed12c83fc
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Bug 8585: Exception with invalid QoS Alert params

Root cause - When invalid config parameter is set through karaf shell,
in-memory configuration is updated before data validation. Invalid configuration
later gets rejected but this makes configuration inconsistent. Any subsequent, configuration
is rejected until problematic parameter is set to valid value.

Fix - Update the QoS Alert configuration parameters only after validation.
Same as done for configuration through REST.

Change-Id: Ia46664850f3c7abe0ce74f76cf7e35bdbd79c7c6
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

Fix checkstyle problems not detected by the current version

This change is required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle

Change-Id: I8f9c20211a2936cf964f2cf51f8e41ffa180bb07
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7451 - guarding NPE

added few null checks to avoid nullpointerexceptions

Change-Id: I8b077f1d87d79bfe3804d8daabdf391a5358f301
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Bug 8581: DNAT failure with openstack/ocata

Problem Statement:
===================
DNAT traffic is getting failed because of missing DNAT flows in switch
(DPN). This issue was observed in openstack stable/ocata only.

Solution:
==========
Due to invalid check in NeutronPortListener.update(), certain FIP DS entry
where not populated and as a result, required DNAT flows where not installed.
Changes done to create required FIB DS entries properly.

Change-Id: Ia11a7e1d28288d762d26c3897d1612401d4ee8fa
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

elanName is null

Check for elanName. If null, return.

Change-Id: I72c931477c6d8ae464a654095383f8537ef6748e
Signed-off-by: Janki Chhatbar <jchhatba@redhat.com>

Bug 8586: Alarm raised due to stale stats polling

Root cause - When port stats polling is enabled, first poll returns
old data (Rx bytes and Dropped bytes) when QoS alert was not enabled.
Alert may get logged if stale stats has dropped % > threshold value

Fix - Alert should not be logged based on first poll data. Use first poll data
to set the baseline counts and raise alert on subsequent polls based on the delta
of counts between the polls.

Change-Id: Ic12f3d41e7c0277109acc51ab548e1a7d0b4d13b
Signed-off-by: Arun Sharma <arun.e.sharma@ericsson.com>

Replace LOGGER by LOG

Replace LOGGER by LOG to follow the
OpenDaylight recommendations [1].

[1]
https://wiki.opendaylight.org/view/BestPractices/Logging_Best_Practices

Change-Id: I038b8fc9a16b7443fcdefcad1bc12283151ba503
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Clean up Optional uses

This patch reverts Optional and related streaming uses where they
result in more convoluted code than necessary (or, in the case of
Optional, where Optional instances are created only for conditions).

Change-Id: I8d0b2c456ff0a477e440fb029b8495476aabed27
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Replace logger and log by LOG

Replace logger and log by LOG to follow the
OpenDaylight recommendations [1].

[1]
https://wiki.opendaylight.org/view/BestPractices/Logging_Best_Practices

Change-Id: Ia4f5a1c1692133efa0e49873ba954a360572989b
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 8589: Fix regression in Ipv6PktHandler

Following this change [1], Ipv6PktHandler is not getting
invoked for RS/NS meant for router interfaces. This patch
fixes this issue.

[1] https://git.opendaylight.org/gerrit/#/c/54389/

Change-Id: I5b98b742623df2552205f8bbb24d70ae17ba1325
Signed-off-by: Sridhar Gaddam <sgaddam@redhat.com>

Use named constants in ElanUtils

... instead of commented true or false.

Change-Id: I7f0738f11666f9433ae0942b2be538514d08311b
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Use direct comparisons instead of Objects.isNull

There is no reason to ever use Objects.isNull() directly, it’s
equivalent to “== null”. Its documentation explains its purpose: “This
method exists to be used as a Predicate, filter(Objects::isNull)”.

Change-Id: I86123fec65c09a1d26906935cad6df79750fa2d8
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bug 7451 - VPN service cleanup of Table 17

Whenever a VM is deleted (or) a VPN where a VM
resides in is removed, the dispatcher-table
entries for that VMs VPN Interface were not
getting removed.

The fix here makes sure that dispatcher Table
(Table 17) gets cleaned up by VPN Service.

The related ELAN patch to clean up dispatcher
table for ELAN service entries is here;
https://git.opendaylight.org/gerrit/#/c/57767

Change-Id: I6460b8a12a37236275e72606bc58c4b59db440cb
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

SNAT performance improvement for Controller-Based SNAT

Description :
Following changes were done to address SNAT performance improvement
in following area.

1) Seperate Queue for first SNAT packet, retry packet and Flow removed
event on Idle Timeout.
2) Multithreading: using 25 threads for processing first SNAT packets, 25
thread for processing retry packets and 1 thread for processing Flow
Removed event.
3) Updated checkIpPortMap API to have direct query based on routerid,protocol,
internalip+port criteria thus preventing unnecessary looping.
4) Moved the code of getting externalMac Address and external address to
get executed only for first packet, thus preventing the retry packet from
executing it which is unnecessary.

Change-Id: Ib34aa8726295381d0e4ffd1f3cec74292d422c4f
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Bug 8417 - [l3vpn_ecmp] Deleting MPLS GRE Tunnel port doesn't delete the

bucket entry in the group

Change-Id: Ic7287a9611bf27453713e47f6dde11373fea3172
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Minor Checkstyle fix Custom Import Order error

This shows up as red in Eclipse, but the CLI mvn build missed this.
This is due to a more recent version of Checkstyle.

required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle

also includes minor removal of (1) extra parenthesis, (2) generic <>
clean-up and (3) missing @Override from automatic clean-up which happens
on auto-save (technically unrelated to Checkstyle).

Conflicts:
vpnservice/qosservice/impl/src/main/java/org/opendaylight/netvirt/qosservice/QosAlertManager.java

Change-Id: Ic650ffe9ac42cfbc7cad4c1c31c7553faf5fe603
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Bug 7451 - Leftovers in dispatcher table when unbind and ietf-state delete

           occurs simultaneously

Change-Id: Id3c44e94da203ddda9dbc3e6fb8b6885523c7fa3
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 8498 - ICMP traffic from DHCP NS not blocked when ICMP rule to allow
VM-VM traffic is added in learn mode.

The  wrong filter table to be used for learn ingress/egress service is
fixed.

Change-Id: I15e8841314c24b10556128bac354ebab1d80a698
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Add JUnits for InterVpnLinkLocator

 + Only 2 so far:
    -  testFindInterVpnLinksSameGroup
    -  testSelectSuitableDpns_moreDpnsThanIvpnLinks

 + It adds an L3vpnCatalog that contains several VPN test data
   to play with

 + Idem with InterVpnLinkTestCatalog

Change-Id: I1c6866ec5fe2570a469dd9ea9e5a6591d11dffa4
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Fix a few non-null collections

In general, methods returning collections should never return null.
This patch fixes L2GatewayConnectionUtils and propagates the non-null
results.

Change-Id: If249aec9b8494de736c362f0ea04139151670d68
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bug 8376: Fix DHCP for external tunnels

DHCP for external tunnels does not work with neutron
DHCP server.

ODL is installing rules to punt DHCP packets received
on external tunnels to controller, even when ODL
DHCP is disabled. But, since ODL DHCP is disabled,
no DHCP response is sent by ODL. And, with these
rules installed, neutron dhcp server never sees the
DHCP packet. As a result, VM will not get IP
address.

Do not start listeners that are responsible for
installing DHCP rules to punt DHCP packets to
ODL controller when ODL DHCP is disabled.

Change-Id: Ib0e243a9bca5866e50a5d1cb4c70cd4441dd01e6
Signed-off-by: Victor Pickard <vpickard@redhat.com>

BUG:8341   IAE seen in CSIT logs when port/vlan name is not matching UUID

Change-Id: If90482540cc5cc7c28d16397f84a39f3774eab7b
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

bug 7599  hwvtep ucast mac add performance improv

Changes for performance improvement.
Added new Cmd TunnelIpCommand and also changed function getKey to return
actual key Identifier in place of String value.

Change-Id: I0fd84c87db6feefcba4c119fe0c05193751ab8cb
Signed-off-by: Akash <a.k.sahu@ericsson.com>

elanmanager-impl <dependency> interfacemanager-impl <scope>test

It is wrong to directly depend on interfacemanager-impl from
elanmanager-impl.  This may only be done with <scope>test, for component
tests.

Remove the sole usage of a util from ITM - genius has the same util.

Change-Id: I0dda82184d547cefa5a447361111438ed3fe40c1
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Use named constants for PolicyAceFlowWrapper

Instead of commenting each use of true or false, use constants.

Change-Id: Ibf8b19b67dc54e99e6caffe8056a0e04b62601c2
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Adjust tunnel state listeners for logical tunnel

If the logical tunnel exists between the source and destination DPNs it will
be set as the lport-tag of register6 in the remote nexthop actions.

Spec: https://git.opendaylight.org/gerrit/50779/

Change-Id: I98ff89efd047e056529e4bbdf06493e548c561d1
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>

BUG:8232 updating BGPVPN with the List of RDs

       *Problem: When a new Extra-route is added and if there are not enough RDs
        for the VPN then the new extra-routes are not handled.
        These unprocessed extra-routes will be handled during vpninstance update
        (ie when new RDs are added(Update) to the existing VPN)

Change-Id: Ic380b6aaabdd0d44484c2ec1a33de7cde9c01366
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>

Fix Unused log and private final

Delete unused log and their imports.
Change initailized private final var to
private static final in order to use less memory.

Change-Id: Ieecaf3cf75ac67b55e2bfd87c1c78eb1bbd77782
Signed-off-by: mcauffiez <mcauffiez@inocybe.com>

EVPN RT2 : Silent host changes

When the flow created on packet received from silent host expires,
prefix is removed from macEntry container and prefix is withdrawn,
There is no necessity to withdraw prefix from here, it will be
duplication.

Change-Id: Iab970736a182efaddf4dfc44a5a11377ca22b111
Signed-off-by: Yugandhar Reddy Kaku <yugandhar.reddy.kaku@ericsson.com>

Bug 8508 : Id-manager exception during releasing id for router

Description : Even though opendaylight-semantics is disabled,
the following EVPN NAT use-cases require VNI to be allocated from
OpenDaylight VNI Pool:

a. All traffic on external network will be using a VNI allocated for the
external-VPN aka Internet VPN.

b. Similarly all traffic from Non-NAPT switch to NAPTSwitch will be using
the VNI allocated for the router from the OpenDaylight VNI Pool.
For VNI-L3-L3 Fowarding feature also, a VNI must be allocated for every
router from the OpenDaylight VNI Pool as that VNI will be used for
Non-NAPT to NAPTSwitch traffic.

This fix makes that allocation as only release alone was done earlier.

Changes done to allocate a vni for router(when router-gatway-set done)
and been released during ext-router entry is deleted. In CSIT(having
non-EVPN use-case) were executed, an attempt to release the non-allocated
id was done during when ext-router entry is deleted resulting in exception.
Change done to make sure an id is allocated when router-gateway-set is done to
prevent such occurance of exception.

Change-Id: Ic1e696d6b4c8d0c56eac8b16ed0731d072b5b135
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Bug 8539: IPv6 L3 Forwarding broken.

The merge of the following review;
https://git.opendaylight.org/gerrit/57145
broke L3 Forwarding for IPv6 addressed ports.

The fix here addresses the breakage by making
sure that L3_GW_MAC_TABLE entries are filled
correctly for both IPv4 and IPv6 router-interface
ports, before embarking on writing PING
Responder flows for IPv4 based router-interfaces.

Change-Id: Ieb190edda470264006609c62e4af9fc0d94d4a2c
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug 8485 : EVPN was also coming as Layer_3 VRF

Problem:
Even after configuring EVPN Layer 2 VRFs in quagga its coming as EVPN Layer 3 VRFs.

Solution:
Changes to modify the layer type was missing in VPN engine

Change-Id: I8f45b06c02f3920512f16563d77b13b8ba076a9d
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

Bug 8358 -  Local next hop group not deleted after VM migration

Issue:

When a VM is migrated from one DPN to another, the FIB flows are not
programmed for the VM in the DPN to which it is migrated.

Analysis and Fix:

The l3nexthop container is used to check whether the local next hop group
is already present for a prefix. If it exits, no new group is created and just
the reference count is increased. When this prefix is removed, this
reference is decreased & the group is removed when the reference count is
0. It was observed that for the first VM creation, the reference count is
2(it should be 1) because of redundant calls to createlocalnexthop. A
check has been added to prevent this.

Also some other minor fixes have been done related to ECMP

1. The addition/deletion of extra route also changes the above mentioned
flow reference count. This caused problems while just updating the
extraroutes(the flow reference count keeps increasing and so the local
next hop group would never be deleted). Now, the addition/deletion of
extra routes doesnt change the flow reference count in the l3nexthop DS.

2. When there is no change in routepath of the extraroute(updated extra
route also present in the same DPN), there is no change in odl-fib DS and
so handling has been added to implicitly call the refresvrfEntry which
would update the FIB flows.

Change-Id: I4d7b435aed73268383b36e1c635c499c71e54c60
Signed-off-by: gobinath <gobinath@ericsson.com>

Bug 8412 - NPE while adding and removing elanmacentry

1. While advertising RT2 route prefix is Optional, hence added a safer
null check. if prefix is not present, still go ahead and advertise RT2
route
2. During withdraw if prefix is not present, return with error log.
3. Remove the elanName null check as its never null.

Change-Id: I3009bb9d19f3413d448031ff47771a986ef10746
Signed-off-by: Janki <jchhatba@redhat.com>

BUG 8537: Get destination from interface remote ip

Genius will not set the tunnel destination IP address until the very
last action. Classifier egress cannot expect to have this IP address
available on it's own pipeline.

Solve this by setting the IP address on REG0 on the egress service
binding actions. This will be the remote IP of the bound interface in
case of tunnel or the local IP otherwise.

To avoid overuse of registries, a REG is no longer used to set the
SFF IP address on table 101. This IP will be set directly on the path
specific flows of table 223 if needed. Previously this REG was being
used to know if the SFF was remote by comparing with the local IP in
these flows but that is not needed as it can be known at configuration
time.

Change-Id: If7a474391ed2d643cc3b3f34743d60c5f418b2e5
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Replace toUpperCase().equals by equalsIgnoreCase()

Change-Id: I54ac6903362c15c6caab05810861de852d2619a5
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>

Bump versions by x.y.(z+1)

Change-Id: I54e2105583481ca281691c735d81d027350d4711
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>

Bug 8475 - Non-external vlan provider networks do not work

Patch - 2 for handling of review comments on master

Spec link:
http://docs.opendaylight.org/en/latest/submodules/netvirt/docs/specs/vlan-provider-enhancement.html

Change-Id: I4fc70ba20927257046e45ffe5d577106fcc8d09c
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>

Bug 7939,8082: Table 19 flow nightmares

Table 19 flows rules aren't getting recycled appropriately
whenever VMs are moved into a routed subnet and then again
backed off.

The fix here attempts to make sure that Table 19 flow rules
are handled consistently when a subnet moves in/out of a router
and also when a router moves in/out of a bgpvpn.

Change-Id: Ief94b1b856a69c631052f11644db6a0b091f2aa7
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7939 : SubnetRoute functionality for bgpvpn broken.

IPs behind Neutron Interfaces are not discoverable for
router-associated-to-bgpvpn use-cases.

With this fix we are disallowing creation of subnetRoutes for internal
subnets off neutron routers.
More specifically, for just neutron routers, subnetRoutes won't be published.
However, if such subnets are part of a router that is within a bgpvpn, the
subnetroutes feature will continue to work.

For the PNF review that requires subnetRoutes to be available,
the fix here needs to be further tweaked to make PNF discovery on internal
subnets of a VLAN-network to work.

PNF Review referredin commit message here is:
https://git.opendaylight.org/gerrit/53984

Change-Id: I507aae2e3d1cbb72457f32b48e598ef0f6cda703
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>

BUG-8388: On egress, filter out SF bound packets

Classifier should not handle packets that are already classified and
bound towards the SFF. For this, we set C1 to 0xFFFFFF on the ACL flow.
Then on beginning of egress pipeline, if C1 does not have that value
we assumed that's a packet bound for the SF and already classified so
it is resubmitteed to dispatcher. Otherwise, C1 is reset to 0 and
handling continues.

Change-Id: I1427a9bbec7fb0c8e36c241b727d39de450bc1ed
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>

Bug 8451: afi parameter is not the good one used in pushRoute and withdrawRoute

The afi parameter used as parameter is calculated based on the prefix nature,
that is to say either IPv4 or IPv6.

Change-Id: I6d34488e43d4dc589ce842fcd1675d24033c9f28
Signed-off-by: Noel de Prandieres <prandieres@6wind.com>

Bug 8471 - Traffic between a VM without FIP and a VM with FIP is not
working

1)Hairpinning changes are done for conntrack based SNAT, now it matches
on external subnet id that external n/w id.
2) The packets are submitted back to table 21 from 47.
3)L3_GW_MAC_TABLE (Table 19)  replace the external n/w id with external
subnet id for the packet address to router external interface
4)L3_FIB_TABLE (Table21) flow now matched external subnet id  instead of
n/w is along with external ip address.

Change-Id: Ib4de831a58638ed9d4aed9c78450e3d7bebf6222
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Bug 8475 - Non-external vlan provider networks do not work

Spec link:
http://docs.opendaylight.org/en/latest/submodules/netvirt/docs/specs/vlan-provider-enhancement.html

Change-Id: Iad7479835a7bd97a4f827c8365601d2a20efe477
Signed-off-by: Ravindra Thakur <ravindra.nath.thakur@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug8484: Non-NAPT Group action is drop for router associated with BGP-VPN

Problem Description:
======================
Non-NAPT to NAPT VM Traffic is getting dropped at Non-NAPT Group level
when router is being associated with private BGP-VPN.

Solution:
===========
When ever Vxlan tunnel got "down" followed by "up" event between Non-NAPT to
Napt Switch happened, Non-Napt group got re-created through tunnel
interface state listener. In this use case is not been properly having the
code logic for if the router with private bgp-vpn configured.

Added the proper code for creating tunnel bucket information for Non-NAPT
groups got re-created during tunnel state changed.

Change-Id: I509eeda9041d5e1b170e45554b3d9f661e6091fd
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>

Bug 8473: Reboot fixes for ECMP with extra-routes

* It includes changes in the extraroute-rds-map container. Nexthop is
  added as key which is used to retrieve the allocated rd for an extra
  route. This is useful during reboot to know which rd is allocated for
  an extra route and this change avoids re-allocation of rds during reboot.

* Some minor fixes for reboot is also added in this review.

Change-Id: I31e8d153be947810a66ed3e2ccc686eee876bea8
Signed-off-by: eceghkl <manu.b@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug 8474: Enable openstack-vni-enforcement on data plane

Change-Id: I5f87df4b45e35a3b857f616437b6ede4c0b5f029
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug 8474: Local FIB entries for VxLAN VPN interfaces not programmed

Local FIB entries for VxLAN networks on router-based VPNs are not getting
programmed when VNI flag is turned to true, as makeConnectedRoute() wasn't
being called.
This commit fixes the same for create and delete cycles.

Change-Id: Ib81ac9aa1270921c11fc35cfb3352afdd5aff577
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug8471 : L3_GW_MAC_TABLE entry for FIP not removed.

Description : L3_GW_MAC_TABLE entries for Floating-ips where not removed
after FIP disassociation. The flow-key generated during removal was
different then during creation of this flow.

Change-Id: I6dc9d7dbb365d9e1a815241b01139658ebb59d12
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Bug 8451 : Problem with EVPN RT5 exchange between ODL and QBGP

Problem:
ODL was always advertising encap type as MPLS and protocol type as ANY
irrespective of L3VPN or EVPN RT5

Fix:
Added the missing code to handle the encap type and protocol type

Change-Id: Ia4e27b62c33977ae5f5ce155695851f97bc47788
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

Bug 8025: Use RPC with barrier when sending SNAT packet out

When configuring SNAT flows and then sending a packet-out to those
flows, use RPCs with barriers before the packet-out to ensure the
flows are configured once the packet-out is sent.

Change-Id: I330df570857317e50bf5dc404e9691a0053f3f54
Signed-off-by: Alon Kochba <alonko@hpe.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Bug 8294 : fib-show cmd throws an exception on usage

When using fib-show, an exception was triggered when parsing FibEntries
from a VRF context.

Correcting the existing fix. Instead of break , changing to continue.

Change-Id: I4637ac14b7fd5bdb9ccb7978d1a2b40538098dfd
Signed-off-by: HANAMANTAGOUD V Kandagal <hanamantagoud.v.kandagal@ericsson.com>

BUG-3874: BGP Manager / fib-show shell cmd triggers an exception on usage

When using fib-show, an exception was triggered when parsing FibEntries
from a VRF context. A sanity check is done in order to prevent accessing
null entries.

Change-Id: If296e3b4770518f53c789130565db8960ca63ef6
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

EVPN RT2 Installing flows for RT2 received in prior

1. When EVPN is created, vpn instance will be created and bgpmanager will
register with quagga bgp. hence BGPmanager will start receiving RT2 routes
from peer. Bgpmanager will write into macvrfentry(odl-fib.yang).
macvrfentrylistener is listening to macvrfentry and which installs dmac
flow for the received RT2 route only if EVPN is attached to network.

2. Now if EVPN is created and NOT attached to network. BgpManager will
still receive RT2 routes from peer, which will be written into
macvrfentry, but since evpn is not attached to network yet, hence DMAC
flows are not installed. Once Evpn is attached to network we need to go
through all the macvrfentry with the matching RD and install dmac flows.

3. when EVPN is detached from network, we go through all the macvrfentry
for that RD and uninstall DMAC flows.

4. async read is done so that even if it fails to read first time it tries
multiple times till either read is success or failure.

Change-Id: If4cb1b0fb924e7defa4e8c119dcf249b3f9b24f6
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>

EVPN RT2 : Receive RT2 BGPManager changes

On receiving RT2, bgpmanager has to updated the macVrfEntry in fib.yang.
Also when the RT2 is from a new TEP, same has to be updated in elan
external tep DS.
A sample cli is added to simulate the RT2 from QBGP as well.

Change-Id: If193af109a3a08d08d6c6f32e715c71f045e5af8
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

EVPN RT2 : Listener for evpn to network associatn

When EVPN is created, even if no network is associated to it, BGP routes
will come from quagga which will be stored in FIB as MacVRF.
Now when the network association happens, all these routes should be
examined and corresponding elan DS has to be updated.

Change-Id: I53b75890005c67469cab512a3bf5824b6265f06d
Signed-off-by: Vyshakh Krishnan CH <vyshakh.krishnan.c.h@ericsson.com>

bgpmanager VPNv6 shell command update as well as OAM changes

bgp-network command is modified so as to support the ability to
configure prefixes with afi parameter, to be 1 ( IPv4) or 2 ( IPv6).
Moreover, this extension permits retrieving the number of prefixes VPNv6 stored
on quagga, like it has been done for VPNv4

Change-Id: I90a043d5d2a77789b0a5fe376994b201830ad5fa
Signed-off-by: Noel De Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

bug 7599  hwvtep ucast mac add performance improv

Adding batching to improve performance

Change-Id: I2844f7418a8f088064baafe4804a9c1a6b3a8f73
Signed-off-by: Akash Kumar Sahu <a.k.sahu@ericsson.com>

Bug 8014 l2gw connection broken

listener registration is broken in previous async datachange listener
refactoring change.

Adding the remote ucast macs immediately instead of waiting for mcast
to appear in operational datastore.

creating logical switch in config datastore , irrespective of if it is
found in operational datastore or not.

Change-Id: I7916ec4d457e189d28bf48eb3c11836dd6c311f6
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>

EVPN RT2 DMAC flow install

1. When EVPN RT2 route is recieved from peer, BGPManager will write into
macvrfentery to which MAcVrfEntryListener is listening to.
and based on add or delete DMC flow entry is added/deleted.

2. ElanEvpnUtils contains utility methods which access elan related info

3. EvpnFlowUtils contains utility methods which helps
in installing flows in DMAC.

Change-Id: I6c6f1b407d4d209985be964011323019b6965a20
Signed-off-by: Riyazahmed D Talikoti <riyazahmed.d.talikoti@ericsson.com>
(cherry picked from commit cf7e6b3e276e6ea62886945b15c9c20e97b7d954)

Thrift changes to support IPv6 calls over Quagga BGP stack

Quagga BGP stack enhancement to support IPv6 leads to Thrift interface
modifications, which specifically include the following Type changes:
a) new AFI parameter - IPv6
b) pushRoute() includes new AFI parameter
c) withdrawRoute() includes new AFI parameter
d) getRoutes() includes new AFI parameter
e) onUpdatePushRoute() includes new AFI parameter
f) onUpdateWithdrawRoute() includes new AFI parameter
Thrift interface changes to include information on afi type.

Change-Id: Id4aea3809e58910c57e588fa6018119911b57834
Signed-off-by: Noel De Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

BUG-8387: Set tunnel id to 0 on egress towards SFF

When classifier is forwarding packets to the first SFF through a
tunnel, set the tunnel key to 0 as that what the SFF expects.

Change-Id: I02830f46cfe1507463aaabc562ee522646f778d3
Signed-off-by: Jaime Caamaño Ruiz <jaime.caamano.ruiz@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 8370 - Fib flows are not deleted after extra route deletion

Issue:

When the extra routes associated with a router are updated, the FIB flows
corresponding to the old extra routes are not deleted.

Analysis:

While there exists handling to update the loadblancing group corresponding
to the extra route, the handling to remove the FIB flows for the extra
route when the extra route is completely removed is missing.

Fix:

Changes have been added to the deletelocalfibentry to remove the FIB flows
of the removed extra route.

Change-Id: I77b5b9d46173b1c2613c28a495bc6a85bff52d9a
Signed-off-by: gobinath <gobinath@ericsson.com>

Fix ivpnlink bgp route leaking

 + After splitting the InterVpnLink refactor commit into 3 smaller
   ones, this line was missed.

Change-Id: Ic31fde68e18a0cb86948301c7e549fab42561168
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>

Bug 8340 - Rules with IP prefix not getting programmed when there
are multiple SG associated with a port

1)Remote filter table programming during port update is fixed.
2)Ingress and Egress filter  table no is fixed in the  respective
classes.
3)AclInterface Map logic is updated to prevent duplicate entries.
4)Test added to cover interface with intially one acl and later two.

Change-Id: If5af5ced20ed81ec27201bd4ec0def91715ebda9
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>

Refactor InterVpnLink. Part THREE

 + This one implements the final purpose of this refactoring:
   taking the route leaking out of the fibmanager to a higher level,
   in this case it's the BgpManager who now takes care of the route
   leaking in case of BGP routes and the VpnManager for static and
   connected routes.

Change-Id: Ie73a7061a26cfa3b95fad633533d44fff87ad6fb
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

BUG-8331: BGP Manager labels values are passedin correct order to QBGP

L3 Label Parameter is swapped with L2 Label when calling QBGP

Change-Id: Ib5d3260a696078e6b3d1de9cd0b84eaf0ebfef7d
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

BUG 8375 - Malformed NSH packets get dropped

- The NSH NextProtocol (NP) field must be set to 3 for ethernet
- Since the field isnt set, its set to 0 by default, which causes
  the packets to get dropped by the SFF OVS.

Change-Id: I4d13b86ff563b76a992c5c88791015053f5688e8
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

UT - Adding Netvirt classifier GeniusProvider tests

- added GeniusProviderTest.java
- added TestInterfaceManager.java to stub the Genius InterfaceMgr
- added TestOdlInterfaceRpcService.java to stub the Genius RPC calls
- added GeniusProviderTestParams.java with common test params for above 3 files
- changes to GeniusProvider for errors that the UT flushed out

Change-Id: I60e85811959319bf6e163641506aa2f26d9975d5
Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>

BUG:8345 & Bug:8347

   * The code-review contains below Fixes
   BUG:8347 -> NPE at org.opendaylight.netvirt.elan.internal.ElanSmacFlowEventListener.onFlowRemoved
   BUG:8345 -> OptimisticLockFailedException for elanDpnInterface
   Operational Datastore while deleting elan-instance and elan-interfaces

Change-Id: Id930e511f1f0814e17ca1f8cbbda43e9fd3d7e6f
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>