Fix allowable Unix ports range 1024 - 65535

The well known ports / system ports are actually only in the range
between 0 - 1023. The port 1024 should be usable. Also now handles the
max allowable port of 65535.

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

BUG 2371 : Leader should reset it's snapshot tracking when follower is restarted

This patch adds a new protocol to InstallSnapshot. It the InstallSnapshotReply returns
a failure and the chunkIndex is -1 then the Leader will reset the FollowerSnapshot so
that when the next heartbeat occurs the Leader would start sending chunks from the beginning.

Change-Id: I0d5f0a4230209856ecf9bcef46220ae348f52b5d
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Bug-2397:Provide a mechanism for stakeholders to get notifications on Raft state change"

Merge "Bug 2435 - Controller/MD-SAL throwing ModifiedNodeDoesNotExistException exception when statistics manager augment queue statistics to the node connector. Exception was thrown because parent (queue) container was not present when statistics manager tried to augment statistics. Fixed the issue by creating empty queue container and 'merge'ing it to the node connector before augmenting the statistics."

Merge "Fix checkstyle warnings in netconf-usermanager"

Merge "Fix checkstyle warnings in netconf-tcp"

Merge "Make sure we include maven plugin in execution"

Merge "BUG-2259: mark AbstractBindingAware* as deprecated"

Merge "BUG-2340 Fix improper cleanup of resources in netconf ssh handler"

Bug 2372: Removing duplicate call in loadBalancerPoolNorthbound to loadBalancerPoolInterface.addNeutronLoadBalancerPool(singleton);

Change-Id: I0b5790b6d363c85bcd0d2bb739a0de14c8516800
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

BUG-2259: mark AbstractBindingAware* as deprecated

These classes encapsulate OSGi-activated clients, which are deprecated
in favor of using config subsystem activation.

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

Make sure we include maven plugin in execution

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

BUG-2340 Fix improper cleanup of resources in netconf ssh handler

This commit also fixes proper reconnect scheduling for netconf-connector.

Change-Id: I1d848af57fddc35e73596fc47f8c7318b8747130
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Bug 2435 - Controller/MD-SAL throwing ModifiedNodeDoesNotExistException exception when statistics manager
augment queue statistics to the node connector. Exception was thrown because parent (queue) container was
not present when statistics manager tried to augment statistics. Fixed the issue by creating empty queue
container and 'merge'ing it to the node connector before augmenting the statistics.

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

Merge "Optimize xsql's use of collections"

Merge "Netconf-cli compilable and included in project"

Fix bug 2413 NPE for group and meters

* group/meter features
* Device RPC features Calls for group/meter depend on FlowCapableNode data change event (create event only)
* Remove Meter's and Group's feature calls from perm. stat. collect
* Add API methods for additional registration new StatCapabTypes for a specific Node statistic collecting process
* Fix NPE for features
* refactoring group/meter stats update processing (Target: Don't block the RPC Notification threads as soon as is possible)
* Fix NPE List processing
* queue - add delete processing (becasue queues has to be edit outside and we are not cleaning it)
* add validations input values
* remove unnecessary validation input values (StatRpcMsgManagerImpl)

Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
Change-Id: If10a0ea0bb098ca70cd19ce32a7ca5790159aae5

Fix checkstyle warnings in netconf-usermanager

Change-Id: Id4cc11b487f98e6b33985c16f7a95a434c01dd74
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-tcp

Change-Id: I95e776dde2645288734f7687c22fc158ee7b486d
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-ssh.

Change-Id: I01cee96432edcf361b463f89c648823042165b9e
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-netty-util

Change-Id: I123c7cbff15c1267a1bdb537e100c1bf2f9894d7
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Merge "Fix expression deprecated warning on ${version}"

Bug-2397:Provide a mechanism for stakeholders to get notifications on Raft state change

A notifier actor is spawned from the RaftActor inherited implementation, in our case Shard or ExampleActor .

Its injected into the RaftActor.RaftActor notifies the notifier with the role changes. The Notification message is local to Shard or Example.

The idea is for any implementation to create a notifier, inject it and issue a notification from that notifier on a role change.

A sample example notifier, listener and notification is provided to show how a listener can register with the notifier and get notifications form the notifier.

Notifier and Notifications are assembled in commons, to be shared along with other apps who might need similar logic. It can be override by specific implementations.

Has been tested with the TestDriver, with separate actorsystems for listner and notifier

Tests have been added.

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

Fix expression deprecated warning on ${version}

${version} is deprecated and ${project.version} should be used instead.
This patch fixes the expression used.

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

Cleanup temporary test files

A properly defined test should correctly cleanup after itself when the
tests have completed execution.

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

Merge "Bug 2267: Reuse leaf and leaf set entry builders in NormalizedNodeInputStreamReader"

Merge "Bug-915: Adding static document generation. Currently the API Explorer can only be accessed from a running server. With this change, the API Explorer will be generated as static html pages at build time in the features folder."

Merge "Fix checkstyle warnings in logback-config-loader."

Netconf-cli compilable and included in project

Change-Id: Ia08396f52c98ea2661e13f60a3787b18c4ae50d6
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in logback-config-loader.

Change-Id: If926826e233000f771518d966493ee8b100a76e4
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-monitoring.

Change-Id: I57cdbd26a841cb8fa6b0435089832c635302594b
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-mapping-api.

Change-Id: Ic0e339063aa79bc0c0338c33486e43fcdd29ef56
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-it

Change-Id: I73f4325f6ff3ca7139b4e4ccf459fa8a10e36696
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-impl.

Change-Id: I921b51e79475bdc7a0a03398bb6a709836bffdb2
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Merge "Fix checkstyle warnings in config-util"

Optimize xsql's use of collections

Rather than performing class comparisons, we perform a simple
instanceof check... and then rather than adding items one at a time, we
use Collections.addAll().

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

BUG-2255: introduce PingPongDataBroker

This is a forwarding DOMDataBroker implementation, which ensures that
there is at most one transaction pending for a particular transaction
chain. This is not useful in all scenarios, but if the users can
restrain themselves to use the supported semantics, they can use it.

Change-Id: I4a9ca12254763829f99e270d6716cd034b4b5ea9
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Fix checkstyle warnings in config-util

Change-Id: I5a8caa79f38081b65f2e25f9aaecc403584fa394
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Merge changes If7159c39,I6c84b2fe

* changes:
  Deprecate ForwardedBackwardsCompatibleDataBroker
  Migrate from using the deprecated SchemaContextProvider

Merge "Migrate FRM/SM to use model API SchemaContextProvider"

Bug-915: Adding static document generation.
Currently the API Explorer can only be accessed from a running server.
With this change, the API Explorer will be generated as static html
pages at build time in the features folder.

Change-Id: I0a898e1a3d9753fd2b046d690173838be057d6ea
Signed-off-by: Alex Fan <railor33@gmail.com>

Deprecate ForwardedBackwardsCompatibleDataBroker

It implements a deprecated API only, so rather than suppressing the
deprecation warnings, deprecate the entire class.

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

Migrate from using the deprecated SchemaContextProvider

org.opendaylight.controller.sal.dom.broker.SchemaContextProvider has
been deprecated, the services away so it is no longer used.

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

Migrate FRM/SM to use model API SchemaContextProvider

Simple migration, eliminates the need for deprecation suppression.

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

Rename 01-mdsal.xml to 01-md-sal.xml

This renames the feature definition to ensure that the file distributed
in karaf matches the name that is stored in the SCM.

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

Merge "Remove raw references to Map in XSQL"

Merge "BUG-8: mark deprecated classes as such"

Bug 865: Changed Config subsystem module for ForwardedBindingDataBroker.

Implementation of new Bidning Data Broker was wired to deprecated APIs
and used broker to discover dependencies which did not allowed for
full-fledged customized wiring. Reworked configuration to explicitly
allow injection of each required depenency and to not use
deprecated APIs for dependency injection.

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

Merge "BUG-650: Split out CommitCoordinationTask"

Merge changes Ia7cd7eda,If961a029,I9f6a57a2,Id8551489

* changes:
  Fix checkstyle warnings in config-persister-file-xml-adapter.
  Fix checkstyle warnings in config-persister-feature-adapter
  Fix checkstyle warnings in config-persister-directory-xml-adapter
  Fix checkstyle warnings in config-manager

Merge "Fix checkstyle warnings in config-api"

Fix checkstyle warnings in netconf-client

Change-Id: I94d8c31a16f0755451a3bebad5a8792a97d426d2
Signed-off-by: Marian Dubai <mdubai@cisco.com>

BUG-650: Split out CommitCoordinationTask

The coordination task takes most of the coordinator implementation. We
do not want to expose it, but at the same time we want to evolve task.
Split it out and eliminate the unneeded atomic state updates, as it is
only ever touched by a single thread.

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

Merge "BUG-650: remove executor abstraction"

Bug-2277: fix the Leader test failing in jenkins

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

BUG-8: mark deprecated classes as such

These classes refer to various deprecated interfaces of MD-SAL,
triggering a lot of warnings about deprecated constructs. Since these
classes themselves have been superseded, mark them as Deprecated,
lowering the amount of warnings we get.

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

Remove raw references to Map in XSQL

This adds the proper types to map, removing some casts and improving
efficiency slightly. Fixes quite a few of eclipse warnings.

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

Fix checkstyle warnings in config-persister-file-xml-adapter.

Change-Id: Ia7cd7eda114225cfc35ec301535d5aaec0fa8794
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in config-api

Change-Id: I56eedb15f3c5892731a757bdb06c34d9d15efab7
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in config-persister-feature-adapter

Change-Id: If961a029b48d85b58ca10b9b71f5fc29d41dbe9c
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in config-persister-directory-xml-adapter

Change-Id: I9f6a57a22b470359c7eb038e77736376cd2ac3ee
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in config-manager

Change-Id: Id8551489d678ebb35908f052a571303fc6c68999
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Merge "Remove unused warnings suppressions"

Merge "BUG-868: do not use YangInstanceIdentifier.getPath()"

Merge "Do not retain module through NotificationListener"

BUG-650: remove executor abstraction

This patch removes sameThreadExecutor from the commit path, eliminating
associated overhead. Relevant benchmarks show improvement pretty much
across the board:

BEFORE                                                            millis        error
write100KSingleNodeWithOneInnerItemInCommitPerWriteBenchmark    2213.735       77.597
write100KSingleNodeWithOneInnerItemInOneCommitBenchmark          171.524        2.289
write10KSingleNodeWithTenInnerItemsInCommitPerWriteBenchmark     164.282        1.391
write10KSingleNodeWithTenInnerItemsInOneCommitBenchmark           14.161        0.196
write50KSingleNodeWithTwoInnerItemsInCommitPerWriteBenchmark     982.697       29.397
write50KSingleNodeWithTwoInnerItemsInOneCommitBenchmark           93.233        2.174

AFTER                                                             millis        error   delta
write100KSingleNodeWithOneInnerItemInCommitPerWriteBenchmark    2138.900       75.844   -3.4%
write100KSingleNodeWithOneInnerItemInOneCommitBenchmark          177.839        3.997   +3.5%
write10KSingleNodeWithTenInnerItemsInCommitPerWriteBenchmark     158.666        1.090   -3.5%
write10KSingleNodeWithTenInnerItemsInOneCommitBenchmark           13.022        0.105   -8.0%
write50KSingleNodeWithTwoInnerItemsInCommitPerWriteBenchmark     935.490       30.395   -4.8%
write50KSingleNodeWithTwoInnerItemsInOneCommitBenchmark           89.907        1.204   -3.6%

Furthermore it cleans up and marks FIXMEs for defunct statistics. These
will need to be replaced with implementation which does not assume
underlying implementation.

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

BUG-868: do not use YangInstanceIdentifier.getPath()

This method has been deprecated, so stop using it.

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

Merge "Fix typos and missing space"

Merge "BUG-2381: do not retain references to DependencyResolverManager"

Make NetconfMessageTransformUtil constants final

All of these constants were missing 'final' keyword, making them not
really constant.

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

Fix typos and missing space

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

BUG-2381: do not retain references to DependencyResolverManager

Instead of creating an anonymous subclass, define a proper class and
pass it the three required fields.

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

Remove unused warnings suppressions

Remove suppressions which do not do anything -- thus fixing eclipse
warnings.

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

Do not retain module through NotificationListener

Analysis shows we are retaining Module instances via
NotificationListener instances. This turns out to be caused by anonymous
class retaining a reference to AbstractDynamicWrapper, which holds the
module for hashCode/equals purposes.

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

Merge "Set auth timeout along with idle in SshProxyServer"

Merge changes Ic434bf4a,I05a3fb18,I47a3783d,I8234bbfd

* changes:
  Fix checkstyle warnings in netconf-cli
  Fix checkstyle warnings ietf-netconf-monitoring
  Fix checkstyle warnings for netconf-testtool
  Fix checkstyle warnings for config-netconf-connector

Merge "BUG-2243 Fixing invalid hello message handling"

Merge "BUG-2218: Keep existing link augmentations during discovery process"

Merge "Bug 2273: Removed unbuilt third-party code."

Merge "Remove unneeded embedded dependency"

Merge "Do not retain NetconfConnectModule reference"

Merge "Use ImmutableSet/Map in ServiceReferenceRegistry"

Merge "Fix raw references to Iterator"

Merge "Fix raw ServiceReferences and ServiceRegistrations"

Merge changes Ife66b5fe,I174ea168

* changes:
  Fix checkstyle warnings for config-persister-impl
  Fix checkstyle warnings in netconf-api

Merge "Bug-2277 : Isolated Leader Implementation"

Merge "BUG-2304 Fix leafref/instanceIdentifier processing in restconf -> netconf pipeline"

BUG-2243 Fixing invalid hello message handling

When invalid message before hello message, session dropped after hello message

Change-Id: I2c92012d3ba25069693a35b76d105407d200a833
Signed-off-by: Marian Dubai <mdubai@cisco.com>

BUG-2304 Fix leafref/instanceIdentifier processing in restconf -> netconf pipeline

Resolve the leafref type based on type being referenced by the xpath.
Fix tests testing wrong instance identifier behavior
Fix subtree filtering in netconf northbound when identityrefs are involved

Change-Id: I561e0c92550dd6639a9f9fbd1909695041c378e4
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Fix checkstyle warnings in netconf-cli

Change-Id: Ic434bf4aac666c312e41781588aa9fca94acca4e
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings ietf-netconf-monitoring

Change-Id: I05a3fb185ae53128c31f8565a9489f7d0399febe
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings for netconf-testtool

Change-Id: I47a3783d9b89cadc6549d6f4784508cefb149059
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings for config-netconf-connector

Change-Id: I8234bbfdb7092f6423cedc7f95de07995871756d
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings for config-persister-impl

Change-Id: Ife66b5fea7bb6d1fe293d6181d6da7221e296731
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Use ImmutableSet/Map in ServiceReferenceRegistry

Rather than wrapping collections in unmodifiable references, perform a
copy to obtain a properly-constant collection. These references are
usually retained for longer time, so opting for memory efficiency is
better than using sparse collections.

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

Remove unneeded embedded dependency

yangtools:util already embeds the dependency with a different version,
causing a potential clash. Since sal-dom-broker is not a direct user,
this will work just fine.

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

Do not retain NetconfConnectModule reference

Anononymous classes retain a reference to the module, which means we
leak quite a bit of memory if the client is not shut down.

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

Fix raw ServiceReferences and ServiceRegistrations

Adding a windcard is enough to reduce the number of eclipse warnings.

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

Fix raw references to Iterator

Adds type declarations to fix eclipse warnings.

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

Set auth timeout along with idle in SshProxyServer

Change-Id: I4d7de2ca3f5e2025856646b825040b79ca344f34
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>