Merge "Unify maven-bundle-plugin version at 2.4.0"

Merge "BUG-972: correct Precondition"

Merge "Add serialVersionUID to Serializable"

Merge "BUG-868: stop using InstanceIdentifier.builder()"

Add serialVersionUID to Serializable

Serializable classes have to have serialVersionUID. Add it where it's
missing.

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

BUG-868: stop using InstanceIdentifier.builder()

The static helper method has been deprecated. Use either the non-static
one, or child()/augmentation() helpers.

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

BUG-981: remove deprecated use of constructors

This removes the use of all constructors, such that we initialize in one
go.

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

Merge "BUG-893: do not store classPool"

Merge "BUG-642: get rid of sneakyThrow"

Bug 935 Fixed, added order key in instruction and action as flow yang model expects it.

Change-Id: Ib7a82db71f4915496574ec2b8731cb0161351309
Signed-off-by: Amit Mandke <ammandke@cisco.com>

BUG-893: do not store classPool

This prevents storing a reference to the classPool. It is not necessary
and it makes thinking about accesses much easier.

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

BUG-642: get rid of sneakyThrow

This patch rids us of unneeded usage of Exceptions.sneakyThrow. It
generally hides problems and is only useful for generated (or sloppy)
code.

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

BUG-891: do not drop events silently

As it turns out, infrastructure silently drops things on the floor if
exceptions happen. When these exceptions occur, they hint at major
problems. Promote tham to warnings.

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

Merge "BUG-969 Move fake read on nodes/node to separate thread."

Merge "Making it easier for cygwin users to run OpenDaylight using run.sh. With this change, the users will only have to add the following line to their .bashrc to make it run: export CLASSPATH=".;lib/*;""

BUG-969 Move fake read on nodes/node to separate thread.

This will prevent blocking of config thread.

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

BUG-972: correct Precondition

This is just a tiny adjustment of a Precondition, which should have been
triggered before this access. isPresent() returns true/false, which will
be autoboxed, so will always pass checkNotNull. use checkArgument()
instead, which will ensure that isPresent() == true.

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

Unify maven-bundle-plugin version at 2.4.0

This removes superfluous version declarations, fixing about 70 eclipse
warnings. At the same time it bumps the version 2.4.0, to unify it -- we
were using both 2.3.7 and 2.4.0 at the same time.

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

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

Merge "Upgrade enunciate version to 1.28   -Convert XmlElementRef to XmlElement to allow doc generation to work properly   -Add swagger module dependency to enunciate"

Upgrade enunciate version to 1.28
  -Convert XmlElementRef to XmlElement to allow doc generation to work properly
  -Add swagger module dependency to enunciate

Change-Id: Ia41e7733bd44965951ca83aaa7738e2df74151da
Signed-off-by: mrajvaid <mrajvaid@cisco.com>

Felix Web Console authenticates using HTTP-Bridge, which is similar to web-gui

By default web console perform its own authentication using default user and passowrd
If user password is modified via web, felix console can not authenticate

Change-Id: I38a277ffabdfac652325b83fe80facffc043c669
Signed-off-by: mrajvaid <mrajvaid@cisco.com>

Bug-590: Packet loss on first time ping test

Bump severity of log in exception code path.
Undo copy and paste mistake, which caused unwanted line left behind.

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

Bug:969 - Add fake read to block thread to sync with BI broker.

Block netconf client until Binding Independent broker loads inventory
models.

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

Merge "Fix netconf-connector filter construction"

Merge "Add missing exception to log message in SchemaAwareRpcBroker"

Fix netconf-connector filter construction

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

Merge "Initial mdsal and config karaf features"

Merge "Increase mdsal's notification queue size, make it configurable."

Initial mdsal and config karaf features

This loads cleanly into karaf :)

Instructions for checking:

cd controller/

mvn clean install

cd opendaylight/distributions/opendaylight-karaf/target/assembly/bin
sh karaf

at the karaf prompt

feature:install odl-mdsal-restconf
feature:list -i

(odl-mdsal-restconf will pull everything else through).

PatchSet2:  Fixed failure to use odl- prefix in features for -all

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

Add missing exception to log message in SchemaAwareRpcBroker

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

Merge "FRM should use async function on southbound event"

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

Merge "Bug 933: Fixed incorrect update type when Link was deleted."

FRM should use async function on southbound event

- On reacting to southbound events (switch up/down, ports added/removed, ...)
  FRM is supposed to use asynchronous calls for flows installation/removal.
  This is needed because these events may affect a large number of flows,
  for which we do not want to slow down the processing via sync calls.
  Also, logic has been we never do process synchronously a southbound event.
- In case of port up, FRM was still using a synchronous call.
  This patch is for taking care of that.

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

Bug 967: Statistics Manager should delete flow, not augmentation only.

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

Increase mdsal's notification queue size, make it configurable.

Add System property mdsal.notificationqueue.size to allow tweaking
of queue size.
Increase default queue size from 10 to 1000.

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

Bug 933: Fixed incorrect update type when Link was deleted.

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

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-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 a flow that would cause it to forward the packet to its destination w/out involving the controller.

The race in this case is caused by Simpleforwarding handling the packet before HostTracker is fully aware
of the destination host. In such case, the forwarding fails to determine where to send the packet and drops it.

The proposed change in this commit takes a stab at handling the race by keeping the packet until either
1) the destination is known by HostTracker or 2) a max time expires. In order to accomplish that, the change
includes the creation of a thread that will periodically age out any packet that have been pending for too
long. The change also adds a concurrent map for efficient lookup among the pending packets. There is a
limitation in this implementation: it will keep at most 1 packet pending per destination ip address. This is
believed not to be a problem, because of the short amount of time it takes for hostTracker to learn about the
destination after the packet is put int the pending map.

Change-Id: I7ec1de69f7841eb7ca5603715b54685be9781401
Signed-off-by: Flavio Fernandes <ffernand@redhat.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>

Making it easier for cygwin users to run OpenDaylight using run.sh.
With this change, the users will only have to add the following line
to their .bashrc to make it run: export CLASSPATH=".;lib/*;"

Change-Id: Icc1576ea1c46568d43f11d90eebecb06c74db184
Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.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>