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

This is a follow up patch of this one https://git.opendaylight.org/gerrit/#/c/33209/
This revert 2a301c41e506c834c63b53aa62df68cd83385553

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: Ide2f24890d6b9fa577bba793d0b1f0b8ea85a222
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.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: Id8281d352583d1d2ad8cf37a62ceb5a9431be3c0
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Netconf-topology module unit tests

Change-Id: I8258ce4973758ef20d503712147e34ee78a1cd95
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>
(cherry picked from commit 32e6bedfee7fa221f28aea8694ed36965c9b905b)

Merge "BUG 5413 -  RestPerfClient: Hang on java.util.concurrent.CancellationException." into stable/beryllium

Merge "Bug 5475 - File descriptor leak on netconf connector reconnects" into stable/beryllium

Bumping versions by 0.0.1 for next dev cycle

Change-Id: I4b2e0eaae93dd58449b712f4483e5efd1bf9dfb0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Release Beryllium-SR1

Change-Id: I9cbba03941d57eb03c583179c19aa6ac8ecbee72
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

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: I6c70fa669dfceadeaa0b388894f7042bb44fc586
Signed-off-by: adetalhouet <adetalhouet@inocybe.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>

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: Ib66b6d0e49fc32709f1163e517240ec0bddd0a60
Signed-off-by: Martin Ciglan <mciglan@cisco.com>

Bug 5345: Unable to use get-schema RPC due to unsafe class casting

This patch represents class cast fix for Beryllium SR1 code base only
due to using of deprecated structures.

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

Replace mdsal model bundles with feature

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

Auto create mixin list-parent nodes from netconf northbound

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

Merge "Sal-rest-connector unit tests switch to new yang parser." into stable/beryllium

API Doc Explorer crashes if a yang file on a mounted resource does not have a version

The fix for bug 3782  was checking for dates less than 1970 ( default date when there is no version).
The fix is to check for less than or equal to 1970. The existing code works fine for US time zones
because all US time zones are behind GMT and hence the local time for EPOCH date results in year 1969.
However when in other time zones such as in Europe the year part of the EPOCH date results in 1970.
This is a simple fix to correct that.

Change-Id: I671e602cc5f99cdc3f940073609028b8f2349de9
Signed-off-by: Balaji Varadaraju <bvaradar@brocade.com>

Merge "BUG 5269: Allow capability override in topology" into stable/beryllium

Merge "Bug 4114 - netconf connector replace causes transaction chain failure" into stable/beryllium

Sal-rest-connector unit tests switch to new yang parser.

Change-Id: I4efa8e265ca2827033cb0cd3f0b8be3924616842
Signed-off-by: Peter Kajsa <pkajsa@cisco.com>
(cherry picked from commit 2dd8dce2a8260edb330a7ade345e42b06538a414)

Bumping versions by 0.0.1 for next dev cycle

Change-Id: I738fcc7b2bd45b91921282a9018f2ce379e51491
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Release Beryllium

Change-Id: I94ed921f16c34ce35b8b6d4437071d7782d79787
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 5346: Add odl-config-netty to odl-netconf-client

Feature odl-netconf-client applies 01-netconf.xml config file,
which expects capabilities provided by config file 00-netty.xml
from odl-config-netty feature. But odl-config-netty is not part of
odl-config-all feature, so it needs to be specified directly.

Also, added more specific capability dependency to 01-netconf.xml
and updated dependencies for netconf features pom.xml

Change-Id: Idb8aa24e7adeecd1d6a7d9bee8fb0d8006828831
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

BUG 5269: Allow capability override in topology

Previous patch was for cfg subsystem configuration only,
this adds the ability to override devices capabilites
to netconf connectors configured through topology.

Also fix typo in UserPreferences.

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

Bug 4114 - netconf connector replace causes transaction chain failure

In order to avoid race condition when txChain perform delete then write
transactions in a row to the same path, perfom the delete as a blocking
transaction so once done, write can be perform without any race condition

Also reset the txChain after a failure.

Change-Id: If0c51be8fe80106a6722abb93dbe4b47548fce1b
Signed-off-by: adetalhouet <adetalhouet@inocybe.com>
(cherry picked from commit 76cce56c2bf85efde7f360f2af779a75b57141ee)

Merge "Add ietf-netconf-monitoring-extension to base schema" into stable/beryllium

Merge "BUG-3934 Don't omit updated data from notification" into stable/beryllium

Add ietf-netconf-monitoring-extension to base schema

Monitoring extension must be present in base schema context
to be able to call get-schema to download models from device.

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

Merge "Bug 2806 - Immediate and infinite reconnect attempts during negotiation" into stable/beryllium

Merge "Move NetconfHelloMessage to netconf-api" into stable/beryllium

Merge "bug 5096 Clean up transaction in case of failure" into stable/beryllium

Merge "Fix tests according to review comments." into stable/beryllium

Bug 2806 - Immediate and infinite reconnect attempts during negotiation

Set connection to success, after successful netconf negotiation.

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

Move NetconfHelloMessage to netconf-api

Make sessionPreferences return NetconfHelloMessage.
Move NetconfHelloMessage and NetconfHelloMessageAdditionalHeader test to netconf-api.

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

bug 5096 Clean up transaction in case of failure

In case of transaction fail, clean up failed transaction,
so it is posible to start new. Add more detailed error message.

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

Bug 4100 - Extend sal-netconf-connector base schema context

In case, that device supports notifications, but doesn't provide
model for them, model from BaseSchema in NetconfMessageTransformer
will be used as fallback.

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

BUG-3934 Don't omit updated data from notification

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

Fix tests according to review comments.

https://git.opendaylight.org/gerrit/#/c/33949/
https://git.opendaylight.org/gerrit/#/c/33972/
https://git.opendaylight.org/gerrit/#/c/33849/

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

BUG 5089: Fix incorrect slave schema repo behaviour

Need to use the correct schema repo in slave schema providers.
Introduced by the patch for BUG 4577.

Change-Id: Iba64841ba0f2f875f4da455cb3025266cf7b6c5e
Signed-off-by: Tomas Cere <tcere@cisco.com>
(cherry picked from commit 0dbc2a1f8597733e72086b0e9dfa191089e3eeab)

Merge changes Ib7024e5e,I1b1003c1 into stable/beryllium

* changes:
  Ignore netconf tools from sonar report
  Topology node writer UnitTests

BUG-5269 Enable device capabilities override

So far, merge was the dafult strategy. From now on, override
is also possible.

Useful when it's necessary to completely replace device models
with user/app models.

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

Bug 5175: RPC called on mount point not being correctly handled when multiple revision-date coexist

 The ControllerContext#toQname was just looking for module using the module name. In scenarios where you have multiple revision for the same module, the method was returning the latest schema. This fix is to retreive the schema based on the revision date

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

Bug 4948: Catch RuntimeException from ensureParentByMerge

Made the calls to PUT and POST more robust. Caught the RuntimeException from
 the merge call in the EnsureParentsByMerge. If this fails then started
a new transaction and continue with the original PUT. This ensures when a device
is unable to handle the empty list/container merge it will still succeed with the
original request. Similar change has been made for POST.
This bug was fixed in stable Lithium. I am pushing the same fix on Beryllium.

Added FIXME statements to figure out and catch specific Runtime exceptions
thrown by Netconf instead of RuntimeException.

Change-Id: Id18a97857e46f44014dbd61cbfa86285457a2905
Signed-off-by: Balaji Varadaraju <bvaradar@brocade.com>
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

BUG 5125: Fix NPE on module's with unknown namespace

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

Bug 3934: Websockets: Scope ONE doesn't work correctly

Bug fix for vcmts system manager.

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

BUG-5019: adjust to changed NormalizedNodeStreamWriter API

This patch just provides the information which is readily available.

Change-Id: Ie56c880c6480d6be750d4efdeeceea6036509bb8
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 0ddfd274ee0274d2a3e5e5a883fa3cbdc3df1916)

Bug 3934: Websockets: Scope ONE doesn't work correctly

Make sure data in data-change-event is wrapped in data element
per sal-remote.yang specification

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

Bug 3934: Websockets: Scope ONE doesn't work correctly

When processing an event which contains a capture of changes in a data subtree,
path and operation are captured to DOM Element, but data itself is ignored.

Change-Id: I11e6a5be5f5f4a025cf3edeb181140c573227a94
Signed-off-by: Martin Ciglan <mciglan@cisco.com>
(cherry picked from commit 898916bcccbae02956efa6175118c7521080a2d2)

Add more context when data parsing fails

This adds information about the XML element which failed to parse. Also
cleans up some eclipse warnings.

Change-Id: I8accc396bc166de26a9a1efe3678e648ac17b28e
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Merge "BUG 5014 - init topology for config datastore" into stable/beryllium

Ignore netconf tools from sonar report

No need to include these.

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

Topology node writer UnitTests

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

Merge "Bug 4577 Allow specification of a distinct schema cache directory per netconf device" into stable/beryllium

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

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

BUG 5014 - init topology for config datastore

this adds initialization for both clustered and non-clustered version
of the feature.

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

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

A leaf is added to odl-sal-netconf-connector-cfg.yang to allow specification of
a schema cache directory relative to the "cache" directory.  The leaf defaults to
"schema", so the default directory for loaded yang files is "cache/schema".  It
is useful to specify a distinct cache directory per netconf mount to avoid potential
model conflicts caused by revisionless import statements.

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

Stop failing netconf connection on revision mismatch

When there's a mismatch between model revision and what the device
reports we should omit this model and not fail the whole connection.

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

Bump scala version to match version bump in controller.

Should only be merged after the version bump in controller.

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

Merge "BUG 4968: Clear transaction after proxy tx is submitted." into stable/beryllium

Merge "Fix netconf-config-connector NetconfMappingTest" into stable/beryllium

Merge "Init topology-netconf when clustered topology starts" into stable/beryllium

Fix netconf-config-connector NetconfMappingTest

Since commit operation does not fail when no transaction is
present, remove test expecting exception in such case.

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

Bug 4455 - Inconsistent COMMIT operation handling when no transactions are present

Return positive response for commit operation in netconf mdsal
northbound even if no candidate transaction is open for session.

Change-Id: I2483a1e4da533566ce318bab0858af2a4fb3c6d3
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>
(cherry picked from commit 5a7a791d64aa5de004dd526c2cb6331fc0d58522)

BUG 4968: Clear transaction after proxy tx is submitted.

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

Init topology-netconf when clustered topology starts

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

BUG 4930: Make keepalive executor definition actually non-mandatory

Yangtools started enforcing mandatory statements and keepalive
executor was not actually non-mandatory since name still inherited
mandatory true from the config model.

While we're here cleanup netconf connector module from leftovers
of the old topology-netconf connector design.

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

Update .gitreview for stable/beryllium

Change-Id: Ifc8f47f76f33348c54c73686982f1719827cdf98
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix setting connection timeout milliseconds parameter

Populate connection timeout milliseconds parameter in netconf topology with
corresponding value from netconf node.

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

Bug 4129 - available capabilities are not updated after sideload

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

Merge "BUG 4547: Make sure all channel writes are from a netty thread."

BUG-2062: use newly-introduced ordered leaf-list methods

This adds explicit handling of ordered nodes, so that the ordering is
really retained.

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

Merge "Fix yang source provisioning from MD-SAL netconf northbound"

BUG 4547: Make sure all channel writes are from a netty thread.

When we were in restconf thread some writes were executed from jetty thread,
we need to make sure all channel writes are coming from netty thread to
preserve message ordering.

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

Merge "Add logging for RPC calls."

Merge "Cleanup features a bit"

Add logging for RPC calls.

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

Fix yang source provisioning from MD-SAL netconf northbound

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

Merge "Revert "BUG 2594 - PUT method returns wrong status on create""

Revert "BUG 2594 - PUT method returns wrong status on create"

This reverts commit 93816f7aeacd89785f083a82036bf6a7d90c316a.

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

Merge "Make mdsal datastore persist through different sessions"

Make mdsal datastore persist through different sessions

Each device needs cached set of datastore operations.

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

BUG-4709 Fix first keepalive execution

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

Cleanup features a bit

Use artifacts imports in pom.xmls (partial conversion) and use odl-netty
feature.

Change-Id: Id50719e7e1b84fcd7b9048d3fb7d6476793b12ce
Signed-off-by: Robert Varga <robert.varga@pantheon.sk>

Merge "BUG 3015 - restconf: Un-helpful error message presented when no service provider is there to handle the RPC call."

Merge "BUG 2594 - PUT method returns wrong status on create"

Merge "BUG 3274 - Generate multiple configs for same port"

Merge "Bug 4709 - Add keepalive response timeout to KeepaliveSalFacade of netconf connector."

Merge changes from topic 'netconf-clustering'

* changes:
  Move mount point registration after schema resolution
  Cluster schema resolution pipeline
  Clustered sources resolution
  Change onNodeUpdated to first cleanup previous state
  Use normal identify messages first
  Use lock in topology node writer
  Prevent NPE's on failures
  RemoteDeviceDataBroker proxy

Merge "Fix empty revision handling in NETCONF capability"

Merge "Bug 4846 odl-mdsal-apidocs swagger UI uses deprecated TokenAuthFilter"

Remove unused jersey properties

Jersey dependencies are inherited from odlparent.  This
removes some now unused jersey properties.

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

Bug 4846 odl-mdsal-apidocs swagger UI uses deprecated TokenAuthFilter

Switches from TokenAuthFilter, a ContainerRequestFilter, to AAAFilter, a
javax.servlet.Filter.  This allows use of Shiro Realms including LDAP.

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

Bug 4709 - Add keepalive response timeout to KeepaliveSalFacade of netconf connector.

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

BUG 3274 - Generate multiple configs for same port

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

Merge "Formatted code for Bug-3890"

BUG 3015 - restconf: Un-helpful error message presented when no service
provider is there to handle the RPC call.

* added catch in checkRpcResponse when cause of error is unimplemented
rpc service

port of :
https://git.opendaylight.org/gerrit/#/c/23277/2

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

BUG 2594 - PUT method returns wrong status on create

Port of:
https://git.opendaylight.org/gerrit/#/c/19009/

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

Move mount point registration after schema resolution

Also a couple other minor bugfixes.

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

Cluster schema resolution pipeline

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

Clustered sources resolution

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

Pull in com.typesafe:config from odlparent

Change-Id: I42b74d58b3e522c141ede246911b35e59f6ce734
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Change onNodeUpdated to first cleanup previous state

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