Use right version for statistics pom

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

Fix cherry-pick of 56902

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

Fix cherry-pick of 56875

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

Fix cherry-pick of 58749

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

Fix funny character in ebgp.yang (slanted quotation mark)

Change-Id: If8b5aa13c1884685f0d283ad2e622be42aa29f22
Signed-off-by: Josh <jhershbe@redhat.com>
(cherry picked from commit e9421c8ae44ce642277d37f80c6d4f00f3eb8011)

Adding Log statements, helps during debugging

Change-Id: Iaff8515aeff1fed46b4552029c3520ed79015826
Signed-off-by: kiranvasudeva <kirankumar.v@altencalsoftlabs.com>

Bug 8800: Fix AclServiceStatefulTest newInterfaceWithMultipleAcl()

The expected objects of that tests were broken, but due to Bug 8800, the
test did not fail because of that.  (This needs to be reviewed and
merged before the actual root cause fix for Bug 8800 can go it.)

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

Bug 8838 - aclservice NPE's

Fixes NPE in programConntrackRecircRules in Egress and Ingress Acl
serivce.

Interface is now deleted from the cache after notify is called.

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

Bug 8838 - aclservice NPE's

Ignoring notification for ports which are not present in AclInterface
cache.

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

Fix NullPointerException in QosInterfaceStateChangeListener

2017-06-27 18:29:37,596 | ERROR | eChangeHandler-0 |
QosInterfaceStateChangeListener  | 344 -
org.opendaylight.netvirt.qosservice-impl - 0.4.1.SNAPSHOT |
Qos:Exception caught in Interface Operational State Up event

java.lang.NullPointerException: the Function passed to
Optional.transform() must not return null.
        at
com.google.common.base.Preconditions.checkNotNull(Preconditions.java:226)[38:com.google.guava:18.0.0]
        at
com.google.common.base.Present.transform(Present.java:71)[38:com.google.guava:18.0.0]
        at
org.opendaylight.netvirt.qosservice.QosInterfaceStateChangeListener.getNeutronPort(QosInterfaceStateChangeListener.java:117)[344:org.opendaylight.netvirt.qosservice-impl:0.4.1.SNAPSHOT]

Change-Id: Ic2a6750ac605dedeb61572689f62a97c811b8fa2
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
(cherry picked from commit a32ac7563dda41f0b581b4e93826bee7ff4bee20)

Use manual setters instead @Immutable in IdentifiedAceBuilder.xtend

The mix of Immutables.org with Xtend.org seemed cool when I originally
came up with this 1 year ago, but after having run into issues with
in-IDE compilation order in Eclipse EVERY TIME I want to touch something
in aclservice, this got enough on my nerves to finally simplify this.

The issue basically is that the Xtend IDE and Eclipse APT processor seem
to struggle about the correct order in which Xtend active annotations
and APT processors should run (although it did, always, work, reliably,
in the mvn CLI build); Eclipse IDE could never get this right, and there
was too often red ink errors for (supposedly) missing
ImmutableIdentifiedAceBuilder... so basically this mix of Xtend with
Immutables.org probably wasn't a good idea.

The Xtend active annotations @Accessor (as well as @Data) really do
something quite similar to Immutables.org (yet let us keep the "pretty"
object literal initialization syntax of Xtend); however they generate
"real" setters, which do not "return this;", so instead it's simplest to
just hand write those few setters, for now.  Later if this pattern is
used more widely, we could have a new Xtend active annotation, such as
@Builder.

This also, and strictly speaking kind of unrelated, changes the double
indirection of Builder -> Immutable -> DataObject to a simpler to
understand single level, without the intermediate Immutable (which makes
these Builders used to compose objects more easily in tests not thread
safe, which is perfectly fine, actually).

dito for IdentifiedInterfaceWithAclBuilder

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

Spec to support dualstack VMs in L3VPN

This specification brings the ability for a BGPVPN configured VPN to
distribute IPv4 and IPv6 addresses within the same VPN, and by using two
distinct routers to respectively handle IPv4 and IPv6 routing issues.

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

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

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

Bug 8793: Null Pointer Exceptions in Carbon CSIT

This commit fixes NPE observed in bindAcl() of AclService

Change-Id: I7397677f0e63a56e0e023eb4b2697ae1ef095714
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Bug 8793: Null Pointer Exceptions in Carbon CSIT

Fixes NPE in  programConntrackRecircRules and syncSpecificAclFlow in
AclService

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

Bug 8793: Null Pointer Exceptions in Carbon CSIT

Many NPEs are seen in Carbon CSIT on ACLService
and NAT Service.

This review fixes some of them.

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

Adding logs in ACL.

Adding logs in ACL which can be helpful for debugging.

Change-Id: I29b9ed92cedd06e4c325117e4419b01e190dc217
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

Fix NPE in aclService test

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

Fix clear bug in BgpConfigurationManager's interVpnLinkService

The assignment to the field is just self-assigning the current value of
that field (null), not the injected argument, due to a typo... :-(

This could never have worked - isn't this kind of coded tested?! ;)

Found chance related to a closer look in the context of
https://git.opendaylight.org/gerrit/#/c/59205/.  We really should be
working on enabling FindBugs (which, probably, would have spotted this)
on more projects in netvirt... is anyone volunteering to take this on,
if I can provide some coaching and would do reviews?

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

Revert "Neutron Port allocation for DHCP Service"

This reverts commit 30cba431cf065d25897dffe01cd5c4febc63e14a.

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

BUG 7824: Handling Optimistic lock exception for elanmodule

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

Remove sfc-translator MdsalUtils::put()

Replace MdsalUtils::put() with
SingleTransactionDataBroker::syncWrite(). MdsalUtils is no longer used
at all and can be removed.

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

Remove sfc-translator MdsalUtils::merge()

Replace MdsalUtils::merge() with
SingleTransactionDataBroker::syncUpdate().

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

Remove sfc-translator MdsalUtils::delete()

Replace MdsalUtils::delete() with
SingleTransactionDataBroker::syncDelete().

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

Remove sfc-translator MdsalUtils::read()

Replace MdsalUtils::read() with
SingleTransactionDataBroker::syncReadOptionalAndTreatReadFailedExceptionAsAbsentOptional()
(which is itself deprecated, but I didn’t want to redo the
error-handling in this patch).

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

Remove NatUtil::read()

As a step in the MD-SAL utilities clean-up, remove NatUtil::read(),
simply replacing it with
SingleTransactionDataBroker::syncReadOptionalAndTreatReadFailedExceptionAsAbsentOptional()
(which is itself deprecated, but I didn’t want to redo the
error-handling in this patch).

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

Use documenting constants for put()

WriteTransaction provides constants for put()’s createMissingParents
parameter, this patches uses them throughout.

A few transaction-related collection constructions are replaced with
singletonList().

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

Remove old NetVirt

This is no longer used, let’s just remove it.

Change-Id: Idc36a582a39279caace8981e235a3ebb426a6af4
Signed-off-by: Stephen Kitt <skitt@redhat.com>
(cherry picked from commit 6ce3e7cc4ad11e10c33d62497e32ea5303724972)

Fix NPE triggered after disabling SG on a port

When port-security is disabled on a Neutron port, removeAclInterfaceMap
is invoked, but the entry in aclInterfaceMap is not deleted as there is
a difference (i.e., securityGroups element) in the port info present in
aclInterfaceMap and the one passed as argument. Because of this, the stale
entries remain in the aclInterfaceMap and are causing NPE issues for
subsequent operations. This patch fixes this issue by matching on the
interfaceId and deleting the corresponding entry.

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

Bug 8398 - communication between two tenant network failed while using
default SG (stateful mode)

This commit fixes issues observed in
https://git.opendaylight.org/gerrit/#/c/57705/

Depends-On: 0c01eacb8100ee5a224d52e44362fce98bd875f5
Change-Id: I827e99835c36509bad5b48f03034103d377102f8
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Bug 8398 - communication between two tenant network failed while
using default SG (stateful mode)

This fixes the remote ACL indirection flows for the case where a
router/bgpvpn is involved in communication. It enables the dispatcher
tables to write vpnId to the metadata so that Remote ACL flows can
correctly match and send the packet to the intended destination.

The fix makes use of the vpn footprint service to program ACL flows for
every host addition/deletion.

Depends-On: b3e21d27e8f593b4ca326dd93239db38e1ead4cd
Change-Id: I1cbee44f951a75d4bbb382fed4e882162016c946
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

Bug 8645: Alarm not cleared while deleting the bgp neighbor

Clearing the alarm during the deletion of bgp neighbor.

Change-Id: I0692b18c3850536ffac73decc0d6b4d23523e963
Signed-off-by: Gowri R <gowri.r@ericsson.com>

bug 7599  hwvtep ucast mac add performance improv

Batching changes for Dmac flows.
Making sure all macs installed at lower case to resolve duplicate mac
entries issue.
Deleting entries related to logicalSwitch in case of
l2GwConnectionDeletion.(More explanation in inline comment)
Fixing minor bug in HA (Only parent nodes needs to be updated in cache not
child node hence returning)
ElanInstanceListner has been updated to cluster listner as in case of
cluster reboot / or other situation cache should be cleared form all
nodes.

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

VrfEntryListener refactoring using chain of responsibility pattern

This is a secod review where RouteInterface handling is separated into a
new handler class RouterInterfaceVrfEntryHandler.

Change-Id: If4bc6ded02e23dc12f6217a7c412ac5cc1188f50
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>

Neutron Port allocation for DHCP Service

The feature enables DHCP service to make use of the
newly created Neutron ports provisioned by Openstack
networking-odl driver at the time of Subnet creation.
Additionaly an ARPResponder flow is also introduced to
service the ARP requests for these ports during VM
DHCP renewal.

All ArpResponder handling for Router interface gateway,
floating IP and DHCP ports will be moved to ElanService,
introducing a pipeline change for ARP handling.

LPort Dispatcher Table(17) -> ARP Check Table(43) -> ARPResponder Group
-> ELAN Base Table(48), ARP Responder Table(81), Controller
-> Egress Dispatcher Tabel (220)

Change-Id: I8ebbe05c37f5e7e4c5ae1dd1d17faa12965d5e65
Depends-On: I2dce5e253d3c94180dd18e51125c5c18170b5c69
Signed-off-by: Achuth Maniyedath <achuth.m@altencalsoftlabs.com>
Signed-off-by: Vijayalakshmi Chickkamenahalli Nagaraju <vijayalakshmi.c@altencalsoftlabs.com>
Signed-off-by: karthik.p <karthik.p@altencalsoftlabs.com>
Signed-off-by: Achuth Maniyedath <achuth.m@altencalsoftlabs.com>

Bug 7939 - CSIT Sporadic failures - Flow(s) missing in VPNService suite on
compute node

Fix ensures that table-19 is completely cleaned up on swap, before add the
vpn and router interfaces to the bgpvpn/routervpn instance

Change-Id: I9b106492234caa80ba9b2d961bb05a728cf5af27
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

BUG 8553: Rules are not deleted/added to the SG.

  Problem: Event not processed when we delete last rule
  and also addition of first rule to the SG.

  Fix: Skip only if both before and after ACL list are empty

Change-Id: I7f8ccb10adbdb1b8c1cab0ca7af8a2dcc2105aea
Signed-off-by: arthi.b <arthi.b@hcl.com>

Bug 8743 : Fix NAT Service to safely use vpn-to-dpn list.

Description : Its been observed the vpn-to-dpn list been not updated
properly resulting in missing dpn-state with active state. As a result,
installation of default route rule been missing resulting in SNAT/DNAT
failure.

Changes are done to make use of vpn footprint service to update the
vpn-to-dpn list whenever floating-ip and external fixed ips are added
and deleted.

Change-Id: Ic55fc5364ef45fbbe7ce63a9feef51bf39332541
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>
Signed-off-by: cgowdru <chetan.arakere@altencalsoftlabs.com>

Move fibmanager-shell blueprint.xml from OSGI-INF/blueprint to org/opendaylight/blueprint

This may (?) fix the SFT failing to find the DataBroker, as discussed on follow-up to
https://lists.opendaylight.org/pipermail/netvirt-dev/2017-June/004770.html; and if
this does fix that SFT problem, then this was an indirect impact of
https://git.opendaylight.org/gerrit/#/c/54885/.

This also reverts both https://git.opendaylight.org/gerrit/#/c/59105/ private field,
and the availability="optional" from https://git.opendaylight.org/gerrit/#/c/59051/

Change-Id: I856c0f4311a02a5380512496619aee74cf70ee2e
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
(cherry picked from commit b0a5086f16acf9d9bd5a38eea1a0715625744aa0)

Netvirt wide blueprint xml path change: /OSGI-INF/ -> /org/opendaylight/

Also have removed fake "availability=optional"

Change-Id: Iae7caa155eeffa7c831822ac600bbc6283a7e2fc
Signed-off-by: Abhinav Gupta <abhinav.gupta@ericsson.com>

Bug 8179 - Data validation failed for path bgp/networks

Issue:

Data validation failed exception during withdrawing prefix from BGP.

Analysis:

The issue occured while trying to delete an element from the
bgp/networks which was not in the desired format. The DPNid is
used as rd in case of extra routes with internal VPN. The prefixes in
the internal VPN are not be advertised to the BGP. The condition for
this already exists(rd is the vpnName here). For extra routes, DPNIDs
are used as rds for ECMP and they should also be advertised. This was
already handled but there was no handling to avoid the withdrawal from
the BGP in case of the removal.

Fix:

Condition has been added to avoid calling withdrawPrefix() for the extra
routes removal in case of the internal VPN.

Note: Also some other errors in logging have been fixed.

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

Bug 7824: Handling CSIT Exception under Elan module

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

Bug 8710:  Bgp afi related bug fixes

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

VrfEntryListener refactoring using chain of responsibility pattern

This is a first review where BGP Route handling is separated into a new
handler class BgpRouteVrfEntryHandler.

All common methods are put into a new superclass i.e VrfEntryHandler. Each
of the subclasses like BgPRouteVrfEntryHandler will extend the superclass.
This is done to avoid circular depency on VrfEntryListener.

As part of chain of responsibility pattern , VrfEntryListener will pass on
the Vrf object to the first handler in a chain based on the RouteOrigin
type. Once the first handler completes processing the Vrf obj , it will
pass on the same to next handler.

In the current review BgpRouterVrfEntryHandler will process Routes of type
"RouteOrigin.BGP". BGP routes need not be processed by any other handler ,
hence next handler is not defined in the chain.

In the subsequent patches , when new handlers like
LocalRouteVrfEntryHandler, SubnetRouteVrfEntryHandler etc will have to
pass on the Vrf obj to next handler which will be
ImportExportVrfEntryHandler.

Change-Id:  I538085bfcda7733b5f3ba5ffe49d541619ac40ec
Signed-off-by: ehvkand <hanamantagoud.v.kandagal@ericsson.com>

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>