Removed RPC get-node-ip-address from flow-node-inventory

this RPC is not needed. IP address is stored in datastore.
Removing of implementation should be merged first:
https://git.opendaylight.org/gerrit/#/c/10382/

Change-Id: I749a65edf2a774d3a91cb210eb9bc9189a983e45
Signed-off-by: Martin Sunal <msunal@cisco.com>

Fixed dependencies for features-nsf.

features-nsf depends on features-ad-sal and features-base. However,
pom.xml for features-nsf lacked these dependencies.

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

Merge "BUG-1622 Fix handling of non-downloadable schemas from remote netconf device"

Merge "BUG-1621 Fix reconnecting."

Merge "Cleanup DOMStoreReadTransaction/DOMDataReadTransaction exists API documentation"

Merge "BUG-1493: activate recursion elision"

BUG-1622 Fix handling of non-downloadable schemas from remote netconf device

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

BUG-1621 Fix reconnecting.

Netconf-connector did not reconnect after recent changes (Ssh mina integration)
SshHandler was in pipeline after listener in reconnect, thats why the listener received no event about session down.

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

Cleanup DOMStoreReadTransaction/DOMDataReadTransaction exists API documentation

This commit also cleans up the usage of ForwardedBackwardsCompatibleDataBroker from DataBrokerTestCustomizer

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

BUG-1520 coverage tests for config-api

Change-Id: I497ad8717ad98ca58f34a1b5052b779481b6a665
Signed-off-by: Filip Tehlar <ftehlar@cisco.com>

Merge "Adding sal-compatibilty feature"

Merge "BUG-650: fromYangInstanceIdentifier() can return null"

BUG-650: fromYangInstanceIdentifier() can return null

Properly interpret a null return as an absent identifier.

Clarified cast in #toBinding(Entry) and explained
why it is necessary, even if Eclipse reports
it as unnecessary (difference in various java compilers).

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

Merge "Simplify YangInstanceIdentifier references"

Adding sal-compatibilty feature

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

Add example Unit test to Toaster

Added an example unit test to the toaster application
which uses a live DataBroker. This is intended to show
how you can use the base test helpers to get access to
a live MD-SAL databroker.

Change-Id: I090b5806bf7bead086be3e85a5d42c64110aa8c2
Signed-off-by: Devin Avery <devin.avery@brocade.com>

Simplify YangInstanceIdentifier references

Now that the (binding) InstanceIdentifier and YangInstanceIdentifier
have different names, we can import both at the same time, clarifying a
bit the code.

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

Merge "Bug 1576: Handle remote failures for write Tx async"

Merge "Bug 1616: Issues with using container and default values in config yang"

Merge "BUG-1618 Handle pending writes in ssh netconfclient"

BUG-1618 Handle pending writes in ssh netconfclient

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

Merge "Added feature for sal-mdsal-xsql"

Merge "Added IP address of the OF switch to flow-node"

Merge "BUG-1521 Increase test coverage of netconf-netty-util"

Merge "BUG-1568 Remove unused VirtualSocket"

Merge "BUG-1521 Netconf-monitoring test coverage raise"

Merge "BUG-650: remove unneeded sort()"

Merge "Fixing incorrect case for interface method neutronLoadBalancerPoolMemberDeleted"

Added feature for sal-mdsal-xsql

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

Merge "Bug 1536: Fixed minimum values of SET_TP_SRC/DST."

Merge "Get some basic unit testing in place for the RaftActor class"

Merge "Do not use protobuf serialization for FindPrimary and it's responses"

Merge "Added missing dependency sections that helps if the cache is missing the required dependencies."

Merge "Improve SnapshotBackedReadTransaction.close()"

BUG-1521 Increase test coverage of netconf-netty-util

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

BUG-1568 Remove unused VirtualSocket

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

Merge "Bug 1234: Force to use legacy OF plugin if it is installed."

Merge "Archetypes for karaf support"

BUG-1521 Netconf-monitoring test coverage raise

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

BUG-650: remove unneeded sort()

The need for sorting entries by depth disappeared with the migration to
new codec infrastructure. Tracing shows some 44% CPU being spent here,
so this should provide a marked improvement with large data changes.

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

Improve SnapshotBackedReadTransaction.close()

This improves visibility of close() operation, resulting in improved
close/read collision detection.

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

Archetypes for karaf support

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

Added missing dependency sections that helps if the cache is missing the required dependencies.

Also added a missing northbound bundle and filled up missing versions

Change-Id: I7076d50fd5f1e8bc90f753e74b59936ce46db75d
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Merge "BUG-1568 Remove ganymed Ssh client adapter"

Merge "BUG-1568 Ssh Handler for netconf client Mina implementation"

Bug 1234: Force to use legacy OF plugin if it is installed.

This patch enables to prioritize AD-SAL protocol plugin services,
and gives higher priority to legacy OF plugin than sal-compatibility.

This patch never affects integration.git artifacts because legacy
OF plugin is filtered out if MD-SAL OF plugin is installed.

Change-Id: Iee40a6f724db2105f2ec36014a10872bad824e86
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Fixed possible NPE in TopologyManager FlowCapableTopologyExporter"

Merge "Make InMemoryDOMDataStore.ready() unsynchronized"

Merge "BUG-1601 Fix null pointer in netconf connector for empty schemas node"

BUG-1493: activate recursion elision

Previous patch prepared the infrastructure for elision, but it did not
actually activate it. This patch flips the switch on.

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

Merge "BUG 1597 - Do not use toString as serialized version of YangInstanceIdentifier/PathArgument"

Merge "Add type of transaction being created to log message"

BUG-1568 Remove ganymed Ssh client adapter

Change-Id: Id4ec6140df3e91fd8a219b03ceabca67732af568
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-1568 Ssh Handler for netconf client Mina implementation

This implementation of Ssh Handler is asynchronous and based on Apache Mina SSHD.

+ Add Mina SSHD to distribution.

Change-Id: I51f51e2eb56b5f97a87c3d068ea90f8a44adca09
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Workaround for karaf bug KARAF-3069"

Merge "Fix for possible NPE if Bundle is stopped."

Fixing incorrect case for interface method neutronLoadBalancerPoolMemberDeleted

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

Modified Base Features to follow proper naming conventions

Following the comments of merged patch this fixes the Base Features
naming conventions so that it is : odl-base-xxx. Changes on dependent
feature files are also included.

Change-Id: I46683d1c6d8a8cf17a22c3909b2a1d6a2ee3ff89
Signed-off-by: Mathieu Lemay <mlemay@inocybe.com>

Merge changes Ib0aadbff,I765ffc07

* changes:
  Add descriptions to features.
  Small change to make everything executable in target/assembly

Merge "Keep a cache of QName"

Add descriptions to features.

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

Trial patch to use stock jersey-servlet

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

Small change to make everything executable in target/assembly

It is already if you unzip the zip file.

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

Merge "Added features-test for Karaf features that cover Network Service Functions and ADSAL Northbound APIs"

Merge changes If391c483,I964eedcd

* changes:
  Added features-test for ADSAL Layer  Karaf features and fixed some minor issues that caused the failures before.
  Added features-test for base Karaf features and fixed some minor issues that caused the failures before.

Get some basic unit testing in place for the RaftActor class

The two tests that were added do the following,
    - test construction
    - test finding a leader when the current raft actor is the leader

This commit also specifies the proper includes and excludes for code coverage

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

Do not use protobuf serialization for FindPrimary and it's responses

FindPrimary, PrimaryFound and PrimaryNotFound are local messages so there
is no advantage to serializing and deserializing them into protobuf messages

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

Add type of transaction being created to log message

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

Keep a cache of QName

QName creation is expensive so keep of cache of 10,000 QNames around so that
we can lookup QNames that were previously created

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

Workaround for karaf bug KARAF-3069

https://issues.apache.org/jira/browse/KARAF-3069

More recent maven-bundle-plugins are putting

Require-Capability: osgi.ee; filter="(&(osgi.ee=JavaSE)(version=1.6))"
into MANIFEST.MF.  This causes them to not
be loadable in karaf when using the equinox framework (though
felix works fine).

As noted in the bug, this is the workaround.

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

Make InMemoryDOMDataStore.ready() unsynchronized

Tracing datastore operations has revealed some contention on the
datastore, specifically when users call WriteTransaction.ready(). It
seems this method does not need synchronization at all, as it is not
directly changing any state.

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

Merge "Bug 1262 - Document Transaction Chaining of MD-SAL"

Added features-test for Karaf features that cover Network Service Functions and ADSAL Northbound APIs

Also fixed a minor issues that caused the failures before.

Change-Id: I178f36f2ff53bbdd4d47864a3a153acab91b0a41
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Added features-test for ADSAL Layer  Karaf features and fixed some minor issues that caused the failures before.

Change-Id: If391c4837972062b4d90e5a84606ba5a74a60b24
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Bug 1576: Handle remote failures for write Tx async

TransactionProxy
  - Modified TransactionContextImpl to cache Futures from put/merge/delete
    ask operations.

  - On ready, combine all previous cached Futures and the
    ReadyTransactionReply Future and transform to one Future that
    extracts the cohort actor path from the ReadyTransactionReply. Any
    Future failure results in a failed cohort actor path Future with
    the cause propagated.

  - The cohort actor path Future gets passed to the ThreePhaseCommitCohortProxy.

  - Modified read and exists methods to also wait and verify async any
    previous cached put/merge/delete Futures.

ThreePhaseCommitCohortProxy
  - On canCommit, it first asynchronously builds the list of cohort actor
    paths from the Futures passed from the TransactionProxy. If any
    cohort path FUture fails, canCommit fails and propgates the failure
    cause.

Added extensive unit tests for both classes to cover all cases.

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

Bug 1616: Issues with using container and default values in config yang

Fixed issues as outlined in bug 1616.

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

BUG 1597 - Do not use toString as serialized version of YangInstanceIdentifier/PathArgument

The code has been modified to stop interpreting YangInstanceIdentifier.toString
and PathArgument.toString as a serialized versions of those objects. The toString
implementations of those objects has now been moved into PathUtils.

- NormalizedNodeGetter and NormalizedNodeToNodeCodecTest have been modified to use
PathUtils.toString instead of YangInstanceIdentifier.toString or PathArgument.toString

- Added tests to cover all the utilities in PathUtils

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

Added features-test for base Karaf features and fixed some minor issues that caused the failures before.

Change-Id: I964eedcdb6b1d109f5ab3e494a7d0202ff66643b
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

 BUG 1606 - A read is attempted on a WRITE_ONLY transaction

Change-Id: I896d976ff387ecdc9ecfe6e4cb2ead75b15c15f5
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Merge "changed the label for seed node config"

Merge "Configuration service uses an inconsistent hashcode in the clustered data structure"

Configuration service uses an inconsistent hashcode in the clustered data structure

- Using Enum in the Key field of a clustered data structure can lead
to propagation issues, because the clustering services implementation
relay on a consistent hashcode to be calculated. So replaced the Java
Enum used in configuration.implementation bundle with String.

Change-Id: Id36ac46c9f38859d5325fc4f2e8674c9c3828ec4
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

Bug 1262 - Document Transaction Chaining of MD-SAL

Improves and clarifies the transaction chain concept and related APIs.

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

Merge "Incorrect name of elements in yang case nodes"

BUG-1601 Fix null pointer in netconf connector for empty schemas node

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

Merge "Resolving Karaf Northbound API issues with a few related fixes"

Merge "Removed getClass comparison if FlowComparator."

Merge "Remove unused getter"

Merge "BUG-868: remove unused things"

Merge "BUG-868: remove deprecated call"

Resolving Karaf Northbound API issues with a few related fixes

1. Upgrading Apache Tomcat Patch version from 7.0.32 to 7.0.53 (https://git.opendaylight.org/gerrit/10110)
2. No impact to the legacy distribution.
3. Fixing the Jackson - Jersey binding by fixing the Classloader confusion where multiple bundles exports javax.ws.rs
4. Adding a temporary hack for the Http authentication by adding a karaf-tomcat-security bundle
   The existing security bundle is rift with dependencies into a lot of deeply chained AD-SAL NSF and needs a lot of pruning.
   Hence in order to make progress in Karaf testing and development, Have bypassed it temporarily and we will fix it soon.

Change-Id: I132002c0b4e389f50968c7fd08521157900183d3
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Small fix on dependencies for karaf features.

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

Remove unused getter

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

BUG-868: remove unused things

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

BUG-868: remove deprecated call

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

Changed gossiper test to use akka system's LocalRefProvider

This improves unit test's performance and reliability

Change-Id: I6e669c5994aacb0c6ec3da0a321110572aa29f31
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Added more tests for Bucket store

Changed bucket store tests to use akka's LocalRefProvider from ClusterRefProvider
to improve performance. Also added more unit tests.

Change-Id: Iadc6b5967074651a99027f10538aee88a7c4ec84
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>

Added IP address of the OF switch to flow-node

- adds IP address to flow capable node
- adds routed RPC:
get-node-ip-address - returns IP address of a flow capable node

Change-Id: Ie1f3459ddbe898cc2aa8cc488437666c566246d6
Signed-off-by: Martin Sunal <msunal@cisco.com>

changed the label for seed node config

Change-Id: Iafbd7d781f0fb41bff8e0194954809febecf909c
Signed-off-by: Basheeruddin Ahmed <syedbahm@cisco.com>

Merge "Karaf odl-nsf-managers feature fails due to missing topologymanager shell. Fixing that."