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>
Avoiding hash collisions of a match with its reverse

Change-Id: I6b6942e90b85028af9ec98cb27cf17cd0377c033
Signed-off-by: Asad Ahmed <asaahmed@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>
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>
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>
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>
Merge "Log transactions being allocated"
Bump yangtools artifacts to reflect their uptodate versions

Change-Id: I9f90d50b1cb3955f04b54353dc771a19048ca209
Signed-off-by: Robert Varga <rovarga@cisco.com>
Merge "Default DataBroker does not read subtrees from Mountpoint"
Merge "Move NetconfUtil to netconf-util."
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>
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>
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>
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>
Merge "Avoiding hash collisions of a match with its reverse"
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>
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>
Merge "Fix ClassCastException in logging bridge stop"
Merge "Bump yangtools artifacts to reflect their uptodate versions"
Merge "Optimization - calculate the subnet prefix only once."