.gitignore .factorypath created by m2e-apt

https://github.com/vorburger/opendaylight-eclipse-setup now bundles
https://github.com/jbosstools/m2e-apt, which creates .factorypath files
all over the place.

Also .checkstyle created by Eclipse Checkstyle plugin.

Change-Id: Ibc9dc3d6a471f216d1e3a9adf76523e4061a0099
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge "Remove unused isMaster variable in ClusteredNetconfDevice"

Merge "BUG-865: Do not use QName.cachedReference()"

Merge "Bug 5529 - Restconf M5 - Clean-up sonar issues"

BUG-865: Do not use QName.cachedReference()

QName.intern() is the modernized method. Also use it for all constants,
allowing greater reuse inside the JVM.

Also create NodeIdentifier and LeafNode constants for things that are
invariant, improving GC behavior.

Finally import YangInstanceIdentifer.{NodeIdentifier,PathArg} to make
the code more readable.

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

Bug 5529 - Restconf M5 - Clean-up sonar issues

Remove critical issues

Change-Id: Ie14934864976b5f9496af836a81293f052f4666d
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bug 5911 - Restconf draft11 - bad parsing

- fix for leaf-list deserialization
- fix for YangInstanceIdentifier.EMPTY serialization and deserialization
(YangInstanceIdentifier.EMPTY is serialized to single slash and single
slash is deserialized to YangInstanceIdentifier.EMPTY)
- fix: parser recognizes all reserved characters according to rfc3986
- fix for list keys serialization
- fix for percent encoded characters decoding
- add check if leaf list instance value was supplied, error otherwise
- fix: module name before identifier is required only if node was not
defined in parents module

Change-Id: I48a944ed6139da5b7be0835cc99e219e11dc0072
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Merge "Bug 5708: Schemaless netconf mount point"

Improve logging in netconf test tool

This adds debug logs to TesttoolNegotiationFactory to help distinguish
cached operation service from newly created one. Also logs passed
parameters to netconf test tool.

Change-Id: I52dbc86a926724f97490ef2d939da22e71a3925a
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Remove unused isMaster variable in ClusteredNetconfDevice

isMaster is never used, so it should be removed.

Change-Id: I5419d281062e9d25cf91e7a710cec06c609765e9
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Fix bug 5810, 5812: 1. only when node's config statement is true, such apis as GET/PUT/POST/DELETE config are added for this node; 2. when there are two or more top container/list nodes whose config statement is true in module, make sure that only one root post link is added for this module."

Merge "Bug 5526 - Testing - Rest implementations - Streams"

Merge changes from topic 'parser-post-removal'

* changes:
  BUG-865: Remove reference to ExtendedType
  BUG-865: remove references to obsolete types

Fix bug 5810, 5812:
1. only when node's config statement is true, such apis as
GET/PUT/POST/DELETE config are added for this node;
2. when there are two or more top container/list nodes whose config
statement is true in module, make sure that only one root post link is added for this module.

Change-Id: I96d0b485c5943ad9bee63327c5d10e3cd0e5cb77
Signed-off-by: Nafee Chan <nafee.chan@gmail.com>

Bug 5526 - Testing - Rest implementations - Streams

Unit tests for RestconfStreamsServiceImpl

Change-Id: I7c120cda262786b898e8918381747f98cb935329
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

BUG-865: Remove reference to ExtendedType

ExtendedTypes are no longer generated by the parser
and the class has been deprecated. Remove references
to it so it can be removed.

Change-Id: Id74fd89b5a8dcd3489f4b0856716e39324a75cc9
Signed-off-by: Jakub Toth <jatoth@cisco.com>

BUG-865: remove tests dependend on pre-Beryllium parser

Removed tests have had unclear logic and were removed.

Change-Id: I7410a6f3e7959f709947222636b75e8fcad66afe
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Merge "Bug 6023 - Add default address setting"

BUG-865: remove references to obsolete types

TypeDefinition subclasses in model.util are deprecated.
Use their model.util.type equivalents available through BaseTypes.

Added one negative test into NnToXmlTest.

Change-Id: Id5bda10663b12e782a24731bc73d3e771ea242f1
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Merge "Bug 5527 - Testing - Provider"

Bug 6023 - Add default address setting

Adds default address and ssh private key path setting for config
subsystem netconf endpoint.

Change-Id: Id3169847a26f50e701aa5acfb592d9092abeeaf6
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Bug 5526 - Testing - Rest implementations - Schema

- unit tests for RestconfSchemaService
- small fix for found bug in validation

Change-Id: I37f14db187c27be4192f1fdbd7984bcbc5e927fd
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bug 5527 - Testing - Provider

Unit tests for RestConnectorProvider

Change-Id: I97c592808a5e33b127b1de4b44b9b209d4353caf
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Fix of build breakage caused by changes in Yangtools

- use the RevisionSourceIdentifier implementation of SourceIdentifier
abstract class

Change-Id: I8ffceca436c1477a676e87f69e31ce7c4722c061
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

BUG 4968: Clear transaction after proxy tx is submitted.

Change-Id: I2038a25cdd0ae17e6517a1f0ac375850a3c17479
Signed-off-by: Tomas Cere <tcere@cisco.com>

Fix broken tree of modules in sonar analysis

Change-Id: Ie978c3696e814b6765d340736c5853c13e12bd02
Signed-off-by: Tomas Cere <tcere@cisco.com>

Fix XPath expression in NetconfMappingTest

Netconf replies constructed in config subsystem netconf endpoint now
contain correct namespaces when in DOM format. Fix XPath expression in
NetconfMappingTest to contain these namespaces.

See https://git.opendaylight.org/gerrit/#/c/39492/

Change-Id: I5ec33403059a9f0f046fda6f00122d52a0885ffa
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Netconf Karaf CLI code clean-up

- Fix show-device command argument checking
- Remove default values for strings to make Help look cleaner
- Check for null node to avoid NPE

Change-Id: I5376e76e252cbe3743410f84c8b20722a483e20d
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Merge "Bug 5912 - Restconf draft11 - utils"

Replace old yang parser usages in netconf.

Change-Id: Ice3cbfe4863f14f6c1f75df212c62d1e370c5297
Signed-off-by: Tomas Cere <tcere@cisco.com>

Merge "Add logger implementation to netconf-impl test scope"

Merge "Add Karaf CLI for NETCONF CRUD commands"

Merge "Bump ietf-yang-library module to newest draft"

Add logger implementation to netconf-impl test scope

Also rethrow exceptions on failures in ConcurrentClientsTest to
make issues more apparent.

Change-Id: I3e907568c6a384f9512a81c05ae798e11972f607
Signed-off-by: Tomas Cere <tcere@cisco.com>

Merge "Clean up a lot of JavaDoc etc. to conform to current Checkstyle rules"

Merge "Name Logger field LOG instead of _logger and make it static (Checkstyle)"

Merge "Log exception instead of using printStackTrace, and warn instead debug."

Clean up a lot of JavaDoc etc. to conform to current Checkstyle rules

Change-Id: I1b5613ad00633e6b59626dacca08f8c458f076f3
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Name Logger field LOG instead of _logger and make it static (Checkstyle)

Change-Id: If57f5b6dda95345f0ae3fe4584507aef1c3e0e08
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Log exception instead of using printStackTrace, and warn instead debug.

The formatting etc. changes are done automatically now (eclipse-setup).

Bug 5980 root cause does not actually show up in log with this, but
still seems like a good idea.

Bug: 5980
Change-Id: Ib07f029b5350928e3f0d3c70bcc9dbc9cc3a318d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Replace old yang parser usages in restconf

Change-Id: Ib82a0577592f2e443120aea16bdfa329eba4c43b
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bump ietf-yang-library module to newest draft

Bump ietf-yang-library yang module to confrom to
draft-ietf-netconf-yang-library-06 described in
https://tools.ietf.org/html/draft-ietf-netconf-yang-library-06

Change-Id: I765d14f1401f06aec33fe08299cc6df1413df0b4
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Bug 5912 - Restconf draft11 - utils

Fixing bugs of Draft11 implementation found by unit tests
in RestconfSchemaServiceTest:
- wrong condition
in ParserIdentifier#toSchemaExportContextFromIdentifier
- slash at the end of mount point identifier
in ParserIdentifier#toInstanceIdentifier
- wrong condition at the end of the input
in YangInstanceIdentifierDeserializer#create
- module names validation

Change-Id: Ib4031fd6d2830a49845191d1792e695a19d04e93
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bug 5708: Schemaless netconf mount point

Schemaless mount point can be used to connect to devices
which doesn't provide their schemas. In that case,
both rpc input and output must be of anyxml type.

Data broker and transactions works with anyxml too. Read transaction path
Yang instance identifier is transformed to xml filter structure
used in get-config rpc. Resulting data are selected from rpc-reply
according to path, wrapped in data element(to be able to return lists)
and returned as anyxml. Write transaction path is transformed to
xml structure. Data are appended to the last element of this structure.

This is done by SchemalessRpcStructureTransformer. It is strategy
used in NetconfBaseOps class. For standard mount points
NetconfRpcStructureTransformer is used.

Change-Id: I38c70fc6b676941ef6d731a52a6b12ae7c10eec6
Signed-off-by: Andrej Mak <andmak@cisco.com>
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Improve SAXTransformerFactory instantiation

This is just a correctness fix to follow the API contract.

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

Merge "Bug 5526 - Testing - Utils - Validation"

Fix for the Bug 3880 - inconsistent code fragments in netconf

Change-Id: If9094546b18c11cf1c2755253bc33be1c5a12907
Signed-off-by: Dhevendran <dhevendran.k@tcs.com>

HOTFIX - Fix upgrade of jackson in ODL parent

Change-Id: Ic2efcb0b825b9bf82899863e8bfe282c30b9369c
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Add Karaf CLI for NETCONF CRUD commands

Change-Id: I200151c3f8eea34d667f4a5187b69365931e0cb0
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>
Signed-off by: Alexis de Talhouët <adetalhouet@inocybe.com>

Bug 5526 - Testing - SchemaContextHandler

Unit tests for SchemaContextHandlerImpl.

Change-Id: Iff1330700ecc0209aae2d6d1cf6d745b582ca506
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5526 - Testing - Utils - Validation

Unit tests for RestconfValidation.

Change-Id: I7b85149e6b634616503c06917d01e98cea6698ac
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bug 5900 - Jenkins RaceContiditions - problem between Jersey and RestconfProvider

Change-Id: I2b87b53c0679eede0fdbdea9de69568c14367ea7
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Merge "Implement getSessionListener in DummyMonitoringService"

Merge "Refactor Logger in RestCodec"

Implement getSessionListener in DummyMonitoringService

Returning null caused errors during connecting to testtool.

Change-Id: I62efccf5042c2e9084654c44262a19afbaeb1d3b
Signed-off-by: Andrej Mak <andmak@cisco.com>

Bug 5526 - Hotfix - Fix to register provider of restconf after init
restconf application in Jersey

  * moving load on startup of new restconf draft impl to 0

Change-Id: I394393d9ddaec6c935fcdd23e4eab6e47913c0f8
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5527 - Re-implement ControllerContext.toInstanceIdentifier() method

  * create YangInstanceIdentifier serializer and deserializer
  * fix mount points

Change-Id: If97ced6cb14a11d9cd037f4b93a8f1c8fc984561
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Merge "Bug 5526 - Added new servlet to the web.xml for new restconf implementation"

Merge "Remove unused jersey dependencies in RESTCONF and yanglib features"

Remove unused jersey dependencies in RESTCONF and yanglib features

Cleans up jersey dependencies in RESTCONF and yanglib features.

Change-Id: I0f0846ce77f20155429327a3d87ec7882422e582
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Add contextual logging to WebSocketServer and clean up formatting

Just adds some logging to WebSocketServer and cleans up formatting.  Since
the port is assigned as part of createInstance(...), the DEFAULT_PORT public
constant was removed to further support encapsulation.

Change-Id: I7f6922fd08d54d982d267ed50c860b76a31d0014
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Fix WebSocketServer allowed port range

Operating systems generally limit binding to ports below 1024 to
super users.  However, 1024 is fine for normal users.  This change
augments the precondition check to allow for port 1024 if it is
specified.

Change-Id: I876ff1694371e0c71b94806a6a1b7dbf3b27950f
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Bug 5526 - Added new servlet to the web.xml for new restconf
implementation

  * application for new servlet
  * register SchemaContextListener via SchemaContextHandler

Change-Id: Ifb42aed73f7524a07174c0c5e966ca44bcb228d8
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Change naming in testtool payload template

Change-Id: Ib252a846645e6392222c8fe619cbe6bf0234d754
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Merge "Bug 3864: Notify netconf monitoring about changes in session"

Merge "Update comment and remove unwanted FIXME in SchemaSetup"

Update comment and remove unwanted FIXME in SchemaSetup

I don't think there is anything to fix here as the API
is defined as is. So I believe it works as designed.

Change-Id: Id750be7a8e2cf2a8756f056623118a9eba7f9914
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Bug 5526 - Remove jersey from features for restconf

  * Jersey initialization is in the AAA project

Change-Id: I52c9ee5c481fa843915821cd4d295363c5634ea9
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5526 - Wrap providers of both implementations of restconf.

Change-Id: Ic90c49f50d63d6ca99b149e69de236ef3172cb59
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5526 - Prepared provider for new implemantation

Change-Id: I8d89b01b39d32b9491c7c4293c20820720a47b5c
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5526 - Supported MediaTypes from Draft11 in all writers from
old restconf(reusing them in new)

Change-Id: Ibbc34d8bdd3b057c0d344764d8bf5c6021bb2ae7
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5526 - Implementation of SchemaContextHandler

Change-Id: I8575639228e498a16b350a4134b9928089091b4f
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5526 - Implementations of services

  * used soft reference for schema context for holding the current schema
  * util classes for static methodes and constants for implementations
    of services

Change-Id: I5ddccc91feaf3ebfd7f8b40c2f954c4506995622
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 2343 - ODL remote YANG sources repository plugin

ODL plugin functioning as a remote YANG sources repository.

Plugin takes YANG schema sources from specified directory and provides
them through web service.

Change-Id: I93370af60e53dc0acafb844422adeea2abccf7a0
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Bug 2343 - Sideload of models for nodes from remote yang sources repository

Add support for sideloading schemas for devices from remote yang
schemas sources repositories.

Remote repository is queried for list of provided models and URLs
representing their schemas resources. These schemas are preregistered
to device schema registry as fallback schemas.

Change-Id: Iba3d77dd1a868a89a2f260ed751e36250fa30a47
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Refactor Logger in RestCodec

Add missing static keyword to Logger objects.
Rename Logger constants to match naming conventions.

Change-Id: I86cb061496ea2d422be406a74ef259cf00020db6
Signed-off-by: Dileep Ranganathan <dileep.ranganathan@intel.com>

Merge "Add NetconfDevice builder class"

Merge "Find the most recent notification"

Add NetconfDevice builder class

Parameters in constructor are reduced by creating a builder class.
Now we can use a NetconfDeviceBuilder in constructor which checks
if all needed parameters are present and sets all the variables
as before.

Change-Id: I8bf2c40b80c26faa99b299b7b7f98e824a963f3a
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Merge "Add some validations and new arguments"

Merge "Add a batching system in Netconf testtool"

Find the most recent notification

If multiple revisions for same notifications are present
it will pick the most recent one. It will iterate through
all of the notifications and it will pick the one with most
recent revision date. StripNotification function is also moved
to NetconfMessageTransformUtil class together with
EVENT_TIME_FORMAT which is needed for this function.

Change-Id: Ibd9fe800d75f877ead03162b42895b322288b62e
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Bug 5526 - Mark old services methods with @Deprecated

Change-Id: I8f65897d45327d2bf257f51fc541f9fc1210cc08
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Add some validations and new arguments

It validates some new parameters.Also thread pool size
and Ip of socket address is configurable. We can set
them using program arguments.

Change-Id: I4af24500ca0650c0c5a922d41b292c573ac78400
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Add a batching system in Netconf testtool

We can now set how many devices will be configured
in one request. It can be done asynchronously or
synchronously. It s configured using json file
instead of XML.

Change-Id: Ic4d666b21771e8cebdaaa1fabb246ac482bd8ef9
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Bug 3864: Notify netconf monitoring about changes in session

According to https://tools.ietf.org/html/rfc6022#section-2.1.4,
netconf-state/sessions should contain number of received rpcs, notifications etc.
Because of performance reasons, data tree is not updated after every message, but
NetconfSessionMonitoringService periodically publish updates with changed sessions
to its session listeners. MonitoringToMdsalWriter writes changes to datastore.

Change-Id: I5cdc2a18a036541fd23f430be6e19ba395ca62c9
Signed-off-by: Andrej Mak <andmak@cisco.com>

Merge "Fix for Eclipse to not choke on this line of Java Generic (was red) "

Fixed 3 tests which were clearly put in wrong directory

Physical directory location and Java package declaration must match

Change-Id: Ie13abea34024aca40a0f47625d4ed2f6ad8a42ee
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Fix for Eclipse to not choke on this line of Java Generic (was red)

JDT has subtle differences to undocumented JLS re. generics vs javac

Change-Id: Ifd2f5c5e104d112026092ef9be8428271e2f1d91
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 5676 - Mark old restconf draft like depricated

Change-Id: Iba80160b2dec0af4af64d50dfae3ec35276bd70d
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Merge "Make rest-perf-client modify the {DEVICE_PORT} tag in payloads"

Merge "Fix NetconfDevice schema setup"

Merge changes Ia25fc20d,I0b2ff209,I33768fc9

* changes:
  Bug 5526 - Handling schema context for restconf
  Bug 5526 - Restconf services api + RestconfConstants
  Bug 5676 - Base draft util class for media types and modules

Fix NetconfDevice schema setup

I don't know exactly where, but it seems like SchemaResolutionException
can wraps a MissingSchemaSourceException thus breaking the schema setup
mechanism.

Change-Id: I2fb20294e95d877598199b02d6e6474781addc0b
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

Merge "Bug 3863: Add onSessionStart and onSessionEnd notification support"

Bug 5526 - Handling schema context for restconf

Change-Id: Ia25fc20d73b844960fe5d9a6b845f73690390aac
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5526 - Restconf services api + RestconfConstants

Change-Id: I0b2ff2099dfaa8e064893525c7ea4a56168bbcdb
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 5676 - Base draft util class for media types and modules

Change-Id: I33768fc91cedc55549e7e2ee62c74e58ece738ed
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Fix test failures in NetconfClientDispatcherModuleTest

The implementations of NettyThreadgroupModule and NettyThreadgroupModule
HashedWheelTimerModule in the controller were recently changed - they
now obtain the service instance via the OSGi service registry therefore
several BundleContext methods must be mocked appropriately.

Change-Id: I83c77b0e5199ba73f4fd9e9d3f5ae93a0222f7a7
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Merge "caching the same capabilities for devices using Interner."