Bug 8533: Not possible to invoke RPC on mount points with new Restconf

- put identifer of RPC into InstanceIdentiferContext
instead of mount point identifer which leads to fail
in unit test

- when working with mount point use mount point's
schema context to find RPC module, previously used
controller's schema context leads to NPE

- repaired imports in RestconfApplication to use
new implementation

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

Merge "Bug 8289 - 409 in cluster restperfclient test" into stable/boron

Merge "Bug 8405: Add close check to NetconfDevice" into stable/boron

BUG-8085: create missing parent augmentation node

Augmentation nodes do not exist in serialized form
(e.g. in edit-config message), but are required by DataTree-based
DOMDataBroker implementations, so should be created if data from
augment is present.

This patch creates missing augment nodes by issuing merge on augmentation
before put operation (based on current behaviour for ListSchemaNodes).

Change-Id: If657ae96e914fc46617099042a833c7d4d5883b7
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>

BUG 8566 direct writes to ordered list fail

Incorrect handling of direct writes into a mapEntry which tries
to merge an empty parent list instead of an exists check but doesnt
correctly handle the possibility of an OrderedMapNode as a parent.

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

Bug 8455: Yang Patch response is not having the error details

- add error-info to error message

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

Bug 8289 - 409 in cluster restperfclient test

To ensure message ordering we need ReadWriteTransactionActor. This
way we are sure that all reads and writes will be executed in
the order that they are sent.

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

Bug 8405: Add close check to NetconfDevice

Since schema resolution runs in its own thread, it is possible,
that handleSalInitializationSuccess is called when
NetconfDeviceCommunicator was closed meanwhile. Add check to
prevent this.

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

Bug 8311 - Apidoc: Incomprehensible 500 id model is wrong
Bug 8266 - Apidoc explorer is broken after installing Boron SR3

Depending on path if it is relative or absolute module has to be
resolved different ways. If it is relative path it will have to
be resolved through DataSchemaNote which returns namespace and
revision of module in which node is situated. If it is absolute
path we need TypeDefinition which returns a namespace and
revision of original module.

Change-Id: I522ea66383ba1d65e6674d5e8ff67ec31e019303
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
(cherry picked from commit 5ee44740e99f60768f488b7c8fb00873ed758a79)

Merge "Bug 8084 - FilterContentValidator.getKeyValues creates invalid YII key values" into stable/boron

Bug 8197: Deregister schema sources on actor stop

NetconfNodeActor registers schema source provider
to schema registry. When mountpoint is removed, this
registration should be removed too. If it isn't,
following issue can happen:

1. Create mountpoint for device1
2. Master actor for device1 is registered as mod-1.yang provider
3. Delete device1
4. Create mountpoint for device2
5. Master actor for device2 is registered as mod-1.yang provider
6. Register slave -
   schemaContextFactory.createSchemaContext(sourceIdentifiers)
   is called
7. Since dead device1 master is still registered as provider,
   ask in ProxyYangTextSourceProvider timeouts
8. After timeout device2 master is queried
9. Device 2 slave mountpoint registered

This delays slave mountpoint registration.

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

Remove references to controller's sal-remote

This model has been migrated here, so all references should
be removed.

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

Refactor netconf clustered topology tests

- split long tests into smaller
- remove duplicated code

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

Merge "Bug 8152: Add way to configure idle timeout" into stable/boron

Bug 8152: Add way to configure idle timeout

Clustered apps should be configurable via datastore
to allow config distribution to all nodes according to
https://wiki.opendaylight.org/view/Using_Blueprint#Using_the_Datastore
Add such configuration for clustered topology.

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

Do not include duplicate models

Adding same module into the CrossSourceStatementReactor guarantees
namespace conflicts, which are flushed out by BUG-7424 handling.

Make sure we do not add same modules multiple times.

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

Bug 8152: Transaction is already opened

This issue happens, when for some reason transaction
submit or cancel message isn't delivered to master
node. With current implementation, only one device
transaction can be opened at the time, so submit or
cancel delivery failure will lock device forever.

To prevent this, this patch introduces write trancaction
idle timeout. Write transaction actor will be stopped
and its device transaction cancelled, when no message
is received for given time. Cancellation unlocks device,
so mountpouint is usable again.

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

Change handling of netconf cluster transactions

Own actor is created on master for every transaction.
Proxy transactions use this actor for mount point access.
Creation of proxy transaction initiates sending of message
to the master node actor. Master node actor starts
transaciton actor if needed.
Both master and slaves communicate with device via
proxy transactions.

For write transaction, submit and cancel cause actor stop.
There can be single instance of read actor, which can be
reused for all transactions.

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

Close read-only transactions

As noted in API, read-only transactions are resources and must
be closed just as any other external resource.

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

BUG-7868: perform checks before starting modifications

The codepath for makeNormalPost() performs sub-optimal emulation
of create().

This patch changes the logic to first check for presence and then
perform modifications. This saves cycles as continuous modify/read
cycles incur heavy penalty on DataTree and also allows batching
of modification requests.

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

Bug 8115: Change URI decoding from ISO-8859-1 to UTF-8

Change-Id: I3bcc7cae197aa267a43c8598f54050e987712c9a
Signed-off-by: lbeles <lbeles@frinx.io>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

Merge "Bug: 8037 YANG Patch using "replace" instead of "merge"" into stable/boron

Merge "Bug 5581: Optimize subtree filtering" into stable/boron

Merge "Bug 7812: NPE when NetconfDeviceSalProvider.close" into stable/boron

Bug 8084 - FilterContentValidator.getKeyValues creates invalid YII key values

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

Bug 5581: Optimize subtree filtering

If single list entry with key is specified in filter,
only that specific entry is read from datastore, thus
filtering has smaller input to process.

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

Bug 8072: Fix decoding of URLs with external identityref

Mount point schema (rather than global) should be used
to decode identitirefs defined in external modules.

Change-Id: I51f135fe0cd5ceee39d5a2e6f0fc62631cf4437b
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>

Fix LibraryModulesSchemasTest failure

LibraryModulesSchemasTest#testCreateInvalidModulesEntries() tests
filtering invalid modules entries during resolving
ietf-netconf-yang-library/modules-state/modules node.

Resolution goes as follows. We parse xml document containing
ietf-netconf-yang-library/modules-state retrieved from the device with
help of DomToNormalizedNodeParserFactory. Then we filter invalid modules
entries from the parsed modules list node. Module entry is considered
invalid if its revision or schema url are invalid.

However, change in yangtools
https://git.opendaylight.org/gerrit/#/c/53553 causes that parsing
invalid revision will fail whole parsing xml process with unchecked
exception.

For now, do not test modules with bad revision. This is intended just as
a temporary solution. We will also need to slightly change our
ietf-netconf-yang-library/modules-state/modules node resolution logic.
Opened BUG 8071 to track that.

Change-Id: If12ae8d7bf94528088aac81e78b24058c5152283
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>
(cherry picked from commit 95452d33026b37907b541dc47006426e3b3e35ad)

Bumping versions by x.y.(z+1) for next dev cycle

Change-Id: Ic71e33faab853911061c97cde8ada0cc5f3efeee
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Bug 7812: NPE when NetconfDeviceSalProvider.close

Clustered connector close logic could be called twice on master.
Add boolean guard field to prevent this.

Add null check to netconf device sal provider close method
to prevent NPE.

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

Bug: 8037 YANG Patch using "replace" instead of "merge"

Change-Id: Ic53b0649b52529e9bf244b82d85582d343aa95e9
Signed-off-by: Wenbo Hu <huwenbo1988@gmail.com>
(cherry picked from commit c82fa3134d0f582f7e49f5b6cfe8e50a02d77f19)

Merge "Bug 7906: Fixed json parsing on patch request" into stable/boron

Merge "Bug 7295 - Incorrect handling of device transactions in clustered setting" into stable/boron

Merge "Clear out tx reference immediately after submit." into stable/boron

Bug 7906: Fixed json parsing on patch request

Currently lists with simple objects (string, int, etc.) are not
supported in the patch request. This commit adds support for that.

Change-Id: I4310a683aa42271b914c5dd8c3349bb2bac9e971
Signed-off-by: David Goldberg <gdavid@hpe.com>

Bug 7987: Json HTTP PATCH: Problem parsing simple leaf value

Change-Id: I8094797fc2a26bd809631d995a29e111320e664e
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
(cherry picked from commit aca66f6d6b8065b88d73bec17822ac6d226d9199)

Fix autorelease fail

- fix regression caused by https://git.opendaylight.org/gerrit/#/c/53297

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

Bug 6856: Rpc definition should implicitly define input/output

Corresponding patch for netconf.

Change-Id: I041a90e222578f43144d42cd72dd2f9457362a4a
Signed-off-by: Igor Foltin <ifoltin@cisco.com>
(cherry picked from commit 23f46e2b1c14be9ee7be61d3285d52cc27cfaefc)

Merge "Bug 7686 - Make notifications defined by yangs automatic loaded for streaming" into stable/boron

Bug 7728 - modules-state conflict kills rest-connector-default-impl

-improved workaround:
reset transaction chain to be able to continue writing another data
after transaction fail

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

Bug 7728 - modules-state conflict kills rest-connector-default-impl

- this is just Carbon workaround for bug
- ignore error when another node is already putting the same data

Change-Id: I091393c27cb4ab8d7db04257ffe0e922d6487843
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
(cherry picked from commit e95dcf13b6b1c442ef9a0fa69533cb2ee6eed38c)

Fix error reporting for PUT/POST

If a datastore failure occurs for PUT or POST, restconf reports the
error message "Problem while PUT operations" which is neither useful
nor grammatically correct.

It used to report the underlying error info however this was broken by
https://git.opendaylight.org/gerrit/#/c/40235/ which changed the code to
use a Future callback with a CountDownLatch but doesn't propagate errors
from the Future. I don't understand why the code was changed in this manner
when it worked fine and was much simpler using checkedGet. I changed the
code back to use checkedGet, in several places.

Change-Id: I2e917d7eedc569702cdf3a54c4aa0321fe229ca1
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>
(cherry picked from commit 0f3e7394ddba30db976ec5b0afe0db64c6186a32)

Bug 7686 - Make notifications defined by yangs automatic loaded
for streaming

  * make notifications define by yangs initialized by restconf
server
  * subscribe to notifications define by yangs by getting of
location specfici stream
  * fix test

Change-Id: Ib2a3787e5d0252b7200190643f848a180260d167
Signed-off-by: Jakub Toth <jatoth@cisco.com>
(cherry picked from commit 159f2a9cfd97531452c174cede9a4f3ab1262935)

Bug 7295 - Incorrect handling of device transactions in clustered setting

Transactions on mounted device in clustered setting are delegated to
leader node and leader node actually configures the device.

Implementation, hovewer, depens only on one transaction at a time.
Delegated calls can interleave together, for example one node can submit
transaction of the other node.

Explicitly open a new tranasction before applying any operations to it.

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

Clear out tx reference immediately after submit.

When sumbmitting multiple transactions after each other
you could get into a state in which a transaction was submitted
and the reference wasn't cleared out in the callback resulting in
attempts to use an already finished transaction.

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

Remove blocking get in read transaction

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

Merge "Bug 5581: Merge empty map instead of exist check" into stable/boron

Bug 5581: Merge empty map instead of exist check

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

Merge changes from topic 'cherry-pick form master to RFC8040' into stable/boron

* changes:
  Clean up Rfc8040
  Bug 7735 - Update restconf models by RFC 8040
  Bug 7429 - Upgrade docgen in restconf to the latest draft of ietf-restconf
  Bug 7206 - Chinese improper code problem.
  Bug 5679 - implement new service RestconfService
  Bug 5679 - add new module ietf-restconf
  Bug 5679 - implement ietf-restconf-monitoring - cleanup
  Bug 5679 - implement ietf-restconf-monitoring - streams
  Bug 5679 - implement ietf-restconf-monitoring - capabilities
  Bug 5679 - prepare constants for ietf-yang-library model
  Bug 5679 - add new module ietf-restconf-monitoring to restconf
  Bug 5679 - fix generating of schema by ietf-yang-library and path of RestconfSchemaService
  Bug 5679 - ietf-yang-library module implemetation
  Bug 5679 - prepare constants for ietf-yang-library model
  Bug 5679 - new module 'models' for new models in restconf
  Bug 5679 - renamed package rest.services to base.services
  Add missing yang - sal-remote
  Do not attempt to construct invalid QNames
  Bug 7231 - Upgrade ietf-restconf draft17 to draft18
  Bug 7207 - Bad parsing of identityref in typedef for last ietf-restconf draft
  Bug 6951 - Implement Query parameters - with-defaults
  Bug 6947 / Bug 6948 - implement point and insert query params
  Bug 4883 - implement query parameter - filter
  Bug 6949 / Bug 6950 - Implementation of start-time and stop-time query parameters
  Bug 6935 - RPC in latest draft doesn't work - problem of parsing to instance identifier
  Bug 6995 - Change event notification subscription usability PART2
  Bug 6995 - Change event notification subscription usability PART1
  Bug 6903 - Implement Query parameters - fields
  Bug 6895 - Implement Query parameters - depth
  Bug 6382 - add apidoc for latest restconf draft
  Bug 6731 - add valid example values
  Bug 6931 - Fix unsupported specific type of leaf
  Use UTF-8 as the standard output
  InstanceIdentifierCodecImpl JUnit test
  Bug 6679 - api explorer creates false examples
  Bug 6272 - support RESTCONF PATCH for mounted NETCONF nodes
  Bug 6746 - Restconf: Not working GET operation on mount points
  Fix close() in provider

Bug 5581: Minor Exi decoder optimization

Initialize DomSource with new document. Otherwise new
document is created by SAX2DOM class, which is more
expensive as it includes creating new
DocumentBuilderFactory etc.

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

Clean up Rfc8040

- use URI.create() as we know the format is correct
- use SimpleDateFormatUtil
- make sure constants are constant
- QName contants should be interned
- throw ExceptionInInitializerError on constant init failure

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

Bug 7735 - Update restconf models by RFC 8040

  * rename base class Draft18 -> Rfc8040

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

Bug 7429 - Upgrade docgen in restconf to the latest draft of ietf-restconf

  *upgrade paths from 17 to 18

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

Bug 7206 - Chinese improper code problem.

Change-Id: Iedad5f4cfce270216811788b217bced62ff88428
Signed-off-by: Geng Xingyuan <geng.xingyuan@zte.com.cn>

Bug 5679 - implement new service RestconfService

  * update constants of RestconfModule in Draft18 by latest draft
    of ietf-restconf
  * implement servic for getting yang library version
    * path - /restconf/18/yang-library-version
  * test for new implementation
  * move tests from ../rest/services/impl to ../base/services/impl

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

Bug 5679 - add new module ietf-restconf

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

Bug 5679 - implement ietf-restconf-monitoring - cleanup

  * cleanup of both listeners (data-change, yang)
    * create new common abstract classes with common methodes
    * fix broken tests
  * add support of listener to listen on stream with both output
    types (XML JSON)

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

Bug 5679 - implement ietf-restconf-monitoring - streams

  *removed service and tests for getting streams
  *implemented put/delete specific stream to operational DS
  *new mapping for data-change and yang notifications
  *new tests + fixed old

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

Bug 5679 - implement ietf-restconf-monitoring - capabilities

  * make constants in QueryParams full (with BASE_URI path)
  * mapping capabilities
  * put mapped capabilities to operational DS in provider
  * added + fixed tests

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

Bug 5679 - prepare constants for ietf-yang-library model

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

Bug 5679 - add new module ietf-restconf-monitoring to restconf

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

Bug 5679 - fix generating of schema by ietf-yang-library
and path of RestconfSchemaService

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

Bug 5679 - ietf-yang-library module implemetation

Make modules data as data in state DS
  * remove modules service
  * remove tests of modules service impl
  * remove mapping modules according to ietf-restconf
    model

  * implemetation of new mapping modules according to
    ietf-yang-library model
  * (SchemaContextHandler) mapping modules on every new
    schema context and update DS(state) with new data
  * added test for mapping
  * fix tests by new implemetation

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

Bug 5679 - prepare constants for ietf-yang-library model

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

Bug 5679 - new module 'models' for new models in restconf

  * added new module to models - ietf-yang-library with
    model ietf-yang-library.yang
  * added dependencies to sal-rest-connector and features
    on ietf-yang-library

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

Bug 5679 - renamed package rest.services to base.services

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

Add missing yang - sal-remote

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

Do not attempt to construct invalid QNames

This fixes an attempt to instantiate a QName with a colon
in its local name. Two of the failing tests are straightforward
fixes.

The RestconfOperationsServiceImpl is more involved, as it requires
faking module imports to keep encoding working.

Change-Id: Iceed2310e94a16189d80fe57b58510fbfc45e17a
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 7231 - Upgrade ietf-restconf draft17 to draft18

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

Bug 7207 - Bad parsing of identityref in typedef for last ietf-restconf
draft

  * value for identityref in typedef has been added as moduleName+value
    * moduleName:modulName:localName
  * fixed by create value by moduleName + localName of value

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

Bug 6951 - Implement Query parameters - with-defaults

  * fixed tests

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

Bug 6947 / Bug 6948 - implement point and insert query params

  *fixed test

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

Bug 4883 - implement query parameter - filter

  * added test

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

Bug 6949 / Bug 6950 - Implementation of start-time and stop-time
query parameters

  * added and fixed tests
  * added yang notifications to latest restconf draft impl

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

Bug 6935 - RPC in latest draft doesn't work - problem of parsing
to instance identifier

 *added parsing of rpc identifier
 *test for rpc

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

Bug 6995 - Change event notification subscription usability PART2

  * added new yang for leaf with location for subscribing on stream
  * changed return type of subscribing stream request from Response
    to NormalizedNodeContext
    * implemented leaf with location of stream
  * implemented support of adding new headers on data return
    * added new statement 'headers' to NormalizedNodeContext

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

Bug 6995 - Change event notification subscription usability PART1

  * added leaf of notification-output-type to grouping for use
    both augmentations of notifications:
      * data-change notifications
      * yang notifications
  * implemented support of output type for data-change
    notifications (in draft02 and in latest draft)
  * fixed tests to support output of notification

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

Bug 6903 - Implement Query parameters - fields

- parse and verify fields parameter from URI
- nodes written to output are limited in
ParameterAwareNormalizedNodeWriter according to
parsed fields parameter value
- ParameterAwareNormalizedNodeWriter is used as writer
even if no parameters are used
- fields parameter is used together with depth
parameter, when node is selected by fields parameter
then value of depth parameter has no effect on selected
node and its *ancestors*
- contains unit tests for fields parameter parser and
ParameterAwareNormalizedNodeWriter class
- fixed depth for map nodes in way that when map node is
target and depth is 1 no map entries children are written
to output
- added support to read data (GET) from restconf/data path

Change-Id: I3896dba6faa3fdbcb00e07fdfd7b3fdc02ba57d9
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Signed-off-by: Jakub Toth <jatoth@cisco.com>

Bug 6895 - Implement Query parameters - depth

- depth URI query parameter for new Restconf
- parse and verify all GET parameters in one method
- added tests for depth parameter writer
- added unit tests for content parameter

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

Bug 6382 - add apidoc for latest restconf draft

Create a swagger api explorer for latest ietf-netconf-restconf draft

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

Bug 6731 - add valid example values

Add API Explorer Model Schema Data Type 'Click to set as parameter
value' valid example values taking in consideration -> default values,
patterns, data types, min and max value.

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

Bug 6931 - Fix unsupported specific type of leaf

  * add value to YangInstanceIdentifier as type of leaf
  * converting of value of leaf to String in
    YangInstanceIdentifierSerializer
  * fix tests according to parsing value of leaf by type

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

Use UTF-8 as the standard output

Do not rely on environment-specific encoding, but always
use UTF-8 as the target encoding.

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

InstanceIdentifierCodecImpl JUnit test

Test serialize and deserialize methods of InstanceIdentifierCodecImpl class

Change-Id: Id40a25e2ed5828cc78368f4baa9dfdd205f078d3
Signed-off-by: matus.kubica <Matus.Kubica@pantheon.tech>

Bug 6679 - api explorer creates false examples

Api explorer created false examples when yang has two data nodes with
same name in different containers or lists. It compared name of data
nodes rather than whole path to the data node which result to creting
examples of same types even if they should be different.

Fix examples by comparing the path to data node rather than just name
of the node.

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

Bug 6272 - support RESTCONF PATCH for mounted NETCONF nodes

- solution for Restconf draft17
- make mount point service available when identifier is parsed
- added unit tests

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

Bug 6746 - Restconf: Not working GET operation on mount points

- support for reading data on mount points
- improved merge for config and state data:
when identifiers for data in config and state are the same
then merge must by done at one level down, otherwise state
overrides config or vice versa
- identiers are parsed to identifiers context with reference
to mount points when data are behind mount point
- adapted and added unit tests

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

Fix close() in provider

 *added condition on get() from handler

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

Fix autorelease - stub channel and eventLoop

Since netty version has been bumped, change are required in the unit test.

Change-Id: I7ff42c9538f4fdfadf1c1677ced81b1afa767e31
Signed-off-by: Alexis de Talhouët <adetalhouet89@gmail.com>

Bumping versions by 0.0.1 for next dev cycle

Change-Id: If0d03515044dedb8aaaba3e6806e361bf154cd40
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Bug 6911 - RPC support in singleton

- master invoke RPC and send back to slave.

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

Add mdsal-singleton-common-api to singleton pom

Change-Id: I4159b4ef312f5d924426af82e1bf02f07ee8a5bc
Signed-off-by: Tomas Cere <tcere@cisco.com>
(cherry picked from commit 485d8d0ef7de77ffd15dd2913227a4c368100d1f)

Remove old clustered netconf topology implementation

Since new netconf clustering implementation resides in
netconf-topology-singleton module, old implementation is not needed
anymore. This patch gets rid of it.

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

Bug 7172 - Correct error-info for missing-attribute errors

Add bad-attribute and bad-element information to error-info when
constructing missing-attribute rpc-error.

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

Bug 7240 - Restconf returns Status.Ok if delete fails

  * add a private class for setting Throwable object
  * if operation failed, set Throwable object to object created from
    private class
  * throw RestconfDocumentedException in Restconf (doesn't in hijack
    thread) if Throwable object isn't null
  * add test

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

Bug 6324 - Notifications stream output is not same as restconf data

  * serialization NormalizedNode to JSON with gson codec from yangtools
  * added test

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

Set mdsal version to Boron-SR2 version

Patch https://git.opendaylight.org/gerrit/47394 pulles in the incorrect
mdsal-artifacts version for current stable/boron. This patch updates the
version to the correct one fixing autorelease nightly build.

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

Merge changes I0e30376d,I7b95f664 into stable/boron

* changes:
  Add logging in tx facade along with the RemoteDeviceId
  Move SubmitFailedReply in the appropriate package

Merge "Use SerializationUtils to (de)serialize NormalizedNode and YangInstanceIdentifier" into stable/boron

Bumping versions by 0.0.1 for next dev cycle

Change-Id: I8522635565af46576554223d2367656dbe03c7d9
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>