ACL: Updated ACL related constants in NwConstants

+ As the pipeline is w.r.t switch, updated ACL ingress/egress constants to
align w.r.t switch instead of VM.
+ Earlier constants are deprecated, will be deleted when the references
are updated in netvirt.

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

Bump versions by 0.1.0 for next dev cycle

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

Merge "Use odlparent dependency management for guava"

Merge "Merge L3Vpn service Id check for Arp Learning"

Use odlparent dependency management for guava

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

fixed some formatting problems

Change-Id: I85d3e24b1a280b840dedc265e8aa64da59154378
Signed-off-by: David Goldberg <gdavid@hpe.com>

Merge L3Vpn service Id check for Arp Learning

Change-Id: I965c3997b7437219f1f2bc449bf8ee0435499563
Signed-off-by: Achuth <achuth.maniyedath@ericsson.com>

GetEgressACtions giving wrong actions for tunnel interfaces

Description: Packets to egress via tunnel need not go via Egress
Dispatcher Table, since currently we don't have egress binding on tunnels.
Making getEgressACtions() API return normal egress actions for tunnel
ports, and returning goto egressTable action only for l2vlan interfaces

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

Merge "Having maximum id block size"

Merge "Fix cpu hog in JobQueueHandler thread"

Merge "Add service names for priority configuration"

Merge "Log message improvements in ResourceBatchingManager"

Add service names for priority configuration

Change-Id: I98c25db1bd9d4db9ab9174f9b8edec0f7f3ccb4e
Signed-off-by: Konsta Pozdeev <konsta.pozdeev@hpe.com>

Merge "Support for binding multiple egress services on the same interface"

Having maximum id block size

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

Log message improvements in ResourceBatchingManager

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

Fix cpu hog in JobQueueHandler thread

When the jobEntriesMap is empty, this thread consumes
as much CPU as it can, spinning and wasting cpu
cycles.

Add a small 500ms sleep, when the jobEntriesMap is empty.

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

fix compilation error due to a change in openflow plugin

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

Fix for bug in cli elaninterface:show

The cli "elaninterface:show" listed the operational state of the
interfaces as "UP" even after their DPN is disconnected from the
controller. Only handling for the states "UP" and "DOWN" were present in
the code. Changes have been made for handling of the "UNKNOWN" state.

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

Support for binding multiple egress services on the same interface

Description : Adding support for binding multiple egress services on the
same interface. Whenever an egress service is bound on an interface, the
following operations will happen:
1) an entry will be created in Table 220(Egress Dispatcher Table) to match
on the lport tag and service index, and take the packet to the respective
egress table pipeline
2)Based on the service priority, entries in table 220 will be reordered,
so that highest priority service will match first in the pipeline
3)Applications should do a resubmit from their pipeline, back to table
220,if the packet is not destined for their pipeline

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

Merge "Set VRF-ID metadata so it would not to override split-horizon bit"

Set VRF-ID metadata so it would not to override split-horizon bit

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

Using Table Ids from NwConstants.java

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

This thing is already done in netvirt module.
https://git.opendaylight.org/gerrit/#/c/42776/

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

Merge "Updated NwConstants with ACL related constants"

Merge "Added phyAddress field in InterfaceInfo"

Updated NwConstants with ACL related constants

+ Added new constant INGRESS_ACL_SERVICE_INDEX for egress service.
+ Wanted to rename EGRESS_ACL_NEXT_TABLE_ID to EGRESS_ACL_FILTER_TABLE_ID,
currently deprecated these constants to avoid compilation issues in
netvirt.
+ EGRESS_ACL_NEXT_TABLE_ID and INGRESS_ACL_NEXT_TABLE_ID will be removed
once references are updated in netvirt.

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

Merge "TCP and UDP matches added with mask in the Nicira extension"

Added phyAddress field in InterfaceInfo

- Included phyAddress field in InterfaceInfo
- Set this field before returning InterfaceInfo

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

TCP and UDP matches added with mask in the Nicira extension

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

Set split-horizon bit to ingress table for external ports

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

Merge "Fix and renable FlowBasedServicesConfigurationtTest tests"

Merge "Add support for physical network on flat/vlan provider networks"

Fix and renable FlowBasedServicesConfigurationtTest tests

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

New Table ids in NwConstants.java

This review includes few new table ids in NwConstants.java file, so that
we can use these table ids globally across all module.

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

Merge "Addition of new table ids in NwConstants.java"

Addition of new table ids in NwConstants.java

To keep all the table ids at the common place, adding few new constants
from CloudServiceChainConstants.java to NwConstants.java file.

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

Use short for the ServiceIndex constants

The ServiceIndex in the yang file is a short, whereas the serviceIndex
constants in the NWConstants file are defined as int. This patch updates
the constants accordingly.

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

Update NWConstants with IPv6 table-id and cookie

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

Merge "Addition of new Table in NwConstants.java"

Fix and renable FlowBasedServicesConfigurationtTest tests

Test was failing because the cache was returning an interfaceState
object from a different test.

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

Add ICMPv6 nd_target match

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

Add support for physical network on flat/vlan provider networks

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

Addition of new Table in NwConstants.java

This review includes addition of new Tables in NwConstants.java file so
that these constants can be used universally across all module.

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

Fixes build breakage caused by change 41849

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

Merge "Make ActionInfo immutable"

Increase max service-binding's to 7

With new services coming up and using the ServiceBinding framework,
the existing limit of 3 needs to be appropriately updated.
This patch increases the max-service binding value to the maximum
supported value.

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

Document method and format changes

Minor changes: Javadoc for a method and formatting changes. Remove an useless null assignment.

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

Merge "Added Ipv6 source and destination match."

Merge "Modification in ResourceBatchingManager.java"

Remove wildcard imports

Remove wildcard imports to comply with Open Daylight's coding
guidelines.

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

Modification in ResourceBatchingManager.java

This change is done to support a change in netvirt that deals with
VPN Engine concurrency fixes.

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

Added Ipv6 source and destination match.

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

Remove static block from ResourceManagerUtils

These are usually not a good idea for anything but very trivial code -
just in case anything goes wrong within the static, it leads to hard to
find problems, because they'll occur during class loading instead of
when methods are called. See e.g.
https://git.opendaylight.org/gerrit/#/c/39715/ for an example.

On a separate note, it is also wrong to use e.getMessage() in a LOGGER.

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

Merge "Fix icmpv6 matchbuilder"

Merge "Checkstyle fix"

Merge "Checkstyle clean-up src/test/java "

Checkstyle fix

preparing for https://git.opendaylight.org/gerrit/#/c/41305/ with:

* Utility classes should have private constructor

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

Fix icmpv6 matchbuilder

This patch addresses a typo in matchbuilder and allows us to
program an icmpv6 flow specifying icmpv6-type/code value.

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

MDSAL module sync-up

Providing a simple batching infrastructure
for modules that use MDSAL DataStores

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

Merge "Modification in JUnit test case file"

Merge "Load config system files in etc/opendaylight/karaf"

Modification in JUnit test case file

After few merges went in InterfaceManager module, JUnit test case for
vlanInterfaceConfigurationTest.java started failing. For Jenkins build to
pass commit https://git.opendaylight.org/gerrit/#/c/42216/ ignored those
test cases.

This review again considers those test cases and gives a fix to pass all
of them.

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

Merge "Increase max service-binding's"

Merge "ITM module sync-up"

Merge "Multilple fixes in InterfaceManager module"

Merge "Implementation files for resourcemanager bundle"

Merge "add MatchFieldType for tcp flags"

Merge "Multiple match support for nicira extension"

Increase max service-binding's

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

Checkstyle clean-up src/test/java

Required to be able to run (enforce) CS on src/test as well.

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

ITM module sync-up

Below review gives a fix for the following issue:

FIB and Routes are not re-written on TEP Changes

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

Multilple fixes in InterfaceManager module

This review deals with following fixes:

1> VM Migration Flows are not installing.
2> Elan flows are not getting installed
3> Fix for live provisioning of neutron ports.
4> Ignored 3 test cases in VlanInterfaceConfigurationTest.java file.

Beacuse of one of the previous merges in InterfaceManager module, these
JUnit test cases started failing. Because of that all Jenkins build were
failing. Ignored for now and later will fix for the same.

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

Merge "Upstreaming MdSalUtil interal code changes to ODL"

Support for same OF port Name across different DPN

Description: Dpnid is prefixed to the Parent Inte-
-rfaces(<dpnid>:<interfacename>) in the Interface
State Operational DS, that fixes issue of,if two
compute nodes are managed by a single ODL and both
define "eth1" as the interface for external
network, their entries will overlap and the
interface-state subtree will contain only one
entry.

Change-Id: I2bf4f4462d9b440e86523f15f3770aecd4059799
Signed-off-by: echegow <chetan.arakere.gowdru@ericsson.com>

Implementation files for resourcemanager bundle

This change contains all the implementation files for resourcemanager.

Change-Id: I4278cc9cd121a9d32c48b8f83e1a3e7ff5965ec8
Signed-off-by: karanrajsingh <karan.raj.s.singh@ericsson.com>

Upstreaming MdSalUtil interal code changes to ODL

Description :- Compared the MdSalUtil module between the internal code
base and ODL master and upstreaming the code that has not yet been pushed
to ODL master

Change-Id: I56fa1366f3154b2a870502d103a297995b9ba297
Signed-off-by: HemaTG <hema.gopalkrishnan@ericsson.com>

Merge "Introducing Builders for FlowEntity & Co. beans"

Merge "Changes in different modules"

Merge "Enabling interfacemanager-shell"

Introducing Builders for FlowEntity & Co. beans

I need this for tests I'm writing, and it seems like a good idea anyway,
for consistency with the YANG generated beans.  (Doing all this recent
work on FlowEntity & Co. fixing equals() & hashCode() and toString()
etc. and now this, I'm starting to seriously wonder if all this stuff
shouldn't just simply also be YANG modeled... thoughts?)

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

Merge "Monitor interval range change."

Merge "Upstream : Performance Improvements"

Merge "Avoid deletion for the interface while adding"

Merge "Portname was dispalying somestring and now updating with actual port name from parentref"

add MatchFieldType for tcp flags

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

Changes in different modules

This commit deals with upstream changes in ITM and Interfacemanager module
that was done for bug CSC cluster reboot failure for 100 VPN scenario

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

Enabling interfacemanager-shell

few dependencies, etc. were missing to enable the shell bundle at runtime,
fixed it.

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

Monitor interval range change.

Min interval value changed from 100 to 1000.

Change-Id: I3865d0ffba58b48cd76a58e0d8cd9f20ae318ede
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Addition of new constants in mdsal

Following commit includes few additional constants in following file:

1> MetaDataUtil.Java

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

Portname was dispalying somestring and now updating with actual port name from parentref

Change-Id: Ie7b5043116a1ed1c16c3897de689c7dc220f8da5
Signed-off-by: Sharath Kumar V <sharath.kumar.v@ericsson.com>

Upstream : Performance Improvements

IDMANAGER

- Allocating Id block to child pool when the pool is created
- The childPool entry updated for that particular instance for which lock
  manager is not required.
- Added a check before debug log messages
- The id entry in parent pool is updated directly so lock manager is not
  required
- Batching transactions where ever posssible

DATASTOREJOBCOORDINATOR

- The JobQueueHandler waits until a new job is enqueued. Earlier it used
  to sleep for 1 second and again check for new jobs.

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

Merge "Supporting DHCP as a service"

Merge "Implement missing equals() & hashCode() for FlowEntity & Co."

Multiple match support for nicira extension

Modfied nicira match types to support multiple nicira extension matches
in the same flow.

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

Implement missing equals() & hashCode() for FlowEntity & Co.

PLEASE CAREFULLY CODE REVIEW BY SEVERAL PEOPLE.

This is required to compare FlowEntity instances e.g. in tests.  It's
also a Good Idea in general to have all "value" objects always properly
implement equals() & hashCode() ..

The MoreObjects2 is just a novel pattern I came up with for how to keep
equals() implementations short and concise.

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

Avoid deletion for the interface while adding

Avoid deletion for the interface when add comes
from the same dpn.

Change-Id: I0fc66ba7943e0e83c8bfee57baa6ec1dacea63f8
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>

update even when original schema has no DPNs

Change-Id: Iee9692ce18445e7b9a7af08b75e581c892bd6f43
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>

Supporting DHCP as a service

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

Merge "ArpUtil module sync up"

Merge "Add convenience syntactic constructors and setter"