Turned down some logs in StatisticsProvider from INFO to DEBUG

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

Bug 319: Fixed two-phase commit verification of data.

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

Extract dom-serializer-api|impl from binding-broker-impl

Change-Id: I1b2844bfac5062fd15c1f011c3104b3dddb33b51
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Minor fix in flow IP Matching

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

Merge "Gerrit contains following minor enhancements: 1) Send statistics requests whenever new flow capable node connects to the controller 2) Clean up transaction-id cache for expired Ids 3) Remove Tx id when last part of multipart response received."

Merge "fix for frm to not act on duplicate mode change notifications"

Merge "Relaxing a strict check in updateNode in SwitchManager to handle a condition in which the adaptor fails to call the addNode."

Small fix to finally fix dryrun.

Turns out that the lack of directory hiearchy matching
our pom hierarchy requires some redeclaration of
properties.

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

fix for frm to not act on duplicate mode change notifications

Change-Id: If3ec14399bd0fc52a26463f901967b6708b7f352
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Relaxing a strict check in updateNode in SwitchManager to handle a condition in which the adaptor fails to call the addNode.

The existing ADSAL code is already robust enough to handle the scenario of NodeProps not containing the a node Key.
Also, there are cases in which an application might just want to see an update event for processing
(especially when the addNode events are missing).

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

Gerrit contains following minor enhancements:
1) Send statistics requests whenever new flow capable node connects to the controller
2) Clean up transaction-id cache for expired Ids
3) Remove Tx id when last part of multipart response received.

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

Merge "Fix for NullPointerException"

Small fix for version skew to fix dryrun

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

Merge "Remove dependency on Exificient due to licensing incompatibility"

Merge "Pull zeromq initial configuration out of 01 config file"

Merge "Make MD-SAL use opendaylight parent"

Merge "Yangtools version police: remove references to non-current versions"

Merge "Added test for MouontPoints and URI"

Merge "Reduce informations from logging.bridge"

Remove dependency on Exificient due to licensing incompatibility

Exificient is licensed under GPLv2 with no intention to add the
classpath exception (witnessed in at
http://sourceforge.net/p/exificient/discussion/856595/thread/6017013e/).
Remove all dependencies on it, pending reimplementation using a more
open library (OpenEXI looks like a good candidate).

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

Make MD-SAL use opendaylight parent

Since MD-SAL code does not comply with coding style enforced by the
checkstyle plugin, disable its execution for now -- BUG-272 tracks the
need to re-enable it.

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

Yangtools version police: remove references to non-current versions

Unifies the version property to yangtools.version, force restconf to use
the versions specified in the md-sal parent and bump its reference to
restconf-api to 0.6.0-SNAPSHOT.

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

Pull zeromq initial configuration out of 01 config file

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

md-sal: sync commons-lang dependency

topology-lldp-discovery is currently specifying 2.6 commons-lang
dependency whereas the controller is dependent on 2.4.  Bring md-sal in
sync w/ the controller by dropping back to 2.4.

Change-Id: Ifb340ceef25f866cf606c4733ade80e78aea0e31
Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Added test for MouontPoints and URI

Change-Id: Ibb18ac011f0523e18595e4fff0900a492d3c5e25
Signed-off-by: Ladislav Borak <lborak@cisco.com>

Merge "Remove powermock dependency from md-sal."

Merge "Provide single point for enforcing config naming convention"

Remove powermock dependency from md-sal.

Remove unused dependency.

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

Reduce informations from logging.bridge

Logging bridge maps 1-1 the log level from OSGi logging service to
slf4j. By design the OSGi framework set al INFO level all the modules
start and stop, which by pure chance, is also inline with our
guideline. Even though the compliance is satisfied there is a
usability issue because the console is flooded with many start stop
events given there are many bundles hence setting to WARN level the
log level for logging.bridge

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

Provide single point for enforcing config naming convention

- Provide generic ConfigurationObject class which other will extend
- Class provides method for validate string based on default private regex
- Regex enforces 1 to 256 alphanumeric chars with addition of -+/*.()[]@
- Default regex can be overwritten with one set in config.ini
- Have configs which need name validation to extend new class.
  Change is bacward compatible, existing config file are correctly
  deserialized

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

Small fix to make dryrun happy.

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

Merge "Binary masks in match types yang model changed to numbers. Plugin code review corresponding to this change: https://git.opendaylight.org/gerrit/#/c/4638/"

Binary masks in match types yang model changed to numbers.
Plugin code review corresponding to this change: https://git.opendaylight.org/gerrit/#/c/4638/

Patch3: Fixed masks to use the same type as values (MacAddress for MacAddress and
IPv6FlowLabel for IPv6FlowLabel) rather than arbitrary uint* values.

Signed-off-by: Deepthi V V <deepthi.v.v@ericsson.com>
Change-Id: Icbc84312c0a2f26738ab9d84f6af34783c4aad5d
Signed-off-by: Ed Warnicke <eaw@cisco.com>

Merge "Fix config transaction handling in netconf."

Merge "sal-remote yang"

Merge "Provide callback feature when modal is visible"

Provide callback feature when modal is visible

Change-Id: I50cc978f1a32bca94d53db84d2c281ddf9099d52
Signed-off-by: Sapan Shah <sapshah@cisco.com>

Merge "Add support to resolve docker container names for clustering support. Currently on works when all containers are on a single docker host"

Add support to resolve docker container names for clustering support. Currently on works when all containers are on a single docker host

Change-Id: Ie141e80712e15e037d39ae788a739cb4d51db403
Signed-off-by: David K. Bainbridge <dbainbri@ciena.com>

Fix for NullPointerException

Change-Id: Id272d14bc1b248780ce533af00073c104a2503b0
Signed-off-by: Pramila Singh <pramisin@cisco.com>

Fix config transaction handling in netconf.

When config manager fails a transaction because of optimistic lock failure, netconf
must clean up this transaction as well. Fix TransactionProvider to abort transaction
if commit fails. Only exception is ValidationException where user should
have an option to fix the configuration and commit again.

Turn off netconf client reuse in persister: this narrows the possibility of
hitting concurrent modification with other config transactions. Each retry
to push a snapshot is now done with a newly obtained client, so new transaction
will be started each time.

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

sal-remote yang

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

Include the incomming port in PacketOut message.
Without this flood messages do not omit the incomming port on the switch.

Change-Id: I932d00c1f17b95678cb73136aefa05fcc5718a95
Signed-off-by: Elek Jani <elekjani@gmail.com>

Fixed incorrect null check.

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

Merge "Change target to ${project.build.target} in a bunch of pom file. Add some xtend-gen source dir to the build path. The commit purpose is to eliminate errors in eclipse."

Merge "Make configuration push timeout configurable"

Merge "Gerrit contains following changes : 1) Fix for bug 284. Added functionality that listen to data store *remove* changes and cleanup the relevant statistics from operational data store. 2) Added functionality that listen to config data store *create* changes for Flow,Group, Meter & Queue and send statistics request to switch to get respective stats. 3) Added functionality to periodically remove stale stats from operational data store. Clean up thread invokes after every two cycle of stats collection. 4) Removed unnecessary local caching."

Merge "md-sal: commons-codec artifact version sync"

Gerrit contains following changes :
1) Fix for bug 284. Added functionality that listen to data store *remove* changes and
cleanup the relevant statistics from operational data store.
2) Added functionality that listen to config data store *create* changes for Flow,Group,
Meter & Queue and send statistics request to switch to get respective stats.
3) Added functionality to periodically remove stale stats from operational data store.
Clean up thread invokes after every two cycle of stats collection.
4) Removed unnecessary local caching.

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

Make AbstractSessionNegotiator react to exceptions

While the SessionNegotiator is present on the pipeline it should react
to exceptions happening on the channel by failing negotiation.

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

Introduced advanced strategy for parsing of YANG schemas

  - This strategy increases convergence time, because it does not
    parse files with unsatisfied dependencies.

  - New strategy for schema parsing in runtime uses two separate parser
     Minimalistic - is used when YANG source is discovered and scans only
     for module name, revision and dependency information
     Full - parses set of YANG sources and creates model representation.

     When bundle is added / removed - tracker gets yang sources,
     registers it to SchemaResolver, which immediatelly uses
     Minimalistic parser to obtain base information.

     Then context of all known sources and their dependencies is sorted
     and filtered only to use sources which has their dependencies resolved.

     Full parser is then invoked only for models, which have dependencies
     resolved.

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

Merge "Fixed bug when global RPCs we're not forwarded"

Merge "Added DEBUG log statements for transaction translation."

Merge "Fixed inappropriate uses of log level INFO"

Merge "fix typos and some Java improvements in ClusterManager"

Fixed inappropriate uses of log level INFO

This is done per the recommendation here:

https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings

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

Change target to ${project.build.target} in a bunch of pom file.
Add some xtend-gen source dir to the build path.
The commit purpose is to eliminate errors in eclipse.

Change-Id: I90727a0096154d7633f4b6cc309dcc09c9a51678
Signed-off-by: Elek Jani <elekjani@gmail.com>

Fixed bug when global RPCs we're not forwarded

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

Added DEBUG log statements for transaction translation.

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

Merge "Optimization - calculate the subnet prefix only once."

Merge "Bump yangtools artifacts to reflect their uptodate versions"

Merge "Fix ClassCastException in logging bridge stop"

fix typos and some Java improvements in ClusterManager

Fix typos in error messages, use StringBuilder instead
of StringBuffer, append strings more efficiently,
and move a local variable to be a constant.

Change-Id: I23a4436449909cf02a6fce47e0f1f9a0ce80f0e4
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Optimization - calculate the subnet prefix only once.

Before this change, subnet prefix was calculated on every
'isSubnetOf' call. Since the prefix is not changing unless
the subnet address itself changed, it's sufficient to do it only once
and keep it as part of the Subnet object.

Marked it as transient so it will not be serialized - it can be
calculated from the rest of the info of Subnet anytime after
deserialization.

Change-Id: Ie16bf80713820b566864bea0ff46298c93cc2d47
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Merge "Avoiding hash collisions of a match with its reverse"

Minor bugfix in Topology Manager

- This fixes an incorrect assumption that subsequent edge-ADDED updates
  will always have equal edge properties

Change-Id: Ib124a4c260d7fce6bb8464f267c2e88cf0b35384
Signed-off-by: Yevgeny Khodorkovsky <ykhodork@cisco.com>

Fix ClassCastException in logging bridge stop

  - Additional fix to handle NPE during logging bridge start

Change-Id: Id6d4b727dd08db18a91b509cad99450fd41a5171
Signed-off-by: Prasanth Pallamreddy <ppallamr@cisco.com>

Make configuration push timeout configurable

This patch adds the option to specify a the pusher timeout, such that it
can be made either shorter or longer than the default 2 minutes.

Another improvement is the use of System.nanoTime(), making the deadline
work in face of calender time changing.

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

unitests: Remove extending TestCase from unitest classes.

Extending TestCase which is a Junit3 format, and using Junit4
annotations at the same time in same class causes confusion,
and can also lead to runtime problems.
An example for a problem - putting @Ignore on a test case which starts
with word "test" and is located in a class that extends TestCase will
actually NOT ignore the test because it will be interpreted as Junit3
and not as Junit4.
Removing extending TestCase will make the unitests to be interpreted
as Junit4 style and leave no room for confusion.

Change-Id: I2365f0921d64667a5489c42eeb53adadc45efdaa
Signed-off-by: Alissa Bonas <abonas@redhat.com>

Merge "Move NetconfUtil to netconf-util."

Merge "Default DataBroker does not read subtrees from Mountpoint"

Bump yangtools artifacts to reflect their uptodate versions

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

Merge "Log transactions being allocated"

Move NetconfUtil to netconf-util.

Implement simple acceptance netconf client test that can connect to a remote machine. Clean up
helper functions from netconf-impl and persister-impl to netconf-util.

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

Default DataBroker does not read subtrees from Mountpoint

Changed default behaviour of DataBroker, which involved
also read from remote Netconf nodes to return only
data from local store. To access Netconf device
as a datastore one must get MountInstance for it -
MD-SAL instance exclusive for that device.

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

Log transactions being allocated

We do get logs when the transaction processing starts, unfortunately we
do not get the context in which the transaction was created. This fixes
the deficiency by logging when the transaction is first allocated. Also
adds explicit logs about transaction state transitions.

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

md-sal: commons-codec artifact version sync

md-sal is using both 1.7 and 1.8 versions of commons-codec.  The rest of
the controller is using 1.7, so bring md-sal in line with that.

Change-Id: I2a0e38874f857c30f34bf4f0e65becc35bc46f28
Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Avoiding hash collisions of a match with its reverse

Change-Id: I6b6942e90b85028af9ec98cb27cf17cd0377c033
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Changed port number for test server in ClientRequestHandlerTest
When tests are run in parallel, the port number in ServerRequestHandlerTests and ClientRequestHandlerTests
could have conflicted. Changed the port number to avoid conflict.

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

Merge "More defensive RPC handling in DOM Broker"

Merge "sal-remoterpc-connector: sync ch.qos.logback:logback-classic version"

More defensive RPC handling in DOM Broker

  - RPCs registered to DOM broker now must comform to the schema
    - at the time of registration of RPC implementation,
      the YANG schema must be present for that RPC.
      This does not affect RPC registrations for Binding Aware Broker.

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

Merge "Commented out Sys.outs in runtime generated code."

Merge "Update to new version of the artifacts proposed by jenkins-controller-version-changes-11"

Merge "Fix bug in hello message: module with more revisions."

Merge "fix of Bug 314"

Merge "Fix: removed tabs from yang files."

Merge "Tests for loading data with augmented schema nodes with equal names"

Merge "Clean up unnecessary imports in 01-md-sal.xml ."

Merge "The config now is loaded only from the coordinator. The previous way of loading the config only if empty had some side-effects."

Merge "Migrate all artifact versions to dependencyManagement"

sal-remoterpc-connector: sync ch.qos.logback:logback-classic version

The following commit:

  8079442 Enhancements to remote rpc server. Using zmq router-dealer bridge

introduced a new version dependency on ch.qos.logback:logback-classic.
Let's make this dependency consisten with the rest of the project by
moving from 1.0.12 back to 1.0.9.

Change-Id: I3bd7e3ce7d5c4af36fad8ddf7ee883117d9c4386
Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Merge "InputStream for RSA file should not be static"

Merge "Changed codec for Identityref in JSON transformation"

Merge "OSGI version change to 5.0.0 in ganymed subproject"

Merge "Enhancements to remote rpc server. Using zmq router-dealer bridge to make the server async. Enhancements to remote rpc client. Using zmq router-dealer bridge to make the client async. Added configuration for remote rpc in Configuration subsystem On Server startup reading of routingtable to populate remoterpcs was giving exception - fixed the same Client was not registered with Server and addRPCImplementation was failing - fixed the same ServerImpl was not getting registered as listener to RoutingTable hence announcement was not received on remote - fixed the same Patch 6: Some unit tests were intemittently hanging. ZMQ Test server was not gracefully shutting down."

The config now is loaded only from the coordinator. The previous way of loading
the config only if empty had some side-effects.

Change-Id: I4f1819c60b491a36e1c14a78ec376e2dfcee8583
Signed-off-by: Asad Ahmed <asaahmed@cisco.com>

Merge "Bug fix for bug 328 https://bugs.opendaylight.org/show_bug.cgi?id=328#add_comment"

Bug fix for bug 328
https://bugs.opendaylight.org/show_bug.cgi?id=328#add_comment

Casts node_id to long before instantiatiing an Node

Change-Id: I846bfe11cef49adeea65d9d2cce6a0d7444b93f5
Signed-off-by: Dave Tucker <dave.j.tucker@hp.com>

Migrate all artifact versions to dependencyManagement

- Migrate all the artifacts to use dependencyManagement to centralize
the version changes in controller and make sure the dry-run for
release could actually rewrite those.

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