Bug 2318: Follow-up changes for previous patch 12535

Addressed some comments and other minor changes as a follow-up to patch 12535.

Enhanced the comments in TransactionChainProxy.

Modified Shard to send a failure reply if an runtime exception occurs in
createTransaction. This allows the error to be propagated to the caller
instead of being thrown back to akka.

Modified TransactionChainProxy to throw an IllegalStateException if the
next chain Tx is created before the previous one is readied. Prior, the
Shard would throw an IllegalStateException but from the Future returned
from submit and not on new*Transaction as the API docs state. So this make
it consistent with API contract. This was implemented by introducing a
volatile State field with implemnetations, Idle, Allocated and Closed.

Also, modified TransactionChainProxy to throw TransactionChainClosedException
if the chain is already closed when on Tx create. Again. this makes it
consistent with the API docs.

Added unit tests in DistributedDataStoreIntegrationTest to verify failure
in both cases.

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

Merge "Bug-2301 - Clustering:Snapshots need not be stored in in-mem ReplicatedLog for Installing snapshots"

Merge "Update junit Assert imports"

Merge "Migrade JUnitMatchers references"

Merge "Remove unused imports"

Merge "Fix checkstyle warnings in opendaylight/netconf"

Merge "Fix checkstyle warnings in opendaylight/config"

Merge "Create config-artifacts"

Update junit Assert imports

JUnit 4.11 has moved Assert to org.junit, and deprecated
junit.framework.Assert. This adjust the imports to fix the warnings.

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

Migrade JUnitMatchers references

With JUnit 4.11 JUnitMatchers.containsString() has been deprecated in
favor of org.hamcrest.CoreMatchers.containsString(). Same goes for
hasItem().

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

Remove unused imports

This fixes warnings stemming from unused imports.

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

Fix checkstyle warnings in opendaylight/netconf

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

Fix checkstyle warnings in opendaylight/config

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

Fix warnings in sal-akka-raft

Make sure to add generic arguments to instantiation and do not use
deprecated junit classes.

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

Add generic arguments to Answer

Just use Object as the argument to get rid of the warnings.

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

Merge "Introduce forwarding DOMData classes"

Merge "Fix a typo in javadoc"

Create config-artifacts

This creates a dedicated artifact for hoilding the config subsystem
artifacts, which then can be reused by downstream projects.

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

Merge "Bug 2233 - RPC register exception when rpc has no input"

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

Merge "Fix logback-config warnings"

Merge "Remove duplicate groupId"

Merge "Fix warnings in netconfig-netty-util"

Merge "Fix warnings in netconfig-monitoring"

Merge "Fix cluster test warnings"

Merge "Add serialversionUUID to cluster/datastore/modification"

Merge "Fix raw Class in sal-distributed-datastore messages"

Merge "Fix warnings in clustering"

Merge "Fix warnings in config-manager"

Merge "Fix warnings in config-util"

Merge "Fix warnings in neconf-util"

Merge "Fix warnings in config-api tests"

Merge "Do not override managed versions"

Bug-2301 - Clustering:Snapshots need not be stored in in-mem ReplicatedLog for Installing snapshots

1. Snapshots, once persisted to disk, would not be stored as part of ReplicatedLog in memory
2. An Install snapshot would initiate a snapshot capture and be sent to the follower
3. FollowerLogInformation in Leader has a stopwatch which helps to find if a given follower is active/up.

Rebased

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

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>

Fix a typo in javadoc

Simple typo.

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

Fix logback-config warnings

We do not need to use raw types, so use proper generic arguments.

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

Remove duplicate groupId

Specified groupId is the same as parent's -- so we do not need to
re-declare it.

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

Fix warnings in netconfig-netty-util

Removes the use of deprecated junit classes and raw types.

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

Fix warnings in netconfig-monitoring

Removes the use of deprecated junit classes and raw types.

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

Fix cluster test warnings

Remove use of raw types, suppress serialVersionUUID warnings and the use
of deprecated junit classes.

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

Add serialversionUUID to cluster/datastore/modification

Serializable classes need to define this field. Also fixes the use of
raw NormalizedNode type.

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

Fix raw Class in sal-distributed-datastore messages

Fixes the use of Class in the messages. Also makes sure exported
constants are really constant.

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

Fix warnings in clustering

Fixes the use of raw types.

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

Fix warnings in config-manager

Fix the use of raw types, unnecessary casts and use of deprecated junit
classes.

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

Fix warnings in config-util

Fix use of deprecated junit classes and unused imports.

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

Fix warnings in neconf-util

Remove use of raw types, unnecessary casts and use of deprecated junit
classes.

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

Fix warnings in config-api tests

Move to non-deprecated classes and fix the use of raw types.

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

Do not override managed versions

Versions managed by parent, no need to specify them.

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

Quickfix: Changed used feature to odl-aaa-netconf-plugin

AAA project renamed their netconf related feature without
introducing proper patchset into controller. This fixes
features.xml to reference properly new feature.

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

Merge "Remove l2switch sample"

BUG 2090 : Clustering : Bring akka-raft unit test coverage upto 80%

Change-Id: I7b2b16aa42b655e924942f356602c64ccf6d86b7
Signed-off-by: Moiz Raja <moizr@dhcp-10-154-140-127.cisco.com>

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

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

Merge "Fix for Bug 2290."

Merge "Introduce a benchmark for databroker"

Bug 2337: Fix Tx already sealed failure on Tx commit

Details outlined in bug 2337.

Moved the publishing of the TransactonContext instance in
TransactionFutureCallback#onComplete after the cached operations are
executed.

Also fixed a timing issue in
DistributedDataStoreIntegrationTest#testChangeListenerRegistration that
caused intermittent failures. This was an issue with the test. The
listener registration is done async in the shard so the notification for
the first write commit could occur in the initial notification on
registration which screwed up the test. So I moved the first write
commit before the registration so we expect an initial notification.

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

Merge "Bug 2318: Ensure previous Tx in chain is readied before creating the next"

BUG 2325 : Value type of byte[] not recognized by the NormalizedNodeSerializer

Change-Id: I16eab6cdbf7712624f2c1fafb5bf41107b6ae379
Signed-off-by: Moiz Raja <moraja@cisco.com>
(cherry picked from commit 819b04091a3d4d96612a1036228638a3b4e85d09)

Bug 2318: Ensure previous Tx in chain is readied before creating the next

In TransactionChainProxy, subclassed TransactionProxy (ChainedTransactionProxy)
to hook into the TransactionProxy to capture the cohort Futures on ready
and to customize the returned create shard Tx Future.

The previous Tx's ready Futures are captured and cached in the enclosing
TransactionChainProxy instance. When the client creates the next Tx in
the chain, the Future from the shard Tx create message is wrapped by a
sequence Future that combines all the previous ready Futures. Thus, when
the previous ready Futures complete, the shard Tx create is sent and
that Future is used to complete the returned Promise.

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

Introduce a benchmark for databroker

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

Fix for Bug 2290.

Wrap existing IPv4 code with version test.
Add new IPv6 code with version test.

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

Remove l2switch sample

This sample is openflow-specific and creates confusion with the l2switch
project. Remove it.

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

Bug 2233 - RPC register exception when rpc has no input

* added check that should prevent non routed Rpcs being treated as
routed and and failing at context resolution.

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

Merge "BUG-1953: fix SAL compatility layer"

Merge "Bug 2065 - fixed typo in leaf name"

Fix intermittent failure in DataChangeListenerTest

Fixed an intermittent failure in testDataChangedWithNoSender due to timing if a
DeadLetter is received due to the Monitor message being dropped as the
unit tests don't set up the termination monitor actor. The test expects
no DeadLetter for DataChangedReply so added chanhes to ignore
DeadLetters for other messages.

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

BUG 2296 : TransactionProxy should support the ability to accept a local TPC actor path

In Helium when the ShardTransaction processes a Ready message it sends back a the path of a ThreePhaseCommitCohort actor in the ReadyReply. This path is actually a local actor path, this local actor path is then converted into a remote path by the TransactionProxy.

The fix for Bug 1607 breaks this capability which is required to support forward compatibility in a cluster where a transaction request originates in a node that has been upgraded to Helium-1 and the actual transaction is happening on a node which has not yet been upgraded to Helium-1.

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

Merge "Bug 2294: Handle Shard backwards compatibility"

Merge "Bug 2134: Fix intermittent RaftActorTest failure"

BUG 2134 : Make persistence configurable at the datastore level

- Added a peristent flag in the config sub-system
- If persistent in not true then we ignore recovery messages and not use the
  akka persistence apis for persisting anything
- The unit tests
    - assume that persistence is on by default (which it is)
    - test that if recovery is applicable only then the ShardManager and RaftActor process the recovoery messages.
    - test that when persisting the data-persistence API is used in the appropriate places (see RaftActorTest/ShardManagerTest)

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

Bug 2065 - fixed typo in leaf name

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

Bug 2294: Handle Shard backwards compatibility

Implemented as outlined in Bug 2294.

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

Merge "Bug 2055: Handle Tx create in TransactionProxy resiliently"

Bug 2134: Fix intermittent RaftActorTest failure

We need to wait for akka's recovery to complete before we perform our
tests as akka's recovery runs async and may interfere with our tests and
cause them to fail if the timing is right. This was seen once on
jenkins.

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

Merge "Cosmetics: check in pom.xml files as _sort_pom_ wants them to be"

Cosmetics: check in pom.xml files as _sort_pom_ wants them to be

Change-Id: I08fc2c0284e0396605cec7d218ea4470a312bfc8
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Bug 2265: Modified NormalizedNodeOutputStreamWriter to implement yangtools interface"

Merge "Bug 2252: Terminate ShardWriteTransaction actor on ready"

Merge "BUG 2221 : Add metering to ShardTransaction actor"

Merge "Bug 2086: Adding normalized node  stream reader and writer."

Merge "BUG-2195 Enable features test for netconf connector"

BUG-2195 Enable features test for netconf connector

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

Bug 2086: Adding normalized node  stream reader and writer.

Normalized node and its children are written in recursive manner. Besides the leafNode, leafSetEntryNode and anyxmlNode, all other nodes must call endNode method
while writing object to data stream. This helps to recreate the object.

Recursion  runs in opposite direction while reading the object.

Updated code with review comments

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

Bug 2055: Handle Tx create in TransactionProxy resiliently

    Modified TransactionProxy to utilize the FindPrimaryShard
functionality
    to wait until the shard is initialized. Also added retries to
    the CreateTransaction operation to wait a period of time for the
shard
    to elect a leader. This makes it more resilient to avoid transient
    timing/sequencing failures, particularly on startup.

    In addition, the FindPrimaryShard and CreateTransaction operations
are now
    done async (non-blocking). The futures are combined to yield the
    CreateTransactionReply which is passed to a new class
    TransactionFutureCallback on completion. The
TransactionFutureCallback
    class creates and stores the TransactionContext which subsequent
    read/write/delete/ready transaction operations can access. If a
    transaction operation occurs before CreateTransaction completes, the
    operation is cached in TransactionFutureCallback and executed when
    the CreateTransaction future completes.

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

Merge "Bug-2098- sal-compatibility not get up-to-date flow information"

Bug 2265: Modified NormalizedNodeOutputStreamWriter to implement yangtools interface

The NormalizedNodeOutputStreamWriter was originally written to implement
a NormalizedNodeStreamWriter interface class that was copied from
yangtools. This was done to get access to the NodeWithValue in the
leafSetEntryNode method which isn't specified in the yangtools
interface. However it turns out we don't actually need the NodeWithValue so
NormalizedNodeOutputStreamWriter was changed to implement the yangtols interface
and the copied NormalizedNodeStreamWriter interface was removed.

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

Bug-2098- sal-compatibility not get up-to-date flow information

When cached is passed in as false, we attempt to collect all the notifications in a guava cache and
return the results after we get the last notification.

Depends on the OFP Patch : https://git.opendaylight.org/gerrit/#/c/11655/

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

Merge "Bug 2055: Handle shard not initialized resiliently"

Merge changes I8b1a0281,Ib6342c74

* changes:
  BUG-1612 Remove ganymed third party ssh lib (replaced by mina ssh)
  BUG-1612 Remove ganymed implementation of SSH server wrapper.

Merge "BUG 1839 - HTTP delete of non existing data"

Merge "Fix checkstyle warnings"

Merge "Convert netconf to use artifacts"

Merge "Import yangtools-artifacts"

Merge "Cleanup RpcRoutingStrategy definition"

BUG-1612 Remove ganymed third party ssh lib (replaced by mina ssh)

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

BUG-1612 Remove ganymed implementation of SSH server wrapper.

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

Merge "BUG-1612 Update netconf testtool to use new ssh server wrapper"

Merge "BUG-1612 Implement mina ssh netconf server endpoint"

Merge "Fix failing reconnect test."

Merge "BUG-2184 Fix subtree filtering for identity-ref leaves"