Commented out frequently replayed BGPMIP errorlogs

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

fix compilation error

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

Merge "bug-6326 use new sig for setBridgeController"

Merge "Added Port Range and Ipv6 matches"

Set vpn-id/router-id metadata so it would not to override split-horizon bit

Depends on genius commit https://git.opendaylight.org/gerrit/#/c/42918/

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

bug-6326 use new sig for setBridgeController

depends on: https://git.opendaylight.org/gerrit/42979

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

Added Port Range and Ipv6 matches

1)Added port range match using nicira extension.
2)Added IPV6 source and destination matches
3)UT Fixes

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

Merge "Formatting changes"

Global use of table ids from NwConstants.java

This review includes following things:

1> After merging of https://git.opendaylight.org/gerrit/#/c/42940/, build
of Netvirt started failing. Because ElanStatisticsImpl.java was using one
constant from IfmConstants.java, which was removed in mentioned commit.

2> Removed few table ids from FibConstants.java, NATConstants.java and
VpnSubnetRouteHandler.java then using directly from NwConstants.java.

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

Merge "Periodic RA transmission"

Merge "Add a match to nd_target for IPv6 NS punt flows"

Formatting changes

Change-Id: I3879443332230cfd37aa2cb5f9b52fe276182b55
Signed-off-by: Guy Sela <guy.sela@hpe.com>

Periodic RA transmission

Support added for periodic transmission of the RA messages
from the router interfaces available in the compute nodes.
The unsolicitied RA is sent out only on the interfaces
where the same network is available.

Pending items:
1. Configurable RA interval
2. UT for the periodic transmission
(Will be added in separate patch)

Change-Id: I7b498e88ba6801f564c931abc6b5dfeb2d0b14ed
Signed-off-by: kalaiselvik <Kalaiselvi_K@Dell.com>

Add a match to nd_target for IPv6 NS punt flows

Previous patchset that added support for IPv6 ServiceBinding
programs a generic flow to send all Neighbor Solicitation requests
to the controller. This was done since the support to match nd_target
was missing in genius matchutil at that time.

We now have support for matching nd_target in genius. This patch
modifies the punt flows to send only NS packets that correspond to
the Router interface. Neighbor Soliciation for other VMs would be
answered by the respecive VMs on the network.

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

Merge "Fixes related to ACL service."

Merge "Fix of NPE in DhcpServiceCounters and NatServiceCounters."

Fixes related to ACL service.

+ Egress ACL table is split into two tables (40 and 41).
+ Ingress ACL table is split into two tables (251 and 252).
+ Ingress/Egress filter tables are made to match on lport tag.
+ Packets matching ingress/egress ACL filters are re-submitted to
corresponding dispatcher tables for the next service.
+ Egress (w.r.t switch) acl service priority is changed to value 6.
+ Few more fixes.

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

Fix of NPE in DhcpServiceCounters and NatServiceCounters.

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

Global use of Table ids from NwConstants.java

In my previous patches for global use of table ids, I missed one table id
to remove from NATConstants.java.

Removing that particular table id in this patch and using directly from
NwConstants.java.

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

Merge "Using Table Ids from NwConstants.java"

Merge "First use of the infrautils counters in netvirt."

Using Table Ids from NwConstants.java

This review includes changes in entire netvirt module, to use table ids
from NwConstants.java.

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

Add external ports to L3VPN models

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

First use of the infrautils counters in netvirt.

Added install and remove flows counters for Dhcp and SNAT default flows.

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

Support update/delete operations for provider networks
Add external ports to remote BC group buckets

Depends on genius commit https://git.opendaylight.org/gerrit/#/c/41943/

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

Merge "Write netvirt:1 to mdsal when netvirt is ready"

Write netvirt:1 to mdsal when netvirt is ready

This new module will write netvirt:1 to mdsal when all the
netvirt services have started. There are 10 services that
are required. Some services are in the Service Registry and
can be referenced in the blueprint.xml. Other services can
only be found via the config subssystem beans. Once we move
the rest of netvirt to blueprint we can deprecate the config
related code.

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

Create ELAN interface for physnet ports

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

Create physnet bridges+ports and patch to br-int

NB: This commit depends on changes in the ovsdb
project: https://git.opendaylight.org/gerrit/#/c/41946/

AND NOW ALSO:
https://git.opendaylight.org/gerrit/#/c/42693/

Assume "provider_mappings=physnet1:ABC". ABC will either be (in this
order) (1) the name of a port already configured on br-int, (2) the name
of a bridge already configured on OVS, or (3) the name of a physical
interface. So for each option:

(1) ABC is a port already on br-int
At OVSDB node add/update event:
  no op
When adding the ElanInterface to the ElanInstance:
  ABC is the name of the port, use that

(2) ABC is the name of a bridge on OVS
At OVSDB node add/update event:
  patch the bridge to br-int, br-int side port name is ABC-patch
When adding the ElanInterface to the ElanInstance:
  ABC-patch is the name of the port, use that

(3) ABC is the name of a NIC
At OVSDB node add/update event:
  add the port directly to br-int
When adding the ElanInterface to the ElanInstance:
  ABC is the name of the port, use that

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

Merge "Flat/VLAN network type support"

Merge "Support Service Binding for ipv6service"

Merge "Security Group: Allowed address pair changes"

Merge "Add NetvirtItUtils.verifyFlowByFields"

Merge "Fix and disable cloud-servicechain config"

Support Service Binding for ipv6service

This patch supports ServiceBinding with genius for an interface
that is having an IPv6 address and is connected to a router.

IPv6Service uses genius NWConstants for the serviceIndex value.

Pending (will be taken up via a different patch):
  1. Flow to punt Neighbor Solicitation currently does not include
     the nd_target, as support is missing in MatchUtils.
  2. ServiceBinding for existing ports when a v6 subnet is added to
     a network and associated to the router.

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

Fix and disable cloud-servicechain config

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

InterVpnLinkNodeListener log message is happening about
8 times per second.

Changing it from info to debug.

Change-Id: Id0f8fe9b0a9c939daaf75a5614beb7855417dc7e
Signed-off-by: Andre Fredette <afredette@redhat.com>

Added license headers in cloudservicechain test

Also included fix from https://git.opendaylight.org/gerrit/42760
(reflection of NwConstants change in Genius) to enable
successful compilation.

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

Merge "Fixed NPO and and some other changes"

Security Group: Allowed address pair changes

    - Updated Neutron port listener to pass Neutron port's MAC + Fixed Ips
      as allowed address pairs to Acl Service
    - Moved all SG utility methods to NeutronvpnUtils

Change-Id: I44467157e9b5085b198d1a107f2f121c7ef814a4
Signed-off-by: unknown <shashidhar.raja@ericsson.com>

Removed unnecessary changes

Dependencies of interfcemanager-shell not required on pom.xml
of vpnservice in netvirt.

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

Flat/VLAN network type support

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

Merge "Updated to support allowed address pair with ACL service"

Fixed NPO and and some other changes

1)Fixed NPO in case of empty access list

2)Added code to support ether type.

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

interfacemgr-shell was renamed to interfacemanager-shell

See:
https://git.opendaylight.org/gerrit/42028/

Change-Id: Iadf8c11f79b6fac8835a1776ff398435b4b94750
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Clean up programDhcpMatches()"

Merge "Redefining DHCP_TABLE as 19"

Updated to support allowed address pair with ACL service

    - Updated to program flows with ip + MAC matches of VM and allowed
      address pairs in both ingress/egress directions

Change-Id: Ie50d136d5d05541873406d35a283741516cfa622
Signed-off-by: unknown <shashidhar.raja@ericsson.com>

Redefining DHCP_TABLE as 19

    Table 25 is already used by DNAT, so DHCP table is changed to 19

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

Merge "Adding cloud-servicechain module"

Merge "AclServiceOFFlowBuilder UT added"

Merge "Fix for DHCP with Metadata"

AclServiceOFFlowBuilder UT added

Change-Id: I52d6f2779139edba1012044caf5541ef8033e71a
Signed-off-by: Ravit Peretz <ravit.peretz@hpe.com>

Upstreaming changes in Netvirt

(Dependent on https://git.opendaylight.org/gerrit/#/c/42540/)

This commit includes VPN Engine Concurrency Fixes also some SNAT and DNAT
regression issues.

1. We have made multiple fixes here for safe concurrency.

2. We have moved all helper methods that deal with VRFEntry
resource into FibManager from VPNManager.

3. We have gracefully handled deletion of bgpvpn regardless of
how many ever external routes are available in such vpn.

4. We have fixed NATService to directly use FibManager to
program SNAT/DNAT VRFEntries.

5. As of this review, all local VM routes WILL NOT be batched for
write into VRFEntry Config DataStore.

6. All the external routes imported from BGP, WILL BE batched and
written to VRFEntry Config DataStore.

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

Fix for DHCP with Metadata

Fixing patch https://git.opendaylight.org/gerrit/#/c/40518/
The patch does not work after changes on the dhcpservice
module for adding support in dhcp as a service.

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

Clean up programDhcpMatches()

The method as it currently stands is rather opaque: using it requires
knowing how it works, since the data structure it returns needs to be
completed. This patch replaces it with two methods whose return values
can be used as-is.

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

Merge "Remove unused netvirt dependencies"

Merge "Fix for SingleFeatureTest wiring for neutron model"

Merge "code refactor for aclservice"

Remove unused netvirt dependencies

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

Fix for SingleFeatureTest wiring for neutron model

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

code refactor for aclservice

1. remove dupliate code in EgressAclServiceImpl and IngressAclServcieImpl
by using base class
2. constants are moved from AclServiceUtils to AclConstants
3. classes are moved to meaningful packages
4. class names are changed to make it more meaningful

Change-Id: If51e1a83db45d04c871799b563490c81a3573161
Signed-off-by: gujele <imanilyadav@gmail.com>

Adding cloud-servicechain module

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

Merge "NeutronVPN: Acquire lock from local JavaVM"

NeutronVPN: Acquire lock from local JavaVM

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

Upstreaming changes in netvirt

This commit includes following changes:

1> Fixing Port Up/Down Event handling for ELAN
2> Parallelization done in Arpcache

Parallelization of adding and removing arp cache done with the help of
DataStoreJobCoordinator.

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

Add NetvirtItUtils.verifyFlowByFields

This method checks that, for a flow which
exists in config, a flow with the
same priority and match fields exists in
operational. The flow is retrieved from
configuration datastore by flow-id and
then compared to flows in the operational
until a match is found...or not.

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

Multiple fixes in various modules

This review includes following fixes or changes:

1> BGPManager using the batching infrastructure
to batch and apply VRFEntries to Config DS.
(both bgp advertisement and bgp withdrawals).

2> FibManager using the batching infrastructure
to batch and apply flow rules to Inventory Config DS.

3> VPN Refactoring - Concurrency for VPNInterfaceManager

4> Changed the code in dhcpservice such that we add only
direct and macvtap vnic_type to the neutron port cache.

5> Fix for bgp display commands to work in COMCLI

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

Add missing license headers

Change-Id: I1c04ef692375fdc741858b77c1ae76b985da3721
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix build broken due to Checkstyle, and prepare for more CS to come

preparing for https://git.opendaylight.org/gerrit/#/c/41305/ with:
* Utility classes should have private constructor
* potential future problems re. hidden fields
* inner assignments

and also for the just merged
https://git.opendaylight.org/gerrit/#/c/40957/, which I think that
something here just got added and missed in the verification for
c/40957, so was about to cause build failures - so this one urgently
needs to get merged ASAP!

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

Merge "Bug 6066 - Improve the logging."

Merge "Fix Sonar (soon Checkstyle) constant name"

Merge "BGPManager FIXMEs fixed"

Merge "Vpnmanager module sync up"

Merge "Update eos to use service module: entity-ownership-service"

Bug 6066 - Improve the logging.

Change-Id: Ia695cde0f870768d85405d389a85d77fd636e46f
Signed-off-by: ananthip <ananthip@hcl.com>

Vpnmanager module sync up

This commit contains upstreaming of VpnManger module alongwith dependent
commits into fibmanager and neutronvpn manager

Change-Id: I631d9f3652c863180804cdf426d15e6e3ea1cfbc
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>

Fix Sonar (soon Checkstyle) constant name

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

Update eos to use service module: entity-ownership-service

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

Merge "Finish renaming odl-netvirt-vpnservice features."

BGPManager FIXMEs fixed

SInce odl-fib.yang is updated now, dependent classes in FIB are available,
hence the FIXMEs have been fixed.

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

Finish renaming odl-netvirt-vpnservice features.

Change-Id: I21a03c992e91ada7cd6d0764d0c8aaf01e7aa408
Signed-off-by: Andre Fredette <afredette@redhat.com>

NeutronVPN module sync up

Added 9 FIXMEs in VPNManager which came up as errors upon updating the
yang models for dependent NeutronVPN code.
This will be fixed in the upcoming VPNManager module sync up commit.

Fixed unit test

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

Change in NextHopManager.java

After name modification of one of the constant in ITMConstants.java
file in the commit https://git.opendaylight.org/gerrit/#/c/42034/2 ,build
of netvirt was failing.

This review gives a fix for that.

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

Merge "Add line tracking for UT"

Add line tracking for UT

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

Merge "Add Host Config functionality for Neutron"

Merge "Upstreaming DHCP L2Gateway connection removal fix"

Merge "FibManager module sync up"

Merge "Reduce IT logging"

Merge "Skip v4 subnets/ports in ipv6service"

Skip v4 subnets/ports in ipv6service

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

Merge "Add new feature called odl-netvirt-openstack to replace odl-netvirt-vpnservice openstack"

FibManager module sync up

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

Reduce IT logging

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

Remove odl-netvirt-* features.

Change-Id: I4f1af745eaa73ef3fb44b370bde048d3c30dc39b
Signed-off-by: Andre Fredette <afredette@redhat.com>

Add new feature called odl-netvirt-openstack to replace
odl-netvirt-vpnservice openstack

Change-Id: I1c102063c4bc4c85cba85621cfa90058145c017e
Signed-off-by: Andre Fredette <afredette@redhat.com>

Upstreaming DHCP L2Gateway connection removal fix

- Earlier when the l2gwconnection config is removed, the dhcp flows were
  deleted but if it is added again before the Logical Switch gets removed
  from the TOR, an event is never received for adding back the DHCP
  entries and we will never have any designated DPN. This has been fixed
  by moving the remove logic to the Logical Switch listener itself.

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

The odl-sfcofl2 feature has been renamed

- Now its called odl-sfc-openflow-renderer

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

Add Host Config functionality for Neutron

Patch to read host configuration data from OVSDB
external_ids and populates ODL neutron northbound
mdsal data as per yang model defined in the patch
https://git.opendaylight.org/gerrit/#/c/36767/.
Complete details of the feature available at
https://github.com/openstack/networking-odl/blob/master/doc/source/devref/hostconfig.rst
Patch for original netvirt code
https://git.opendaylight.org/gerrit/#/c/40143/

Change-Id: I7da8c8e5cc18d661b21c70381990b7c2a7ebb40c
Signed-off-by: Ritu Sood <ritu.sood@intel.com>