Merge "Bug 4957 Wiring new TxChainManager lifecycle"

Merge "Bug 4957 Role lifecycle support for TxChainManager in DeviceContext"

Merge "Bug 4957 Fix RoleChangeLifecyle (init, close, life)"

Merge "Bug 4957 Fixing DeviceCtx lifeCycle"

Merge "Bug 4957 TxChainManager lifecycle startup cleaning"

Merge "Bug 4957 RoleContext updated with initialization"

Merge "Bug 4957 No empty transaction for every connection fix"

Merge "BUG-5192: FRM skips reconciliation"

Bug-4866 - [Clustering]: Switch state resync is not
happening after controller restart [Routed RPC issue]

Change-Id: I6ca6b8c2a4a38b908571300eab0fda3cda078af7
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Bug-5198 : He Plugin:Inventory manager is deleting

the node from inventory data store, when switch disconnect from slave node.

Change-Id: I19155e84444b57eab94157c34eaa32952fecf3f2
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 69da70673ed8afe48eec65ed5ffab750e6cac444)

Bug-5160 - He plugin : Device ownership change should not send nodeAdded notification

Change-Id: I027101f22a6a5e38ff2ebce6ab603b6c92928c17
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 3fc75a38d7b8883fcfd4cedffdeca7c4e55c1577)

Fixed following issues
1) Statistics collector to stop writing when
device is not connected and rpc's are deregistered
2) Stop send default slave role
3) Disable aggregate statistics (working on better implementation)

Change-Id: I3cb9e347e0cbe3f18222380c784ef4edc724b449
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 5c3f50ee20b4899f763e42fa4d640717106dc0c0)

Send node details notification up to the application,
once switch role is configured. Currently application were getting notified when switch Description message
received by the controller, and application assume that switch is configured and ready to use.
This patch, send the desc request when switch role is configured and it's ready to use

Change-Id: I8a02c209c4796b7aa01161257d523139fdc9c881
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit 636fd4b5cdf0751c684b0c6996523a9fedb5c63a)

Fix table 0 initialization in Inventory-manager

Patch 2: OF plugin should not explictly try to cleanup the
operational data store (inventory), that's the job
of Inventory-manager.

Change-Id: Ia864727fd7fd0e3a5d8bc242d82c5f23a6cbcea6
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit d41e9c63a6df9397ec6bdb855d507682f77dde7e)

Table features support for existing openflowplugin

Patch 2 : Fixes javadoc error

Change-Id: Ibabecb59848c5df470aace8f93b355d1e700d261
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit fa7311ff2ba8a07a578861fbbb2481400213cd29)

Set entity ownership service as mandatory true

Patch v2: Disabled openflowplugin internal IT tests,
because openflowplugin module uses EOS service and there
is no implementation for In Memory Data Store. Openflowplugin
IT loads old openflowplugin implementation and In Memory Data
Store, but after making old openflowplugin clsuter aware, it uses
EntityOwnershipService, and that's where these IT tests fails,
because there is no EOS implementation for IMDS. Apart from that
these integration tests are bare minimal and we do have all these
tests upstream CSIT.

Change-Id: I8d458d8083f0a1095d6992f635f433d9117250ca
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit ab1676bbf9c203dcebfdfe6098a9c0e1ea3f89e2)

Clustering support for existing design (FRM,IM,SM) - add retry mechanisms to avoid race conditions between cluster role change and yang notification process

Change-Id: Ie83dceae61d483f4ab34e305062db36d296b8b08
Signed-off-by: Hariharan_Sethuraman <Hariharan_Sethuraman@dell.com>
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit f43a6bcc47e197288e3ab060ef60b9278895fa3f)

BUG-5192: FRM skips reconciliation

    - if FRM starts upon dirty DS/operational
      then update part of dataChangeEvent is used
      and reconciliation is forced

Change-Id: I486b467cb278b86c06d497f7fe598263476f635b
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>
(cherry picked from commit d19e54f3f0f9a85e87e70ca3fb97a2cb7a1bab85)

Bug 4957 Wiring new TxChainManager lifecycle

* TransactionChainManager constructor change
* OpenflowOwnershipListener should be a last in MASTER
  Node connection close instance (clean OwnershipListers)
* RoleManager - check registration for RoleCtx
* RoleContext - should hold a last DeviceCtx instance for MASTER

Change-Id: I55977f0bf1af3ade36f02f7c097a4472d4989ae7
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 4957 Role lifecycle support for TxChainManager in DeviceContext

* DeviceContext
 - add onClusterRoleChange method for role change
 - add onInitClusterRoleChange method for role change
   in new connection initialization phase
 - modify onDeviceDisconnectedFromCluster method
* TransactionChainManager
 - add activateTransactionManager method
 - add deactivetTransactionManager method

Change-Id: Ibfcb3ddfa74067152a65e5f5c3122f14ebb9d617
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 4957 Fix RoleChangeLifecyle (init, close, life)

 - OpenflowOnwerhipListener
   - add jdoc
   - remove own threadPool
   - add unregistration method
 - RoleManager
   - add check w/Exception for RoleChangeListener
   - add check w/Exception for RoleContext
 - RoleContext
   - add notes and deprecated markers

Change-Id: I0269d0568a67619a1329f596ec8a02007e331bb0
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 4957 Fixing DeviceCtx lifeCycle

 - validate unique DeviceCtx registration in DeviceManager
 - onDeviceContextLevelUp has to stop PacketInFiltering for slave too
 - simplification of teardown
 - invocation of closeHanders in reverse order

Change-Id: I6f4f172e9cb7497c91a03712a4c5d571e99b7f41
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 4957 TxChainManager lifecycle startup cleaning

 - when proceeding connectionContext to deviceManager
   we rely on fact that for given nodeId there must NOT
   exist deviceContext. If there is one then connection
   will be refused because previous session is not fully
   tore down
 - any exceptions thrown by proceeding connectionContext
   will result into disconnection
 - first thing to do for new deviceContext is to build up
   close mechanism

Change-Id: I860768c69849652d36ed9078016ffd9bf5f20c6a
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 4957 RoleContext updated with initialization

 - introduced blocking waiting for device role
 - we probably need blocking call to EntityOwnershipService
   for Netty thread

Change-Id: I4a89f07b9594975dae77219af09a9909f4a39dcd
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug 4957 No empty transaction for every connection fix

Every new connection populate transaction in posthandshake
process. So in closing time every (MASTER / SLAVES) instances
make concurent modification submit.
We would like to prevent to have some data for SLAVES in
postHandshake process in transactionManager.

* move initialization NodeInfo data from DeviceManagerImpl
to new util class : DeviceInitializationUtils

Change-Id: I2e70e53e7b21409fe61270690a502d8222ccaec7
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug-5159 : Statistics transaction failure exception don't include node details

Change-Id: I6d8847e447a7add183cc1ab1298309ef724ff747
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

BUG-2625: update equinox

This bumps dependency to equinox 3.9.1

Change-Id: If55f3853cfc8c21f9770dd7b4e69b7a61dd2a91a
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-5001: eliminate executor

Keeping an executor around just to execute tasks synchronously simply
does not make sense. Refactor the code so that it executes in the
calling thread.

Change-Id: Icc5e16f4475b8b461a1a6e8baca102ba66c2565a
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-5001: clean up state transitions

This is preliminary code cleanup, reducing visibility and moving methods
around so their scope and assumptions can be analyzed.

Change-Id: I2699a19a897635447c54eb4beeac3cc70c64f1f5
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bump yangtools to 1.0.0-SNAPSHOT

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

Bump version to correct Boron version

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

Merge "BUG-4117: initial commit for new App Old-Notification-Supplier"

Bump versions by 0.1.0 for next dev cycle

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

Merge "Cluster aware forwarding rule manager Forwarding rule manager can fail in 3-node cluster if actionNodes cache is not populated properly during the shard leader change. This patch properly populates the internal activeNodes cache"

Merge changes from topic 'BUG-3774'

* changes:
  BUG-3774: 100k flows initial stats fail - fix
  BUG-3774: 100k flows initial stats fail - logging

Cluster aware forwarding rule manager
Forwarding rule manager can fail in 3-node cluster
if actionNodes cache is not populated properly
during the shard leader change. This patch
properly populates the internal activeNodes cache

Change-Id: I74c11d92b6d9cef6f001422dcb624ccdc61fd68b
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Fix checkstyle error for java 8 build success

Change-Id: I6240c6b9e7b329ad16387c6d50c21601fc9067c0
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

BUG-3774: 100k flows initial stats fail - fix

 - synchronized all handshake entry points
 - added sweeping barrier after handshake succeeds
 - fixed unit test

Change-Id: I651cd2244c1f58e53503e7af46e19254975d5418
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit d45ce11aa158cea9bff6648f529dac978758ab27)

BUG-3774: 100k flows initial stats fail - logging

 - added stats schedule and polling logs
 - added much more logging
 - added try-catch to statistics processing

Change-Id: If953ff9d72c06cd57bd3158a407bfdfdcd370860
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit 134eb753830bb774ddf3257c6a3e86cfcbdbeffc)

BUG-4132 add ip src,dst and icmp type match attrs

ip src, ip dst, and icmp type fields are required
to construct the proper response to ICMP echo
requests. OVS 2.5 will support these fields.
Changes in ovsdb to follow.

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

Merge "Added Conntrack Support in Nicira Extenstion."

Added Conntrack Support in Nicira Extenstion.

1)Added Support for Conntrack action
2)Addded support for conntrack state and conntrack zone match.

Change-Id: I3472a95b9fe46b102bee846c369e00535a24f0a0
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>

Merge "BUG-4062: Flows,Groups,Meters not getting deleted during switch flap"

Merge "Added support for TCP and UDP port mask in Nicira extention"

BUG-4762 Sal-port-service registered as rpc service

Change-Id: Id16f060d24052e1dde34323238bf63aeeb89f5a9
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

BUG-4117: initial commit for new App Old-Notification-Supplier

 - we wish to keep backward compatibilility for all old Openflow notification

Note: Original commit https://git.opendaylight.org/gerrit/#/c/25965/
      has to be broken to a chain of smaller commits

(fixed version to Be, adapt changed deps, add copyright)

Change-Id: I6a63329611a04299d09ba265077de1e12e29b4e8
Signed-off-by: Martin Bobak <mbobak@cisco.com>
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 9c94939ff57c2f816b2978c9b763dd971ba932d2)

Add missing license headers

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

Merge "BUG-4118: Li:backward compatibility - rpcs - final wiring"

Merge "BUG-4118: Li:backward compatibility - rpcs - stats services"

Merge changes If902096e,I5a639b30

* changes:
  BUG-4118: Li:backward compatibility - rpcs - transformers
  BUG-4118: Li:backward compatibility - rpcs - initial proposal

Merge "Statistics collection fix"

Merge "BUG-2188: To populate the port_number of switches - yang model"

Merge "BUG-4236: resolve TODOs - part 2"

Merge "BUG-4236: resolve TODOs - part 1"

Merge "BUG-4715: SalEchoService return value fix"

BUG-4715: SalEchoService return value fix

Change-Id: I6e288652efcadfd0a21b47372039aed5a55cc207
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

BUG-4763 Green unit tests with exceptions

Change-Id: I92e6ebb2ffcdcabcecfabe5ab9b341e68739a373
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Bug-4757:Li: TransactionManager does not get closed and future connections are rejected.

Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Statistics collection fix

 - cleaning up flow registry
 - keep stats scheduler alive even if connection is not in state WORKING

Change-Id: If1f03b224f497653769fc2cca815afdc9029df95
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Added support for TCP and UDP port mask in Nicira extention

Change-Id: I4c89c8ee57860e6439c66f74606b62076b0bfae6
Signed-off-by: Aswin Suryanarayanan <aswin.suryanarayanan@hp.com>

Cbench DropTest performance improvement

* PacketIn in cbench test always contains Drop Action instructions
  so we'r prepared the DropActionInstructions for every PacketIn
  like static value.

Note: This approach are not reflecting a real world when
      we make new Instructions for every PacketIn normally.

Change-Id: I86f84768c65e1564f39d5b4ac18df937a66cfa01
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 18623d984cbaa67e83bb5929118c5dd1bd656f78)

Merge "Use odlparent-lite as artifacts parent"

BUG-4236: resolve TODOs - part 2

 - MessageHandler removal

Change-Id: Ief7a46898c81f1f0c309dd2a2c609230bedd7929
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4236: resolve TODOs - part 1

 - SystemNotificationsListenerImplTest#testOnDisconnectEvent4
 - OpenFlowPluginProviderImpl#close
 - RpcContextImpl-members
 - NodeStaticReplyTranslatorUtil#nodeGroupFeatureTranslator
 - DeviceManagerImpl#initializeDeviceContext
 - DeviceContextImpl#processPacketInMessage
 - OpenflowProtocolListenerFullImpl#onHelloMessage
 - AbstractService#handleServiceCall
 - ConnectionManagerImpl#onSwitchConnected
 - OpenflowProtocolListenerInitialImpl#*
 - SinglePurposeMultipartReplyTranslator#getGroupActionsSupportBitmap
 - StatisticsGatheringUtils#gatherStatistics
 - StatisticsManagerImpl#calculateTimerDelay
 - MessageIntelligenceAgencyImpl#getCounters
 - MatchingFlowsInTableService#buildRequest

Change-Id: Ic7dd5caf3d2e7e3b61c823731101040887f6249c
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Bug 4646: Parser accepts invalid models

minor fix of invalid yang files

Change-Id: Iaafef327a1376b37595cdcfc58f2183574ec9e2b
Signed-off-by: Filip.Gregor <fgregor@cisco.com>

Use odlparent-lite as artifacts parent

We created odlparent-lite to provide a minimal pom which allows projects
to publish to the correct server. Without this ${project}-artifacts may
fail to deploy to Nexus.

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

BUG-4118: Li:backward compatibility - rpcs - final wiring

 - wired backward compatible services depending on configuration
   parameter

Change-Id: I4b439857c62d8e9dd81a3668476f67d7ffcc949b
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4118: Li:backward compatibility - rpcs - stats services

 - added flow statistics services with notification emitting feature
 - added group statistics services with notification emitting feature
 - added meter statistics services with notification emitting feature
 - added port statistics services with notification emitting feature
 - added queue statistics services with notification emitting feature
 - added tests

Change-Id: Iffc6637e6f1a81be11cb9ef9479557837195c678
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4118: Li:backward compatibility - rpcs - transformers

 - added general stats to notificatin transformers
 - added abstract service providing transaction aware result
   and async notification

Change-Id: If902096e1bd1041aa9ae1d5e23cd45fcc5eef5ea
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4118: Li:backward compatibility - rpcs - initial proposal

 - added config subsystem parameter for stats rpc control
 - API changes for delegating of new parameter, service lookup

Change-Id: I5a639b30a83fd9b025da5c7e1faffdc63eb1315b
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Merge "Remove antlr4-runtime-osgi-nohead"

Remove antlr4-runtime-osgi-nohead

This bundle was removed from yangtools so no longer exists. See:
https://git.opendaylight.org/gerrit/28272/

Change-Id: Iaea7f5022ac1e4706cddcface7726bbdf67ac8ed
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Adding the unit tests for StatListenCommitFlow.java for Bug#3957

Change-Id: Iba725bc4bdd0063a72c5800673c5c58c8e31b5e5
Signed-off-by: MonikaVerma <monika.verma1@tcs.com>

Merge "BUG-4565: queue stats error"

BUG-4283 experimenter msg support - service expose part

 - added sal-experimenter-service exposing experimenter message
   sending and listening to experimenter notification

Change-Id: I24e0917f5e33f98dd1c11a32b37db47709483873
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4283 experimenter msg support - exceptions part

 - added conversion related exceptions for experimenter message converters

Change-Id: I593f8896d138619b93de68f07be9b86b2ae899b3
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4283 experimenter msg support - provider part

 - added provider infrastructure for experimenter message converters

Change-Id: I12e393ddbf1a4439763bc6922958005cec8a7bf4
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Merge "BUG-4283 experimenter msg support - registration part"

Bug-4433:OF Version 10 does not get ommitted for clustering

Change-Id: I928be12d3dd1b28f6cf110f6a6f99f14d993b35c
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Handle cancel txn exception for slaves and device disconnect exception

During the device initialization process, if its encountered that the role for this node is slave
we cancel the txn which is built during the process.
Now since ping-pong txn does not have a cancel, we drop the txn and recreate it. This throws an exception
Since this is an expected exception, it would be advisable to catch and log it in debug.

Also when a node gets disconected from device, the EntityOwnershipService does send a notification. Which trigers a role change.
A check is made to check if the device is connected before moving ahead with the role change.

Change-Id: I76572f27274f5c5f08117fcfd1590d1042049735
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Inventory Cleanup fails as txn could not be created

Cleanup of the inventory is done post closure of txn manager.

Hence if the status of the txn manager is shutting-down, get the WriteTx directly
to perform a datastore cleanup

Change-Id: If9dc9449f7e54e54ea4159376217297f6998e289
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

BUG-2188: To populate the port_number of switches - yang model

Change-Id: I222fdae39986794c020ddc3a0ff1cab33b67be90
Signed-off-by: Michal Rehak <mirehak@cisco.com>

RPC Registration changes for clustering

1. Unregisters rpcs for master/equal to slave scenario as well.
2. Made RpcManager as the device-context-closed-handler , instead of the RpcContext
3. If Role Change fails after several attempts on a switch, the device context is closed. This is to avoid rpcs getting registered in wrong nodes.
4. Removed debug exception for device close in RoleManagerImpl
5. Addition of Logs

Change-Id: Ie5af7466b1422698d3c3287195d237fe1407b023
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Getting latest state from Ownership RPC

Takes care of two scenarios
1. Switch connects to a second controller, after first controller is master.
   In this case it wont get an Ownership Changed notification

2. Controller which is slave restarts and registers a candidate. Still it wont get a notification.

Notification is only sent when there is a change in the ownership in the OwnershipService

Change-Id: Ifdc5023a711bb66eb36f4d03e294456776810c50
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

RoleManager,SalRoleService Impl and Test

1. Created a Role manager and RoleContext which would get invoked as part of the device init phase
2. Introduced dependency to EntityOwnership service , to get the ownership (implemented in controller)
4. RoleManager requests ownership from EntityOwnership and on notification makes a role change call SalRoleService
5. SalRoleService makes a role change to the device
6. Tests

Change-Id: Ie443d519be985b867d5c6f9ce8473fbd6cdf4dca
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>
Signed-off-by: Moiz Raja <moraja@cisco.com>

moving "context not found" log message to INFO level

Change-Id: I1673074575edd013c3032b8a02268c944a39e141
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

BUG-4565: queue stats error

 - queues do not have any other inspect message but statistics query,
   here 1 parent was missing when writing queue stats into DS/operational
   - addapted data assembly in order to create queue node
   - added TODO: we need to clean existing queues before writing fresh ones
 - fixed test

Change-Id: Icfcb692100b2a0aa666c2e578b09cd903e59fd80
Signed-off-by: Michal Rehak <mirehak@cisco.com>
(cherry picked from commit b706170bb88018010bde3d8bd84ef1bbf4662943)

BUG-4062: Flows,Groups,Meters not getting deleted during switch flap

Change-Id: Ie06d8a61763b64143106f3d924e856934b9d7b26
Signed-off-by: Muthukumaran Kothandaraman <muthukumaran.k@ericsson.com>

Fixed mising ; in presence statements.

Change-Id: Ica105d65f3d49ca6f03c7b326d8338888a3ea937
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

BUG-4283 experimenter msg support - registration part

 - added registration point for experimenter message converters
 - depends on OFJava https://git.opendaylight.org/gerrit/#/c/28813/
 - checkstyle fix

Change-Id: Ibb8294c522335a0f5df783fbf92b617e45f29930
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4551: impact of YT BUG 2399 - action model fixing

 - added presence element to empty action containers
 - added sleep in order to follow port openings with some time reserve

Change-Id: Ia96c0f821455600a412df56c3d353b76184f26ef
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Merge "Add missing information from table features"

Merge "Fix a few spelling mistakes in log output"

Merge "Bug-4442 : PacketOut action headers are not implemented or ignored"

Add missing information from table features

Some information from table features were missing on MD-SAL Operational
DataStore:

  1) Instructions on <instructions> and <instructions-miss>

  2) Actions on <apply-actions>, <apply-actions-miss>, <write-actions>
     and <write-actions-miss>

Change-Id: I31d6a84c821ae4671ae2ee079935ffc959c61ef0
Signed-off-by: Renato Aguiar <renato.aguiar@hp.com>

Openflowjava-util rename reflecting change

 - openflowjava change: https://git.opendaylight.org/gerrit/#/c/28333/

Change-Id: I1a5fcae9e16c7d9bdd187ce588a509442816d6c1
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Merge "sal role service and role injection"

Merge "Deleted unnecessary file."

Bug-4442 : PacketOut action headers are not implemented or ignored

Change-Id: I5a31898035e857ef677fc163dc3670cb977e8b4e
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

sal role service and role injection

role cannot be assigned to the whole of openflowplugin.
Since we are scaling switches across a cluster, the role would be assigned per device.

Added a new sal-role yang, rpc and service.

This would be the first of subsequent commits.

Made changes to merge to master - header and testchanges

Change-Id: I0a681485c5ccb8bc839b3b18f3efb648b88bd86b
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>
(cherry picked from commit f6d2398e08b0a9b2d7bb5c533e9ca17a5041a74f)