BUG-1520 coverage tests for config-manager

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

Fixed bundle dependencies for features-netconf.

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

Add getID() to NeutronRouter_Interface

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

Merge "Include JMX Counters and resetTransactionCounters"

Bug 1441: Added proper handling of reading JSON list items.

Fixed handling of reading nested data in JSON codec.

Initial integration contained bug, which did use JSON
codec correctly, which ended in 500 for reading list item.

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

Fix missing yang-data-util in IT

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

Merge "BUG-1617 Add netconf-auth to netconf api feature"

Merge "Improvement to opendaylight-configfile-archetype"

Improvement to opendaylight-configfile-archetype

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

Merge "Bug 1593 - Flow Statistics manager is updating store with incorrect key Statistics manager cache the metadata related to stats data of all the entities (flow,group, meter,queue) like when was their respective stats were updated in operational data store. If statistics manager see that stats are not being updated from last two stats fetch cycle, it consider it stale and remove it. Object that it uses as keys for hash map cache contains some variant data (counters/durations) that varies in every stats fetch cycle. It creats duplicate entry in the cache for the same entity. After two fetch cycle statistics manager find old entry stale and removes it, but in actual that's not the case. This exception is thrown when it tries to remove the stats related to next duplicate entry (which was already removed ). Fixed it by using invariant key for caching the metadata to avoid the duplication."

Merge "Bug 1569 - [DEV] Too small variable for OF-Port-Number"

Merge "Small fix to xsql dependencies"

BUG-1617 Add netconf-auth to netconf api feature

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

Merge "FlowCapableTopologyExporter uses merge instead of push when link discovered"

Merge "BUG-868: migrate ClassLoaderUtils user"

Merge "BUG-1442: integrate XML/JSON Normalized Node writers with Restconf"

BUG-868: migrate ClassLoaderUtils user

Moves to yangtools.util, which is not deprecated.

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

BUG-1442: integrate XML/JSON Normalized Node writers with Restconf

Integrated XML and JSON NormalizedNodeWriters for returning
data from Restconf. This writers does stream processing
and writes data into input stream without any intermediate
representations.

This will help with larger data sets, since it theoretically
reduces memory footprint of reading data from MD-SAL.

Change-Id: I36564b7a5a1bc50d60e1cb6d5b2b45f92c2caf0b
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

Added sal-rest-docgen to features.

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

FlowCapableTopologyExporter uses merge instead of push when link discovered

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

Fix version in samples/northbound/loadbalancer

Otherwise autorelease breaks

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

Include JMX Counters and resetTransactionCounters

added ReadFailedTransactionsCount
added AbortTransactionsCount
and
resetTransactionCounters

Note:
Could not get the details where InMemoryDatastore counters
are -- need to reset those counters as part of separate
commit

included copyright headers in files that didn't had

patch 4:
Review comments closed

patch 3:
removed a comment that is not needed.

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

Increasing timeout for test

Also releasing the lock after message processing.

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

Changed key actors to use bounded mailbox

1. Changed key actors in clustering components to use
bounded mailbox.

2. Implementation of Bounded mailbox queue changed to
 use deque as its required by Shard Actors.

3. Cleanup of hard coded values.

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

Merge "Optimized version of NormalizedNode Serialization"

Merge "Moving protobuff files to sal-clustering-commons"

Merge "Bug 1598: Cleanup stale ShardReadTransactions"

Merge "FRM performance refactoring: * Interface definitions for FRM functionality  * ForwardingRulesManager (old provider)  * ForwardingRulesCommiter (DS changeListener/sender for SalServices)  * FlowNodeReconciliation (reconcil contract definition) * Centralization Provider/Manager bundle functionality  * fix possibility not unique transaction ID  * implement active NodeHolder (remove unnecessary Operation/DS check) * Listeners  * identify correct data by wildCarded InstanceIdentifier  * move registration functionality to listener class  * remove all unnecessary functionality  * functionality fragmented * Cleaning  * move FlowCookie to StatistManager (performance killer, fixing update Flow functionality,   possibly speed up delete functionality)  * cleaning unnecessary object creation * Added MockTests"

Merge "Added AAA AuthN filter to RESTConf web.xml"

Merge "Bug 1636: Fixed invalid model of inmemory-datastore configuration."

Moving protobuff files to sal-clustering-commons

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

Add toString() to Neutron_IPs

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

Merge "XSQL Karaf Command"

Merge "Bug 1636: Config Netconf Connector did not serialize service type"

Merge "BUG-1617 Extract AuthProvider from netconf ssh"

FRM performance refactoring:
* Interface definitions for FRM functionality
* ForwardingRulesManager (old provider)
* ForwardingRulesCommiter (DS changeListener/sender for SalServices)
* FlowNodeReconciliation (reconcil contract definition)
* Centralization Provider/Manager bundle functionality
* fix possibility not unique transaction ID
* implement active NodeHolder (remove unnecessary Operation/DS check)
* Listeners
* identify correct data by wildCarded InstanceIdentifier
* move registration functionality to listener class
* remove all unnecessary functionality
* functionality fragmented
* Cleaning
* move FlowCookie to StatistManager (performance killer, fixing update Flow functionality,
 possibly speed up delete functionality)
* cleaning unnecessary object creation
* Added MockTests

tests PASS: https://jenkins.opendaylight.org/integration/job/integration-master-csit-base-of13/
(Base-OF13/AD_SAL_NSF/TopologyManager/getTopology sometimes fail)

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

Small fix to xsql dependencies

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

XSQL Karaf Command

Change-Id: I6296313c2b136bc83fff01739a4e40fc5a57cd2d
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Bug 1569 - [DEV] Too small variable for OF-Port-Number

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

Bug 1636: Fixed invalid model of inmemory-datastore configuration.

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

Bug 1636: Config Netconf Connector did not serialize service type

Config Netconf connector did not serialized service type
in list of service dependencies properly, which resulted
in invalid string 'prefix:null'

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

Merge "Bug 1088 - flow id in operational not matched; after removal not removed completely Added comparator to compare ethernet address string in case (upper/lower case) agnostic way"

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>

Bug 1088 - flow id in operational not matched; after removal not removed completely
Added comparator to compare ethernet address string in case (upper/lower case) agnostic way

Change-Id: Ia7ed9bc098d65bc8dff754fbb50b0eade2605136
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.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>

Added AAA AuthN filter to RESTConf web.xml

Change-Id: I78d57335e805165c99038b4a6f89f8c5229253a5
Signed-off-by: Liem Nguyen <liem_m_nguyen@hp.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"

BUG-1617 Extract AuthProvider from netconf ssh

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

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>

Optimized version of NormalizedNode Serialization

This commit contains the following,
- Conversion of NormalizedNode to Protocol Buffer using simple tree traversal
- Encoding of QNames and other values to small sized codes to reduce the size of the protobuf message
- Also we now do not use SchemaContext for decoding a NormalizedNode

In addition to these changes I have snuck in an unrelated change to the pom files
to get the appropriate jacoco settings in it.

Change-Id: I6ab48c9f53cf76d6d0c865cf6e5522ff27a30e3f
Signed-off-by: Moiz Raja <moraja@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>

Bug 1598: Cleanup stale ShardReadTransactions

For read-only Tx, a FinalizablePhantomReference is created with the
TransacionProxy instance as the referent and is added to a static
FinalizableReferenceQueue. The FinalizablePhantomReference is subclassed
to hold the TransactionProxy's remoteTransactionPaths map. When the
TransactionProxy instance in GC'ed, the FinalizablePhantomReference is
notified and calls closeTransaction on each TransactionContext to clean it up.

Also to handle potentially stale Tx's due to disconnects, nodes prematurely
shutting down etc, I set set an idle time out (default 10 min) on ShardTransaction
actors via setReceiveTimeout. If the actor is idle with no messages
after the timeout, the actor self-destructs.

I made the idle timeout configurable via the config yang file. This
setting needs to be passed down from the module, thru the ShardManager
to the Shards to the ShardTransactions. I created a ShardCOntext class
to hold the idle timeout and other data that's passed down. This will
also make it easier in the future if additonal config data needs to be
passed down.

In the config yang file, I created a data-store-properties grouping to
avoid having to duplicate the config properties for the operational and
config data stores.

Many unit tests fail when running from Eclipse due to creating anonymous inner
Creator class instances in the static props methods. The akka code doesn't like this -
if the Creator instance class is enclosed in another class it expects
the class to be static. However this runs fine when running unit tests
from mvn and when running the production controller. I suspect the
difference is b/c the JDK compiler generates the anonymous class as
static since they're enclosed in static methods but the Eclipse compiler
doesn't. Anyway, to avoid this I refactored all the anonymous inner
Creator classes to private static classes. I think this is safer and
will avoid potential future issues with different JDK compilers or JDK upgrades.

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

Merge "BUG-1568 Remove ganymed Ssh client adapter"

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

Bug 1593 - Flow Statistics manager is updating store with incorrect key
Statistics manager cache the metadata related to stats data of all the entities (flow,group,
meter,queue) like when was their respective stats were updated in operational data store.
If statistics manager see that stats are not being updated from last two stats fetch cycle,
it consider it stale and remove it. Object that it uses as keys for hash map cache contains
some variant data (counters/durations) that varies in every stats fetch cycle. It creats
duplicate entry in the cache for the same entity. After two fetch cycle statistics manager
find old entry stale and removes it, but in actual that's not the case. This exception is
thrown when it tries to remove the stats related to next duplicate entry (which was already
removed ). Fixed it by using invariant key for caching the metadata to avoid the duplication.

Change-Id: Iad5823ab70938a66fd3d024dc921b74e0852c204
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

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"