Fix checkstyle warnings ietf-netconf-monitoring

Change-Id: I05a3fb185ae53128c31f8565a9489f7d0399febe
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings for netconf-testtool

Change-Id: I47a3783d9b89cadc6549d6f4784508cefb149059
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings for config-netconf-connector

Change-Id: I8234bbfdb7092f6423cedc7f95de07995871756d
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings for config-persister-impl

Change-Id: Ife66b5fea7bb6d1fe293d6181d6da7221e296731
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Fix checkstyle warnings in netconf-api

Change-Id: I174ea168e5b264cd99c2faa9ab182859806ef12f
Signed-off-by: Marian Dubai <mdubai@cisco.com>

Merge "Updating features archetype to talk about user-facing features."

Merge "Netconf testtool fix multiple feature file modification"

Merge "BUG-2345 Move bouncycastle bundles from netconf-connector to netconf feature"

Bug 2368: MD-SAL FRM may update/remove unexpected flow entries.

FRM should always set strict flag into update-flow/remove-flow
RPC input.

Change-Id: I6027442b56f4cd93cfc3f954dc8501bb57e2d64a
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Bug-2136 - fix for is-local-path

Change-Id: I38a58683c94c54e9f399df56aa586e3916f75e41
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Merge "Bug 2347: Minor fixes to correct log output"

Merge "Bug 2021 - Continuous WARN log nodeConnector creation failed at node: OF|00:00:xx:xx:xx:xx:xx:xx"

Merge "BUG 2353 : Handle binary, bits and instanceidentifier types in NodeIdentifiers"

BUG-2345 Move bouncycastle bundles from netconf-connector to netconf feature

Apache mina sshd bundle is located in features-netconf and bouncycastle in features-netconf-connector.
This is wrong because mina sshd depends directly on bouncycastle but marks the dependency as optional which is probably not correct because this can result in runtime errors instead of resolution problems.

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

BUG-2370 Reset schema context on disconnect in nc

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

Netconf testtool fix multiple feature file modification

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

Bug 2347: Minor fixes to correct log output

DOMConcurrentDataCommitCoordinator#handleException hard-codes the error
mapper to CAN_COMMIT_ERROR_MAPPER however this should be passed
in corresponding to the phase.

In TransactionCommitFailedExceptionMapper, the opNames for
PRE_COMMIT_MAPPER and CAN_COMMIT_ERROR_MAPPER are reversed, should be
"preCommit", "canCommit" respectively.

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

BUG 2353 : Handle binary, bits and instanceidentifier types in NodeIdentifiers

This fix properly serializes and deserializes NodeIdentifier attributes of the types
  - binary (byte[])
  - yanginstanceidentifier
  - bits (set)

Change-Id: I612d40f9730c939be0594496c26370db96ea6449
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Bug 2347: DOMConcurrentDataCommitCoordinator uses wrong phase name"

Bug 2347: DOMConcurrentDataCommitCoordinator uses wrong phase name

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

Merge "Bug 1029: Remove dead code: samples/clustersession"

Merge "Bug 1029: Remove dead code: p2site"

Bug 2021 - Continuous WARN log nodeConnector creation failed at node: OF|00:00:xx:xx:xx:xx:xx:xx

This fixes the bug in ToSalConversionsUtils.actionFrom() that mistakenly
passed an AD-SAL-style NodeID to a conversion utility (fromNodeConnectorRef)
that was expecting a MD-SAL-style NodeID.

Patch 8: Add unit test to exercise code path that exposes the issue

Change-Id: Icc107bd785d6558a3351f139c1b13b8737b56ae7
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Signed-off-by: Colin Dixon <colin@colindixon.com>

Merge "Move adsal into its own subdirectory."

BUG 2302 : odl-clustering-test-app should not be part of the odl-restconf-all feature set

This patch does the following,

1. It removes odl-clustering-test-app from the restconf features
2. Move odl-clustering-test-app to the md-sal feature (just like odl-toaster)
3. Removes all unneccessary dependencies from odl-clustering-test-app
4. Removes all the clustering configuration files from the odl-clustering-test-app feature

This now allows the odl-clustering-test-app to be used with/without clustering. It also
eliminates the problem is inadvertently installing the odl-clustering-test-app when installing
odl-restconf-all.

Change-Id: I8021f88c967576aa8cf8da54dbfa8f4464233bc3
Signed-off-by: Moiz Raja <moraja@cisco.com>

Move adsal into its own subdirectory.

Moved various adsal components into an opendaylight/adsal subdir,
gave it a parent pom, and changed the root parent pom to have the
opendaylight/adsal as a module.

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

Merge "BUG-2329 Add test for anyxmls inside rpc resonse for netcfon-connector"

Merge "BUG-2254 Enable schema-less rpcs invocation in netconf-connector"

Merge "Make idle timeout configurable in ssh proxy server"

BUG-2254 Enable schema-less rpcs invocation in netconf-connector

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

Merge "Fix raw references to SimpleNode"

Merge "Fix raw reference to SessionNegotiatior"

Merge "Fix raw use of Gauge"

Merge "Add serialVersionUID to Serializable classes"

Merge changes Ic7e83322,I2482a37c

* changes:
  Fix a non-generic reference to Comparable
  Fix non-generic references to ArrayList

Merge changes I525c2070,Ib920ff67

* changes:
  Fix neutron pagination
  Reuse NeutronObject comparator

Merge "Use generic-aware empty collections"

Make idle timeout configurable in ssh proxy server

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

BUG-2329 Add test for anyxmls inside rpc resonse for netcfon-connector

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

Fix raw references to SimpleNode

Just adds a wildcard.

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

Fix raw reference to SessionNegotiatior

Add a wildcard and be done with it.

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

Merge "Fix raw references to Promise"

Merge "Fix raw reference to Predicate"

Merge "Fix raw ProtocolService references"

Merge "Fix a raw ServiceReference"

Merge "Fix a non-generic reference to DataContainerNode"

Merge "Fix non-generic references to NormalizedNode"

Merge changes I0fe4b78d,I3d3bc71b

* changes:
  Fix non-generic reference to org.opendaylight.yangtools.yang.data.api.Node
  Fix non-generic references to java.util.Map

Merge "Remove dead code in RuntimeMappingModule"

Merge "Do not override managed versions"

Merge "Fix raw types in Neutron northbound"

Remove dead code in RuntimeMappingModule

RuntimeMappingModule.RuntimeGeneratedMappingServiceProxy is private and
not used locally. Remove it, as it also uses deprecated classes -- thus
eliminating quite a few eclipse warnings.

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

Use generic-aware empty collections

Rather than referring to the instances directly, use the generic-aware
factory methods.

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

Fix raw use of Gauge

It is a generic type, so add type declarations to fix warnings.

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

Merge changes I9865d0cd,Ic71d525f,Ib8faba91,Ia10e5ec9,I35591747,If456a131,I9f8709cc

* changes:
  Fix non-generic references to List
  Hide XSQLBluePrintRelation.addToResult()
  Fix non-generic references to Set
  Fix HashMap references
  Fix raw references to java.lang.Class
  Fix raw references to NotificationListener
  Fix non-generic references to HashSet

Merge changes I52c2aaab,I2509ed5e,Ic1d8a4e5,I73730f79,Ica1959e5,I0ce61b07

* changes:
  Fix a raw Dictionary reference
  Fix a raw Enumeration reference
  Fix a non-generic CheckedFuture
  Access static methods via class
  Remove unused local variable
  Fix non-generic references to Optional

Merge "Do not create Map subclasses"

Fix raw references to Promise

Adds the proper type, so we do not have the warnings.

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

Fix raw reference to Predicate

Adds Object as the input type, fixing a few eclipse warnings.

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

Fix raw ProtocolService references

Adds a wildcard type to suppress an eclipse warning.

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

Fix a raw ServiceReference

Add the type argument to fix an eclipse warning.

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

Fix a non-generic reference to DataContainerNode

Just add a wildcards, as it's only used in a cast.

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

Fix non-generic references to NormalizedNode

Adds wildcard type arguments to get rid of eclipse warnings.

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

Fix non-generic reference to org.opendaylight.yangtools.yang.data.api.Node

Adds the proper wildcard to suppress warnings.

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

Fix a raw Dictionary reference

Fix the cast to match the variable declaration.

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

Fix non-generic references to java.util.Map

Adds proper arguments. Also eliminates unneeded casts to Map.Entry,
caused by unnecessary loss of type.

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

Fix a raw Enumeration reference

Adds the proper type argument.

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

Fix a non-generic CheckedFuture

This adds proper type, and actually amkes the code cleaner.

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

Access static methods via class

Invocation is slightly faster and fixes eclipse warnings.

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

Remove unused local variable

It does not use generic arguments, it's unused, and has overall no
impact (aside from eclipse warnings).

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

Fix non-generic references to Optional

Just adds arguments as needed.

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

Add serialVersionUID to Serializable classes

Each serializable class withouth serialVersionUID produces an eclipse
warning. It also means that the class has undefined versioning and
cannot be be upgraded -- this patch fixes all such classes.

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

Fix a non-generic reference to Comparable

Fixes a warning stemming from not specifying the object type.

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

Fix non-generic references to List

Adds arguments as appropriate.

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

Fix non-generic references to ArrayList

This patch adds the proper type specification.

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

Hide XSQLBluePrintRelation.addToResult()

Also fixes non-generic references to List.

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

Fix non-generic references to Set

Adds arguments as appropriate.

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

Fix HashMap references

This patch fixes missing generic arguments not being added to HashMap
references.

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

Fix raw references to java.lang.Class

Fixes eclipse warnings just by adding either a wildcard, or the proper
type.

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

Fix raw references to NotificationListener

The restconf broker has raw references, so modify the type definition to
allow passing the type down.

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

Fix non-generic references to HashSet

Adds arguments and changes to use ImmutableSet.of() where that
is more straighforward.

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

Do not create Map subclasses

Rather than using subclasses for initialization, which require a serial
version, populate them normally -- thus fixing warnings.

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

Do not override managed versions

This fixes pom.xmls to not override managed versions. Should not result
in any version shift.

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

Fix neutron pagination

This patch fixes a logic mistake which results in dead code being
reported. startPos is guaranteed to be non-null by virtue of being
always assigned.

The problem is that the detection that the marker is not found is not
correct, as binarySearch() will report the negated insertion point.

Also improves performance by using a primitive type instead of an
encapsulated object.

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

Fix raw types in Neutron northbound

Adds proper use of types and suppression magic to the factory method.

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

Reuse NeutronObject comparator

This is a static object, which is thread-safe and can therefore be
reused betwen instances and requests.

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

BUG 2339 : TransactionChain id created by the Clustered Data Store are not unique

The fix is to use a simple atomically incrementing integer to form the transaction chain
identifier instead of using the system time.

I also added a few log statements in there which were helpful in debugging the issue.

Change-Id: Ie1465f1640d6be0ff9da0f66477ebafc7ebff137
Signed-off-by: Moiz Raja <moraja@cisco.com>

Updating features archetype to talk about user-facing features.

This includes both giving advice about what should/shouldn't be
considered user-facing feautures as well as suggestions for -rest and
-ui features that provide RESTCONF and DLUX functionality to the base
user-facing feature.

Change-Id: I7a37c74adad8443cc50984b172c74e97cec13c84
Signed-off-by: Colin Dixon <colin@colindixon.com>

Bug 1029: Remove dead code: samples/clustersession

opendaylight/samples/clustersession/ is dead code that is not being
built and hasn't for some time.  Its complicating keeping
versions up to date and verifying that they are up to date.

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

Merge "Bug 2318: Follow-up changes for previous patch 12535"

Merge "BUG 2317 : StatisticsManager does not unregister from yang notifications on close"

BUG 2303 : Remove the implementation of AddRaftPeer/RemoveRaftPeer

It is both unused and incorrect

Change-Id: Ie0a9844ccf4de7dae68586954031599b55646c47
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Bug-2301 - Clustering:Snapshots need not be stored in in-mem ReplicatedLog for Installing snapshots"

Merge "Update junit Assert imports"

Merge "Migrade JUnitMatchers references"

Merge "Remove unused imports"

Merge "Fix checkstyle warnings in opendaylight/netconf"

Merge "Fix checkstyle warnings in opendaylight/config"

Merge "Create config-artifacts"

Update junit Assert imports

JUnit 4.11 has moved Assert to org.junit, and deprecated
junit.framework.Assert. This adjust the imports to fix the warnings.

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