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>

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>

Merge "Update fixme comments"

Merge "Remove unneeded todos and fixmes"

Merge "Add description to northbound notification yang files"

Update fixme comments

NetconfBaseOps - remove, source node is and has to be wrapped in a choice
CreateSubscription - update comment
MonitoringToMdsalWriter - issue not present anymore

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

Optimize NetconfMonitoringServiceImpl getSchemaForCapability

Update mappedModulesToRevisionToSchema only on capability change,
so getSchemaForCapability doesn't need to compute it every time
it's called.

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

Add support for notification subtree filter

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

netconf-testtool, autoconfiguration through restconf and topology model

It starts all the devices, creates a restconf request and
sends PUT request to odl which sends us some response.
It also gives us time of how long did it take to configure
all the devices. All this can be done synchronously or
asynchronously.

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

change to static function

Function get in TestingNetconfClient is static.

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

Merge "Use java 8 DateTimeFormatter in NetconfServerSession"

Merge "Refactor NetconfEventSource"

Add PasswordCredentialAuth interface

Prevents warnings during compilation since CredentialAuth is generic.

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

Merge "Avoid relying on the timezone"

Remove unneeded todos and fixmes

AbstractNetconfTopology - default values should not be stored
ClusteredNetconfTopology - default values should not be stored
BaseTopologyManager - we want to keep successful connection alive
KeepaliveSalFacade - if anything fails we log the exception and reconnect
NetconfDeviceRpc - no need
NetconfMonitoringServiceTestImpl - test is already implemented in another package
NetconfTopology - no need
TransactionProvider - now, every session is nicely isolated

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

Refactor NetconfEventSource

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

BUG 5413 -  RestPerfClient: Hang on java.util.concurrent.CancellationException.

Make timeout for executing requests customizable via parameter --timeout.
Check for cancelled tasks.

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

Merge "Remove outdated todos"

Merge "Add javadoc link in LoginPassword"

Merge "Code cleanup: unify writeNormalizedNode functionality"

Merge "Use transaction chain in SalNodeWriter"

Merge "Add netconf message to error info"

Merge "Remove outdated fixme"

Add netconf scale-util

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

Remove outdated todos

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

Bug 4964: Akka 2.4.1 version bump

The akka/scala version definitions are now in odlparent so removed the
local definitions.

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

Reimplement NetconfDevice schema setup without recursion

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

Add description to northbound notification yang files

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

Add netconf message to error info

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

Add javadoc link in LoginPassword

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