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>

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>

Add logging in tx facade along with the RemoteDeviceId

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

Move SubmitFailedReply in the appropriate package

SubmitFailedReply is a transaction related message, hence
should resides under the .transactions package

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

Merge "Add serialVersionUID to all java.io.Serializable messages" into stable/boron

Merge "Update netconf-topology-singleton.xml file formatting" into stable/boron

Merge "Remove unused imports" into stable/boron

Use SerializationUtils to (de)serialize NormalizedNode and YangInstanceIdentifier

Controller CDS implementation provide utility method for this,
we can re-use them, as it appears it's keeping a ThreadLocal
holding NormalizedNodeDataOutput instance

Change-Id: Ia106f33d04f937ac8dacf89cb6837a157eee8ae1
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
(cherry picked from commit 4f326302b5dcb41b1da291a0240e2b8808a63893)

Remove unused imports

Change-Id: I8079b664133e88a3b35f21a3c41e84e890a8ff46
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
(cherry picked from commit aa53b02a7523e31dc986ae6f55d469158f1a0938)

Update netconf-topology-singleton.xml file formatting

- Inline the closing statement
- Rename to bean to be more accurate

Change-Id: I2e2052b7b86b12139b2573c0e3828730cc8d606d
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
(cherry picked from commit fc135be045cdf98fbfa5eecef352e3fc43329fb3)

Add serialVersionUID to all java.io.Serializable messages

Change-Id: I4928273c0d65e782162d099ddbae36a0c9fe1c63
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
(cherry picked from commit 337584868377e343ea943a9268da336325628c70)

Add the RemoteDeviceId at the begining of the log

So it's easier to grep and find things related to one netconf
connector, and also, to keep things consistent with how it's
already done in sal-netconf-connector

Change-Id: I29ee41c699de8287fd6e00f3a04cacb3d17c5fd1
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>
(cherry picked from commit 518b77c6c392a18270297705634dd5bff260ea29)

Merge "Bug 6714 - Use singleton service in clustered netconf topology" into stable/boron

Bug 6714 - Use singleton service in clustered netconf topology

Implemented clustered netconf topology with using clustered
singleton service.

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

Bug 6256 - OpenDaylight RESTCONF XML selects wrong YANG model for southbound NETCONF

- dataSchemaNode can contains multiple childs with the same name,
thus when looking for child also namespace has to be compared
- added unit tests

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

Merge "Fix tests after merging Change 47121 to Yangtools" into stable/boron

Merge "Bug 5615 - Netconf connector update overwriting existing topology data" into stable/boron

Fix tests after merging Change 47121 to Yangtools

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

Bug 6272 - support RESTCONF PATCH for mounted NETCONF nodes

- distinguish between operation on server or on mount point
- when operates on server then use global schema context and
data broker
- when operates on mount point then use schema context and
broker from mount point
- if broker on mount point is missing then PATCH operation fails
with global error
- contains unit tests

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

Add xml config dependency to features pom

Configfile added here [0] also needs to be listed as pom dependency,
otherwise distribution-check would fail.
Also added to netconf-artifacts.

[0] https://git.opendaylight.org/gerrit/#/c/44243/10/features/netconf/src/main/features/features.xml

Change-Id: If016aadb03154d83ddd0afa680e8d1d8acd88217
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
(cherry picked from commit c2272d825c810de0a023661f054cd0ce023e0a77)

Merge "Bug 6023 - Adress for config subsystem netconf endpoint is not configurable" into stable/boron

Bug 6023 - Adress for config subsystem netconf endpoint is not configurable

- added NetconfConfigurationActivator for loading ManagedService
- updating configuration from karaf netconf.cfg file
- edited NetconfTCPActivator, NetconfSSHActivator for run with
  cfg values

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

Bug 6936 - Fix post request

When post request sends data which already exist, it response with
data already exist.

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

Unit test for PostDataTransactionUtil class

Change-Id: I38e1d29daa1598609e1a706b16d1bb2f24012f0f
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@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>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>

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>
(cherry picked from commit de60ce77c83f1f09bb804f61124fe5c9f439c376)

Bug 6848 - update url pattern of restconf from 16 to 17

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

Bug 6848 - repackage providers for jersey+create xml
and json reader for restconf draft17

  *fix tests
  *rename tests part from draft11 to draft17

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

Bug 6848 - upgrade XML media type

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

Bug 6848 - upgrade namespace of notification container

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

Bug 6848 - Renaming to draft17

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