Bug 6765 : Overriding in_port in table0 with Zero value

Description : OVS split-horizon logic will drop pkts by default if value
of in_port and out_port is same. But, if we have two vlan-trunk member
VM for which trunk interface is same, the connectivity between these two
VM cannot be established as OVS drops such packets. So, it is required to
override the value of in_port so that split_horizon check is bypassed.
The in_port value will be overrriden using load(ex: load:0->NXM_OF_IN_PORT[]).

Change-Id: Iab5b87558beecee76a011bad33e1867311dc4a29
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Depends-On: Ica0a1b2da27fc189b8906462a4420473215a22af
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Bug 6557 : NPE thrown during Interface-mgr RPCs call

Description : When an get-dpid-from-interface and get-port-from-interface
RPC call is made on a interface object for which interface-state doesn't
exists, NullPointerException is thrown. Instead of NPE, proper error
message is required to be logged.

Change-Id: Iac989c673957eb30eca1d842381e8f2f4edf0a5b
Signed-off-by: Chetan A G <chetan.arakere.gowdru@ericsson.com>
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>

Merge "Adding srcMac to odl-arputil since ARP src mac is currently set to the OF port's mac internally and cannot be set externnaly. (e.g. in the case of GARP for floating ip we would want to use the floatingIp port's logical mac)"

Adding srcMac to odl-arputil since ARP src mac is currently set to
the OF port's mac internally and cannot be set externnaly.
(e.g. in the case of GARP for floating ip we would
want to use the floatingIp port's logical mac)

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

Fixed the use of index NwConstants

There was a change to support L2 before L3
https://git.opendaylight.org/gerrit/#/c/43126/
Change the use of constnats to the new way of work

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

Merge "Fix Dependency Injection IdManager appears to have to have a LockManager"

Merge "Table Id constant for L3VPN GW Table"

Merge "Bug 6539 : Exception seen during OVS disconnect"

Merge "add send packetout from controller"

Bug 6539 : Exception seen during OVS disconnect

Description : Data Validation exception been observed for port-names
prefixed with dpnid(Ex : br-int,br-ex).This is due to removeIngressFlow
and unbindDefaultEgressDispatcherService API getting called on these port
which should be avoided during OVS disconnect.

Change-Id: Ie631a3e40dc39cd0255968813f8334cc24ba6327
Signed-off-by: Chetan A G <chetan.arakere.gowdru@ericsson.com>

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

Table Id constant for L3VPN GW Table

* Changed id from 19 to 13

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

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

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

Removing Blind imports across the module

This commit removes blind imports across the genius module.

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

add send packetout from controller

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

Bug 6493 :Interface-Manager performance optimizations

Description: starting to use batching for tunnel interface configuration
             separating out transaction batching across shards
 optimizing service binding calls for non northbound interfaces
 fixing exceptions in interface-manager while ODL was shutting down

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

Bug 6490 : After VM migration, L3VPN flows are failed to program

L3VPN is processing both remove and add event parallely.
Because of this race condition occurs and in ends up in
removing the flows programmed for table 21 on new DPN.

This fix has introduced a delay of 10sec for VM migration
events for remove and add in interfacemanager code.

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

Bug 6479 : Exception from Ifacemgr while feature load

Description: When feature odl-netvirt-openstack is installed, an
NullPointer Exception is observed. This is due to
InterfaceInventoryStateListener getting triggered before the
entityOwnershipService is initialized in IfmClusterUtil which is used by
this listener. Changes done to initialize this entityOwnershipService
first followed by registration of Listeners.

Change-Id: I4aa2f1178eca10754d0f935d477b4246a0f4821a
Signed-off-by: Chetan Aakere Gowdru <chetan.arakere.gowdru@ericsson.com>

Add learn table constatnts

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

Remove unused fields in close() that caused NPE

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

Remove req-cap for eos

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

Merge "Table Id constant for L3VPN GW Table"

Bug 6439 : lower-layer-if information is missing in Op ietf-interfaces:state for the Tap interfaces

Moving to clustered DTCL in interface-manager to avoid race conditions
encountered during scale testing, when both northbound and southbound events
are triggered simultaneously.

Change-Id: If4a98acf50312a64ae03aaccc4562e5782716b2f
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: unknown <efaseel@IN00116280.ericsson.se>
Signed-off-by: Faseela K <faseela.k@ericsson.com>

Table Id constant for L3VPN GW Table

added table cookie and Id for L3VPN GW Table

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

Merge "add learn action"

Merge "Intro. new TestIMdsalApiManager implements IMdsalApiManager"

add learn action

this commit depends on https://git.opendaylight.org/gerrit/#/c/42702/

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

Merge "ACL: Removed deprecated ACL constants in NwConstants"

Fix Dependency Injection IdManager appears to have to have a LockManager

Therefore it's constructor should expect one, just like it does for
DataBroker (AKA "constructor injection") instead of having a separate
method for it (AKA "setter injection").

Unless a LockManager is optional for an IdManager, which doesn't seem to
be the case.

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

Merge "Bug 6388: Support of Non-unique names across OVS"

ACL: Removed deprecated ACL constants in NwConstants

+ All the references in netvirt is already updated to use non-deprecated
constants in the below commit.
https://git.opendaylight.org/gerrit/#/c/43640/

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

Bug 6388: Support of Non-unique names across OVS

Description: When two nodes connect and those two nodes have interfaces
with the same name (They all do, br-int), each time one of those
interfaces gets added to the InterfaceState, it will erase the previous
one. Changes done to check VM migration only tap/vhu/tun ports and prefix
dpnid for all other type of interfaaces

Change-Id: I9e7d930e294836b2a9be6facf5e0d5847ce1cffe
Signed-off-by: Chetan Aakere Gowdru <chetan.arakere.gowdru@ericsson.com>

Merge "log cleanup - remove errors on resource manager pool init"

ArpUtilImpl fixes

- Add egress output actions when sending ARP request
- Fix NPE in getPortFromInterface

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

Intro. new TestIMdsalApiManager implements IMdsalApiManager

and publish a test-jar which is then used via <type> e.g. in aclservice
testing (moved here so that it will be available to other projects as
well later)

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

log cleanup - remove errors on resource manager pool init

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

.gitignore .factorypath created by m2e-apt

https://github.com/vorburger/opendaylight-eclipse-setup now bundles
https://github.com/jbosstools/m2e-apt, which creates .factorypath
files all over the place.

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

Merge "Install flows into Table 36 only for LLDP."

Install flows into Table 36 only for LLDP.

ITM will not install flows into Table 36 for BFD monitoring.

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

Bump ietf versions to ...10-SNAPSHOT

Bump versions according to:
https://lists.opendaylight.org/pipermail/release/2016-August/007731.html

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

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"