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 "Changes in different modules"

Merge "Enabling interfacemanager-shell"

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"

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."

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"

ArpUtil module sync up

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

Merge "IFM: InterfaceStatecache not updated correctly"

Merge "ITM: Fix for tunnel ports not getting deleted."

IFM: InterfaceStatecache not updated correctly

1. CacheInterfaceStateListener is listening on CONFIGURATION DS changes
but it should be Operational as state changes are only made on operational
DS.
2.cache.remove() methods in InterfaceManagerCommonUtils use remove(key,
oject) to delete from cache. This is not an accurate representation of
deletes on DS where we only use key and leads to inconsistent behavior.
This fix changes those methods to only use key [interface name] to remove
from cache.

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

Merge "Utility method to get port termination point is added in HwvtepUtils.java"

Upgrade ietf-{inet,yang}-types to 2013-07-15

Change-Id: Ia762d7514d2426b3062a4e8d79ca7c9c5d35226f
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Merge "Upgrade ietf-{inet,yang}-types to 2013-07-15"

Utility method to get port termination point is added in HwvtepUtils.java

This utility method is used in netvirt project

Change-Id: Iff23f3011d9c06a4f854ad22d653a8c822149899
Signed-off-by: Shashidhar R <shashidhar.raja@ericsson.com>

IFM: Add caching to optimize DS reads

This adds caching for Interface and InterfaceState to avoid reads on
Config and OperationalDS. For now it deprecates use of API that needs
InterfaceKey/InstanceIdentifier, a later cleanup activity should remove
all references to them.

This also adds a minor change to retry when submitting to DSJobCoordinator

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

Merge "Remove genius_checkstyle.xml fork"

Merge "ITM: Dont use IdManager to generate tunnel name"

Merge "Added new matches and constants."

Remove genius_checkstyle.xml fork

Let's just use the shared one from odl-parent by inheritance.  As almost
no
projects in Genius currently "enforce" Checkstyle anyway, this change
will just cause people to see probably slightly different Checkstyle
messages, but nothing on the build should fail.

Only resourcemanager-impl already enforced Checkstyle compliance to fail
the build.  This change thus includes two CS related fixups to it.

Subsequently people, incl. perhaps me, may then start fixing more
projects, and turn on enforcement of Checkstyle in (more) Genius
projects, gradually.

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

ITM: Dont use IdManager to generate tunnel name

This replaces call to IdManage to get Id part of Tunnel name with
generated uuid. IdKey becomes the seed for generated uuid. Tunnel
interface name is 'tun' followed by first 11 alphanumeric characters of
uuid generated.

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

Initializing bean List fields with empty lists make them easier to use

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

ITM: Fix for tunnel ports not getting deleted.

During deletion of transport zone, tunnel ports (on ovs) connecting to
l2gw devices were not getting deleted.

Root cause: During deletion of transport zone, there was a logic fetching
transport zone from config DS which actually doesn't exist.

Fix: Changed the logic to pass transport zone object from
TransportZoneListener to the worker.

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

Merge "JUnit test case for ITM"

JUnit test case for ITM

JUnit test cases for TepCommandHelper.java

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

Merge "Monitor Interval range changed. Min value 1 sec."

Monitor Interval range changed. Min value 1 sec.

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

Issue : BFD Session with 100ms interval is not getting established

Analysis:

As per ovs code & bfd rfc :
 /* RFC 5880 Section 6.8.3
  * When bfd.SessionState is not Up, the system MUST set
  * bfd.DesiredMinTxInterval to a value of not less than one
  * second
  * (1,000,000 microseconds).  This is intended to ensure
  * that the
  * bandwidth consumed by BFD sessions that are not Up
  * is negligible,
  * particularly in the case where a neighbor may
  * not be running BFD. */

   whatever Tx interval we configure less than a second, ovs is taking 1
sec.
  So this is an expected behavior

Fix:
  Since its an expected behavior no need of fix.
  But we don’t provide options to configure Rx interval and decay min
  rx in ODL, making changes just to pass enable and min_tx instead of
  passing all the unconfigured parameters as well. And let the ovs
  take the default min_rx and decay_interval.

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

Upgrade ietf-{inet,yang}-types to 2013-07-15

Change-Id: Ifb05f4be62744715a6f2726416827d2db9e04670
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Merge "Making all these fields final and explicitly initialize to null"

Merge "Getters of List<T> in Beans should not return List<? extends T>"

Making all these fields final and explicitly initialize to null

This makes the intention of the code in these classes much clearer to
read (the idea of the API as-is seems to be that always only either of
the 3 is set).

Removal of setInstructionValues in InstructionInfo is only consistent
then.

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

Merge "Implement VxLAN-GPE tunnels in Genius"

Merge "lport-tag interface map information lost during cluster reboot"

lport-tag interface map information lost during cluster reboot

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

Getters of List<T> in Beans should not return List<? extends T>

see
http://blog2.vorburger.ch/2016/07/generics-in-list-fields-of-beans.html

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

Add convenience syntactic constructors and setter

For long instead of BigInteger, and int instead of short.

Both are handy e.g. when programmatically creating FlowEntity
instances in test code.  I don't see any downside to alternatively
offering this, to make test code quicker to write and less verbose to
read.

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

Fix Flow/GroupEntity & Co. toString() to be useful

Fixes a number of strictly speaking slightly separate problems:

  1. FlowEntity/GroupEntity: The way the parent class super.toString()
was appended at the end instead of at the start was visually just very
ugly and confusing.  Removal of toString in AbstractSwitchEntity
accordingly, because .. that can never get called now.

  2. The use of the private internal "polish notation prefix" instead of
the classes public (as in getter and setter) field names  was just ugly
and confusing.

  3. ActionInfo, NxMatchInfo, MatchInfo, InstructionInfo and BucketInfo
were all simply completely missing a toString().  This is very bad
because they are contained in FlowEntity (GroupEntity), and thus need
toString as well.

  4. The use of Arrays.deepToString & Arrays.toString is because Guava
18 which ODL uses as of now does not do this automatically (so you get
"[J@493968a9" etc.).  With the switch to Guava 19 proposed in
https://git.opendaylight.org/gerrit/#/c/41515/ this will not be required
anymore.  It doesn't hurt either then, so for now to make this proposed
change not require that upgrade yet and keep things separate, we must
use it for all array (but not List) type fields.  FTR: The minor
disadvantage of Arrays.(deep)ToString is that it renders
omitNullValues() ineffective, so once we are successfully on Guava 19 on
master, it would leader to shorter = nicer toString() output to remove
them again.

This overall change should, of course, not have any impact anywhere,
because toString methods are ONLY supposed to be used for debugging and
logs; any code that relies on e.g. parsing their output would be a bug.

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

Added new matches and constants.

1)Added ICMP and ARP Mathes
2)Added new constants.

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

Implement VxLAN-GPE tunnels in Genius

Now Genius can create Transport Zones using VxLAN-GPE tunnels. The
new type is defined as "odl-interface:tunnel-type-vxlan-gpe". The creation
of VxLAN-GPE tunnels requires a switch with that capability implemented,
as the unofficial OVS v2.5.90.

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

Adding new API's to MDSAL manager

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

Merge "Ingress/Egress Service Binding Renderers"

Implementation yangs for resourcemanager

This patch contains implementation yangs and starter files for resource
manager bundle.

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

Ingress/Egress Service Binding Renderers

Description: Adding a generic model, so that based on the
service mode, specific renderers can be invoked
to do bind/unbind service.
eg : If service-mode is ingress,
FlowBasedIngressServicesConfigBindHelper.bindService()
will be executed.

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

Rework LockManager.getLock() in iterative fashion

getLock currently uses recursion, which limits the retries to the
available stack depth and causes some code duplication. Rewriting them
using loops feels clearer to me and simplifies the code.

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

Merge "Drop Apache Felix gogo packages"

Merge "Drop LockManagerUtils.convertToMillis()"

Merge "Use odlparent-lite for aggregator"

Merge "Nova Evacuation doesn't program Flows and groups"

fix mdsal set ips ActionTypes to use netmask

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

Nova Evacuation doesn't program Flows and groups

Nova Host Evaucation doesn't program flows and
groups on target host.
Interfaces state will be set to unkown if the connection
is lost between controller and swith.

Even in evacuation case, switch will lose connectivity
with the controller and the VM present in those Switch
will be evacuated to the new switch.

So, all the flows and groups should be in the new switch.
Events comes from new switch is updating existing entry
instead of deleting and add.

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

Drop Apache Felix gogo packages

These appear to be unused in Genius, and are obsolete anyway.

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

Change OFP feature dependency

 - changed odl-openflowplugin-all to odl-openflowplugin-flow-services-ui
   since it installs same features/bundles and we would like to remove
   OFP-all feature

Change-Id: I9c6b879a4e483d7e747ebc811a4ffce07a14fd82
Signed-off-by: Andrej Leitner <anleitne@cisco.com>

Merge "Datamodel changes for Egress Service Binding Support"

Datamodel changes for Egress Service Binding Support

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

Merge "DC Gateway IP Persistence"

DC Gateway IP Persistence

Building external tunnels to DC Gateway for new TEP addition to an
existing mesh.
Deleting external tunnels from DPN when the last available tep is deleted.

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

Drop LockManagerUtils.convertToMillis()

It's simpler (and shorter) to use the underlying TimeUnit method.

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

New resourcemanager bundle with yang models

This review contains initial changes for resource manager and relevant
yang models.

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

Adding RPCs in ITM which will be made use of by L3VPN
for handling ITM tunnel state events.

Change-Id: I5cee3945acb27569f0a4f048d8aa9121f5b9757b
Signed-off-by: Chintan Apte <chintan.apte@ericsson.com>

Merge "Removing Table 0 entry for the vm migrated to new dpn."

Merge "Modified Match and Action utilities to support Nicira extension required for connection tracking."

Removing Table 0 entry for the vm migrated to new dpn.

Description: After VM migration, table 0 entry for VM
exists in older dpn as well. NodeConnector for older
DPN is passed for OvsInterfaceStateRemoveHelper

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

Use odlparent-lite for aggregator

Change-Id: Ia505280cf494b810357178b44b3889a0809b37fb
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Dynamic resource-ids allocation

This patch handles the dynamic allocation of a range of resource-IDs. The
allocateIdRange rpc call returns an ordered list of resource-IDs as per the number
of IDs requested.

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

Merge "Add support for vxlan-gpe in YANG files"

Fixing issues with tunnel deletion when switch is disconnected

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

Modified Match and Action utilities to support Nicira extension required
for connection tracking.

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

ITM: Implementation of BFD monitoring

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

Add support for vxlan-gpe in YANG files

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

Merge "Fixed reverse tunnel delete on TEP delete and NPE"

BFD monitoring support for internal tunnels

Description : Currently Genius uses only LLDP monitoring for internal
tunnels. Enabling the flexibility to configure tunnel of choice, either
LLDP or BFD

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

Fixing Tunnel Delete Issues and some RACE conditions
Description:
   1. Releasing if-index on tunnel-deletion
   2. Removing transaction batching across shards for tunnel deletion
   3. Handling tunnel deletion while a switch is stopped.
   4. Interface-manager southbound events were synchronized on tap port
      name, and northbound events were synchronized on interface-name.
      However in a nova delete VM cases, both the northbound and
      southbound
      events can be triggered in any order, and this can cause
      RACE conditions.

   5. Handling the same for port update scenarios

   6. Optimization to handle interface
      creation/deletion/updation
      parallelly on southbound events

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

Fix for the Bug 5533 - NumberFormatException in AlivenessProtocolHandlerLL.java

Problem Statement : The NumberFormatException was observed in VPN service if 3 node cluster are started at the same time in the class AlivenessProtocolHandlerLL.java. This was reported in vpnservice and genius.

Analysis :  By looking at the implemenation of AlivenessProtocolHandlerLLDP.java, there is a unused local variable (i.e. packetId ) in handlePacketIn() mentod which caused the issue. This will be definitly fixed if the assignment is removed since the

Fix Detail: The unused variable (i.e. packetId ) in handlePacketIn() from the class AlivenessProtocolHandlerLLDP.java is removed to avoid this issue

Change-Id: Idbca6a6ba31626842ded7d82f00e9693a0dcbf5f
Signed-off-by: Dhevendran <dhevendran.k@tcs.com>

Fixed reverse tunnel delete on TEP delete and NPE

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

Move fcapsmanager and application to genius

These two need to be in genius, not vpnservice or netvirt.

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

Adding ITM unit testcases and syncing mdsalutil changes from vpnservice

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

Postman collection for l2vlan interfaces, tunnel interfaces and ITM
transport zone CRUD operations

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

Merge "Syncing the bug fixes that went in vpnservice after the initial porting of modules to Genius"

Syncing the bug fixes that went in vpnservice after the initial porting of
modules to Genius

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

Merge "Add menu link for Java apidocs"

fixed the directory structure for mdsalutil module

Signed-off-by: Vivek Srivastava <vivek.v.srivastava@ericsson.com>

Merge "Porting the modules from vpnservice to genius"

Porting the modules from vpnservice to genius

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

Add menu link for Java apidocs

The site.xml allows you to control your left menu bar for your Maven
Site.

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

Add .gitignore

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