Merge "BUG-2573: mark CompositeNode-based RPC APIs obsolete"

Merge "Marked as deprecated all classes within protocol-framework."

Migrate to MoreObjects.ToStringHelper

Objects.ToStringHelper has been deprecated and moved to MoreObjects.
Adjust to this change.

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

Merge "Bug-2692 : Eliminate InitiateInstallSnapshot"

Marked as deprecated all classes within protocol-framework.

- this part of controller will be removed between M3 and M5
- if you're using it, rewrite your code to use netty directly

Change-Id: I864b914d885e15029bca903829c683306780a69d
Signed-off-by: Dana Kutenicsova <dkutenic@cisco.com>

Remove source element from get netconf rpc

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

Bug-2692 : Eliminate InitiateInstallSnapshot

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

Fix Guava 18 upgrade issue , Objects to MoreObjects

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

Merge "Fix reading operaional data for netconf mountpoint"

Merge "Do not override odlparent properties"

Merge "Remove xtend.version property"

Fix reading operaional data for netconf mountpoint

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

Do not override odlparent properties

These properties are defined upstream, no need to define them here.

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

Remove xtend.version property

It is no longer needed, as odlparent has been fixed.

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

Add more context if a config push fails

This adds the current state to the error message.

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

Fix neutron implicit osgi import

Neutron northbound requires osgi-core but fails to mention it in
pom.xml. Fix that.

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

BUG-1173: ensure compatibility with Guava 17+

This migrates the sole user to the Guava-15+ API, as the API is removed
in version 17.

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

Define xtend.version until it is remove from odlparent

https://git.opendaylight.org/gerrit/15313 will remove references to it,
but until that happens, we need to provide an override.

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

BUG-1173: ensure compatibility with Guava 17+

This migrates the sole user to the Guava-15+ API, as the API is removed
in version 17.

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

BUG-1173: remove explicit guava version

Removes an explicit mention of 14.0.1 in favor of odlparent-provided
version.

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

Make sure config-api brings in osgi-core

config-api is used in config-generated code, which needs osgi for
compilation. Unfortunately augmenting all current users is prohibitive,
so just make sure all users inherit osgi-core as a compile-time
dependency.

The real fix is to create a parent, which will pull osgi-core into
artifacts which actually invoke yang-jmx-plugin.

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

BUG-2573: mark CompositeNode-based RPC APIs obsolete

With the advent of the NormalizedNode-based APIs, the users should be
moving towards implementation of those.

This is a placeholder, to be merged once a reference implementation is
available.

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

Merge "BUG-2573: create DOMRpcRouter"

Merge "BUG-2626: activate enunciate only on jdk-1.7"

Merge "BUG-2626: Do not override enunciate version"

Merge "Bug-2674 Notification support for netconf testtool"

Merge "BUG-2573: add NormalizedNode-based RPC API"

Bug-2674 Notification support for netconf testtool

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

Merge "(Fix: Bug 2711) - Match RPC actor system's Netty TCP settings with existing cluster DATA actor system setting to avoid oversized Gossip RPC messages."

Merge "Try to eliminate random failures in RaftActorTest"

Merge "Add slf4j SimpleLogger properties file for unit tests"

Merge "Added LeaderElectionScenariosTest class"

Try to eliminate random failures in RaftActorTest

This patch attempts to add the following enhancements,
1. All test actors get created using a TestActorFactory which when closed
   automatically kills the actors created by it
2. Provide a method to auto generate a friendly actor name/id for cases
   where we need to use such names

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

(Fix: Bug 2711)
- Match RPC actor system's Netty TCP settings with existing cluster DATA actor system setting to avoid oversized Gossip RPC messages.

Change-Id: I3e2c13cc23aa29052c5ed5096a887c563469d59e
Signed-off-by: mark.mozolewski <mark.mozolewski@hp.com>

Merge "Neutron to return ServiceUnavailable if no providers registered"

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: Iefa092a35bdc178f632373eac3b84578028d3768
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Merge "Mark yang generated source location as a source directory."

Merge "Stop scheduling the Install Snapshot check"

Merge "(Bug 2035) - Increasing default Akka config for auto-down of unreachable nodes from 10sec to 5min to avoid locking out nodes during transient network blips, etc. "

Merge "BUG 2705 : No need to ReadData asynchronously in ShardTransaction"

BUG-2626: activate enunciate only on jdk-1.7

The plugin is not supported with JDK8. Create two parent pom.xmls, one
of which defines the activation on JDK7 only and the second one
centralizes the AD-SAL definitions for it.

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

BUG-2626: Do not override enunciate version

The overall version is 1.28, so do not downgrade it.

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

BUG 2705 : No need to ReadData asynchronously in ShardTransaction

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

Merge "Do not duplicate OSGi dependencyManagement"

Merge "BUG-2673: Introduced new more low-level DOM Data Change APIs"

Merge "BUG-1511 - Datastore: cleanup ListenerTreeAPI"

Merge "BUG-2632 Netconf connector (optionally) reconnects after a schema change is detected"

Merge "BUG-2600 Dynamic schemas in netconf server"

Merge "Bug 2684 - Fixing support for Neutron binding extended attributes"

BUG-1511 - Datastore: cleanup ListenerTreeAPI

- split ListenerTree two static inner classes to ListenerWalker and ListenerNode
- fix refences [ResolvedDataChangeState]
- improve performance of ListenerWalker.close()

Change-Id: Ide82bc6caeaca1aa5c9080f134d570cf7a1571d6
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-2673: Introduced new more low-level DOM Data Change APIs

As it turned out abstraction provided by Helium Data Change
API was CPU costly and calculated lot of unnecessary data,
and did not provide necessary navigation for some use-cases.

This new API contract defines more low-level data change
APIs, where application is more in control, how it will
process / consume / decompose Data Change by providing
DataTreeCandidate.

Change-Id: I468c9583c7999bc02101a18a8adf5dc1a084f1d9
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "BUG-2627: do not duplicate descriptions"

Merge "BUG-2627: JMX interfaces need to be public"

BUG-2627: do not duplicate descriptions

This class effectively creates a repeateable annotation. Unfortunately
in Java 8, we get twice as many annotations, which leads to duplicate
entries. Instantiate an intermediate Set, so we suppress potential
duplicates.

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

BUG-2627: JMX interfaces need to be public

Java 8 enforces the specification's required for MXBean interfaces to be
public. The mock interfaces were not public, hence were failing the test
automatically.

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

Merge "Initial heartbeats not sent by Leader"

Stop scheduling the Install Snapshot check

We only schedule it once and even that is unneccessary
because we check if a snapshot needs to be triggered in
sendUpdatesToFollower and that should be good enough

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

Merge topic 'bug/2676'

* changes:
  Store the datastore type in dataStoreContext
  Refactor DataStoreContext
  BUG 2676 : Apply backpressure when creating transactions

Merge "Run SingleFeatureTest on features/netconf"

Add a diagnostic message to figure out why test fails

Adding a message to figure out why testFakeSnapshotsForLeaderWithInInitiateSnapshots
fails sometimes

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

Store the datastore type in dataStoreContext

This way we do not need to pass it to DistributedDataStoreFactory,
DistributedDataStore and ShardManager as it is already present in
dataStoreContext

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

Refactor DataStoreContext

It was getting difficult to maintain because we had the same field defined
in the Builder and the DatastoreContext and we were passing the fields along
using the DatastoreContext constructor. This meant that every new setting that
we added required multiple changes to be made to DatastoreContext.

Refactored for the Builder to create the DatastoreContext upfront and directly set
it's fields.

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

BUG 2676 : Apply backpressure when creating transactions

This patch uses a rate limiter to limit the number of write transactions
that can be created on the datastore at any given point of time (within a
second).

We also have added a metric that is trackable using JMX which will show
exactly how much time we take in committing a transaction.

Could this rate limiter have been on the DataBroker? Yes it could have been
but I think we need to go granular on this one. Ideally the ratelimiter would
be for every shard transaction but it's not trivial to do in a way that would
actually block the consumer for now - this is because creating a transaction on
Shard itself is an async operation.

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

(Bug 2035)
- Increasing default Akka config for auto-down of unreachable nodes from 10sec to 5min to avoid locking out nodes during transient network blips, etc.

Change-Id: I3528b34373af192ff6b58b214ba9313b379e9828
Signed-off-by: mark.mozolewski <mark.mozolewski@hp.com>

Run SingleFeatureTest on features/netconf

Change-Id: Ibc60948413f633b4954ff603ad6a951658a95132
Signed-off-by: Ed Warnicke <eaw@cisco.com>

BUG-2632 Netconf connector (optionally) reconnects after a schema change is detected

A notification listener is used for schema change detection

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

BUG-2600 Dynamic schemas in netconf server

Netconf server does not preserve snapshots of schema contexts for each netconf session anymore

Every netconf session is using current SchemaContext
+ Use notification manager to notify about capability changes

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

Reduce ShardManager logging

ShardManager emits info logging on SchemaContext changes which yields a
lof of output that isn't that useful. Reduce to debug.

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

Merge "Fix issue with TransactionProxy throttling test failures"

Merge "Bring features/neutron into the same parent dir as everything else"

Fix issue with TransactionProxy throttling test failures

Make the measurement a little bit more precise by using nanoTime
instead of currentTimeMillis

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

Bring features/neutron into the same parent dir as everything else

This will ease breaking the code out to the neutron northbound project

Change-Id: Idf887c14cdbc7e77b74f4964b516ac16ab6877e6
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Bug 2701: Fixed dependencies for features-netconf.

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

Merge "Bug 2669: Use slf4j Logger instead of akka LoggingAdapter"

Merge "BUG-2679 Workaround for wrong nagasena encode/decode with reused transmogrifier"

Merge "BUG-2310 - widen yang model for netconf-node."

Merge "BUG-2599 Netconf notification manager initial API and Impl"

Merge "Fix missing log argument"

Merge "Standardize on pax-exam 4.4.0"

Merge "BUG-2288: implement DOMNotificationRouter"

Merge "BUG-2627: Fix ambiguous lookup in test case"

Merge "Bug-2692:Avoid fake snapshot during initiate snapshot"

Added LeaderElectionScenariosTest class

    New unit test class that tests various leader election convergence
    scenarios.

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

Add slf4j SimpleLogger properties file for unit tests

Added a simplelogger.properties to sal-distributed-datastore and
sal-akka-raft to configure the logging format and to enable trace logging
for the appropriate pacakges.

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

BUG-2627: Fix ambiguous lookup in test case

Looking up by java name "stream" matches two potential beans, which
means that the returned value depends on underlying collections. This
breaks when upgrading to Java 8, for example.

Fix by performing an explicit prefix name, which returns the right bean
everytime.

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

Merge "Optimize install snapshot, by making a call at end of installsnapshotreply"

Merge "Add Distributed DataStore as a dependency of the Simulator"

Merge "Optimized AppendEntries sending to follower by adding AppendEntry call at end of reply, if needed."

Merge "Add signature marker and version to NormalizedNode streaming"

Optimize install snapshot, by making a call at end of installsnapshotreply

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

Optimized AppendEntries sending to follower by adding AppendEntry call at end of reply, if needed.

This will help to reduce the lag of a follower and leader does not have to wait for heartbeat time to
send a message to follower. This way, we can tune heartbeat interval in better manner.

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

Initial heartbeats not sent by Leader

When a Leader instance is created, it immediately sends out heartbeats
to the followers. However recent changes broke this behavior as
sendAppendEntries now checks if the follower's timeSinceLastActivity >=
heartbeat interval. However, in this case, timeSinceLastActivity is 0 so
the initial heartbeats aren't sent out. Also, on replication, it seems
that sendAppendEntries should always send AppendEntries. So we want to
disable this checking in these 2 cases. We only want check
timeSinceLastActivity if it's initiated by a scheduled heartbeat.

To resolve this, I changed the Leader constructor to directly call
sendAppendEntries instead of scheduling SendHeartbeat with delay 0. I
now pass in the interval to sendAppendEntries. The scheduled heartbeat
passes in the actual heartbeat interval and the oher 2 calls pass in 0.

Several unit tests in LeaderTest had to be changed due to the new
behavior.

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

Bug-2692:Avoid fake snapshot during initiate snapshot

When an initiate snapshot is received , we should be setting the snapshot captured flag, in order to avoid a fake snapshot at the same time.

Also a check should be made to avoid another initiate snapshot to occur, in case the capture snapshot takes time.

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

Do not duplicate OSGi dependencyManagement

Both compendium and core are already defined by odlparent, so we don't
need it here.

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

Mark yang generated source location as a source directory.

Configured in plugin mgmt only. Will run automatically when binding-parent is updated. See https://git.opendaylight.org/gerrit/15087 (yangtools).

Change-Id: I846478b374645340006937a107c186cfa30f6975
Signed-off-by: Nathan Harmon <nathan.harmon@hp.com>

BUG-2573: create DOMRpcRouter

This patch provides a default implementation of DOMRpc(Provider)Service.
The implementation is geared heavily towards steady state, e.g. so that
invocations are fast.

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

(Fix: Bug 2529)
- Now ShardTransaction does NOT return CloseTransactionReply for Read-Only transactions.

- Separated ShardTransactionTest cases for RW/WO/RO Shard Transactions to test expected CloseTransactionReply behavior for each.
- Updated ShardTransaction Javadoc since ShardTranactionCreater#create method and ShardTransaction subclasses imply javadoc comment was wrong about applicability to other transaction types.

Change-Id: I46aa6025b0a975c4592344bbee2d917574f2c750
Signed-off-by: mark.mozolewski <mark.mozolewski@hp.com>

Merge "Bug 2410: Use generated serialVersionUID for messages"

Merge "Fix NPE in AbstractLeader#handleInstallSnapshotReply"

Add Distributed DataStore as a dependency of the Simulator

This is because AppendEntries delivers the ModificationPayload from
DDS and without adding a dependency on DDS we get a ClassNotFoundException

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