Compress removals into a single transaction

When node removal occurs, do not cascade into multiple transactions, but
trigger only a single one.

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

Rename NodeStatisticsAger and StatisticsUpdateCommiter

The component's responsibilities have changed, rename them to reflect
this change. The new division of responsibilities is:

There is a single StatisticsListener listens for notifications
carrying the statistics data and routes them towards the appropriate
NodeStatisticsHandler. Once MD-SAL is able to route notifications based
on key, this class should be completely removed.

There are per-node NodeStatisticsHandlers, each of which is responsible
for taking care of lifecycle of statistics for a particular node.

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

Move flow statistics update handling

This finishes the cleanup of StatisticsUpdateCommiter, we now have all
per-node statistics processing guarded by the lock -- e.g. have
predictable and isolated updates to statistics.

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

Merge "Move flow comparison methods into utility class"

Merge "Prepare for transitioning NodeStatisticsAger"

Merge "Simplify StatisticsManagerActivator/StatisticsProvider contract"

Merge "Centralize NodeStatisticsAger creation"

Merge "Optimize statistics cleanup"

Merge "Do not allocate/commit transactions in tight loops"

Merge "Rework NETCONF interfaces"

Merge "Fix thread safety issues in netconf client"

Merge "Cache sessionID to increase readability"

Merge "Remove unused DefaultNetconfOperation marker"

Removed redundant dependency on configuration artifact

Change-Id: Ifae0e3071c08ddc7363af1fdb05d97e2b22e4329
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Merge "Fixed removals from data store did not clean direct-match indexes to child nodes."

Merge "Fixed bug when publisher's thread was blocked until all notifications were delivered."

AuthenticatedUser not to expose internal implementation

Update unit test

Change-Id: I9606742fbdfe3bbbae8bbf789e130d24bd946d82
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Move flow comparison methods into utility class

The whole machinery is not interesting except for the two public
methods. Let's move into its dedicated file and limit its visibility to
package for now.

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

Prepare for transitioning NodeStatisticsAger

NodeStatisticsAger is currently the object which tracks per-node state,
synchronizing our view of what comes from the network vs. what is timed
out. Previous patches have already added partial ordering, this patch
moves the actual modifications in NodeStatisticsAger. This gives us
predictable behavior while keeping us per-switch scalable.

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

Fixed removals from data store did not clean direct-match indexes to child
nodes.

  - Added deepRemoveStrategy, which checks for all quick indexes
    if they are removed correctly.

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

Simplify StatisticsManagerActivator/StatisticsProvider contract

Get rid of the need for the StatisticsManagerActivator to keep around
the Provider session -- it is only ever referenced from
StatisticsProvider.start(). Same goes for the various services which
are set into it.

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

Centralize NodeStatisticsAger creation

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

Provide Switch Information in Troubleshooting Tab

Change-Id: I5fc02b4c6ac5babbe21cc082aa4723882846568a
Signed-off-by: Sapan Shah <sapshah@cisco.com>

Optimize statistics cleanup

Perform a single transaction for cleaning up the flow. Also use
System.nanoTime() for keeping the expiry time and keep it constant.

Furthermore, use HashMap instead of ConcurrentHashMap to gain better
memory peformance. This requires synchronizing between the statistics
update and statistics cleanup -- which actually is okay, as it closes
the race between an update happening precisely when a flow entry would
be retired. With the lockless design that entry could be lost, simply
because we'd get undefined ordering between cleanup and update
transaction commit.

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

Fixed bug when publisher's thread was blocked until all notifications were delivered.

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

Do not allocate/commit transactions in tight loops

A transaction is additive and there's little harm in commiting empty
ones. Move allocation/commit outside of tight loops, such that we get
proper bulking.

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

Rework NETCONF interfaces

NetconfSession and NetconfClientSessionListener are supposed to be
interfaces, not classes. Convert them to such and introduce utility
classes which retain their current functionality. Remove an unused
method and fix type safety warnings while we're at it.

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

Fix thread safety issues in netconf client

This patch introduces a proper asynchronous interface and reworks
internals such that thread safety is maintained.

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

Cache sessionID to increase readability

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

Remove unused DefaultNetconfOperation marker

DefaultCloseSession and DefaultGetSchema do not need the netconf
session. so drop their use of DefaultNetconfOperation.

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

Fix yangtools API incompatibility

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

Merge "Do not use deprecated StringBufferInputStream"

Merge "Fix a warning"

Merge "Remove deprecated API elements from protocol framework"

Merge "Fix typo in match types yang model"

Merge "Changed model versions to dependencies"

Do not use deprecated StringBufferInputStream

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

Fix a warning

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

Rewrite POSSIBLE_ENDS and POSSIBLE_STARTS

These two are really simple byte arrays, initialized them as such
withouth going through strings/UTF8.

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

Merge "Some methods should be final"

Merge "Expose ChannelFuture from sendMessage()"

Merge "Getter methods should be final"

Merge "Fix a warning and make field final"

Merge "Remove use of deprecated Exceptions"

Merge "minor pom cleanup to eliminate duplicate group ID specification when the group ID is the same as the parent pom, just removes some warnings from eclipse, should not be any functional difference"

Some methods should be final

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

Merge "Fix warnings reported in toaster"

Merge "Fix a few eclipse-reported warnings"

Expose ChannelFuture from sendMessage()

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

Getter methods should be final

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

minor pom cleanup to eliminate duplicate group ID specification when the group ID is the same as the parent pom, just removes some warnings from eclipse, should not be any functional difference

Change-Id: I396126408fff54d7120d3ae94b2a5c447d14f8d2
Signed-off-by: David K. Bainbridge <dbainbri@ciena.com>

Merge "Get rid of NetconfMessageFactory"

Merge "Get rid of DeserializerException"

Merge "Get rid of using ProtocolHandlerFactory, part 2"

Merge "Get rid of using ProtocolHandlerFactory, part 1"

Merge "Cleanup dependencyManagement section"

Merge "Remove repositories/pluginRepositories/distributionManagement"

Merge "Remove duplicate properties"

Merge "SwitchMgrNB: URL does not allow nodeconnectorId with forward slash"

Fix for BUG 410

Ensure lower case alpha hex representation is included in MAC
regex match in all cases.

Change-Id: Ic2da17146cf8f9325cc5d951f70d614ec61762a0
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

SwitchMgrNB: URL does not allow nodeconnectorId with forward slash

Change-Id: I489d5ae559b8c1e35a3c7b9749f7476618b2765b
Signed-off-by: Pramila Singh <pramisin@cisco.com>

Merge "Fixed bugs in getUnsigned methods of NetUtils class."

Cleanup dependencyManagement section

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

Remove repositories/pluginRepositories/distributionManagement

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

Remove deprecated API elements from protocol framework

The use of these concepts has been deprecated in version 0.4.0, which
was released. The only in-tree user was netconf, which was refactored to
not to rely on these concepts, so bump the version to 0.5.0 and purge
the  code.

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

Fix a warning and make field final

Drive-by fix
- additionalHeader should be final,
- the class is not serializable, no uuid is needed

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

Remove use of deprecated Exceptions

DocumentedException and DeserializerException are deprecated and were
only used by deprecated interfaces which we no longer use. Stop using
them.

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

Get rid of NetconfMessageFactory

This removes the use of deprecated ProtocolMessageFactory and
ProtocolHandlerFactory. Just instantiate appropriate Netty handlers
instead.

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

Get rid of DeserializerException

We can throw anything, so use NetconfDeserializerException instead.

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

Get rid of using ProtocolHandlerFactory, part 2

Now that we do not rely on passing it around, we can safely keep
NetconfHandlerFactory wired directly to the
ProtocolMessageEncoder/Decoder.

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

Get rid of using ProtocolHandlerFactory, part 1

This gets rid of completely dead code in NetconfSshClientDispatcher. It
already talks directly to the initializer.

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

Remove duplicate properties

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

Merge "Cleanup imports/whitespace in MD-SAL"

Merge "Adding FlowRef,GroupRef and MeterRef for the ErrorNotification Message"

Merge "BUG-362: add some diagnostic information Changed Remote RPC Server Implementation to log an error when host IP address is not found and stop itself. Earlier it would raise exception that the configuration subsystem could not handle."

Cleanup imports/whitespace in MD-SAL

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

Merge "BUG 408"

Fixed bugs in getUnsigned methods of NetUtils class.

 * Fixed bugs that getUnsignedByte(byte) and getUnsignedShort(short)
   handled 0 incorrectly.
 * Added tests for getUnsignedByte(byte) and getUnsignedShort(short).

Change-Id: Ie67fb5c07bed31d16d4a52068dda7735fb7cd5e2
Signed-off-by: Hideyuki Tai <h-tai@cd.jp.nec.com>

Fix typo in match types yang model

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

Changed model versions to dependencies

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

Merge "Fix incorrect validation in SwitchConfig"

Fix incorrect validation in SwitchConfig

Change-Id: Ie9267b97b8742f9bd5ebbd85a4f0dc05bf1cebd9
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Fix warnings reported in toaster

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

Fix a few eclipse-reported warnings

- unused suppresswarnings
- missing serialVersionUuid
- missing generic arguments

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

Merge "Lower logger level in OF1.0 Discovery Service"

BUG 408

- added null safe comparison
- added unit test
- added SLF4j
- added log message in method StrIpToIntIp in case of exception

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

operation, store, path and data properties added to Data change event model

Change-Id: I0b1c39e23506988528614b33c4a49cf8e00802ea
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Merge "Fixed publishDataChangeEvent in 2phase commit"

Merge "Increase timeout for waiting for broker service in sal-binding-it."

Merge "Remove unnecessary warn log from config subsystem"

Merge "Fix for bug #236 and bug #240 Have made changes in opendaylight-table-types.yang"

Merge "Improve netconf negotiator messages"

Merge "Improve logging in NetconfClient, logging-bridge. Increase connection timeout in sal-netconf-connector."

Merge "Maven archetype for config subsystem aware bundles"

Merge "Fix of bug in Notificator class"

Merge "Fix Plugin execution not covered in Eclipse for config subsystem"

Lower logger level in OF1.0 Discovery Service

- Discovery Service has a function to attempt to snoop
  non controller generated or malformed LLDP frames.
  The whole function block is in a try catch block,
  as exceptions are well expected.
  In case an exception is hit though, the function logs
  it as a warn.
- Change the log level to debug, do not print a stack trace,
  print the source MAC and the port on which it was received.

Change-Id: I82b459b16356b0a57934b3c26c993d20a2b53dc2
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Improve netconf negotiator messages

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

Merge "Requestor FRM to update the sw views on remote execution"

Merge "Minor change to resource name regex"