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>

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."

Merge "Use JAXB instead of DOM in updateFeatureFile"

Merge "builder class for path creating"

Merge "builder class for NetconfServerSessionNegotiatorFactory"

Bug 3863: Add onSessionStart and onSessionEnd notification support

OnSessionStart and onSessionEnd support for md-sal netconf.
MonitoringToMdSalWriter is notified about changes in netconf state,
and writes this changes to datastore netconf-state tree.
CapabilityChangeNotificationProducer listens for changes in
netconf-state/capabilities subtree and publishesnotifications
about changes.
SessionNotificationProducer does the same for
netconf-state/sessions/session subtree.

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

Use JAXB instead of DOM in updateFeatureFile

Make updateFeatureFile parse xml with JAXB and update configfile.
Use JaxbUtil to get rid of namespace version compatibility error.

Change-Id: I780fe17f14dff7a1586b9f7982078cfb593c4d83
Signed-off-by: matus.kubica <Matus.Kubica@pantheon.tech>

Merge "configure logback.xml file"

Merge "Use mounted device schema context when resolving mounted device rpc path"

Use mounted device schema context when resolving mounted device rpc path

It wasn't possible to invoke rpcs on device, which were not present
also in controller schema context, so only rpcs like get-schema and
create-subscription worked. Now, device schema context is used to
lookup rpc definition during parsing path, so rpcs defined by device works.

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

configure logback.xml file

log file is created only when scale-util class
generates some logs.

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

Merge "Remove inventory related code from netconf connector"

Bug 2244: Remove rpc-reply reread when applying subtree filter

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

Remove inventory related code from netconf connector

also removed tests from MountInstanceTests due to deleted deprecated methods from NetconfDeviceSalProvider

Change-Id: I2f78db4b37447fa9c4808122107759523cb9dd40
Signed-off-by: matus.kubica <Matus.Kubica@pantheon.tech>

caching the same capabilities for devices using Interner.

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

Merge "Edit todo comments"

Merge "Remove blocking checkedGet call"

Bug 4577 Allow specification of a distinct schema cache directory per netconf device

First iteration done here https://git.opendaylight.org/gerrit/#/c/30163 introduce a BUG:
sideloaded schemas, manually added in cache/schema, and specified in the netconf connector,
where not being taking in account in the specified cache folder.

Change-Id: I6a5a6b23fc56bcc507734a99e48de88597b7ae2a
Signed-off-by: adetalhouet <adetalhouet@inocybe.com>

Bug 3708 - APIdoc explorer URLs contain extra 'node' in REST calls

* fixed mount point path builder that added each list qname twice

Manually cherry picked from controller patch to netconf patch:
https://git.opendaylight.org/gerrit/#/c/25531

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

Edit todo comments

AbstractNetconfSessionNegotiator - remove, constructor is used
                                   only once
NetconfEventSource - class is called YangModeledAnyXmlNode
NetconfServerSessionListener - not a fixme but ordinary comment
NetconfDeviceRpc - not needed
NetconfMonitoringServiceImpl - same capabilities are simply replaced
AbstractGet - method in restconf has different behavior
RuntimeRpc - not same as NetconfUtil.writeNormalizedNode()

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

Use the moved ConcurrentDOMDataBroker

ConcurrentDOMDataBroker has moved packages in controller
update the test that uses this class.

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

Merge "Replace mdsal model bundles with feature"

Merge "Add IPv6 to NetconfHelloMessageAdditionalHeader"

Merge "Use keyed list as a selection node in filter"

Merge "add partial match for non-module capabilities - to enable deviations in base capabilities"

Merge "Bug 3866: Support for Restconf HTTP Patch"

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

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

Merge "Bug 5580 - RestPerfClient: Invalid requests rate reported when a thread times out"

Replace mdsal model bundles with feature

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

Merge "Call CreateSubscription close method on session end"

Merge "Removed embedding on javanet stax-utils"

Merge "Remove deprecated NetconfMessageHeader class"

Merge "Bug 4940 - correctly implement default-request-timeout-millis"

add partial match for non-module capabilities - to enable deviations in base capabilities

Change-Id: I06fbeaeb63cca9ad8b2e7b9aa8c75b3a1ce40334
Signed-off-by: Giles Heron <giheron@cisco.com>

Call CreateSubscription close method on session end

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

Add IPv6 to NetconfHelloMessageAdditionalHeader

Address is no longer validated via regex.
Instead, InetAddresses.isInetAddress(address) is used.

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

Bug 4940 - correctly implement default-request-timeout-millis

This is a follow up patch of this one https://git.opendaylight.org/gerrit/#/c/33206/
This revert 09343d18f43b408b830aa563d94e5b203692a02e

The implementation made in previous patch was blocking transaction for a given timeout,
but the blocking wasn't handled at the right place, ending up blocking restconf/user thread.

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

builder class for NetconfServerSessionNegotiatorFactory

-Constructor parameters are reduced. It has only one parameter,
 which is builder class NetconfServerSessionNegotiatorFactoryBuilder.
 NetconfServerSessionNegotiatorFactory also validates in constructor
 if every required parameter is present.

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

builder class for path creating

Paths can be created with new path creator class
instead of using constant strings.

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

Do not pull yang-jmx-generator in

This plugin is used only during compile-time (and is pulled in via
plugin configuration), do not declare it as a dependency.

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

Remove deprecated NetconfMessageHeader class

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

Use keyed list as a selection node in filter

Keyed list couldn't be used as selection node in filter
before, because filter node was parsed to normalized nodes
and yang codecs don't allow keyed list without key.
Filter content is not transformed to normalized nodes now,
only validated against schema context.

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

Removed embedding on javanet stax-utils

javanet stax-utils is now installed as a bundle via a feature so no
longer needs to be embedded in sal-rest-connector.

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

Bug 5580 - RestPerfClient: Invalid requests rate reported when a thread times out

If some threads sending requests fail or time out, skip calculation of
requests per second value and do not log it.

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

Merge "Remove unused aaa dependency from netconf-connector feature pom.xml"

Remove unused aaa dependency from netconf-connector feature pom.xml

Removes the features-aaa dependency as it is not needed.

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

Bug 5638 switch netconf to use odl-aaa-shiro isntead of odl-aaa-authn

odl-aaa-authn has very limited uses now, since odl-aaa-shiro has
included most of its features by default.  Switch netconf to use
odl-aaa-shiro instead of odl-aaa-authn, to get most up to date
aaa implementation.  odl-aaa-authn should be deprecated in a
separate patch.

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

Merge "Optimize NetconfMonitoringServiceImpl getSchemaForCapability"

Merge "Add support for notification subtree filter"

Merge "Add support to RESTCONF to dynamically specify Filters"

Merge "Reimplement NetconfDevice schema setup without recursion"

Merge "change to static function"

Bug 3866: Support for Restconf HTTP Patch

- JSON body reader/writer in place
- XML body reader/writer in place
- refactored BrokerFacade and RestconfImpl code to apply HTTP Patch within transaction
        in order to be able to rollback in case of error(s)
- review comments implemented

Change-Id: I974c88fc846dfa6fd9b1672d431c0f246c904877
Signed-off-by: Martin Ciglan <mciglan@cisco.com>

Merge "Revert "Add PasswordCredentialAuth interface""

Merge "Bug 5475 - File descriptor leak on netconf connector reconnects"

Revert "Add PasswordCredentialAuth interface"

This reverts commit 57bfaf2a0146d33a9a504b5a48c5525916a44b79.

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

Remove blocking checkedGet call

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

Add support to RESTCONF to dynamically specify Filters

This change builds of work that the AAA team took on to help add dynamicism to
RESTCONF.  The javax.servlet-api specifies a means to define chains of Filter(s)
in front of a web Servlet.  However, configuration of filters is not supported, and
there is no easy existing mechanism to add Filters to a prebuilt ODL
distribution.  The issue involves the fact that Jersey Servlets define Filter
chains within web.xml, which is static and included inside of the odl-restconf
bundle.  This makes it very hard (if not impossible due to lack of
DynamicImport-Package declarations in restconf's pom) to alter the web.xml file
after compilation.

AAA added a filter called CustomFilterAdapter, which recreates the javax
filter chain-of-responsibility pattern.  The dynamic filter chain functionality
has very little to no overhead if left unused, but could be used to add
Filter implementations that allow for enhanced logging of HTTP/S requests/
replies, as well as other various functions.  This aids in debugging real
world deployments, as well as enables enhanced Accounting and Auditing
of the controller.

This functionality has been raised as vital by members of the AG.

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

Bug 5475 - File descriptor leak on netconf connector reconnects

Do not start the SshClient each time the AsyncSshHandler is instantiated.

http://mail-archives.apache.org/mod_mbox/mina-users/201603.mbox/%3CCAA66TprZd754Ry8wMYgS7kOJc8%3DvZNP3MA41VYZyVi_oQSv6yw%40mail.gmail.com%3E

Change-Id: If239f87ea4ff707f564f7f10e4f50481033f77fd
Signed-off-by: adetalhouet <adetalhouet@inocybe.com>