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>

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"

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"

Load config system files in etc/opendaylight/karaf

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

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>