Merge "Bug 946: Fixed eclipse errors with nagasena projects"

Merge "Fixed typo in SnapshotBackedWriteTransaction class"

Merge "Breaking up 6360: dummy console"

Merge "Fixed opendaylight-karaf to have resources in target/assembly"

Fixed opendaylight-karaf to have resources in target/assembly

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

BUG-509 transaction future result listening

- replaced transactionResult.get() with future listener

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

Merge "Bug: 627 Added RESTConf API Explorer that dynamically generates API documentation and serves it through swagger UI. Details are documented in this wiki: https://wiki.opendaylight.org/view/OpenDaylight_Controller:MD-SAL:Restconf_API_Explorer"

Merge changes I05e784af,Ie975a2ec

* changes:
  BUG-693 Config module for netconf client dispatcher.
  BUG-938 Fix negotiation error when using exi

Breaking up 6360: dummy console

https://git.opendaylight.org/gerrit/#/c/6360/29

Change-Id: I7915821a74d582f9d41bc200c25e3d7b35cf140b
Signed-off-by: Ed Warnicke <eaw@cisco.com>
Signed-off-by: Mathieu Lemay <mlemay@inocybe.com>

Merge "Breaking up 6360: Simple karaf distro"

Merge "Breaking up 6360: common pom file"

Merge "Bug 949: Fixed support for DataChangeListeners listening directly on Augmentation"

Breaking up 6360: Simple karaf distro

https://git.opendaylight.org/gerrit/#/c/6360/29

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

Breaking up 6360: common pom file

https://git.opendaylight.org/gerrit/#/c/6360/29/opendaylight/commons/opendaylight/pom.xml
Just bringing in the common pom file changes.

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

Bug 949: Fixed support for DataChangeListeners listening directly on Augmentation

Used InstanceIdentifier codec does not support AugmentIdentifier, since it is
designed for CompositeNode format (serialization as defined by YANG Spec).
Instance Identifier for NormalizedNode format allows to directly reference
augmentation, for which support was missing.

Added a special case handling of Instance Identifiers which targets augmentation
directly, since Binding-Aware API and NormalizedNode API allows it.

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

Remove vip from the vips collections instead of members.

Change-Id: I82063e74526a770772fb1eb5d91ab68ea3aeb449
Signed-off-by: Sivasamy Kaliappan <sivasamyk@gmail.com>

Merge "BUG-832 Add schemaContext to notification parsing process in sal-netcon-connector"

Merge "BUG-832 Fix exception from sal-netconf-connector."

Merge "Skip from the surefire execution plugins that can step over guava feet"

BUG-693 Config module for netconf client dispatcher.

netconf/
NetconfClientDispatcher and NetconfSSHClientDispatcher merged into NetconfClientDispatcherImpl.
Introduced NetconfClientConfiguration to contain all configuration attributes for netconf clients.
Introduced interface for NetconfClientDispatchers.
Removed NetconfITSecureTest since it did not test secure connection.

config/
Added netconf-config-api maven module, which contains service definition of client-dispatcher.
Added netconf-config-dispatcher maven module, which contains config module for client-dispatcher.

md-sal/
Updated configuration for sal-netconf-connector with client dispatcher dependency (backwards compatibility preserved).

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

Bug 946: Fixed eclipse errors with nagasena projects

- Uploaded the nagasena OSGi bundle jars to nexus with group ID
org.openexi.
- Changed affected dependencies
- Removed thirdparty/org.openexi project

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

BUG-938 Fix negotiation error when using exi

EXIConfirmation handler is now added to pipeline before start-exi is sent.

Add test with secure netconf server + exi

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

Bug 946: Fixed eclipse errors in sal-binding-dom-it

Enabled the yang-maven-plugin to run by defult in eclipse

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

Merge "BUG-633: allow overriding of channelFactory"

BUG-633: allow overriding of channelFactory

It turns out netty does not let us override an already-set
channelFactory. Reorder customization to detect when such an override is
necessary.

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

Merge "BUG-634: introduce netty ChannelFactory wrapper"

Merge "BUG 274 REST response instead of NumberFormatException"

Merge "Modify config code generator to write JmxAttribute as static fields."

Merge "Bug 618 - convert xtend to java in md-sal/inventory-manager"

Merge "fix for BUG-621 - convertion from xtend to java in compatibility/flow-management-compatibility"

Bug 933: Fixed datastore processing of nested augmentations.

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

Bug 933: Changed Topology Compatibility to use DataChangeEvent

Topology Compatibility adapter used commit handler to receive
change events and also reading from datastore, which may leaded
to some desynchronization issues. Changed topology Compatibility
to data change listener, which ensures data store is up to date,
when read occurs.

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

Merge "Bug 930 - Add bound to queue in SingletonHandler#NOTIFICATION_EXECUTOR ."

Merge changes I4a9861ed,I61c130b9

* changes:
  Bug 509: Fixed DataTransactionListener support.
  Bug 509: Fixed Backwards compatibility bugs with datastore.

Bug 509: Fixed DataTransactionListener support.

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

Skip from the surefire execution plugins that can step over guava feet

Seems like the com.google.common.collect.* package has been reused in
several jars like:

- guava
- google-collection
- pax-url-aether

Now the netconf-it in the classpath of contains via transitive the 3
of them but only guava is usable by yang-bindings used by the netconf
client test. The presence of the 3 cause unreliability on the test
pass based on who comes first in the classpath of surefire. The fix is
to make sure surefire ignore the other two libraries except guava.

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

BUG-832 Add schemaContext to notification parsing process in sal-netcon-connector

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

Bug 618 - convert xtend to java in md-sal/inventory-manager

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

fix for BUG-621 - convertion from xtend to java in compatibility/flow-management-compatibility

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

BUG-634: introduce netty ChannelFactory wrapper

Adds a wrapper for services exposing ChannelFactory. Needed for
protocols which need to extended netty functionality.

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

Merge "Bug 740: Forwardingrules-Manager support for DataChangeListeners"

Merge "BUG-509: remove the key/value datastore"

Merge "BUG-509: make the tree-based datastore the default"

Merge "BUG-509: Cleanup datastore switching configuration"

Merge "Clean up yang-test to stop modifying source files during build."

Merge "Default auth parameters in config.ini."

Bug 930 - Add bound to queue in SingletonHandler#NOTIFICATION_EXECUTOR .

Add queue capacity with hardcoded value 10. When maxPoolSize + queue capacity
is reached, caller thread will be blocked.
Mark thread as interrupted when catching InterruptedException, propagate it
to caller.
Remove code that pushed first two runnables to queue when core size was
reached - saturate whole threadpool capacity first.

Change-Id: I9c52b52eac127bea0cbb2abf7a65c0165daf8895
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Bug 740: Forwardingrules-Manager support for DataChangeListeners

+ Bug 619 - Eliminate the use of xtend in md-sal/forwardingrules-manager

migration from xtend to java files FRM
add DataChangeListener to FRM (Flow, Meter, Group)
remove FRM handlers (Flow, Meter, Group)
fix delete changes registration (in ImmutableDataChangeEvent -> old DataStore)

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

BUG-832 Fix exception from sal-netconf-connector.

Exception:
Ommiting from BA transaction: /(urn:opendaylight:inventory?revision=2013-08-19)nodes/(urn:opendaylight:inventory?revision=2013-08-19)node[{(urn:opendaylight:inventory?revision=2013-08-19)id=ios-xr-105}].

Fixed by treating capabilities from remote device as String instead of QName

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

BUG 274 REST response instead of NumberFormatException

Raised NumberFormatException is now catched and thrown next as
WebApplicationException which is displayed as return message for REST
call.

Change-Id: I1b4ee74c6a953fe3b64ef47c07477c29d892a87c
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Bug: 627
Added RESTConf API Explorer that dynamically generates API documentation and serves it through swagger UI.
Details are documented in this wiki:
https://wiki.opendaylight.org/view/OpenDaylight_Controller:MD-SAL:Restconf_API_Explorer

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

Bug 509: Fixed Backwards compatibility bugs with datastore.

Fixed bug in write and merge of leaf set children,
which prevented update of leaf sets.

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

Merge "Statistics-Manager - Performance Improvement 1) Introduced statistics request scheduler, to schedule request based on the current transaction load on MD-SAL DataStore. Each node submit all individual statistics request to schedular for execution 2) Send statistics request if there is no MD-SAL trasaction pending. It just keep tracks of the MD-SAL trasaction triggered by statistics-manager 3) Removal of stale statistics is now done based on counter rather then time values. Time based removal will break in case of clustered environment. 4) Code clean up Change-Id: Ie7522d0c60f2c7051dbfcdf9a6657843ef4da743 Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>"

Merge "Fixing an unsorted pom change"

BUG-509: remove the key/value datastore

This patch puts the legacy buggy datastore to rest.

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

BUG-509: make the tree-based datastore the default

This patch switches the default datastore implementation to the
tree-based one.

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

BUG-509: Cleanup datastore switching configuration

This cleans up the configuration, relying on the fact that we can have
the two worlds existing concurrently and only expose one of them.

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

Modify config code generator to write JmxAttribute as static fields.

Make JmxAttribute fields static so that name of the attribute can be
referenced and used to read from JMX.

Change-Id: Ibf2afbdfd8ae78e6ca46025671128bc7256f78d9
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Merge "Resolve Bug:853 - remove groovy from config code generator."

Clean up yang-test to stop modifying source files during build.

In yang-test-plugin remove auto generated headers so that things
like timestamp changes will not be reported as changed files
after build.
Also move an artifact definition in commons pom and sal-binding-it
as enforced by the sort plugin.

Change-Id: I5d851d65c149611800a1f7418eb4b642e3fd8260
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Fixing an unsorted pom change

- This would cause changes every time someone builds

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

Merge changes Ie84ea1ff,Ifa0a97da

* changes:
  BUG-848 Make base capabilities from netconf server configurable
  BUG-848 Fix netconf communication while using CHUNK encoding

Default auth parameters in config.ini.

    Hardcoded default username and password for ssh authenticaton moved from AuthProvider.java to config.ini
    IUSerManager in AuthProvider.java is not static anymore.

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

Merge "Allow to disable default gw feature"

BUG-848 Make base capabilities from netconf server configurable

Refactor ConcurrentClientsTest, add cases for different netconf server configuration (EXI/CHUNK)

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

BUG-848 Fix netconf communication while using CHUNK encoding

Cause:
When using Chunk encoding, the negotiation process lasts longer due to pipeline manipulation.
Messages received while still in negotiation were not processed correctly (This is incorrect, messages should not be processed while still in negotiation).

Fix:
Introduce a message barrier that will forward only hello message until negotiation is completed (implemented in NetconfXmlToHelloMessageDecoder).
Modify SessionNegotiators to replace message barier/hello decoder when negotiation is finished and to process non-hello messages stopped by the barier.

* Exclude pax-url-aether dependency in netconf-it

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

Merge "Fixed typo in test name. Temporary ignore test."

Merge "Resolve Bug:748 - Remove test resources from netconf-impl."

Merge changes I8e90a7d5,I25249fe8

* changes:
  Resolve Bug:807 - Keep ModuleFactory references shutdown purposes.
  Resolve Bug:681 - Fix config module registration to Service Registry.

Merge "BUG 652 leafref CCE & BUG 720 colons problem"

Merge "Bug 460 - Fix warning throughout netconf subsystem"

Merge "Bug 616 - Eliminate the use of xtend in md-sal/topology-lldp-discovery"

Merge "BUG-732 Fix sal-netconf-connector unable to download schemas"

Resolve Bug:853 - remove groovy from config code generator.

Replace 3 groovy classes with java equivalents.

Change-Id: I625f0498c3ccaedf0441c637ac61a9653c98e579
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Merge "Cleaned Commons POM prerequisites for Karaf distribution"

BUG-732 Fix sal-netconf-connector unable to download schemas

BUG#1 NetconfDeviceListener used wrong method to parse rpc results
BUG#2 Parsing rpc result in NetconfMapping was looking for schema to response even if it was not needed
BUG#3 Netconf client sent base netconf capability in wrong format

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

Bug 460 - Fix warning throughout netconf subsystem

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

Statistics-Manager - Performance Improvement
1) Introduced statistics request scheduler, to schedule request based on the current transaction load on MD-SAL DataStore.
Each node submit all individual statistics request to schedular for execution
2) Send statistics request if there is no MD-SAL trasaction pending.
It just keep tracks of the MD-SAL trasaction triggered by statistics-manager
3) Removal of stale statistics is now done based on counter rather then time values.
Time based removal will break in case of clustered environment.
4) Code clean up
Change-Id: Ie7522d0c60f2c7051dbfcdf9a6657843ef4da743
Signed-off-by: Anil Vishnoi <avishnoi@in.ibm.com>

Bug 616 - Eliminate the use of xtend in md-sal/topology-lldp-discovery

Change-Id: Iec8335fadd5f1529b3f04288805bda2f5594cc94
Signed-off-by: Debolina Bandyopadhyay <dbandyop@cisco.com>
Bug 616 - Eliminate the use of xtend in md-sal/topology-lldp-discovery

Change-Id: Ie5d82011676aac2b8750fe1e6e46b89da8c3b4eb
Signed-off-by: Debolina Bandyopadhyay <dbandyop@cisco.com>
Removed @SuppressWarnings("all")

Change-Id: I4e012890813f94ec1edc66b538904a7c8bd2e41c
Signed-off-by: Debolina Bandyopadhyay <dbandyop@cisco.com>

Allow to disable default gw feature

- Default gateway is a good feature for having host discovery, and
  consequently hosts' pings, work out of the box.
  But in real scenarios a controlled network node may be connected to a
  legacy switch's trunk port which exposes thousand of hosts which ARP
  packets would easily stress controller functionality.
  Even though today this can be prevented by configuring a more
  specific subnet gateway, the issue is very much possible at first
  controller boot, and can always re-occur during subnet config manipulation
  if at any point of time we have no user configured subnet gateways.
- This patch provides a way to disable he default gw feature by mean of
  a property in config.ini. Property addition is not included, current
  behavior is unchanged.

Change-Id: I15b63233e3ed46a8a9fb57bb0988a0d4a2984b11
Signed-off-by: Alessandro Boch <aboch@cisco.com>

Cleaned Commons POM prerequisites for Karaf distribution

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

Fix Operation not permitted warning.

This fix removes warning
WARN  o.o.c.n.c.m.c.ModuleElementDefinition - Operation not permitted on
current strategy  while default strategy is merge. Element definition
strategy set to default

that occured on osgi startup.

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

Merge "Bug 451 - Fix netconf exception handling"

Merge "BUG-832 Add initial configuration for controller self mount"

Bug 451 - Fix netconf exception handling

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

Merge "Bug-590: Packet loss on first time ping test"

BUG-832 Add initial configuration for controller self mount

Netconf server for config subsystem is mounted to restconf under id "controller-config" by default

This illustrates the usage of sal-netconf-connector

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

Merge "Bug 509: Added support for merge operation to InMemoryData Store"

Bug-590: Packet loss on first time ping test

In this commit, we are addressing a race that could cause the exception (ie first ip packet) to
be lost. What is observed with this bug is that the non-arp packet -- in this case the icmp packet --
is being 'exceptioned' to the SimpleForwarding code path because the node has not yet been programmed
with flows that would allow it to forward the packet to its destination w/out involving the controller.

The race in this case is caused by the exceptioned packet being received before the rulesDB member
in simpleForwarding module is updated. That member -- rulesDB -- is updated via the
hostTracker::notifyHTClient() callback shortly after. In this particular race, the hostTracker is already
updated with the proper information, and rulesDB is 'stale'. Thus, the proposed fix is to make the forwarding
less strict (ie. ignore rulesDB). Note that both routing and hostTracker are all simpleForwarding needs to
know how to forward the packet.

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

BUG 652 leafref CCE & BUG 720 colons problem

BUG 652 leafref Class cast exception
By default if input data were in format which was possible to parse as
XPath if was done like this. It caused that for leafref which indexed to
instance-identifier the value was parsed as XPath and not as string.

BUG 720 colons problem in JSON
If input string contained colon (:) then it was dealed as namespace:value
and object of type IdentityValuesDTO was created (it is probable that
data could be of type identityref). If data wasn't of type leafref then it
was incorrectly processed as leafref.
The RestCodec was updated to correctly translate data (cases when data
contains ':' but aren't of type leafref)

Change-Id: I0cba06bce1b1c69f6901cc90d2b30a3735e5f57e
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Bug 509: Added support for merge operation to InMemoryData Store

Merge operation support is critical for backwards compatibility
and for Binding-Aware mapping and Restconf, which are not aware
of Mixin nodes representing lists, leaflists and choices.

Semantics of merge operation are similar to subtree modified,
but when node is not existing it creates one based on
state provided by user of the Data Broker APIs.

Merge operation also fixes concurrent creations of mixin nodes
as demonstrated in ConcurrentImplicitCreateTest,
where two separate applications may indirectly create same
root node.

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

Bug 722 - nagasena version bumped to 0000.0002.0038

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

Fixed typo in test name. Temporary ignore test.

Change-Id: Icdae59971bded5e76a181d4ab63132e011a561ca
Signed-off-by: Martin Vitez <mvitez@cisco.com>

Merge "Bug 781: Statistics manager is throwing IllegalStateException"

Merge changes I67420353,I6851f9b9

* changes:
  Bug:701-Have Typed ErrorNotification for different node errors
  Bug:645 - MDSAL Notification pool threads not getting spawned besides the core threads

Resolve Bug:807 - Keep ModuleFactory references shutdown purposes.

Retain ModuleFactory instances even if they disappear from OSGi SR,
so that the already committed instances can be closed cleanly. Also
remove shutdown customization (stopping config-manager as first bundle)
from shutdown-impl.

Change-Id: I8e90a7d593b7e4cc84c7a9a7d3f8bb804daf5f41
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

BUG-732 Add rollback-on-error error-option to edit-config rpcs from netconf-connector

Just in case rollback-on-error capability is supported

+ Add error response warning to netconf-connector

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

Resolve Bug:681 - Fix config module registration to Service Registry.

Rework OSGi service registration during second phase commit: only service references
will be registered into SR, instead of old behaviour when each module's service
interfaces were used. Only one key-value entry is persisted in registration:
{name: name of service reference} .

Change-Id: I25249fe8f9e51570cd3ea963e91145b7e44da57d
Signed-off-by: Tomas Olvecky <tolvecky@cisco.com>

Merge "BUG-732 Fix netconf client deadlock when downloading yang schemas from remote device"

Merge "BUG-758 Eliminate xtend from sal-netconf-connector"