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: If1c787c50bd3054766caaa6b11e6c8023402cce4
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>

Bug 8960: port information for 1 dpn is missing in subnet-op-data and
port-op-data

Upon PL reboot , if southbound event (PortUP) event is received first and
northbound event(OnSubnetAddedToVpn) is received later ,DPN will not be
updated in PortOpData.

Fix :
(1) When northbound event is received , SubnetOpDataEntry is created first
by using clustered wide lock based on subnetId.
Then PortOpDataEntry and subnetToDpn is created.

(2)Incase southbound event is received first,PortOpData entry may not be
available.In such case subnetId is retrieved from Neutron cache and
PortOpData entry will be created by the SubnetInterfaceStateChangeListener
thread.

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

Bump odlparent 2.0.2 to 2.0.4

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

Bug 8863: NPE at VpnFloatingIpHandler

Problem Description:
=======================
The following exception was observed from VpnFloatingIpHandler in CSIT

Error in generate label or fib install process
java.lang.NullPointerException: Supplied value may not be null
org.opendaylight.netvirt.natservice.internal.VpnFloatingIpHandler.lambda$onAddFloatingIp$0(VpnFloatingIpHandler.java:194)[307:org.opendaylight.netvirt.natservice-impl:0.5.0.SNAPSHOT]

Solution:
=========
We need to enhance the pre-condition check in VpnFloatingIpHandler before
proceeding with any DNAT flow installation.

Have added the required null check for floatingIp ADD/REMOVE methods.

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

Bug 8962: Fix non-parameterized LOG statements as per guidelines

Non parameterized LOG statement are suspected to take more CPU cycles than
parameterized one. Hence all non-parameterized logs are enhanced.

Also some more log enhancement for better debuggability.

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

Docs: add supported combinations

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

Bug 8958: Java Null pointer exception for display vpn-config after
deleting a vpn from CLI

When no VPN configuration is present in CONFIG datastore , success was
being returned in GetL3VPNOutput RPC. This lead to null ptr exception.

Now error message is being returned in RPC which is handled appropriately
and null ptr exception is avoided.

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

Bug 8882 - With conntrack SNAT communication with PNF fails
Changed set_field to load action and set only the relevent bits and
other bits are preserved.

Depends on https://git.opendaylight.org/gerrit/#/c/61378/

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

Bug 8937 : High CPU utilization of Java process due to SNAT packet looping

Description:
If we have 2 VMs with same IP belong to two different network and the
subnet associated with different routers having same external network and
external bgpvpn. In such as case, when traffic initiated using same
internal port some both the VMs, one of the VMs doesn't undergo SNAT
translation and the packet gets continously looped between Controller and OVS.

When the packet is punted to controller, check is only done if the packet
is already recevied for an given internal-ip and internal-port. As a
result, VM1 packet is processed properly and SNAT translation happens. But
when VM2 packet arrives, we see with internal-ip and internal-port already
processed and hence we sent back to OVS without SNAT translation which
resulted in looping.

Change done to have a check of router-id along with internal-ip and
internal-port to uniquely differentiate VM1 and VM2 traffic seperatly and
perform SNAT translation properly for both.

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

ELAN service is not unbound when nova delete followed by neutron port delete

* Fix added to listen on interface config remove and invoke unbind service
* Changed elan interface job key while submitting to jc

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

BUG 8883 : LOG enhancement for NAT service module

1) In most of the logs, "NAT Service" is used as a prefix. Since logs
already has the package Name, it would be better to print method name
instead of prefix.

2) Many logs have have string concatenated instead of passing as
parameters in these statement.

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

Bug 7545 - FIP-FIP traffic
between vm in same n/w in same compute is not working in stateful SG
mode.

The patch fixes the FIP-FIP use case.
Add a dummy table to do a ct_lookup if the packet was already send to
the netfilter from the pipeline.

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

BUG 8876 - Fix to remove ELAN interface flows when external network deleted

    - Problem: when deleting external network, the underlying ietf
      Interface might have been removed from the config DS prior
      to the call to delete the ELAN interface. As a result, the call
      interfaceManager.getInterfaceInfo returns null and the
      subsequent call to delete the ELAN interface fails to delete the
      flow entries for external network's ELAN interface.

    - Fix: invoke interfaceManager.getInterfaceInfoFromOperationalDataStore
      when delete External Network's ELAN interface

Change-Id: I764f3516d34a6d79aacff8dbedabb55219799ac9
Signed-off-by: Vinh Nguyen <vinh.nguyen@hcl.com>

Bug 8936: Fix IPv6 IPAM issues when an IPv6 subnet is added to IPv4 network

This patch addresses the following issues in ipv6service.

1. In an existing IPv4 network (which has a DHCP Port and some VMs)
   when you add an IPv6 subnet, the VMs/DHCP Port is not configured
   with the IPv6 address.
2. Similarly, in a multi-node setup when an IPv6 subnet is added to
   an existing IPv4 network and associated to the router, sometimes
   the necessary punt flows in Table_45 are not configured.

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

Revert "Bug 8882 - With conntrack SNAT communication with PNF fails "

This reverts commit 0a1048ddc55621b2b675b7cef3e1b6c808693af1.

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

Bug 8789 - Designated DHCP DPN is missing.

When l2gw connection is creaated and first dpn interface creation
happens at about same time there is a race in dhcp designated listener
which thinks that it has to update mcast only if there are no dpns in
elan.

when an elan dpn is removed, the dpns tep ip should not be removed from
mcast mac if that tep ip is the designated dpn tep ip.

when l2gw is deleted clear its connections also.

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

Add chain egress classifier support

On scenarios where origin and destination of traffic are on the same
node, there is no tunnel id metadata to help steer the traffic back
to the destination on chain egress, thus SFC cannot take care of
this by itself. In a general case, the classifier is in a better
position to steer traffic to destination as it is aware on how it
was intercepted on the first place.

For netvirt classifier and the scenario mentioned above, the
classifier will copy the REG6 to NSH C4 on interception. On chain
egress, SFC will send the encapsulated packet back to the origin node
(only when C2 is 0, which will only happen when origin and destination
node are the same). There, the classifier will restore the
de-encapsulated packet to the egress dispatcher table with the proper
REG6 value taken from NSH C4. REG6 contains the the egress ifindex plus
the nex egress service in sequence from the classifier egress service
to handle the packet. This effectively restore the packet processing
where it left off on interception.

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

Fix some logs

Fix some logs.

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

Replace size()==0 by isEmpty()

Replace size()==0 by isEmpty(). It improves readability and avoid "=0".

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

Bug 8882 - With conntrack SNAT communication with PNF fails

Change set_field to load action and set only the relevent bits. Thus the
lport tag is preserved.

Depends on [1] https://git.opendaylight.org/gerrit/#/c/60870

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

BUG 8914: Fix NPE in sfc.classifier-impl

NPE was due ACLs with no RedirectToSfc action. Such ACLs should be
ignored.

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

Bug 8346 - Conflicting modification for vpnNextHops.

The VpnInstanceListener used to wait for the vpn Interface Operational
data to clean up, before cleaning up data stores related to it.
There was a max wait time calculate, but some time interface op cleanup is
exceeding this wait time. As a result, the vpn instance related cleanup is
done before or at the same time as the vpn interface op cleanup,
resulting in these exceptions.
This fix remove the timed wait for the vpn instance, and cleans up the vpn
data based on notifications fired when both a vpn instance delete is
orchestrated and its footprint is cleared

Change-Id: I54d6369f30aa6d2a0e98df56b80cfcd6b4295978
Signed-off-by: eupakir <kiran.n.upadhyaya@ericsson.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Cleanup

- Replace ArraList by List
- Add override
- Declaration of constants
- Replace size() by isEmpty()

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

Bug 8917 - CSIT Sporadic failures - Arp learning suite - missing flow on
compute node

Remote FIB Flow installation was failing due to OptimisticLockException.
The DJCs involved in this operation is now modified in the L3 Application
to retry on failure.

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

Bug 8875 - fix in handleNeutronPortUpdated

This fixes commit "Bug 6770 - Fixes DjC for NPortCL + snmaps serialized
+ listed changes" 19af79aecc2f217c01faeda721606a77bd1b2504.

NeutronPortChangeListener: handleNeutronPortUpdated method receives
portoriginal and portupdate objects as arguments. So, to check, if
neutron port was really updated, compare portoriginal and portupdate
SubnetIDs.

Change-Id: I7509c61bfa1135e21fa8e0a49ca8e0f1c68d4c72
Signed-off-by: Valentina Krasnobaeva <valentina.krasnobaeva@6wind.com>
Acked-by: Philippe Guibert <philippe.guibert@6wind.com>

Bug 8439: Handling interface update event for ECMP extra routes.

* Up/Down event from Interface state was not handled before this fix.
  Only add and delete event were handled

* With this fix, up/down events are handled. These events are processed
  similar like add/delete events. Vpntoextraroute, extraroute-rds and
  fibentries are updated accordingly which will result in the bucket
  removal/addition from the select group. For the last extraroute of a
  particular prefix, the corresponding flow(table 21) and select group
  are removed

Change-Id: Ieb205ff9e626c67c888ecdf3d5bfbcc679fdeba8
Signed-off-by: eceghkl <manu.b@ericsson.com>

Bug 8884: SNAT traffic is getting dropped when router gateway is removed

Problem Description:
====================
Issue 1
+++++++

SNAT traffic is getting dropped for second router when gateway is removed
from first router.

Consider the following topology.

1) 2 Internal Network + 2 Subnets
2) 2 Routers with each subnet associated to it.
3) One External network and both router associated to same External
Network.
4) One External BGPVN.

when both Routers, if the same NAPT switch is elected, it will have a
common NAPT PFIB TABLE which forwards the outgoing packet. In such
scenario, when the gateway is removed for one of the routers,
this flow gets removed effecting the other routers traffic.

Solution for Issue 1:
+++++++++++++++++++++
Changes done to prevent this NAPT PFIB table flow(47->21) deletion if
there exists
any other router having same DPN as NAPT Switch.

Issue 2:
++++++++
Label of external-fixed-ip is freed from vpnservice pool when NAPT
switchover happens for SNAT.

Solution for Issue 2:
+++++++++++++++++++++
Changes done to prevent release of label allocated for external-fixed-ip
when Switch-Over Happens.

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

Bug 8632: ECMP LB group not updated during tunnel events

* Tunnel State is read from interface operational DS before installing
  select group
* A few NPE also fixed in this review

Change-Id: Id61c20059fcf4dff16f7590b0c439330bb2c46c3
Signed-off-by: eceghkl <manu.b@ericsson.com>

Revert "Revert "Bug 8864: DNAT to SNAT traffic is getting failed on same DPN for VXLAN""

This reverts commit d81c93c38d9cf09236a735f9f18e49822bead43b.

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

Remove not needed Thread.sleep() from aclservice component tests

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

Bug 8838: aclservice NPE's

This commit fixes NPE observed in processInterfaceUpdate().
Similar fix added in applyAce() and removeAce().
Also, unwanted check taken out from updateRemoteAclTableForMultipleAcls().

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

Revert "Bug 8864: DNAT to SNAT traffic is getting failed on same DPN for VXLAN"

This reverts commit e88336d0b929975f84beb448f5fd08e9233dcf9f.

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

BUG 8476 Fix checkstyle error

Fix checkstyle error

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

Bug 8476 Add support for logical switch replication mode

With OVS 2.6, the Logical Switch table, in the hardware_vtep schema, has
a new field, replication_mode. This bug will be used to add support to
netvirt, ovsdb, and genius for setting the replication_mode of logical
switches. The default will be for ODL to set the replication_mode
to "source_node" when creating logical switches.

Refer to [1] for more details on replication_mode.

[1] http://openvswitch.org/support/dist-docs/vtep.5.html

This patch depends on genius patch:
https://git.opendaylight.org/gerrit/#/c/57210/

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

Bug 8892 - LOG.error format error(VpnManagerImpl.java:176)

Change-Id: Ic1c88364cb04a5226f4cf3d7d32c8125db5508f7
Signed-off-by: Freeingo Sure <xiao.fenggang@zte.com.cn>

Bug 8864: DNAT to SNAT traffic is getting failed on same DPN for VXLAN

Problem Description:
=========================
DNAT to SNAT traffic is getting failed on same hypervisor for External
VXLAN Provider Type

Solution:
===========
Have added the follwing flow install/remove code to handle this problem
for external VXLAN provider type (EVPN_RT5)

Install the flow L3_FIB_TABLE (table=21)-> INBOUND_NAPT_TABLE (table=44)
(FIP VM on DPN1 is responding back to external fixed Ip on DPN1 itself.
ie. same Hypervisor){DNAT to SNAT Intra DC traffic}

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

BUG 8880: Trunk port flows not created for subports

Fix for BUG 8020 broke the assumption for trunk sub-ports i.e. Interface
will be already created for Subports at time of NeutronPort creation.

https://git.opendaylight.org/gerrit/#/c/54410/ adds an if condition that
will always be false for VM ports, so we don't create interface at Port
creation. This breaks trunk logic. This also has side effect of increasing
rae conditions by adding too much to do once VM is actually spawned and is
not a good design.

Change-Id: I90e5c60f6d5afe5242cdd0a93d5c35d00a25ab85
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Remove derivation from controller config-parent

Netvirt does not use the config subsystem so does not need to
derive from controller config-parent. I removed the local
config-parent and changed poms to derive from the local
binding-parent.

Change-Id: Ib6ab82d43a021247808f17cefb8d836e5b00b36e
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 8853 - In conntrack SNAT , FIB flows are not created for existing
Non-NAPT switch ports.

When a port is added to the VpnDpnList it is not checked whether it is a
first/last port and handle the fib flows.

VpnFootprint service Api is now used to handle the last port and
add/delete the fib flows.

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

Bug 8753 : Changes to create bgpvpn without VPN target

When bgpvpn is created without VPN targets, it throws
an error as "GetVPN failed due to null" in VPN config.

This is fixed by adding a check of, if VPN target is
null, then also create VPN by avoiding the exception.

Change-Id: I08179cc7a7e9013be7008e0514b98e9e0edc3bd9
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

Bug 8841 : add fib-show option to select entries per address-family or prefix

An enhancement of fib-show is done here to show optionally prefixes
based on address-families or/and prefix/subnet value given in input.
Available values for address-families are : ipv4, l2vpn, ipv6.
Available values for prefix/subnet are IPv4 or IPv6 entries, with or without
prefix length.

Change-Id: I8d1c61149bae7442221dea5b0d04aae0de59f9b5
Signed-off-by: Noel de Prandières <prandieres@6wind.com>

Bug 8683: Aclservice releaseId IdManager Exception

Issue: When VM port is deleted, ACL service calls IDManager to release ID
which in turn throws exception that Id Entries does not exist.

Root cause:
When OVS is restarted, then ACL pools are deleted and created freshly
due to which IDs allocated to SG flows of VM port before OVS restart are
not available in IdManager ACL pools after OVS restart.

Fix:
On OVS restart, Do not delete ACL pools for OVS if VM ports are associated with OVS.

Change-Id: I0a0d36d4d8f25d7157d73292f9a027f67fe6dc0c
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>
(cherry picked from commit f2ddba6f5526e232125180185170e84319d97f7a)

Move statistics into org.opendaylight.netvirt

Now that the BluePrint plugin only considers org.opendaylight.netvirt,
we need to move all NetVirt’s packages into that namespace.

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

Clean up lambdas and streams

* Use method references where appropriate.
* Use lambdas instead of anonymous classes where appropriate.
* Simplify lambdas as far as possible.
* Use forEach() directly on collections instead of streaming them.
* Simplify streaming matchers (!anyMatch -> noneMatch in particular).

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

Revert "temp rm netvirt-sfc feature while sfc is busted"

This reverts commit 17fd01cfd7fa6839035767e3123dd2fdbe476136.

Revert "temp rm netvirt-sfc feature while sfc is busted"

This reverts commit 112ea539ae0ca3b355f15ca0944819b904f1bb5a.

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

Bug 8857 - SNAT Conntrack - VM MAC is used as Source MAC for all
outbound traffic

The vm mac is replaced with router mac address

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

Bug 8810 : BGP Manager / support for EVPN on OAM submodule missing

Despite EVPN is supported on VPNs, there is no statistics coming from
BGP. The changes include the following:
- evpn summary information ( list of neighbors speaking epvn)
- evpn total list of entries

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

Bug 8809 : BGP Manager / neighbor summary for VPNv6 wrongly parsed

Instead of filtering the BGP neighbor IP with IPv6 address, the
filtering is done against IPv4 neighbor addresses. There is no need to
match with IPv6 neighbors, since it is related to underlays tunnels, and
not overlays information like VPNv6.

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

Bug 8811 : BGP Manager / bgp-nbr shell config with afi vpnv6 / evpn

This commit brings some changes related to VPNv6 and EVPN capabilities,
since it is becoming possible to create neighbours with  VPNv6 or EVPN.

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

Bug 8818 : BGP Manager. VPNv6 prefixes injected to QBGP

This issue is related to VPNv6 prefixes under the format prefix/length.
Indeed, the prefix/length format is not recognized by bgp manager when
analysing the prefix. Then it fails and considers it as IPv4.
Subsequently, an inconsistency is detected by QBGP, and it is rejected.
The fix consists in striping the length part so that the prefix can be
analysed correctly.

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

BUG 8868: Override the Aries Blueprint Maven plugin configuration

Override the Aries Blueprint Maven plugin configuration inherited from
odlparent to specify a more restrictive scanPath avoiding the generation
of spurius references in the autowire.xml files that force the SFT to
fail.

This fix will allow [1] to pass enabling again the SFC feature in
Netvirt.

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

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

Bug 8822 : Bgp Manager / bgp-cache dumps unknown address-families

The fix consists in parsing the various afi/safis of neighbour contained
in address-family list, and identify as VPNv6 or EVPN address-families.

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

Bug 8770: Stale NAT entries are not getting removed for external FLAT

Problem Description:
=====================
Stale NAT entries are not getting removed for external FLAT/VLAN network
on following flows

Stale FIB Entry =>L3_FIB_TABLE(21)-> INBOUND_NAPT_TABLE(44)
Stale Tunnel Entry =>INTERNAL_TUNNEL_TABLE(36)-> OUTBOUND_NAPT_TABLE(46)
Stale LFIB Entry =>NAPT_PFIB_TABLE(47)->L3_FIB_TABLE(21)

Solution:
=============
As part of hairpinning feature support, external vpn name was generated
from external-subnet Id instead of external-network Id. To accomplish this
changes have done the changes in all the places to install and remove the
NAT related flows for external FLAT/VLAN use cases.

Change-Id: I0bd0c79ca638cf6664eaa4237334c6524f2d2c1f
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.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>

natservice-impl: propagate upstream non-null annotations

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

Add a private constructor to this utility class

Add a private constructor to this utility class so that it cannot be
instantiated.

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

Bug 8835: Java NullPointerException in display-bgp-config command

Change-Id: I8df4de8620c1147d4d9f30152339e2db840f5787
Signed-off-by: Siva Kumar Perumalla <sivakumar.perumalla@ericsson.com>

Bug 8567: Addition of new flows after addng extra route

  When the extra routes are configured on VMs, ping between
  them works and these extra route IP entries appear in list
  of adjacencies and new flows are added in table 21.

  This is fixed by adding a check of, if an extra route IP
  is present in any of the adjacencies' listed subnets,
  then do not execute ProcessArpLearning.

Change-Id: I49f691b48b7f838bdc42d14ee81daff1e96d7e16
Signed-off-by: eswanit <swati.udhavrao.niture@ericsson.com>

New Yang model container for Neutron DHCP Port service.
Updated spec with correct yang.

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

Enforce non-null collection returns in NatUtil

Collection-returning methods shouldn’t return null; this fixes
NatUtils appropriately and propagates the non-null guarantees.

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

Bug 8844: CSIT Job: NullPointerException from NAT feature

Problem Description:
====================
Retrieving router-name from router-id NAT DS
"odl-nat/router-id-name/routerIds" sometimes returning with empty as
resulting Nullpointer exceptions are been thrown from NAT module. Since
there is no check to handle for null values.

Solution:
============
As part of fixing this Nullpointer exception, we have added the
appropriate check to validate the null values in cll the aller methods.

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

SNAT performance improvement for Controller-Based SNAT

Description : Updated log statements and reduced number of retry
threads required for processing.

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

Revert "Ship aaa-cli-jar.jar in the distribution"

This reverts commit c73a81ee42819e885c9f90ecf02d078649ab182e.

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

temp rm netvirt-sfc feature while sfc is busted

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

Bug 8796: Fix of issue that vxlan port is created with remote_ip set to the node itself

Issue:
  If start L2GW node (Open vSwitch HWVTEP emulator) without setting HA
  configuration and create L2GW GATEWAY and L2GW CONNECTION, after
  update (set) HA configuration for this L2GW Node, vxlan port which
  its remote_ip is set to the node itself will be created.

Cause of this issue:
  If there are multiple tep ips of the node itself in the tep ip list
  which is generated from L2GW devices information,
  the information of the node itself will be set in RemoteMcastMac.

Fix:
  Do not add the node information to tep ip list if the same tep ip
  exists in the list which is generated from L2GW devices information.

Change-Id: I8ca27a32248a9439a1f9d56077d6071fe4de30ac
Signed-off-by: kaoru sueda <k-sueda@zj.nes.jp.nec.com>

Ship aaa-cli-jar.jar in the distribution

This is a port of https://git.opendaylight.org/gerrit/49278 and
https://git.opendaylight.org/gerrit/54388

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

Bug 8791 - L2gateway delete is not clearing l2gwCo

L2gw delete was not clearing associated L2gateway Connection.
CLearing associated l2gwConnection as part of the review.

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

Bug 8790 - Local Macs getting cleared

Local Macs are getting cleared from HA parent node , in case only one of
the HA child node is diconnected.
It should be cleared if both the HA nodes are disconnected .

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

Fix dpnId handling in SRISCListener

In SubnetRouteInterfaceStateChangeListener::add, we call
vpnSubnetRouteHandler.onInterfaceUp() whether we get a dpnId or not.
But onInterfaceUp() immediately returns if dpnId is BigInteger.ZERO,
so there’s no point in doing so, and having both calls in the same try
blocks makes the whole process clearer.

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

Use right version for statistics pom

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

temp rm netvirt-sfc feature while sfc is busted

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

Fix cherry-pick of 59691

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

BUG 8828: Fix NPE when no remote IP on interface

Fix NPE When new netvirt classifier fetches the remote IP address of an
egress interface that does not have one (for example, when it is not
a tunnel interface).

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

Filter notifications for unwanted interfaces

Change-Id: I3e6a82c97bdd8fd467cf5f39cd59fdb88c121223
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Use right version for statistics pom

Change-Id: I71099836cb81f55c9a590a565b4d244e7ed47d3a
Signed-off-by: Sam Hague <shague@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 cherry-pick of 58749

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

Fix cherry-pick of 56875

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

ElanServiceTestModule using parent's bindTypesToInstance

moved up in https://git.opendaylight.org/gerrit/#/c/58094/

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

Bug 8677: Bump odlparent from 2.0.0 to 2.0.2

This lets Windows users build Karaf distributions
locally more easily (without requiring Git Bash/Cygwin).

see https://lists.opendaylight.org/pipermail/odlparent-dev/2017-June/001154.html

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

Fix ActionNxResubmit in FlowEntryObjectsStateless for aclservice

This fixed a test failure I'm seeing locally in aclservice's
AclServiceStatelessTest>AclServiceTestBase.newInterface
(and which should fail on Jenkins as well; if it does not, I do not
understand how it can work on the build; we should find out more why).

The test failure message shown is confusing; with
https://git.opendaylight.org/gerrit/#/c/60219/ it becomes much clearer.

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

Adding Log statements, helps during debugging

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

Remove direct usage of OpenFlowJava models

Remove direct usage of OpenFlowJava models in OpenFlow13Utils and
replace them with usage of OpenFlowPlugin models.

Resolves: bug 8808
See also: bug 8747

Change-Id: I43cc4e17172a53f827b62697804b69f958d3dae1
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

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

Change-Id: If8b5aa13c1884685f0d283ad2e622be42aa29f22
Signed-off-by: Josh <jhershbe@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>

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

This commit fixes NPE observed in bindAcl() of AclService

Change-Id: I7397677f0e63a56e0e023eb4b2697ae1ef095714
Signed-off-by: Shashidhar Raja <shashidharr@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>

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>

Bump odlparent to 2.0.0, not SNAPSHOT

Change-Id: Ib4d6af8ebf62aa2b4537da3d331f951d7fd84a40
Signed-off-by: Robert Varga <nite@hq.sk>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

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

This reverts commit 5c32d2e26885b17e9c74fbb7bdc5195da5700d0b.

Change-Id: Ifb7cd33f6f72ed51b8cd06e2aa566e56d93fb35e
Signed-off-by: Sam Hague <shague@redhat.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>

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>

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>

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>

Remove sfc-translator MdsalUtils::merge()

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

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