BUG 2820 - LLDP refactor

LLDP class field changed from tlvList to mandatoryTLVs, optionalTLVs, customTLVs
Added new create custom TLV value in order to hold security hint for LLDP topology discovery.
Fixed guava bundle import
Fixed apache-lang3 bundle import

Change-Id: I5d0c6b9a9e29213d3f25aa99ff7edd5b30e6c7a8
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Merge "Bug 2820 - LLDP TLV support and testing" into stable/helium

Bug 2820 - LLDP TLV support and testing

- enhancement: deserialization now supports custom TLV
- tests:
  - LLDP.deserialize
  - LLDPTLV.createCustomValue
  - LLDPTLV.getCustomString
  - beautified (checkstyle)

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

Bug 2834: Fix Restconf POST to wait for completion

This changeset introduces a blocking mechanism for Restconf POST to ensure that
the transaction completes before a response is returned, as is consistent with
Restconf PUT behavior.  checkedGet() is utilized to ensure write transaction
completion.

Change-Id: I3a7d7a388d1b3d6eb3d80a9d0880abe0b6657583
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Switched from the WIP custom authn realm to using tomcat-users.xml, also enabled HttpOnly cookies" into stable/helium

Switched from the WIP custom authn realm to using tomcat-users.xml, also enabled HttpOnly cookies

Change-Id: I8bda566670b77e84d16b0cae303e93ea76442cc7
Signed-off-by: David Jorm <djorm@iix.net>

Bug 2820 - problem to add second TLV with type 127.

Custom TLVs aren't saved to map with key type but are stored in
standalone array.

 - fixed list copy (for custom TLV list)
 - extended test
 - involved custom TLV list items into packet length counting
 - published

Change-Id: Ifa1cab17206e1be37022bc8b49f7990649cbd356
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Bug 2699 - websocket does not implement the ws:// protocol defined in
the URI scheme in rfc6455

* modified returned location header entry on get stream to use "ws" protocol

Change-Id: I0b7a6e572a917545371276f09c5949e9b66e4e73
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>

Merge "Bug 2642 - Websocket notifications: No notification received when data are removed" into stable/helium

Neutron to return ServiceUnavailable if no providers registered

opNFV interwork revealed that neutron returns 200 in error cases
where no SB providers are registered and if there is an error getting
the list of SB providers.  Close both of these holes by returning
Service Unavailable Status

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

Bug 2642 - Websocket notifications: No notification received when data
are removed

* modified returned paths to skip augmentation identifiers

Change-Id: I0c3056bb915577787aeda4a482e203c19deb616f
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>

Merge "Bug 2524 - Change event notification subscription: Unable to listen on topology" into stable/helium

Re-enable tests and bump aaa version

This patch is part 2 of 2 patches:

* Increments the version of the depenency on AAA from 0.1.2-Helium-SR2
* to 0.1.3-SNAPSHOT.
* Re-enables the feature tests for the restconf and netconf-conector
  features.

Change-Id: I147f0cb20529af4c7497af1f11e67820745a8d15
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bump versions by 0.0.1 after Helium SR2 release

This patch is part 1 of 2 patches.

* The only version not incrmented is aaa.version, which is left at
  0.1.2-Helium-SR2 since they depend on controller and can't update yet.
* To break the cyclic dependency, this patch temporarily stops running
  the netconf-connector and restconf feature tests.

A second patch (to be run after AAA increments their versions to
0.1.3-SNAPSHOT) wiil update aaa.version and re-enable these tests.

Change-Id: I810aed96a178f30edadf1fb694fe5559bf123392
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Applying Helium SR2 release patch

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

Bug 2524 - Change event notification subscription: Unable to listen on
topology

* added conversion to normalized instance identifier in notification subscription
path
* removed mixin nodes from data change event path

Change-Id: I2ea3713903fc506cf3d28801c563e5c007641ca4
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>

Merge "Revert "BUG-2255: introduce PingPongDataBroker"" into stable/helium

Revert "BUG-2255: introduce PingPongDataBroker"

This reverts commit 94d07bdc6133e3667484b42d8b2e7f10c67863d1.

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

Revert "Bug 865: Changed Config subsystem module for ForwardedBindingDataBroker."

This reverts commit c0aeb7a0cd983f1bae0439c1e162a6013d958af1.

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

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>
(cherry picked from commit f38e20c808607c91dd1e35d49ea59f73524c48b6)
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>
(cherry picked from commit 5764146d24df9d9450ebfbb0aec10cbbcdcc655c)

BUG-2255: Introduce forwarding DOMData classes

These classes are useful for implementing various decorator. Their sole
functionality is to forward calls to a backing instance.

Change-Id: I8a65283ab6e3f9787dc59f43d7b7e8eb37459538
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit eaf9e8dd010eaa743cf1bda842519140086870b2)

Increase timeout in config pusher for conflicting version ex

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

Merge "Remove sonar.profile since it's deprecated in Sonar 4.5" into stable/helium

Remove sonar.profile since it's deprecated in Sonar 4.5

This parameter is deprecated in Sonar 4.5 and should no longer be used.
Instead this profile should be set server side on the Sonar server.

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

Bug 2570: Make NotificationPublishService available

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

Merge "Cache TransformerFactory" into stable/helium

Merge "BUG-2459: Fix netconf-monitoring not reusing JAXB context" into stable/helium

Merge "Make NetconfMessageTransformUtil constants final" into stable/helium

Merge "Do not retain NetconfConnectModule reference" into stable/helium

Stabilizing tests for TopologyManager.

Change-Id: I3babd7b4718587f95010513923efbcfc360797b4
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Merge "Bug 2526: Race condition may cause missing routes in RPC BucketStore" into stable/helium

Merge "BUG 2509 : Removing all journal entries from a Followers in-memory journal causes Leader to send an InstallSnapshot" into stable/helium

BUG 2464 : Shard dataSize does not seem to correspond to actual memory usage

The dataSize that is reported is the "serialized" size of the payload. Since the replicated
log actually contains the CompositeModification object (which may hold on to a lot more memory)
the serialized size is not neccessarily the same as the object size.

To make the data size correspond to memory usage and to actually reduce memory usage this patch
creates a new payload class called CompositeModificationByteStringPayload which only stores the
ByteString which is an order of magnitude smaller. Custom serialization ensures that this object
is written and read correctly.

This patch is backward compatible in that a replicated log containing a CompositeModificationPayload
will be read correctly but is not forward compatible in that if a new controller instance were to send
a CompositeModificationPayload to an older instance it would not work.

To ensure that we do not need to immediately require a conversion from ByteString to PersistentMessages.CompositeModification we maintain a SoftReference to CompositeModification.

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

BUG 2509 : Removing all journal entries from a Followers in-memory journal causes Leader to send an InstallSnapshot

This patch simply checks if an AppendEntries#prevLogIndex is equal to the followers ReplicatedLog#snapshotIndex. If
both are the same it also verifies that the AppendEntries#prevLogTerm is equal to the followers ReplicatedLog#snapshotTerm.
This is sufficient in most cases to prevent a re-installation of a snapshot when the AppendEntries message is supposed
to immediately follow the snapshot.

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

Bug 2526: Race condition may cause missing routes in RPC BucketStore

Changed the RpcRegistry class to derive from BucketStore instead of
creating the BucketStore as a separate actor. This was done because the
copy-on-write update operations in the BucketStore need to be done
atomically so the RpcRegistry needs direct access to the BucketStore in
order to do this safely and efficiently. The RpcRegistry handles the
sematics of the Bucket data - this keeps the BucketStore data agnostic.

Change-Id: Ief96c28775b729f459d324971403222e5a578029
Signed-off-by: tpantelis <tpanteli@brocade.com>

Bug 2415: Fixed intermittent RpcRegistry unit test failures

The tests inject a ConditionalProbe into the BucketStore actor via a
message so it can verify the BuckStore received expected messages from
the RpcRegistry actor. The problem was that the BucketStore actor may
not have been created yet, as this is done async by akka, by the time
the test sends the ConditionalProbe message, in which case the message
went to dead letters resulting in a test failure.

To fix it, I modified the BucketStore to return a response on receiving
the ConditionalProbe so the tests can know deterministically that it
received it. I did this in a retry loop in the tests as I still saw
sporadic failures if just trying once.

After the fix, the RpcRegistry tests ran 900 times successfully.

Change-Id: I7094954c4a38493cc910723a5a6950c073e2f03a
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "Bug 2158: Fixed TopologyManager for edge updates." into stable/helium

Merge "Bug 2517: Catch RuntimeExceptions thrown from the DCL in DataChangeListener" into stable/helium

Bug 2158: Fixed TopologyManager for edge updates.

MD-SAL does not keep the order of notification of data changes. In the
result, TopologyManager sometimes receives a notification of an edge
addition before it is notified of the addition of the NodeConnector
which is the head of the edge. In such a case, TopologyManager ignores
the edge notification, and fails to create correct topology information.
This is one of the root causes of Bug 2158.
To handle this issue, this patch fixed TopologyManager to queue edge
notifications, and wait corresponding NodeConnector updates.

Change-Id: Ifeabc91d856eb9cc88ae3595d14e0a6819ec7454
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Bug 2551 - Statistics collection of random node fails when large number if switches disconnects from controller.

When statistics manager is collecting stats and large number of switches disconnects from controller
,some time it causes OptimisticsLockFailedException while processing the existing multipart reply
present in dataStoreOperQueue. Exception occures because it tries to write data to the Node that
doesn't existing in the data store. While handling OptimisticsLockFailedException, statistics manager
flush existing data store related operation task from dataStoreOperQueue. This queue also holds Operational
task for cleaning up internal data of the disconnected nodes and because it flushes the queue, those
operational task don't get executed and statistics manager assumes that those nodes are yet connected
to the controller. When all the switches connects back with different mode (e.g previsouly Openflow13
but reconnects in OpenFlow10 mode), statistics collection of those node fails.

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

Bug 2552 - Fix statistics manager log messages

Fixed the log message to spit clear information about the warning message.

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

Bug 2517: Catch RuntimeExceptions thrown from the DCL in DataChangeListener

This prevents akka from suppressing/dropping subsequent data change
events if a RuntimeException is propagated to akka.

Change-Id: Ibc90e00c56a3e3b6f0039ff609d639fe8201adf0
Signed-off-by: tpantelis <tpanteli@brocade.com>
(cherry picked from commit 75495f26e7718da7d6f9e0ad06af1caa2e3ebf08)

BUG-2511: disable external entitiy resolution with EXI

OpenEXI transmogrifier can resolve external entities. This should never
be necessary in NETCONF context, so disallow that. The same goes for the
reader, but there we need to add a custom EntityResolver.

Origial work done by Robert Vargag. Cherry-picked from
8350f87e0e1190fcc84099d3b7446a2f1de9e5f9 by Colin Dixon.

Change-Id: Idf4b7faf13063c88624f2ba16f3871679c072b3e
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Colin Dixon <colin@colindixon.com>

Merge "Remove unneeded embedded dependency" into stable/helium

Merge "Fix bug 2449 in sal-remoterpc-connector Regarding to Moiz's comments, replace sender from null to ActorRef.noSender() in RpcManager.java" into stable/helium

Fix bug 2449 in sal-remoterpc-connector
Regarding to Moiz's comments, replace sender from null to
ActorRef.noSender() in RpcManager.java

Register SchemeContextListener with global SchemeService and push scheme
context to RpcBroker

Change-Id: Ia941f25f7b82a8146bf412389fbea5219c76f55a
Signed-off-by: Xin Chang <xin.chang@huawei.com>

BUG-2511 Fix XXE vulnerability in initial config loaders

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

Cache TransformerFactory

Instantiating a transformerfactory is costly, so cache and reuse it.

Change-Id: I43561ef36cdd10ef077b77c71673840d4dc83fbd
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 2fac3594f00d12e35239ecb61cb4454bc8b67d0e)

BUG-2459: Fix netconf-monitoring not reusing JAXB context

JAXBContext can be freely reused, so cache it, as it has significant
footprint in testing tool.

Change-Id: If2f53b9dfd4a1d30390580fd800aa2f5eb4e2b72
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 50b0ecab6c88cc2de12dc7d111ad80a198989543)

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>
(cherry picked from commit 61486fc82a2f4b3dc49ea49102e08680f8f059a6)

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>
(cherry picked from commit b0ab6a3cbc68fbd859151507622c651f823714a5)

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>
(cherry picked from commit 02541f9e4c26b77e9e592f68b103cb71cf99017b)

Merge "BUG-2459: do not instantiate factories in fastpath" into stable/helium

Merge "Bug 2405 - sal-binding-broker tests use openflow model" into stable/helium

Merge "BUG-2511 Fix possible XXE vulnerability in restconf" into stable/helium

BUG-2511 Fix possible XXE vulnerability in restconf

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

BUG-2511 Fix XXE vulnerability in Netconf

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

Merge "BUG-2510: handle RPC route removal" into stable/helium

Bug 2405 - sal-binding-broker tests use openflow model

* removed and recreated openflow protocol specific code from tests

Change-Id: I9dce5d9122974b5d7a58fcb040cb3befc2498e93
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>

BUG-2510: handle RPC route removal

DOM to Binding forwarding ignored removals of RPC implementations. Add
that case back.

Change-Id: I7af1d8b43b9b229cc9a696cfdfc61146f960b374
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 11380c5b8bc0d9831ed182ed758a74f66278df78)

BUG-2510: Remove all registrations when a routed rpc is closed

The API contract specifies that the effects of a Registration are
removed on close(). Unfortunately the routed RPC case is more funny, as
it has sub-registrations, for each added path -- and we have no mention
of the fact that the user should be removing them.

Hence we need to handle the user expecting us to remove any and all
paths which have been registered.

Change-Id: Iedf451c2a481d648173819f895cccb9e63c54d99
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 940751adf968a6789514566281bb0817bbf07878)

BUG-2459: do not instantiate factories in fastpath

This caches the factories in static fields rather than instantiating
them on each call.

Change-Id: I4dfee7724320914f2c7c9724192e4653deea3cde
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit a69a67c76a04a89d870e9f12df7919cbeb0d5001)

BUG 2463 - Changing setting for shard-snapshot-data-threshold-percentage does not work

This was missed by the commit which introduced this setting. It was not read from the config
and set on the DataStoreContext. This meant that the default (12 percent) was used regardless
of the actual setting.

Also changed the name of the stat to InMemoryJournalDataSize instead of just DataSize to be
more clear

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

Merge "Bug 2368: MD-SAL FRM may update/remove unexpected flow entries." into stable/helium

Fix bug 2450 - Statistics collection slow - performance

* fix timeout value for statWaiter to notification (30 sec is mistake - 3sec is correct value)
* add check TransactionId for every notification (prevent unexpected notification for collecting next statistics)
* timeout has to clear TransactionId (prevention for notification from slower statistics processes

* patch 3  - revert the log level msg (debuging issue in StatPermCollectorImpl)
- change an expiration calculation for cached RPC results (StatRpcMsgManagerImpl)
- fix conditions for call notifyToCollectNextStat (Meter, Group)

succesfull tested for karaf-compatible

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

Merge "Bug 2175 - Migrate frm, statistics and inventory manager to config subsystem" into stable/helium

Bug 2175 - Migrate frm, statistics and inventory manager to config subsystem

* migrated statistics manager to config subsystem
* updated tests
* patch resolves Bug 1646 and Bug 2079
* Patch Set 6 - add log.warn msg for a fail loading xml ConfigSubsystem input value + fix MIN_REQUEST_NET_MONITOR_INTERVAL_DEFAULT to 3sec (follow Anil's comment)

Change-Id: I67d398c82c3dafbb2ef759d07a80d17ad6656c1e
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

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

BUG 2437 - Enable snapshotting based on size of data in the in-memory journal

- Changed RaftActor to snapshot based on the transaction count or the size of the
  data in the in-memory journal whichever comes earlier
- The size of data that is used is the rough (not-accurate) size of the Payload
  in the Replicated log entry
- In ShardStats exposed another property which is the data size of the in-memory
  journal
- The snapshot data threshold percentage is configurable using the config sub-system and is
  set to a default of 12%. The reason for setting it at 12% by default is because we have
  a total of 8 default shards out of the box. I could have set this to 16% as toaster is not
  a "real" data shard.
- The snapshot data threshold is calculated as a percentage of the Runtime.totalMemory()
  which is the total memory the jvm considers available for object allocation. From testing
  it appears that the total memory is what would appear in jconsole as the committed memory.

I have not added any unit testing for this - but tested this using the scenario described in
bug 2437 and it seems to work pretty well. The deployment used only 2G of memory and worked
fine for a 7 switch topology and I observed that it had not run out of memory after more than
2 hours.

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

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>

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>
(cherry picked from commit a469dbcec569cc972df0cd57cf725a2173d2604a)

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." into stable/helium

Merge "BUG-2340 Fix improper cleanup of resources in netconf ssh handler" into stable/helium

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

Change-Id: I7fe59ca72505838dd377c4689615cfc161eff179
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.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

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

Merge "bug 2266 : added more types of schema nodes to increase code coverage" into stable/helium

Merge "Bug-2277: fix the Leader test failing in jenkins" into stable/helium

Merge "Bug-2277 : Isolated Leader Implementation" into stable/helium

BUG-1953: fix SAL compatility layer

Sal compatibility produces pure addresses instead of prefixes as
dictated by the model. Fix moth tests and implementation to emit
prefixes.

Change-Id: I1aca04c0fc22aa973d9578fea33550194c6f9264
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit da19620fa7bf6af3ab28c90ebd4df8eec920ec2c)

Merge "Bug 2328: Change ensureParent algorithm." into stable/helium

Bug-2277: fix the Leader test failing in jenkins

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

Bug-2277 : Isolated Leader Implementation

A new RaftState has been added and on a scheduler, we check if the leader is isolated, looking at the majority peer statuses.
If Yes, then it switches the behavior to IsolatedLeader.

On the receipt of each AppendEntriesReply, the IsolatedLeader checks for isolation.
And if no, then switches back to either Leader or Follower.

These changes have been tested with TestDriver  and 3 node cluster and the switching of Leader to IsolatedLeader and back,
on the stopping and reinstating of followers, was successful.

The Isolated Leader check interval can be configured via the config subsystem

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

bug 2266 : added more types of schema nodes to increase code coverage

Change-Id: I7372157ffc2b2e69ff9ae34b44eaeecadbf6b75d
Signed-off-by: Harman Singh <harmasin@cisco.com>

Merge "BUG-2304 Fix subtree filter in netconf-impl for identityrefs." into stable/helium

Merge "BUG 2155 - depth parameter in URI" into stable/helium

BUG-2304 Fix subtree filter in netconf-impl for identityrefs.

There was a bug in prefixed content comparison.

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

Merge "Bug 2233 - RPC register exception when rpc has no input" into stable/helium

Merge "BUG-2370 Reset schema context on disconnect in nc" into stable/helium

Bug 2368: MD-SAL FRM may update/remove unexpected flow entries.

FRM should always set strict flag into update-flow/remove-flow
RPC input.

Change-Id: I6027442b56f4cd93cfc3f954dc8501bb57e2d64a
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 41d00971714a0e52d37c67734a22068103244224)

Revert "Revert "Merge "Revert "Bug 1764 - added service for pushing default node configuration"" into stable/helium""

This reverts commit 64544f998de558cb6219e166a8fdb3f5e3c80589.

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

Revert "Revert "BUG-2254 Enable schema-less rpcs invocation in netconf-connector""

This reverts commit 294bfea81a17c03f8ef11f67c7ea9f9277d46d6c.

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

Revert "Revert "BUG-2254 Make runtime rpcs in config subsystem/netconf handle context-instance attribute with namespaces""

This reverts commit 63146c45f2c982be3703bf3aae137cbd9498ce76.

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

Merge "Bug-2136 - fix for is-local-path" into stable/helium

Merge "BUG 2302 : odl-clustering-test-app should not be part of the odl-restconf-all feature set" into stable/helium

Merge "Bug 2347: Minor fixes to correct log output" into stable/helium

Merge "BUG 2353 : Handle binary, bits and instanceidentifier types in NodeIdentifiers" into stable/helium

Bug-2136 - fix for is-local-path

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