Blueprint fix

Registration of entity type "elan" missed out. This is used by L2Gw in
order to identify the owner for type "elan".

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

Merge "Port cloud-servicechain to Blueprint" into stable/boron

Merge "clean isExternal NPE and string syntax" into stable/boron

Merge "add local_ip to other_config to satisfy auto-tunnel" into stable/boron

Merge "More elan Blueprint clean-up" into stable/boron

clean isExternal NPE and string syntax

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

add local_ip to other_config to satisfy auto-tunnel

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

Merge "ACL Support for IPv6 IPAM" into stable/boron

Merge "Neutronvpn caches require update upon Router/Network changes" into stable/boron

Merge "Fixing external network group table lifecycle" into stable/boron

Merge "Remove elanUtils instance from ElanInstanceManager" into stable/boron

More elan Blueprint clean-up

* Use static ElanUtils methods where appropriate.
* Drop ElanUtils and IInterfaceManager from
  ElanInterfaceStateChangeListener.
* Drop ElanUtils and MdsalUtils from ElanStatisticsImpl.
* Remove ElanServiceProvider.getElanUtils().

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

Merge "Create VLAN trunk once for multiple VLAN networks" into stable/boron

ACL Support for IPv6 IPAM

This patch implements the following support.
1. Programs the anti-spoofing rules not to allow a VM to run an IPv6 Router.
2. Allows the necessary ICMPv6 traffic in the VM-egress pipeline.
3. Allows the necessary ICMPv6 traffic in the VM-ingress pipeline.
4. Fixes the DHCPv6 ports

Pending:
Some of the flows should be made more restrictive by taking into
consideration the MAC address of the IPv6 Router/DHCP server. This
change is required for both V4 and V6 rules and will be handled in
a separate patch.

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

Fixing external network group table lifecycle

Subnet changes will create and remove group tables

Change-Id: I000a295f93af5f1217cc54d5ad2cbf3b3aa3ed9a
Signed-off-by: yair <yair.zinger@hpe.com>

Merge "Bug 6449 - Issues in Service Function Forwarder translation" into stable/boron

Merge "Rename JMX MBean's ObjectName from com.ericsson to org.opendaylight" into stable/boron

Neutronvpn caches require update upon Router/Network changes

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

Port cloud-servicechain to Blueprint

This also enables the feature.

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

Rename JMX MBean's ObjectName from com.ericsson to org.opendaylight

From com.ericsson.sdncp.services.status:type=SvcElanService to
org.opendaylight.netvirt.elan:type=SvcElanService

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

Fix completely broken ElanStatusMonitor JMX MBean

including a test for future non-regression of this problem:

javax.management.NotCompliantMBeanException: MBean class
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX does
not implement DynamicMBean, and neither follows the Standard MBean
conventions (javax.management.NotCompliantMBeanException: Class
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX is not
a JMX compliant Standard MBean) nor the MXBean conventions
(javax.management.NotCompliantMBeanException:
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX: Class
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX is not
a JMX compliant MXBean)
at
com.sun.jmx.mbeanserver.Introspector.checkCompliance(Introspector.java:176)
at
com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.registerMBean(DefaultMBeanServerInterceptor.java:317)
at
com.sun.jmx.mbeanserver.JmxMBeanServer.registerMBean(JmxMBeanServer.java:522)
at
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX.registerMbean(ElanStatusMonitorJMX.java:30)
at
org.opendaylight.netvirt.elan.statusanddiag.ElanStatusMonitorJMX.init(ElanStatusMonitorJMX.java:24)

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

Create VLAN trunk once for multiple VLAN networks

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

Remove elanUtils instance from ElanInstanceManager

With a couple of clean-ups in ElanUtils, ElanInstanceManager can run
with only static method calls.

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

Bug 6449 - Issues in Service Function Forwarder translation

Change-Id: I7ca466ae65aa36c732d74a26def73bd64981b133
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

allow all should be verified by null protocol

Change-Id: I94720d90cba5822a75a5ca3454927303d9eaf66c
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

Refactoring to make ElanStatusMonitor more pluggable for tests

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

wrong goto table in stateless default flows

Change-Id: Ieebb317073dbd4511e35a084d5f79a6caabd2aad
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

Merge "Listen to changes on subnet GW mac addresses" into stable/boron

Merge "Check network type before adding vtep to TZ" into stable/boron

Merge "elanmanager Checkstyle clean-up (94%) " into stable/boron

Merge "Fixes Bug6423" into stable/boron

Listen to changes on subnet GW mac addresses

- Update ext-net flow based on MAC changes

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

Check network type before adding vtep to TZ

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

elanmanager Checkstyle clean-up (94%)

Enforcement on for api, but not yet for impl (coming later; let's at
least merge this already ASAP to save re-re-basing)

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

Fixes Bug6423

1)The dhcp port should not have SG associated. DHCP will be created with
portsecurityenabled=true and later it will be updated. The update even
was getting ignored.
2)Updated few priority values

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

add nextHopPointerPool poolId

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

Merge "pom cleanup, odlparent-lite" into stable/boron

Merge "netvirt-it Checkstyle clean-up (100%) and enforcement" into stable/boron

Merge "Fix bug wherein console commands would not work with Karaf ssh session" into stable/boron

Fix bug wherein console commands would not work with Karaf ssh session

Thank You to Checkstyle for pointing these out.  Please see
https://wiki.opendaylight.org/view/BestPractices/Coding_Guidelines#System.out
for background & details.

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

netvirt-it Checkstyle clean-up (100%) and enforcement

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

Fix (minor) usage of MacNotFoundException

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

Merge "blueprint cleanup" into stable/boron

Merge "bug 5618 - fix creation of default rules in stateless security group mode" into stable/boron

pom cleanup, odlparent-lite

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

Merge "MAC resolution of external GWs" into stable/boron

bug 5618 - fix creation of default rules in stateless security group mode

Change-Id: I7bc8e2c52c9b783e722136840b5478e612e638f6
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>

blueprint cleanup

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

MAC resolution of external GWs

- NeutronSubnetGwMacResolver will send ARP requests periodically to GW ips
- ARP requests would also be triggered by neutron router changes

ARP responses are already handled and saved to the ARP cache by ArpNotificationHandler

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

Fix vrf metadata not to override the SH bit

The original fix for this issue was deleted as part of
https://git.opendaylight.org/gerrit/#/c/43306/ (performance improvements)

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

Merge "change log debug level of elanutils lock print" into stable/boron

Merge "Fix IT default flows validation" into stable/boron

change log debug level of elanutils lock print

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

Fix IT default flows validation

1)Changed the list of default flows due to some
changes in the pipeline.
2)Changed the max time for flow validation method,
because 1 minute was not enough.

Change-Id: Icc7c2ac03fce75a14cb04f185b1350936d39f67b
Signed-off-by: Oded Shvartz <oshvartz@redhat.com>

Add config for aclservice to set security-group-mode

1) add transparent security group
2) load default value from blueprint

To set the default for security-group-mode:
1. Change the value in aclservice/.../aclservice-config_aclservice-config.xml
2. Copy the file into the unzipped distribution before starting karaf:
cd netvirt/vpnservice
mkdir -p ./distribution/karaf/target/assembly/etc/opendaylight/datastore/initial/config; cp aclservice/impl/src/main/resources/aclservice-config_aclservice-config.xml ./distribution/karaf/target/assembly/etc/opendaylight/datastore/initial/config

Change-Id: Ic57c438499f6b529e462b57fbfcbc669c591bfc4
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>

Fix ipv6service main pom.xml version

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

Add missing init-methods

Change-Id: I6de3c5ace68b83e1861433583ae596ea798b2a0e
Signed-off-by: Tali <tali.ben-meir@hpe.com>
(cherry picked from commit b3431d2248f077523fe63aa9d3ebb95ac6a9b5e2)

Add missing init-method to FIB+ELAN services

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

Merge "Bug 6089: Fix to drop Tcp Syn packets after default SG removed." into stable/boron

cleanup elanmanager blueprint 2

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

Merge "Fix exception in IPv6Service for non-neutron ports" into stable/boron

Merge "Optimize iface search in transmitRouterAdvertisement" into stable/boron

Merge "Change controller-dhcp-enabled default value" into stable/boron

Bug 6089: Fix to drop Tcp Syn packets after default SG removed.

 * Removed rule to check legitimate ip/mac pair during
   VM creation because incoming Tcp packets matches this
   rule and it passing packets to next pipeline.
   "cookie=0x0, duration=45.414s, table=40, n_packets=0, n_bytes=0,
   priority=36001,ip,in_port=12,dl_src=fa:16:3e:49:06:f1,nw_src=10.0.0.2
   actions=goto_table:50"
 * Support added for Tcp Syn drop rule in Egress table.

Change-Id: Ifa0e8cb122c16dd12f5d94c98a187dc604bd8c87
Signed-off-by: bharathi.m <bharathi.m@hcl.com>

Merge "ACL: Using non-deprecated ACL constants" into stable/boron

Fix exception in IPv6Service for non-neutron ports

This patch fixes the IllegalArgumentException for notifications
related to non-neutron ports.

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

Optimize iface search in transmitRouterAdvertisement

Currently the periodic router advt timer is iterating over
all the virtual ports to figure out the ports on the network
where the RA has to be sent out. This patch optimizes the
search by using the local cache of (dpnIds <--> List[ofPorts])
maintained per network.

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

Merge "BUG 6387: Traffic drop in 17 in serviceChain" into stable/boron

Change controller-dhcp-enabled default value

Changed the default value to false in order to use the
neutron dhcp agent.

Change-Id: I8be03bde202759964b85f89c125218e2fcd5e8fb
Signed-off-by: Oded Shvartz <oshvartz@redhat.com>

Fix ipv6 version for rc0

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

ACL: Using non-deprecated ACL constants

+ Replaced usage of deprecated constants with non-deprecated constants.

Change-Id: Ibe3b516665f51fb4358bf9eb432c651810717017
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

cleanup elanmanager blueprint

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

Support new actions in new ovs nsh patches

- rebase
- Added pop_nsh to classifier bypass flow

Change-Id: Id12e9c0bea42aa5c20adf34769b366d9b5c0231b
Signed-off-by: Yi Yang <yi.y.yang@intel.com>

BUG 6387: Traffic drop in 17 in serviceChain

 Analysis:
    In a non servicechain scenario, packets get to LFIB and are sent over
    the corresponding port.
    In ServiceChain scenario, there are new Flows in LFIB, with higher priority,
    that sends the packet to 17 which, in turn, is redirecting it to SCF Pipeline.
    However, no SC matches and packets are submitted back to 17. Since packets
    have not changed, they get in a loop between 17 and 70.

 Solution:
    Table-miss in 70-75 tables must change the SI to L3VPN, so that when
    the packet gets back to 17, they are redirected to FIB table.
    Also, entries in 17 that handle packets that match no SC must be
    programmed in all CSS where the VPN has footprint, not just in the
    CSS where the last vSF in the chain is located.

Change-Id: I1cfc2973049746019a310ef3b106a5e416f24f04
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Merge "Exception in SubnetRoute Feature" into stable/boron

Merge "Changing the conntrack default drop and allow flows from per port to per switch." into stable/boron

Merge "blueprint cleanup" into stable/boron

Changing the conntrack default drop and allow flows from per port to per
switch.

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

blueprint cleanup

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

Add blueprint wiring for elanmanager

Change-Id: I1274ef299469602cd8fa4722bc037e4e2785e1ea
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
Signed-off-by: Alexis de Talhouët <alexis.de.t@gmail.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Merge "Fix IT setup" into stable/boron

Exception in SubnetRoute Feature

1. Handle exception in subnetroute feature
specifically for the
SubnetRouteInterfaceStateChangeListener.

2. Handle removal of subnetroutes when
subnets are removed from the router (or)
when the router is disassociated from VPN.

3. Increase the retry count to 3 when trying
deletion of VPNs.

Change-Id: Iaf263c5596bd2b2b061e82597dcdc400dad42f17
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

Fix IT setup

Remove deprecated methods after moving to blueprints.
The method were used when working with config and after
moving to blueprints we get "NOT FOUND Module" exception.

Change-Id: I16a417c58ee9c2b0720ef35675dc98ddc333b3cf
Signed-off-by: Oded Shvartz <oshvartz@redhat.com>

ACL: Enabling ACL service in the pipeline

Change-Id: Ieca5bbefa97f38703c79f592b383fb32bc745fd9
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

Merge "ACL: Adding tracked flows for each of allowed address pair" into stable/boron

Merge "fix blue print to init InterfaceStateToTansportZoneListener" into stable/boron

Merge "IPv6Service Binding using elanTag" into stable/boron

Merge "Using DataStoreJobCoordinator" into stable/boron

Merge "stateless security groups" into stable/boron

Merge "BUG 6363: Further performance improvements for VPN Engine" into stable/boron

IPv6Service Binding using elanTag

This patch implements the following.
1. We do Service binding for a port only when a v6 address is
   configured on the port. IPv6Address could be part of the port
   when its initially created or later (i.e, port updated) after
   the VM is spawned. Both cases are handled and the port is
   unbinded when the v6Address-is-removed/port-deleted. Once
   Service Binding is done, packets would hit the IPV6_TABLE(45)
   for further processing.
2. Optimizes the IPv6 punt flows by matching on the elanTag.
   Previous patch implements punt flows based on the VM macaddress,
   this is not optimal as we have to add VM specific punt flows when
   a new VM is added. In this patch, we take into account the elanTag
   (i.e., network-id) and configure punt flows for
   - icmpv6 Router solicitation requests
   - icmpv6 Neighbor solicitation requests for router interface
3. Multiple IPv6 subnets can be added/deleted (even when existing VMs
   are present) on the network and those edge cases are handled in
   this patchset. IPv6Service maintains a cache of (dpnIds <--> List[ports])
   per network. We use this info, while programing flows when IPv6 subnets
   are added/deleted to handle the edge cases.

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

Using DataStoreJobCoordinator

- Made changes such that the Interface Events make use of
  DataStoreJobCoordinator.
- Batching transactions.

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

stateless security groups

1. refactor Egress/Ingress security group to allow inheritance of Stateless implementation
2. adding MethodInvocationParamSaver to allow asserts on parameters on invoked mocks.
3. current implementation differ between stateless Vs statefull is by
   additional configuration in aclservice-config.yang.
4. blueprint adaptation to support inheritence in security group
4. some check-style fixes

Change-Id: I5a73d5087ada651162ee92a42beeea8d9aaf97ef
Signed-off-by: Shlomi <shlomi.alfasi@hpe.com>
Signed-off-by: Sam Hague <shague@redhat.com>

ACL: Adding tracked flows for each of allowed address pair

+ For tracked packets, now matching on IP + MAC instead of lport tag

Change-Id: I71373273d1ed9ce255c00e4c007f618ca7267fe5
Signed-off-by: Somashekar Byrappa <somashekar.byrappa@ericsson.com>

BUG 6363: Further performance improvements for VPN Engine

This fix removes waits in some critical places for
MDSAL operation to complete:
a. InterfaceStateChangeListener
b. VpnInterfaceManager

Change-Id: Ife821982f458a16f02e6bcf28cab960ad0d94d51
Signed-off-by: Suraj Ranjan <suraj.ranjan@ericsson.com>

fix blue print to init InterfaceStateToTansportZoneListener

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

Add blueprint wiring to natservice

(cherry-picked from commit 954b3f14841bccd4a86bf668890124ce98c47707)

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

Merge "Ignore VpnSubnetRouteHandlerTests" into stable/boron

Ignore VpnSubnetRouteHandlerTests

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

Remove dependency on RpcProviderRegistry

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