Netvirt: Qos Alert patch6

*** patch #6 - Added log4j configuraton support ***

1. Added a new log4j appender with default logging properties in qosalert.cfg
2. Added listener method update in QosAlertGenerator class to handle any chage in qosalert.cfg

Spec Review Link:
https://git.opendaylight.org/gerrit/50689

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

EVPN RT2 advertise and withdraw prefix changes

1) Added elan external tep listener.
When elan external tep is added update the elan remote broadcast
group for all elan dpns.

2) when an elan interface is added , advertise its prefix
when elan interface is deleted, delete its prefix.

3) Advertise prefix when we receive packet in message from silent
host.

4) Withdraw prefix when the smac flow entry of silent host expires

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

Set feature capability for port status update

See: https://bugs.opendaylight.org/show_bug.cgi?id=7718
DEPENDS ON: https://git.opendaylight.org/gerrit/#/c/55854/

Change-Id: Ia83d31c348176947e31e11d383f770ac97390145
Signed-off-by: Josh <jhershbe@redhat.com>

bgpmanager VPNv6 shell command update

bgp-network command is modified so as to support the ability to
configure prefixes with afi parameter, to be 1 ( IPv4) or 2 ( IPv6).

Change-Id: Ie0a5faf7160a049e930d2eb8f89af9d87693ddcd
Signed-off-by: Noel De Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Hari Krishna <hari.i.krishna@ericsson.com>

BUG 8296 - Erroneous Egress Classifier flows

- Egress classifier filter flow now just matches on NSH
  MD Type as set in the Ingress classifier ACL flow
- OpenflowRenderer.renderPath() is now passed the nodeIP
  instead of the destination SFF IP
- Patch Set 2 : fixed minor merge issue
- Patch Set 3 : fixes found by ClassifierEntryTest UT

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

Factor out executor from ClassifierService & UT

Depends on [1]

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

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

Thrift interface changes to support BGP VPNv6

Thrift interface changes to include information on afi type.

Change-Id: Ie82e9a086621a551f19a0faaf9107d630277ae43
Signed-off-by: Noel De Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Hari Krishna <hari.i.krishna@ericsson.com>
Signed-off-by: Vivekanandan Narasimhan <n.vivekanandan@ericsson.com>

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

Change-Id: Ibd4842624d55fd07ae88b0d2ca8094a641f8749d
Signed-off-by: Noel De Prandieres <prandieres@6wind.com>
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Hari Krishna <hari.i.krishna@ericsson.com>

Bug 7866 adding retries for remote dmac programming during tunnel up event

* currently there is no retries in case of OptimisticLockFailedException
while programming remote dmac during tunnel up event. This leads to remote
dmac is programmed with drop action which leads to packet drop. Now added
retries for jc job.
* Also adding retries in case remote BC group fails (during tunnel up event) due to datastore exceptions.

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

EVPN RT2 NeutonVpn changes

1. EVPN RT2 RPC for create, delete, and get EVPN
creation and deletion of EVPN via REST is handled by NeutronEvpnManager.
When create EVPN RPC is received, NeutronEvpnManager will check if
vpninstance already exist with same name if not then it will create
vpninstance of type L2 for EVPN. similarly vpninstance will be deleted for
delete event.

2. Changes for attach/detach network to EVPN
NeutronEvpnManager will add/remove/update evpn name in Elan augmentation so that
EvpnElanInstanceManager will advertised and withdraw routes accordingly.

3. Few previous comments mandatory false from yang removal fix

4. Added ElanHelper file to move commonly used utility methods to avoid
cyclic dependency

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

Bug 8244  In Conntrack SNAT implementation G/w is not reachable when no
port in the router subnet is present in the node

1)Added a the pseudo port to vpn-dpn list.
2)Moved the router cache util to a separate class.

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

stale entry in flow rules, after deleting port associated to qos policy with dscp

When we are delete the VM which has an associated qos policy with it,
the stale entry of DSCP marking use to persist for the table number 90.

The reason for this was that when QosNeutronPortChangeListner is called
rpc called to fetch DPN id use to fail. By moving the code to
QosInterfaceStateChangeLister we can use Interface to find out DPN id.

Change-Id: I485d84806eae2bcc8341518f8186973efb171658
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>

Bug 8310 - SNAT flows not added when the first subnet is added to the
external n/w after router g/w set

1)Check for the presence of external IP during add.
2)Update now considers whether a subnet is added after router g/w set.

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

ClassifierUpdate unit tests

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

Fix remote acl bugs

1. Merging metadatas of lPort and aclId.
2. Handling reg6Match for ingress
3. Fixing dupplicate AclInterfaces in AclDataUtil
4. Adding all ips to remote acl filter table, not only those with remote acl rules
5. Fixing the flows instalation logic for the case of port in more than one SG.
6. Fixing handling of SG that uses itself as remote SG.

Change-Id: I64b0e294317e0ac1a3444192a92f8058f2604f1d
Signed-off-by: Slava Radune <slava.radune@hpe.com>

Netvirt: Qos Alert patch5

*** patch #5 - Port direct statistics polling  ***

Added support of  retrieval of port statistics data using OpenflowPlugin
direct-statistics RPC and log the alert message if packet drop ratio is
greater than the configured threshold value.

Spec Review Link:
https://git.opendaylight.org/gerrit/50689

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

BUG 8284 - Unmatched packets get dropped by Classifier

- Packets that enter the Netvirt classifier that dont get
  matched are dropped.
- These packets should instead be resubmit back to the
  Ingress Dispatcher table.
- The solution is to add a lower priority MatchAny flow
  to the INGRESS_SFC_CLASSIFIER_ACL_TABLE table that
  resubmits back to the Ingress Dispatcher.
- Adding UT for the newly created flow.

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

Fix the error in the egress classifier spec

The table numbers were wrong and could confuse newcomers

Change-Id: Idd57ab06f4c17801fc40dc90b00a410f5f4cfb74
Signed-off-by: Manuel Buil <mbuil@suse.com>

@Immutable FlowEntity with FlowEntityBuilder

This is dependent on the genius change
https://git.opendaylight.org/gerrit/#/c/53763/, which must be merged
simultaneously.

The change in the Xtend re. flowId is because the FlowEntityBuilder does
not have a getFlowId() like FlowEntity, only a setter; see
https://github.com/immutables/immutables/issues/432

Change-Id: Iecbe1ccd2c1c6cb9c914b70c6e44590d748ad739
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 7758: Use Trunk instead of Transparent port for Flat networks

Netvirt should have used Trunk ports for Flat networks, instead of
Transparent ports - otherwise there are issues of mixing Trunk and
Transparent ports together, since Trunks are automatically created
once there are Trunk Members (VLAN networks) on the same port.

Change-Id: If666b672b1158919623db11b5324321f2f7b15b0
Signed-off-by: Alon Kochba <alonko@hpe.com>

Netvirt: Qos Alert patch4

*** patch #4 - Building Qos Alert port cache ***

This patch builds a cache of port stats data having QoS rate-limit rule applied.

A neutron port is added into the cache if -

1. Port QoS policy has bandwidth rate limit rule.
2. If port does not have QoS policy applied, then if network QoS policy
   has bandwidth rate limit rule.

Spec Review Link:
https://git.opendaylight.org/gerrit/50689

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

Bug 8244 - In Conntrack SNAT implementation G/w is not reachable
when no
port in the router subnet is present in the node

Creates an elan interface when a router port is created amd is deleted
on port delete.

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

ClassifierEntry unit tests + javadoc

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

EVPN RT2 elan changes to adv/withdraw RT2 routes..

1. Elan instance changes when network is attached and detached to EVPN
Neutronvpnmanager will add/delete/update evpn augmentaion in elan instance
EvpnElanInstanceManager is listening to the augmentation and based on type
of event EvpnElanInstanceManager will advertise or withdraw RT2 routes.

2. EvpnElanInstanceManager uses vpninstance to get rd to advertise or
withdraw routes.

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

Bug-7718 Operational neutron port status

networking-odl requires feedback as to when a neutron
port can be marked as "ACTIVE". This commit places an
estimation of that information in the operational data
store. For ports that connect to OVS we wait for them
to connect and for basic L2 flows to be configured.
Note that we do not validate the flows in operational
nor do we check all flows. Ports that are implemented
purely as flows are marked ACTIVE immediately. The plan
is for networking-odl to receive notifications of these
status changes via a websocket.

It was decided at the time being
to keep the port status out of the ODL neutron project
since (a) functionally, this issue is mainly an issue for
netvirt and (b) to do it right in netvirt would require
some re-architecting.

Change-Id: Id719e904b277fe4dbb9c3d118d24c3bedf110a33
Signed-off-by: Josh <jhershbe@redhat.com>

Error messages are displayed when the port updated with qos policy stating
that binding are not allowed to the port

Following Error message was seen  when the same port updated with second
qos policy:

2017-04-05 14:42:54,116 | ERROR | pool-39-thread-7 |
FlowBasedServicesConfigListener  | 338 -
org.opendaylight.genius.interfacemanager-impl - 0.1.4.SNAPSHOT | Service
Binding entry update not allowed for:
b193e270-8053-4650-8059-a8ec57713069, Data:
BoundServices{getServiceName=qos.b193e270-8053-4650-8059-a8ec57713069,
getServicePriority=3, getServiceType=class
org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.servicebinding.rev160406.ServiceTypeFlowBased,
augmentations={interface
org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.servicebinding.rev160406.StypeOpenflow=StypeOpenflow{getFlowCookie=67108865,
getFlowPriority=10,
getInstruction=[Instruction{getInstruction=GoToTableCase{getGoToTable=GoToTable{getTableId=90,
augmentations={}}, augmentations={}}, getOrder=1, augmentations={}}]}}}

Use case:

- create a port, say port1
- create qos policy q1 with dscp and bandwidth
- Associate qos policy to the port
- observe the policy applied properly
- update the port again with qos policy q2
- Though the 2nd qos policy (here q2) seen applied properly , below error
  message was seen in ODL side:

  "Service Binding entry update not allowed"

  RCA:

  Qos service binding is happening even during the 2nd qos policy updation
  on the same port on which a 1st qos policy already associated.

Change-Id: Ia3754833d716f1e387bd7bdce00f82ed51a394ce
Signed-off-by: A Vamsikrishna <a.vamsikrishna@ericsson.com>

Bug 8189 - Policy flows are not updated after ovsdb other-config changes

Change-Id: Ic27b87922dd42aac020fa6f7e65bbab53e8c4489
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Bug 8255 - VPN creation with multiple RD via Rest
is not working

fixed this by removing throwing of an error when
multiple RDs are passed to create VPN.

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

Bug 8200:NAPT_PFIB_TABLE(47) with Internet VPN is not programmed for EVPN

This code review will address the following problem for EVPN NAT UCs.

1) NAPT_PFIB_TABLE (47) resubmit to L3_FIB_TABLE (21) {47->21} with
external (internet) VPN Id as match flow is not programmed for if external
network with Provider type VXLAN is configured.

2) When the external router is deleted some of the EVPN stale entry flows are
presented.

3) If more than one floatingIp is available in vpn-to-dpn-list for given
dpn id, do not call for installing INTERNAL_TUNNEL_TABLE (table=36) -> PDNAT_TABLE (table=25)
flow entry with same tunnel_id again and again.

Change-Id: I5a54764da323ed1ac20f91dd40fc36c06ed59ecb
Signed-off-by: karthikeyan <karthikeyan.k@altencalsoftlabs.com>
Signed-off-by: Sam Hague <shague@redhat.com>

BUG 8240 - Fix odl-netvirt-sfc dependent features

- Adding the odl-sfc-genius feature.

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

Fix equals() bug in FederatedNetworkPair found by Checkstyle

This shows up as red in Eclipse, but the CLI mvn build missed this.

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

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

Bug 8235 - Bug in group programming for directly connected subnet routes

Issue:

1. The ECMP feature enables the loadbalancing of traffic to the the extra
routes only but the discovered routes are also getting loadbalanced similar
to the extra routes.

2. This also leads to the issue in case of the discovered routes
containing macaddress(directly connected subnet route), the
loadbalancing groupId overrides the local nexthop group Id
as the parameters used for computing the groupId turns out to be same.

Analysis:

1. Since the discovered routes are also present in the vpntoextraroute
container, the discovered routes are also being currently loadbalanced by
ECMP feature.

2. Also, the loadbalancing groupId is computed from the prefix or the nexthop Ip
depending on whether the macadress is present or not(if mac is present,
the nexthopIp is used and prefix for otherwise). In case of the directly
connected subnet route, the macaddress is present and so the nexthopIp
is used for computing the loadbalancing groupId. This overlaps with
local next hop group programmed for that nexthop.

Fix:

Since the discovered routes have only 1 next hop, this
condition is now used to distinguish between discovered routes and the
extra routes. Only the extra routes are loadbalanced now and the
discovered routes are handled as before(creating a new local nexthop
group).

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

FIB: VNI support in datapath for VxLAN networks

This commit adds support for VNI based L3 forwarding for VxLAN based
provider networks.
Spec: https://git.opendaylight.org/gerrit/#/c/48640/

Following changes are done:

1.
  i)   FIB_TABLE will set the destination network VNI in the tun_id
       field instead of the MPLS label.

  ii)  On egress OVS, match will happen on this VNI in table 36,
       and packet will be taken to ELAN pipeline in order to reach the
       destination VM.

  iii) Table 36 will no longer be programmed by FIB to match on MPLS
       label for internal router-based VPNs whose subnets are part
       of a VxLAN network

2. Some changes are done to program VNI in tun_id for ECMP LB NH
   groups.

3. The commit also incorporates the enforce-openstack-semantics
   flag to enable easy turning on/off of the feature.

4. Some refactoring and cleanup has also been done, majorly to
   remove the local utils/references for MDSAL sync
   write/delete/update APIs.

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

ELAN: VNI support in datapath for VxLAN networks

This commit adds support for VNI based datapath forwarding for VxLAN
based provider networks.
Spec: https://git.opendaylight.org/gerrit/#/c/48640/

a. Unicast
  1. Table 51 now sets VNI in tun_id field for packets egressing on
     tunnel ports
  2. On egress, table 36 now matches on VNI in tun_id field, sets the
     ELAN tag in metadata, and forwards the traffic to table 51

b. Broadcast
  1. Remote broadcast group now sets VNI in tun_id field for packets
     egressing on tunnel ports
  2. On egress, table 36 now matches on VNI in tun_id field, sets the
     ELAN tag in metadata, and forwards the traffic to table 51

Also, integrates the openstack-vni-semantics-enforced flag to enable
easier turning on/off of the feature

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

Bug 8201 - ECMP not working with router based VPN

Issue:

In case of router based VPNs, the loadbalancing groups created for extra
routes are not updated with correct buckets when the extra route is
present behind different DPNs.

Analysis:

The issue is caused as the current design requires list of rds to
differentiate the extraroute present behind different DPNs.

Fix:

In case of the router based VPNs, the DpnIds are used as rds so the
vpntoextraroute is now populated correctly.

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

Bug 8221 - Remove Wait Time To Install L3VPN Groups on OVS

Currently we have a wait of 1.5s to ensure L3VPN groups are programmed
correctly in OVS, before we start programming the L3VPN Flows. This needs
to be optimized further.
This patch makes use of the OpenFlowPlugin addGroup() RPC call to directly
install the group in the OVS. This will ensure that the subsequent installation
of flows will always find the required groups in the OVS.
Group installation via the FRM is still retained to ensure consistency in the
datastore.

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

Remove unused setFibManager

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

cleanup ovsdb-ui bundle

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

Add ovsdb-ui bundle

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

Bug 8142 : DHCP timeout issue.

Issue:
stable/boron not usable - DHCP timeout

Analysis:
more than 5 thread were waiting on BGPConfigurationManager when BGP was
trying to send thrift messages to QBGP and QBGP is not up. BGP was
retrying it again and again. Meanwhile, services which call addPrefix is
getting blocked due to synchronized behavior of addPrefix.

Fix:
APIs exposed by BGP to other services - addPrefix, delPrefix, addvrf and
delVrf is made unsynchronized.

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

Deprecate legacy features

Depends-on: https://git.opendaylight.org/gerrit/55083

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

add odl-sfc-genius to odl-netvirt-sfc

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

BUG 8240 - Fix odl-netvirt-sfc dependent features

- Needed to install odl-sfc-openflow-renderer instead of
  odl-sfc-provider in order to bring in all the needed
  SFC features.

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

Bug 8188: L3VNI without configured UC handled for EVPN

Problem Description:
==================
L3VNI without configured in Internet VPN needs to be handled for EVPN in
NAT feature.
Currently Internet(External) VPN with L3VNI use case only handled for
external VXLAN network (EVPN) in NAT feature.

Internet(External) VPN without L3VNI use case also needs to be handled for
external VXLAN in NAT feature. Since Openstack Operator may configure L3VNI or
may not configure L3VNI.

Solution:
=========
Unconfigured of L3VNI value in Internet(External) VPN, will carve-out
the L3VNI value from OpenDaylight VXLAN VNI Pool to use SNAT and DNAT
flows.
The following UCs are handled based on the L3VNI configured/unconfigured
for GRE and VXLAN provider type.

Ext-Net-Type || Internet-VPN-L3VNI || External and Internal VXLAN uses
===============================================================================
GRE           Configured          Ignore L3VNI, allocate VNI from pool and
                                  use it for Intra-DC only
--------------------------------------------------------------------------------
GRE           Not Configured      A VNI taken from ODL Pool and used for
                                  Intra-DC communication
---------------------------------------------------------------------------------
VXLAN         Configured          Internet-VPN-L3VNI used for both
                                  Intra-Inter-DC communication
---------------------------------------------------------------------------------
VXLAN         Not Configured      A VNI taken from ODL Pool and used for both intra-
                                  inter-DC communication
---------------------------------------------------------------------------------

Note: If VNI value is unable to get from ODL VNI Pool, router-id will be
used as tunnel-id for external network provider type as VXLAN

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

Bug 8241: Fix openstack-sfc ACLs IllegalArgumentException

Fix ace port range to be set only if mandatory lower port
is specified, otherwise we hit an exception.

Change-Id: Ic6206394e4255653a837c2b57cefc538555eefa4
Signed-off-by: Alon Kochba <alonko@hpe.com>

ADDED INFO logs for tempest issue DEBUG

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

Bug 8170 - Loadbalancing nexthop group not updated after VM deletion

Issue:

When the last nexthop present in a DPN is deleted, the loadbalancing next
hop group is not updated. The group still contains the bucket pointing to
the deleted nexthop.

Fix:

There was an error while retrieving the endpointIpaddress for the DPN in
method getEndpointIpAddressForDPN. There was an error in converting the
IpAddress object to string. Appropriate conversion is used now.

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

Set copyright for sfc translator back to Brocade

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

Remove .0 from model.bgp.version

The version bump script ancorrectly added a .0 to the version.
This was a problem as during this same time the vpnservice-features
was using the value without the .0 from a different patch, so they
were not consistent.

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

Bug 8166: Acl interfaces are not set and cached correctly

The acl interface can exist in the cache before an interface
was assigned one. This can happen via the AclInterfaceStateListener.
Also, an interface ID should be set when a new entry is created.

Change-Id: I3e882582731eb273da533ac15dd2c8cc7f27ffe2
Signed-off-by: Yakir Dorani <yakir.dorani@hpe.com>

Use ArpConstants from arputil

Depend-on: https://git.opendaylight.org/gerrit/54890

This avoids duplicating constants and also helps to
break cyclics.

Change-Id: I76689a97d1131d13cffa382bbf4d07d7a3d19044
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 40f3c94a3068d45b5fda6b38500dbce07aadca44)

Name all blueprint.xml file consistently

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

Fix model-bgp version

Change-Id: I419cbebf5df5d38501cf3793aba5ecd806c7f25c
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Bug 8162 - NPE at org.opendaylight.netvirt.vpnmanager.VpnSubnetRouteHandler.onSubnetAddedToVpn

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

Cleanup unused dependencies

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

add interface for VpnFootprintService

This is another attempt to reduce cyclics.

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

Remove legacy utils from new netvirt

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

Bug 8014 - L2Gw connectivity not working

Tunnel creation issue as cache were not populated properly
Fixed .

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

Remove utils.config

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

BUG 8193 - Fix Netvirt classifier egress service port binding

- The Netvirt classifier egress service should bind on egress
  ports, not ingress ports like it does now.
- Since its not possible to know all the possible egress ports
  to bind on before-hand, the egress classifier service will
  bind on all switch ports. It will only process NSH packets
  and return all others to the egress dispatcher.
- Also in this patch, when the SFF is on the same bridge as
  the classifier, the egress classifier will resubmit the
  packets directly to the SFF instead of going through the
  ingress dispatcher, since we dont know the correct ingress
  port to use for the SFF.
- Updated OpenFlow13ProviderTest to reflect the change to the
  egress classifier resubmit.
- Changes from code review comments
- Final changes to make sure the egress binding is working.
  Now using getDpnInterfaceList RPC to get the switch ports
  to bind to.

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

Cleanup version properties

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

Bug 8105: IllegalArgumentException in getSrcIp fix

1. reading IP and ARP payload was wrong, which is fixed by getting payload from
deserialised ethernet data based on IPv4 or ARP, getPayload will return
the right payload as the offsets are maintained in Packet based on
type of payload.
2. IllegalArgumentException was for payload type IPv4 where
getSourceAddress method will return of type int, and
NWUtil.toStringIpAddress will accept input as byte.
The same was working fine with payload type ARP as
getSenderProtocolAddress return byte.
3. Tested these changes by creating 2 ports in neutron network, delete 1
port and send ping packets from deleted port to the other port.
First ARP packet hit and was able to retrieve IP address properly, and
then data traffic hit and was able to retrieve IP address.

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

Bump versions by x.(y+1).z for next dev cycle

Change-Id: I9934b7eaaf36c6726e0f8e6700aca63421f03b5c
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

BUG 8140 : Elect nodes for subnetroute on control path reboot

When a physical switch on the cloud management network in the cloud is
rebooted, it results in disconnection of datapath-nodes from the ODL
controller. Under such conditions, the subnetRoute is removed from all
the DPNs.
However, once the physical switch comes back up after reboot, the
datapath-nodes are all getting reconnected to the ODL controller.  During
this time in a scaled environment, the subnetRoute flow rules aren't
getting reapplied to the datapath-nodes.  This is because the election
process fails in figuring out a designated datapath-node for hosting
subnetroute, thereby the subnetRoute is not getting reconfigured on all
the datapath-nodes.

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

cleanup checkstyle to pull from parent poms

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

Bug-8053 OVS Bridge other_config:hwaddr not preserved for nodes not in config DS

1) Make sure to retain operational DS other_configs for existing bridges
2) Set other_config:datapath-id for all bridges already in operational. This
is important to guarantee dpid does not change even for bridges where the
other_config:hwaddr or other_config:datapath_id vals are not set.

DEPENDS ON: https://git.opendaylight.org/gerrit/53856

Change-Id: I5aa792262230655bf753045290666c73aaa4096b
Signed-off-by: Josh <jhershbe@redhat.com>

UT for SfcProvider and NetvirtProvider

- Added SfcProviderTest.java
- Added NetvirtProviderTest.java
- modified pom.xml to add ConstantSchemaAbstractDataBrokerTest

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

Minor code cleanup in ELAN

Change-Id: I15e80fd1d38b6a4cb96d1c947bddbc1a68bf4180
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Bug 8001: Data validation failed for path CSIT TCs

Problem Description:
====================
On external network update or internet VPN update NAT code will trigger
the ExternalRoutersListener.update() to handle the updated changes
in NAT feature. During this invocation, for getting NAPT switch-id is
retrieved from the "router-to-napt-switch" DS. After retrieved the NAPT
switch-id from the DS, it contains the value as "0". There is no check for
NAPT switch-id validation.

clrRtsFromBgpAndDelFibTs() method is invoked with passing NAPT switch-id
as "0" value is throwing the data validation exceptions.

Solution:
===========
As part of this issue fix, invoked NAPT Switch selection if dpnId value is
"null" or "0" from "router-to-napt-switch" DS then go for electing new
Napt switch for existing router. This will resolve the data validation
exception. Since clrRtsFromBgpAndDelFibTs() method is never invoked with
NAPT switch-id as "0".

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

Bug 8165 - Learnt IP route does not reappear on DC-GW after OVSRestart

On OVS Restart, a learnt IP will have to be deleted by Liveness
Monitor, and then re-learnt once the OVS comes up.
But this IP was not learned again.

Cyclic events were getting spawned while processing removal of
Learnt IPs.  This was causing deletion of re-learnt IPs.

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

Bug 8020 - Conflicting modification on VpnInterface creation for router-interface in newton

Change-Id: I918174b829075c571414bf82c40936cc950af630
Signed-off-by: Tali <tali.ben-meir@hpe.com>

Fix erroneous space in DhcpAllocationPoolRemoveJob import statement

This shows up as "red ink" (error) in Eclipse as a Checkstyle violation.
I'm not sure why the CLI mvn Checkstyle does not fail forr this - it may
be a subtle difference between the exact version of Checkstyle
internally used by the Eclipse plugin and the one used by the Maven
build.

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

Bug 8145 - DNAT to DNAT traffic is getting failed for 2nd FIP for EVPN/VNI

Problem Description:
====================
DNAT to DNAT traffic (Different Hypervisor) is getting failed for 2nd FIP
VM for VXLAN and GRE Networks.

Consider 2 FIP VMs are booted on single DPN.

When disassociation/remove of 1st FIP VM, 2nd FIP VM traffic for DNAT to
DNAT is getting failed.

When remove 1st FIP it will remove the flow 36->25 entry. Hence DNAT to
DNAT communication is not happening.

This issue is existing for VXLAN and GRE (L2_L3_VNI Based Forwarding)
external provider type networks only.

Solution:
===========
For EVPN_RT5 and L2-L3 VNI based forwarding, uses "l3vni" value as
tunnel_id in the INTERNAL_TUNNEL_TABLE (table=36). If mutiple FIPs are
existing in the single DPN, each remote tunnel_id will be setting with
same "l3vni" value. Hence removing one FIP is causing the problem
with other DNAT traffic. To avoid this problem added the fix for removing
the flow INTERNAL_TUNNEL_TABLE (table=36) -> PDNAT_TABLE (table=25) to
check if any FIP is existing on the given DPN. If not exist then only will
remove the flow table36->25.

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

Tunnel Interface State Handling for extra routes

* Handle interface state changes for tunnel
* Handles add and delete event
* Add/remove nhList from vrfentrylistener based on event
Depends-On: I1b40ce3ea8b73707295e86ae9d4ddddf15716b2e
Change-Id: I3c091a538d201ff4f4accc04d5fb35bee482de4d
Signed-off-by: eceghkl <manu.b@ericsson.com>

UT for Netvirt SFC classifier

- Added AclMatchesTest.java
- Added OpenFlow13ProviderTest.java
- Changes to remove DataBroker from OpenFlow13ProviderTest as suggested by Michael Vorburger
- Minor changes were made to OpenFlow13Provider to expose constants used in creating
  flows so that OpenFlow13ProviderTest can verify they were set accordingly.
- Flushed out a bug in OpenFlow13Provider thanks to the UT: The flowId
  was not being set correctly in createEgressClassifierFilterVxgpeNshFlow
- Flushed out a bug in OpenFlow13Utils thanks to the UT: The inport
  NodeConnectorId was not being set correctly in addMatchInPort.

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

Bug 7730:  Traffic drop in ACL during port update for AAP

Issue: ARP flows in ACL tables are missing if AAP with same MAC is updated

Fix:
+ For AAP case, changed the order of programming flows to delete and add
instead of add and delete flows.
+ For port update, programming ARP flows is handled separately.
  - For egress, deletion of ARP flow is skipped if AAP is updated for same
    MAC.
  - For ingress, no action required as ARP flow has match for lport
    tag only. MAC is not used here.

Change-Id: I6c814468d80a601e127c813ee1897f9ed18315bf
Signed-off-by: Somashekar Byrappa <somashekar.b@altencalsoftlabs.com>

Added Nat Config in vpn-service feature pom.

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

Set update-strategy to none.

Check [1] for more info.
https://git.opendaylight.org/gerrit/#/c/54103/

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

Final changes for netvirt sfc classifier

- correctly obtain egress port from genius
- minor changes from previous code review
- Bugs tracked to be fixed in subsequent versions:
    BUG 8127
    BUG 8128
    BUG 8129
- Rebased
- BUG 8128 turned out to be a configuration error of mine.
  IPv4 matching works just fine.
- BUG 8128 has been closed.
- Removed Legacy Netvirt flow utils usage.
- BUG 8129 is partially addressed. Removed legacy netvirt flow utils,
  but in order to remove FlowBuilder usage, several changes to Genius
  will be needed, and will be done in a separate patch.
- minor fix for flow deletion
- made the ACL flow key more unique
- Check for the Vxgpe option on tunnel ports

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

Added a new config parameter for selecting SNAT.

The default is set to controller mode.

The controller mode is disabled wherever applicable when conntrack is
selected.

When conntrack is selected new HA framework is used by default. The
controller mode continue to use the existing HA.

Change-Id: I00bb3ccda43cf382d6f2047298c4b25c82c73194
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Update neutronvpn to use Singleton

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

Update natservice to use Singleton

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

Adding a check for elan-Interface removal

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

Bug 8108: Fix for NPE in Cloud-SC's VrfListener

 + I think all stream()+map() should be reviewed to avoid these
   kind of Exceptions

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

Untangle bad class design: OpenFlow13Provider does not use DataBroker

It's generally best practice to not "reach over" through other services
to obtain injected services, but just let each class have (final) fields
with all of its dependant services.

In this particular case, the OpenFlow13Provider does not need to have a
DataBroker, so it's best to keep this simpler.  This makes writing unit
tests for OpenFlow13Provider more clear; for example, in
https://git.opendaylight.org/gerrit/#/c/54376/.

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

Updating the display-bgp-config command

To display the newly added options for BGP Multipath feature

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

Bug 7633: add K4 features to artifacts

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

update DHCP spec according to the desired feature capabilities

Change-Id: Icdfb529d15f40c688dada4695ea75abbb189037e
Signed-off-by: Shai Haim <shai.haim@hpe.com>

Update fibManager to use Singleton

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

Update cloud-servicechain to use Singleton

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

Disabling enforce-openstack-semantics flag

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

Bug 8025: Correct non-NAPT to NAPT tunnel ID

As part of the vni-based-l2-l3-nat changes, the tunnel
used was changed but the packet out sent should use the
same tunnel ID for NAPT to work from a VM on a non-NAPT
switch.

Change-Id: I7ee2b526531beb84ececd78acf79a42d6177f7ab
Signed-off-by: Alon Kochba <alonko@hpe.com>

Fix natservice-impl for Karaf 4

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

Update dhcpservice to use Singleton

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

External networks subnetroute fix

Commit [1] changed the VpnSubnetRouteHandler APIs from using
yang based notifications to listening on subnetmap changes
from the datastore.

Subnetmap entities for subnets in Flat/VLAN external networks don't have a
vpn-name assigned to them, which caused a null pointer exception in
VpnSubnetRouteHandler after [1]

In this change, subnetmap is assinged a vpn-name, for subnets in
external Flat/VLAN networks, which is the subnet id itself.

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

Change-Id: Ic7593a5f59aa389f19729c5c5701fd9e1e02eea0
Signed-off-by: Tomer Pearl <tomer.pearl@hpe.com>

Remove statemanager's ConfigStateManager

now that all of netvirt is using blueprint

This class (ConfigStateManager) does not seem to be used anywhere.

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

Update ipv6service to use Singleton

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

update AclLiveStatistics to use Singleton

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

statemanager StateManager @Singleton with @Inject instead of <bean> XML

https://wiki.opendaylight.org/view/BestPractices/DI_Guidelines

Change-Id: I9c5607b1eba5bcb9bf74b234a64c7583a272f09d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Command line added to display the current NAPT switches configured.

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