Merge "Move sal-remote to sal-rest-connector"

Merge "netconf-api simple unit test added"

Merge "mdsal-notification simple unit test added"

Merge "sal-netconf-connector unit tests added"

Merge "Bug 6681: Fix md-sal netconf edit-config behavior"

Merge "Remove use of controller/checkstyle.xml (use odl-parent's new Checkstyle)"

Bug 6681: Fix md-sal netconf edit-config behavior

Check for existence of mixin map node. If it doesn't exist,
create it. Only then write map entry.

Before this patch if user tried to create map entry,
but parent mixin map node didn't exist, edit config failed.

Change-Id: I4004a66100eb4e0d0918005c54b06fdebc8ea6dd
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>

sal-netconf-connector unit tests added

Change-Id: If01df83d92eb38106a634ea6c246fc76b6999d2d
Signed-off-by: Rudolf Brisuda <rudolf.brisuda@pantheon.tech>

Merge "yanglib unit tests added"

Move sal-remote to sal-rest-connector

This module is only used by sal-rest-connnector bundle,
moreover, it is intended to create notification stream
and/or register a data change event which is done using
RESTCONF, thus it make sense to move it to the appropriate
project.
I beleive this is a leftover of the migration that happened
when controller was splitted out..

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

Merge "netconf-notifications-impl unit tests added"

config-netconf-connector unit test added

Change-Id: I871d24097801f2bbaff9c984710ea8636d80f609
Signed-off-by: Rudolf Brisuda <rbrisuda@cisco.com>

netconf-notifications-impl unit tests added

Change-Id: I2a73ca3962cfa663d4dfa7dd2035a878f090d54f
Signed-off-by: Rudolf Brisuda <rudolf.brisuda@pantheon.tech>

yanglib unit tests added

Change-Id: I27054db37b52c1ef5a0fc8e79777e78b84932aae
Signed-off-by: Rudolf Brisuda <rbrisuda@cisco.com>

mdsal-notification simple unit test added

Change-Id: Iabf4f59f80ab28871c2fa340a24eb0954dc8fa99
Signed-off-by: Rudolf Brisuda <rbrisuda@cisco.com>

Merge "Address TODOs: use NetconfRpcFutureCallback for RPC callback"

Add Unit test for bug fix 5998 and 5997

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

Bug 5997 - Fix put and post body examples

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

netconf-api simple unit test added

Change-Id: I2bd0e90a1590b65a88bdb3629c1737066fb8bdb8
Signed-off-by: Rudolf Brisuda <rudolf.brisuda@pantheon.tech>

Merge "Bug 5998 - fix operations request example body"

Merge "Add final where needed"

Merge "Clean-up mdsal-netconf-connector"

Merge "Clean-up config-netconf-connector"

Address TODOs: use NetconfRpcFutureCallback for RPC callback

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

Clean-up mdsal-netconf-connector

- add FIXME to address duplicated code
- remove unsed API
- remove iner generic type
- use lambda when possible
- reduce visibility of param when  possible

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

Clean-up config-netconf-connector

- add final and/or this when possible
- remove infer generic type
- use method reference

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

Clean-up models/* pom files

- Remove unecessary dependencies

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

Bug 5998 - fix operations request example body

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

Add final where needed

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

Merge "Fix broken ApiDocGeneratorTest"

Fix broken ApiDocGeneratorTest

Patch https://git.opendaylight.org/gerrit/#/c/43500/ changed
the way constraints are handled in yangtools which causes
the faiure of EdgeTest in ApiDocGeneratorTest.

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

Merge "Clean-up mdsal-netconf-notification"

Merge "Add missing copyright to pom files"

Merge "Do not loose cause when wrapping DocumentedException in NetconfDocumentedException"

Merge "mdsal-netconf-notification unit tests added"

Bug 6664 - upgrade draft15 to draft16 - change media types

  * for XML - yang-data -> yang-data-xml

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

Bug 6664 - upgrade draft15 to draft16 - renaming

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

Do not loose cause when wrapping DocumentedException in NetconfDocumentedException

DocumentedException has a constructor allowing to define the Throwable/Exception,
but the wrapping done in NetconfDocumentedException is loosing this information.
This patch makes sure we propagate the exception correctly.

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

Add missing copyright to pom files

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

Clean-up mdsal-netconf-notification

- remove infer generic type

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

Remove use of controller/checkstyle.xml (use odl-parent's new Checkstyle)

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

Wrap throwable in NetconfDocumentedException for AbstractWriteTx

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

Merge "Unit test for RestconfDataServiceImpl class"

Merge "Unit test for RestconfInvokeOperationsServiceImpl class"

mdsal-netconf-notification unit tests added

Change-Id: Ib6a0ced3975ca72c0b8671c5d51bd005c8cc1a06
Signed-off-by: Rudolf Brisuda <rudolf.brisuda@pantheon.tech>

Merge "Unit test for RestconfStreamsSubscriptionServiceImpl class"

Unit test for RestconfStreamsSubscriptionServiceImpl class

Remove redundant check of empty or null String. This will never
happen since it would fail while getting LogicalDatastoreType
or DataChangeScope.

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

Unit test for RestconfInvokeOperationsServiceImpl class

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

Unit test for RestconfDataServiceImpl class

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

Merge "BUG-5488: Decrease buffer size to prevent memory exhaustion"

BUG-5488: Decrease buffer size to prevent memory exhaustion

When netconf device is connected using ssh, AsyncSshHandlerReader
allocates buffer for each received notification. Under high load,
it is possible, that notifications are received faster than consumed.
In this case, pending notifications tasks are queued
in netty executors queues. It could cause memory exhaustion.

Preallocated buffer size was 8192 B. This patch decreases it to
2048 B, as Apache Mina resizes buffer if it is needed.

Bellow are test results of receiving 100000 notifications as described by
https://wiki.opendaylight.org/view/NETCONF:Testing#NETCONF_southbound_performance_test
using the largest notification - i2rs-notifs-perf100k10prefixes.xml.

Time needed to handle notifications is same regardless of BUFFER_SIZE value,
but max memory needed is lower, if BUFFER_SIZE value is lower.

BUFFER_SIZE = 128,  max heap memory = 1.1 GB  avg = 59640 ms
BUFFER_SIZE = 2048, max heap memory = 1.3 GB  avg = 58450 ms
BUFFER_SIZE = 8192, max heap memory = 2   GB  avg = 59207 ms

Change-Id: I7be5e50aa72e04f5df6fd96988bd588d3c369b01
Signed-off-by: Andrej Mak <andrej.mak@pantheon.tech>

Unit test for PostDataTransactionUtil class

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

Unit test for PutDataTransatcionUtil class

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

Add test for CreateStreamUtil class

Redundant condition removed from CreateStreamUtil class

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

Unit test for RestconfInvokeOperationsUtil class

Fixed so it add rpcError to the result when there is one.

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

Unit test for ReadDataTransactionUtil class

Fixed bug with merging list data. Also delete and change some
redundant data and function calling.

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

Bug 6429 - Fix notification xml output according to RFC

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

Merge "Bug 6343 - Incorrect handling of configuration failures in SAL netconf connector"

Bug 6343 - Incorrect handling of configuration failures in SAL netconf connector

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

Bug 6594 - Unsupported leaf in identityref in leafref

  * fixed incorrect checking of variable

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

Use a methodhandle to extract EnumPair name

This a more efficient replacement for a anonymous filter.

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

Bug 6485 - unstable mount point

  * swap result of operation with CountDownLatch.countDown()

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

Merge "Bug 5615 - Netconf connector update overwriting existing topology data"

Merge "Remove jersey-server artifact"

Remove jersey-server artifact

This artifact prevents correct debuging in Idea or Eclipse

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

Bug 6457 - Delete Cars configuration from Follower produces HTTP 500 Error

  * change InterruptedException with Exception
  * Ignore unstable tests in RestPutOperationTest + TODO

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

Add Gson bundle to odl-netconf-connector feature

This explicitly adds Gson bundle to odl-netconf-connector feature. Gson
bundle needs to be resolved and installed, because sal-netconf-connector
bundle depeneds on it.

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

Bug 5615 - Netconf connector update overwriting existing topology data

- putting only augmented node 'NetconfNode' to operational datastore
- updated tests

Change-Id: I3153613c9041c36f558e73128d40dd75dece0293
Signed-off-by: Rudolf Brisuda <rudolf.brisuda@pantheon.tech>

Merge changes I20578f97,I629c909b,I54f04c99,I9c1e3a28,Ie681c8c5

* changes:
  Bug 6325 - Fix for draft15 update
  Bug 6325 - upgrade draft11 to draft15 - added timestamp & etag
  Bug 6325 - upgrade draft11 to draft15 - change get modules
  Bug 6325 - upgrade draft11 to draft15 - change media types
  Bug 6325 - upgrade draft11 to draft15 - renaming

Merge "Bug 6278: Switch to use odlparent's karaf-parent"

Bug 6325 - Fix for draft15 update

- transaction chain must be updated
in transaction handler

- adapt utility methods to reset
transaction chain

Change-Id: I20578f97c9a8d002c0ebcc6b8a89a152c1e1c47a
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 6325 - upgrade draft11 to draft15 - added timestamp & etag

  *fixed read of data from datastore

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

Bug 6325 - upgrade draft11 to draft15 - change get modules

  *change modules to modules-state
  *chagne modules/module to modules-state/module

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

Bug 6325 - upgrade draft11 to draft15 - change media types

  * remove media types(for +json and +xml in all cases):
    * yang.api
    * yang.datastore
    * yang.data
    * yang.operation
    * yang.errors
  * add new media type:
    * for xml - yang-data
    * for json - yang-data+json

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

Bug 6325 - upgrade draft11 to draft15 - renaming

  *renaming of all occurencies of "draft11" to "draft15"

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

BUG-865: remove String-based getDataChildByName()

Some classes still implement String-based getDataChildByName method.
Get rid of this method implementations.

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

Bug 5553 - Impl get operations

  *package protected impl of container, leaf and module
  *fix for new restconf part too

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

Merge "Bug 6251 - Exclude flawed models from mount point"

Bug 6278: Switch to use odlparent's karaf-parent

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

Bump OpenExi to 2.62.0

This bumps the version to the latest released one.

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

Merge "BUG-865: remove String-based getDataChildByName()"

Bug 6251 - Exclude flawed models from mount point

Flawed models that cannot be parsed by Yangtools are ignored when
building schema context for mountpoint.

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

Merge "Bug 6198 - Use sal-netconf-connector to connet device costs too much time"

BUG-865: remove String-based getDataChildByName()

Replace usage of deprecated String-based getDataChildByName() method
with QName-based one.

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

Bump versions by 0.1.0 for next dev cycle

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

Bug 6358 - Fix merge parents

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

Merge "Bug 6037 - Check if delete request was successful"

Merge "Yang HTTP PATCH for Restconf11"

Bug 6037 - Check if delete request was successful

- adaptation for Restconf draft 11+
- checking if data to delete exists, if not return error 404
- remove not needed method from POST utils
and use method from common transaction utils
- added unit tests

Change-Id: I133c628a2e41f12d1db58fd06ff6c073ebfc6ece
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Yang HTTP PATCH for Restconf11

- support for restconf draft11-15 changes
- support for xml and json media types
- applied fixes for:
- Bug 5897 - PATCH merge operation does nothing
- Bug 5909 - PATCH does not report 409 on
OptimisticLockFailedException
- Bug 5730 - Delete subset of list items using PATCH?
- Bug 5898 - PATCH success "ok" field has wrong JSON value
- Bug 5915 - PATCH with "target":"/" error
- Bug 5509 - HTTP Patch in Restconf doesn't support general
absolute or relative target xpath
- contains unit tests
- Draft02 classes marked as deprecated
- adapt PATCH writers to support error path

Change-Id: I981dc8cbead3c5a7dc06b7b807c17a40a9cfe7b3
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Bug 6037 - Check if delete request was successful

- solution for old restconf
- checking if data to delete exists, if not return error 404
- use callbacks for transactions

Change-Id: I0f4c11cb3dd9c7fc560a4fee06df0d398c0c4f61
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Bug 2594 - PUT method returns wrong status for create resource

  *remove get() and checkedGet() on future objects, add callback
  *fix tests

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

Merge changes I426a2bc1,I5440b866

* changes:
  Bug 6291 - Fix bugs of new Restconf 11 implementation
  Bug 5909 - PATCH does not report 409 on OptimisticLockFailedException

Bug 6322 - Boron: odl-netconf-clustered-topology still depends on distributed-entity-ownership-service

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

Bug 6291 - Fix bugs of new Restconf 11 implementation

- first identifier does not start with slash
when deserialization is invoked
- adapted unit tests
- return error 404 when data missing with GET
operation
- return config data when state data does
not exists with GET operation
- return state data when config data does
not exists with GET operation
- identifier can be null and it is deserialized to
YangInstanceIdentifier.EMPTY

Change-Id: I426a2bc1cb0e908dc189553e464f405ef4ce1457
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Bug 5909 - PATCH does not report 409 on OptimisticLockFailedException

- wait for transaction to be finished using callback

Change-Id: I5440b866eada07ee6540c6b93d2e9174e62a37ce
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Bug 5897 - PATCH merge operation does nothing

- added merge patch operation processing

Change-Id: Ic3f5908c5f016a0680779ed12b4bc26b7b840d9a
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Bug 5915 - PATCH with "target":"/" error

- fixed bug + added unit test (example of usage)
for json and xml
- improved solution of bug5730

Change-Id: I0d8042ffdced97918ff936817513fc30163be577
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Bug 5509 - HTTP Patch in Restconf doesn't support general absolute or relative target xpath

- yang patch for xml media now supports relative or absolute xpaths
- only create, merge, replace and insert operations require value to be present
- if value is not present with create, merge, replace or insert operation
then error 400 is returned
- if value is used with other operations then error 400 is returned
- deleting sublist values instead of full parent
- support for multiple values
- added unit tests

Change-Id: I9828ceb7368760642b22b3a336884b949f0edfcc
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Bug 5509 - HTTP Patch in Restconf doesn't support general absolute or relative target xpath

- dependencies shared between XML and JSON PATCH implementations

Change-Id: Ia649763379fab03019ddfd16eac564da576cb35b
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>