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

Fix empty revision handling in NETCONF capability

If a capability exposed by a netconf server contains revision parameter
but has no value there e.g.
<capability>some-namespace=some-module&amp;revision=</capability>

connection would fail with NPE in netconf connector.

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

Merge "Fix non ietf-netconf* mounted RPC invocation from Restconf"

Use normal identify messages first

Prevents crashing akka if bundles containing our messages
weren't loaded on remote node yet.
Add resyncs on remote node's coming up.

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

Merge "Make test tool work with schemas with no revision"

Merge "Make test tool expose submodule sources"

Fix non ietf-netconf* mounted RPC invocation from Restconf

Invoked RPCs from modules other than iet-netconf* were executed on ODL itself
instead of a mountpoint regardless of whether they were aimed at a mountpoint
or not.

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

Add copyright headers to sal-rest-connector

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

Formatted code for Bug-3890

Change-Id: I7b6c481104ec4445c034cc5a9e560a641e7df93c
Signed-off-by: Shivani Pathak <shivani.pathak@tcs.com>

Use lock in topology node writer

Prevents race conditions from akka callbacks.
Prevent NPE's when deleting node that's not connected.

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

Prevent NPE's on failures

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

RemoteDeviceDataBroker proxy

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

Bug 4773 Package uses conflict between authn and restconf

Add in the javax.annotation dependencies, and make the bundles private
so there is no package uses conflict.

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

Merge "Bug 3896 - direct usage of == instead of .equals"

Bug 3896 - direct usage of == instead of .equals

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

Merge "Formatted code Bug-3894."

Merge "Remove DelegatingNormalizedNodeStreamWriter"

Merge changes I24adce35,Ia5947ae6

* changes:
  Revert "Fix shorthand case statement in netconf notifications model"
  Revert "Better FIXME comment in ietf-netconf-notifications model"

Merge "Bug 4700 - RestPerfClient spits NumberFormatException and hangs"

Merge "Remove unnecessary uses of odlparent properties"

Merge "Bug 4719 Shiro integration into restconf"

Formatted code Bug-3894.

Change-Id: I9cc580133250c12592e8eb2d64629c54fc3d78f0
Signed-off-by: Shivani Pathak <shivani.pathak@tcs.com>

Bug 4719 Shiro integration into restconf

Switches from TokenAuthFilter, a ContainerRequestFilter, to AAAFilter, a
javax.servlet.Filter.  This allows use of Shiro Realms including LDAP.  In
order to run restconf without AAA, you can can still utilize the
odl-restconf-noauth feature.  AAAFilter is disabled by default, and only
enabled when the odl-shiro-act bundle is activated.

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

Merge "Bug 4712 - Fix Keepalive task to check the response of a keepalive request"

Merge "Fix missing NetconfNotificationCollector service"

Fix missing NetconfNotificationCollector service

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

Remove unnecessary uses of odlparent properties

${bouncycastle.version} and ${jersey.version} are no longer
necessary.

This depends on Ifb388ce08e1870b50ce720061550e72137bce798 in
oldparent.

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

Bug 4725 - RestPerfClient: Add support for authentication.

Add HTTP basic authentication support to RestPerfClient.

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

Bug 4700 - RestPerfClient spits NumberFormatException and hangs

Validate destination argument and exit if destination does not start with '/'.

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

Bug 4712 - Fix Keepalive task to check the response of a keepalive request

If response is null, tear down the session.

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

Make test tool work with schemas with no revision

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

Make test tool expose submodule sources

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

Revert "Fix shorthand case statement in netconf notifications model"

This reverts commit da1abe40ffbc95f53149797f7aff46a4510b5db3.

Since BUG 4414 is fixed, use shorthand version of case statement

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

Revert "Better FIXME comment in ietf-netconf-notifications model"

This reverts commit 9940f0ef33ebc84294f278423d33b031c3da1c72.

Since BUG 4414 is fixed, use shorthand version of case statement

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

Remove DelegatingNormalizedNodeStreamWriter

We already have ForwardingNormalizedNodeStreamWriter
in yangtools.

Needs to be merged after:
https://git.opendaylight.org/gerrit/#/c/30914/

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

Fix changed version

Inadvertently changed version while cleaning up features.
This reverts it.

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