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>

Bug 5898 - PATCH success "ok" field has wrong JSON value

- empty value is represented as an array with value null

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

Bug 5730 - Delete subset of list items using PATCH?

- fix: edit is not ignored when end of input is reached
- fix: only create, merge, replace and insert operations require value to be present
- fix: if value is not present with create, merge, replace or insert operation then error 400 is returned
- fix: if value is used with other operations error 400 is returned
- fix: deleting sublist values instead of full parent
- fix: order of patch edit fields is not important anymore (target, value)
- move class StringModuleInstanceIdentifierCodec to own file

Change-Id: I4c3f407858034b49795034da6c93b41f9d891b62
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

Absolute or relative input URI and PATCH target JSON support

Change-Id: I86821e59581968cef5f11ec34ce98757909f7c36
Signed-off-by: Martin Ciglan <mciglan@cisco.com>
Signed-off-by: Ivan Hrasko <ihrasko@cisco.com>

Revert "Bump apache mina to 1.2.0"

This reverts commit 3911a32461cf5817787c807cad676d4164f0cbd4.

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

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

Filter missing sources Identifiers before createSchemaContext in class RecursiveSchemaSetup.

Change-Id: Ic41d4803df0807d799a1e80ceb6639877f006e42
Signed-off-by: Xu Rong <xu.rong4@zte.com.cn>

Bug 3959 - support netconf notification

  *added dependency for converting JSON to XML
  *created new notification listener adapter extended
   by DOMNotificationListener
  *added augment for sal-remote to add new param
   for input of create-notification-stream rpc to
   make a choice between JSON or XML output of
   notification (not mandatory - default is XML)
  *added unit tests

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

Merge "Bump apache mina to 1.2.0"

Add missing license headers

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

Bump apache mina to 1.2.0

Bumps the dependency from pre-release to the current
release. Also adjusts implementation to match changed APIs.

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

Merge "Switch to using StandardCharsets"

Switch to using StandardCharsets

Guava's Charsets should not be used in recent Java
environments.

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

Use upgraded ietf-{inet,yang}-types in test-tool

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

Merge "Organize Import to conform to Checkstyle guidelines."

Merge "Bug 5682 - MDSAL netconf northbound yang library"

Bug 5682 - MDSAL netconf northbound yang library

This adds support for ietf-yang-library for MDSAL netconf northbound
server

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

Bug 6154 - Connection to config subsystem netconf endpoint refused

Makes config subsystem SSH server listen on address 0.0.0.0 instead of
127.0.0.1.

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

Adjust tests to new ietf-inet-types

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

Organize Import to conform to Checkstyle guidelines.

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

Upgrade ietf-{inet,yang}-types to 2013-07-15

Change-Id: I632370c6e9b078817c69a19b3f7c1c17846ce249
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Bug 5526 - Testing - RestconfModulesService

Unit tests for RestconfModulesServiceImpl + repackaging of tests

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

Bug 5526 - Testing - Utils - Mapping

Unit test for RestconfMappingNodeUtil

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

Bug 5528 - Connecting RESTful part to restconf + fix bugs

  *new wrapper for transaction services
  *wrapper for transaction and base services
  *fix bugs
    *fix serializer of YII for create right prefix of namespace

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

Bug 5528 - Subscribing to stream impl

  *preparing dom data broker handler

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

Bug 5528 - Create stream for DS

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

Bug 5528 - Streams service impl - prepare impl by interface

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

Bug 5528 - Invoke RPC impl

  *create RpcServiceHandler for handling service of rpc
  *create factory of rpc result

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

Bug 5528 - RPC invoke impl - prepare impl of interface

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

Bug 5528 - Delete data impl

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

Bug 5528 - Impl Post data

  *added new constant for post data
  *added location to response factory
  *moved ensureParentsByMerge to common util class of Put and Post

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

Bug 5528 - Put data impl

  *make FutureCallbackTx more generaly
  *factory for NormalizedNode(READ) and Response(PUT) from Future
  *fix ReadDataTransactionUtil after remake of FutureCallbackTx

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

Bug 5528 - Read data impl

  *impl RestconfDataService
    *util class for constants
  *wrapper for transaction vars

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

Bug 5528 - Handlers

  *make handlers immutable
    *DOMMountPointServiceHandler
    *SchemaContextHandler
    *TransactionChainHandler
  *common interface for all handlers
  *fix dependencies of these changes

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

Bug 5528 - Preparing enviroment for impl of restful services

  * repackeging
  * fixing changes

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

Bug 5403 - Support yang-library schema resolution

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

Merge "Bug 5526 - Unit testing"

Merge ".gitignore .factorypath created by m2e-apt"

Bug 5526 - Unit testing

Fix for streams service unit testing. Prepare test streams
before tests are run and clean test streams after tests completed.

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

Bug 5528 - API for RESTful - transaction part of CRUD

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

Merge "Add simulated discard-changes to testtool default datastore"

Merge "Stop using blocking invoke for lock and editConfig."

Add simulated discard-changes to testtool default datastore

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

Merge "Propagate reconnectOnSchemasChanged flag to ClusteredNetconfDevice"

Merge "Bug 5708 - Schemaless mount point notifications"

Merge changes Ife737bb3,Ifa1e8774

* changes:
  Bug 5527: Unit testing - serializer
  Bug 5527: Unit testing - codec

Merge "Bug 5527: Unit testing - deserializer"

Merge "Bug 5526 - Testing - Utils - Schema util"

Bug 5526 - Testing - Utils - Parser identifier

Unit tests for ParserIdentifier
+ added check if identifier does not contain multiple slashes

Change-Id: I1d4e446be60133cb5fa432a49fb88cb59f85702c
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bug 5526 - Testing - Utils - Schema util

Unit tests for RestconfSchemaUtil

Change-Id: I375f9fe9c3845c63cee261ed37fbf3cd253e27e1
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

.gitignore .factorypath created by m2e-apt

https://github.com/vorburger/opendaylight-eclipse-setup now bundles
https://github.com/jbosstools/m2e-apt, which creates .factorypath files
all over the place.

Also .checkstyle created by Eclipse Checkstyle plugin.

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

Stop using blocking invoke for lock and editConfig.

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

Bug 5708 - Schemaless mount point notifications

Add support for notifications for schemaless mountpoints.
SchemalessMessageTransformer treats notification's payload as anyxml
node. To conform to API transformer wraps this payload in container node
with notification type as node identifier.

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

Merge "BUG 5634 : Implement concurrent message limit"

Bug 5527: Unit testing - deserializer

Unit tests for YangInstanceIdentifierDeserializer

Change-Id: Ia09e2393fa0e29e31f8c116fe6db14b1736a96bb
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Propagate reconnectOnSchemasChanged flag to ClusteredNetconfDevice

Currently, the "reconnectOnSchemasChanged" is not propagated to
ClusteredNetconfDevice.  This change propagates the value so it
may be used in setting up the NetconfDevice.

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

Merge "Remove unused isMaster variable in ClusteredNetconfDevice"

BUG 5634 : Implement concurrent message limit

This adds optional feature of limiting maximum concurrently sent rpc
requests. When we reach limit, next request will fail. Queue is freed
when reply message is received.

Change-Id: I5c49206ab929300ac973b12d5ccee58c70536c41
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Bug 5527: Unit testing - serializer

Unit tests for YangInstanceIdentifierSerializer

Change-Id: Ife737bb38f43e7855a9b8657fa56bd49adf1be18
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Bug 5527: Unit testing - codec

Unit tests for IdentifierCodec mostly according to examples in restconf specification

Change-Id: Ifa1e8774a69b6f24521cc8a73b8df9720980b43f
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Merge "BUG-865: Do not use QName.cachedReference()"