Remove deprecated CheckedFuture from callhome-provider module

Some other changes regarding checkstyle, sonar and annotations.

Change-Id: I361e2fc1928620ac1ed178f77beb397d5110a761
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Merge changes from topic 'ListenerAdapterTreeMigration'

* changes:
  Transition ListenerAdapter to ClusteredDOMDataTreeListener
  Add unit tests for ListenerAdapter

Transition ListenerAdapter to ClusteredDOMDataTreeListener

Solves this: https://bugs.opendaylight.org/show_bug.cgi?id=9147
This is in general an issue with ListenerAdapter which does not
currently work in a clustered deployment.

ListenerAdapterTest needed to be adjusted for two reasons:
1. There was a bug in ListenerAdapter where if leafNodesOnly
was set the leafNodesOnly logic was not applied in delete
notifications. The new ListenerAdapter code fixes that so
ListenerAdapterTest needed to be adjusted.
2. In DomDataTreeEtc, merge operations cause the key node
to count as "WRITE"-en for notifications even if it was not
changed. ListenerAdapterTest was adjusted accordingly. Bug
for this here: https://bugs.opendaylight.org/show_bug.cgi?id=9179

Change-Id: I3c40e02551d8f04354489e5ffb8c66f111bb3b96
Signed-off-by: Josh <jhershbe@redhat.com>

Add unit tests for ListenerAdapter

Test that json generated by ListenerAdapter is correct
for regular notifications and "leaf nodes only" notifications.

Change-Id: I62fa22b1e30ca6a8632b3d2614558e9ced0abf54
Signed-off-by: Josh <jhershbe@redhat.com>

Fix augmentation node lookup

This fixes an issue where we erroneously checked instanceof
against optional rather than its content -- which is obviously
wrong.

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

Import MappingCheckedFuture from mdsal

yangtools is getting rid of this class, import it from its new home
in mdsal.

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

Merge "Bug 9256: Add websocket server config knob for ip"

Merge "BUG 9112: NPE in karaf cli when device is still connecting"

Merge "BUG-9048 Fix actor crash when writing incorrect data"

Merge "NetConf test tool improved so now it can be used as library not only as CLI tool."

Merge "Replace deprecated Futures.addCallback by the newer version"

NetConf test tool improved so now it can be
used as library not only as CLI tool.

Change-Id: I14ca03d0b2cec0f6fe8b5994b3e195fa6de8a4d2
Signed-off-by: Juraj Veverka <juraj.veverka@pantheon.tech>

Remove unnecessary bouncycastle dependencies

These are provided through the JRE's JCE, and should logically be
thought of as being provided through by the underlying environment
rather than direct dependencies.  This removes the places where
the dependency exists but is not needed.

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

Bug 9256: Add websocket server config knob for ip

Also cleans up a couple of ugly NPE catches.

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

BUG-9218: pull odl-mdsal-model-inventory

Rather than packaging controller's model, use if from the upstream
feature.

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

Replace deprecated Futures.addCallback by the newer version

The method addCallback(ListenableFuture<V> future,
FutureCallback<? super V> callback) is deprecated and will be replaced
by a new its new version addCallback(ListenableFuture<V> future,
FutureCallback<? super V> callback, Executor executor) in April 2018.

Some other minor changes to pass the new version of checkstyle.

Change-Id: I0aaf43e913ab241fd74e53ff91d7852e77b699b3
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

BUG 9112: NPE in karaf cli when device is still connecting

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

Merge "Directly inject CredentialAuth dependency"

BUG-9048 Fix actor crash when writing incorrect data

We need to catch all unechecked exceptions when writing data
in the clustered implementation since these will crash the actor
running and cause problems down the line.

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

Directly inject CredentialAuth dependency

In today's blueprint-oriented OSGi DI, it is more efficient to
express the dependency on AAA's CredentialAuth API rather than to
spin waiting for a ServiceReference using a ServiceTracker.  This
change specifies direct injection of AAA's CredentialAuth Service
through blueprint wiring.

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

BUG-9218: fix features to not duplicate upstream bundles

Restructures dependencies to pull in upstream features which
provide the bundles we need.

Change-Id: I7e1cf306abd100a1a50cc4248de78d85dc91df9c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit ccce725613d3078a0ad18f3f8fbfa5e1d9aea68c)

BUG-9132: don't provide a value for restconf/streams/events

The yang type of events leaf is 'type empty', meaning there should be no value.
As of Carbon performing a GET on restconf/streams causes a cast
exception due to the empty string value here. See EmptyJSONCodec in
yangtools.

Change-Id: If0501b9c5459f4255096a4f070968c87a749407e
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>

Bump odlparent 2.0.4 to 2.0.5

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

Bump netconf to use yangtools 1.2.0

Also reverts I1914426b96029d7d2377f77b9f7df8beb14e607a to downgrade
yangtools properly.

Jira: releng-485
Change-Id: Ie344af870af6a85e859b1e35ca268f5bf988b00b
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge "Adjust test-tool to yangtools changes"

Adjust test-tool to yangtools changes

yang-maven-plugin adds revision to packaged yang file name. Netconf
test-tool should register yang resources with valid filenames, i.e
name@revision.yang, when preparing for assembly of its schema context.

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

Add local MultivaluedHashMap implementation

The JSONRestconfService implementation contains a local UriInfo
implementation, SimpleUriInfo, that uses a MultivaluedHashMap
implementation from javax.ws.rs.core. This class is provided by
the dependency used at compile time however, the runtime bundle
that provides the javax.ws.rs.core interfaces doesn't provide
MultivaluedHashMap. Therefore I added our own local MultivaluedHashMap
class based on the external class. I also refactored SimpleUriInfo into
it's own class as it's used by both JSONRestconfService implementations.

Change-Id: I1af0771327d46c72e55facf14c41038fd583e0ed
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Minor cleanup of blueprint config

The path for private file was left here due to which the netconf module
looks for the private key file and shows error in the log which is a false alarm.
Cleaning up the configuration

Change-Id: Ied9cc137ab67cab158318a6c2f34da19dc174fe0
Signed-off-by: Atul Gosain <atul.gosain@gmail.com>
Signed-off-by: Atul Gosain <agosain@brocade.com>

RPC for netconf node addition. Supports encrypt option for password encryption.

Change-Id: I34beb880cbe2ccc6a3fe8e5f6cd4253f7e1938f4
Signed-off-by: Atul Gosain <atul.gosain@gmail.com>
Signed-off-by: Atul Gosain <agosain@brocade.com>

Merge "Bug 8988 - Check for empty payload properly"

Enable public key based authentication for netconf

Change-Id: Icca97924515537b670688767d41ddd4bd6e1bbf6
Signed-off-by: Atul Gosain <agosain@brocade.com>

BUG 8839: Revert "Make netconf utilize encrypted passwords only"

This reverts commit 1867bf17d781f21473dabcde2ad3f85b9b1defc6.

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

Bug 8988 - Check for empty payload properly

Do not use InputStream#avalaible method to check for empty payload's
input streams in our implementations of MessageBodyReader interface.
InputStream#avalaible method returns an estimate of the number of bytes
that can be read from input stream without blocking. We cannot be sure
the stream is actually empty, even if this method returns 0.

Wrap the payload's input stream in PushbackInputStream and try to read
and unread from it instead.

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

Adjust tests for yangtools changes

yang-maven-plugin now adds revision to the packaged yang file name.
This is introduced in https://git.opendaylight.org/gerrit/#/c/62706/.
Adjust NetconfMappingTest and NetconfMDSalMappingTest to that.

https://git.opendaylight.org/gerrit/#/c/62731/ enforces revisions to be
in valid yyyy-MM-dd format. Adjust Restconf tests to this change.

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

Split Restconf implementations (draft02 and RFC) - features

  * remove restconf rfc8040 implemetation from bierman02
  * add features for restconf-nb-rfc8040

Change-Id: I96110d0721c39acdc88332e748f81de07fd5663d
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - Application

  * add blueprint + web.xml for restconf-nb-rfc8040

Change-Id: Ifd8ea387ace5580ddd36dde59000b7cd69515272
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - providers tests

Change-Id: If8c26aa059508763e4730d69d9b64f594ec9e302
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - move providers

Change-Id: Iab075f8d91001a646783f35aa5fe1550d811d01b
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - tests

Change-Id: I4e20a07ed96e63a1238ade12c7fc3377853ec211
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - move
sal-rest-connector

  * move Patch parts, RestUtil, RestconfSchemaUtil,
    RestconfValidationUtils, IdentityValuesDTO to common module
  * move packages from bierman02 to rfc8040:
    * /restconf/base/services/ ->
        /restconf/nb/rfc8040/services/simple
    * /restconf/common/wrapper/services ->
        /restconf/nb/rfc8040/services/wrapper
    * /restconf/common/references/ ->
        /restconf/nb/rfc8040/references
    * /restconf/handlers -> /restconf/nb/rfc8040/handlers
    * /restconf/parser -> /restconf/nb/rfc8040/utils/parser
    * /restconf/restful -> /restconf/nb/rfc8040/rests
    * /restconf/utils -> /restconf/nb/rfc8040/utils

Change-Id: I8259d69ab2378a25ddcf41d0b6f14dd65afc2507
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - move restconf
errors to common part of restconf

Change-Id: If8786352d59e013aa8da5b22790ee125e23b771b
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - base api

  * create common module for both implemetations
    * move common classes
  * move base api services to restconf-nb-rfc8040

Change-Id: I31d084c5d19da515337f79562bafa520c57a087d
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - Rfc8040

  *move Rfc8040 to the new module restconf-nb-rfc8040

Change-Id: Ica52f23791e8326f20f8449d0e39273b7db8f5c5
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Split Restconf implementations (draft02 and RFC) - Prepare modules

  * rename sal-rest-connector module to restconf-nb-bierman02
  * prepared new module restconf-nb-rfc8040

Change-Id: I401a7fc595251dfd2ae08074096a4a16707c6527
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Merge "Bug 9092: revert to org.json temporarily"

Bug 9092: revert to org.json temporarily

This patch reverts to org.json to convert the XML notification to JSON
in ListenerAdapter. The appropriate fix (coming later) is to serialize
the notification to JSON directly, without going through XML, but in
the mean time this unblocks the release.

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

Adjust for mdsal-binding-dom-codec movement

The codec is moveing its package, adjust for that.

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

Merge "Convert NetconfEventSourceManager to DataTreeChangeListener"

Do not use NamedInputStream

NamedInputStream is going away, as it is useless. Use a proper
File-based schemacontext utility instead.

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

Merge "Convert callhome DCLs to DTCLs"

Merge "Fix failed test - tested string of '..was[n']t..' to '..was[ no]t..'"

Merge "Bug 8989 - Create just one DS for each test-tool's simulated netconf device"

Fix failed test - tested string of '..was[n']t..' to '..was[ no]t..'

Change-Id: I2e49e3d4ce72dfd161807736f9afc9b2a5af3edb
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Convert NetconfEventSourceManager to DataTreeChangeListener

DataChangeListener has been deprecated since Carbon with possible removal
in Oxygen so convert to DataTreeChangeListener.

Change-Id: I23e4318a9973a7cffcaa95c42ac6df7449b59b61
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Convert callhome DCLs to DTCLs

DataChangeListener has been deprecated since Carbon with possible removal
in Oxygen so convert to DataTreeChangeListener.

Change-Id: I5fa84fd609eb7aa145756a5cc3839ed544b77456
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Merge "Simplify boolean expressions"

Merge "Remove unused singleton code"

Merge "Bug 8977 - Failed on binary key type"

Bug 8977 - Failed on binary key type

  * fix validating of uri with input data for binary key type
  * tests

Change-Id: I70ac6e10f9999bc0feca2ba600124fa639eef842
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

Remove unused singleton code

CredentialServiceAuthProvider is not a singleton.  This removes
the INSTANCE Map since it is unused and confusing.

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

Bug 8989 - Create just one DS for each test-tool's simulated netconf device

When using netconf test-tool with option --md-sal set to true, we expect
that each test-tool's simulated netconf device will have its own md-sal
persistent datastore. However, current implementation creates own
datastore for each netconf session.

Map newly created simulated device's datastore to its server socket
instead of netconf session's socket.

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

BUG-2389: remove references to XmlDocumentUtils

This class is going away, do not use it. Also speed up transformation
by sharing XMLOutputFactory. Also fixes NetconfUtil to refer to writer
at its new place.

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

Simplify boolean expressions

* Drop explicit “? true : false” (or the reverse).
* Simplify “if (...) return true; else return false;”.
* Simplify tail “if (...) return false; return true;”.
* Simplify explicit comparisons with true or false.

This was mostly done automatically by IntelliJ.

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

Bug 7176 - Netconf MD-SAL connector is treating prefixed XML as invalid

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

Bug 8968: Convert sal-rest-docgen to blueprint

Removes BundleActivator and corresponding ServiceTracker implementation
in favor of utilizing blueprint DI.

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

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

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

Merge "Remove explicit version declaration"

Merge "Bug 8939: Extract MapEntryNode from MapNode result of XML parsing"

Bug 8939: Extract MapEntryNode from MapNode result of XML parsing

Originally, XmlNormalizedNodeBodyReader expected a MapEntryNode
as the result of parsing an XML input with a root list element.

However, after migrating to the new XML parser, the result of parsing
such an XML input is a MapNode with one MapEntryNode inside.

Therefore, extract this MapEntryNode and use it as the result of parsing.

Change-Id: I778bf8036d7b90f75d2d5028c5a9e31ae4db995e
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>
Signed-off-by: Jakub Toth <jakub.toth@pantheon.tech>

BUG-8923: Bump odlparent to 2.0.4

Adopts changes to netty packaging.

Change-Id: Ibbaebf94838641752cc391c38199c5a597415588
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Remove explicit version declaration

odlparent provides a declaration, no need to override versions.

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

Merge "Avoid null Optional parameter"

Merge "Avoid returning null Optional"

BUG-8923: eliminate another reference to bcprov

bouncy castle is provided by the platform, we should not be pulling
it in.

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

BUG-8923: do not install bouncycastle

These bundles are provided by odlparent's substrate, hence we should
not be mentioning them.

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

Avoid null Optional parameter

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

Avoid returning null Optional

Optional references should never be null, this patch uses
Optional.absent() instead and cleans up the caller.

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

Merge changes from topic 'protocol-framework'

* changes:
  Activate protocol-framework artifacts
  Cleanup build system

Merge "Bug 5825: Migrate restconf to the new XML parser #2"

Merge "Bug 5824: Migrate netconf to the new XML parser #2"

Avoid spurious WARN in NetconfClientConfiguration

Change-Id: I7b010183de605f207451de975fb1cde238460d1e
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Bump all Odlparent dependencies to 2.0.2

Change-Id: I1b51dce8036333ca4562219e7f586048c98e9b8a
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Bug 5825: Migrate restconf to the new XML parser #2

Migrate restconf components to the new XML parser from YANG tools

Change-Id: I2011904cb30f69fdb798f735292a8383049cda6b
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

BUG-7891: Remove odl-netconf-ssh feature prerequisites

This works in my setup. After removing the prerequisite dependency
I do not see anymore the server unhealthy issue.

Change-Id: I54eac80cd987474747ea47f652414245a893470c
Signed-off-by: Luis Gomez <ecelgp@gmail.com>

Bug 5824: Migrate netconf to the new XML parser #2

Migrate netconf components to the new XML parser from YANG tools.

Change-Id: I0cd30ea671239fad986d173bd4be845701d6e968
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

Activate protocol-framework artifacts

This adds protocol-framework to compilation

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

Cleanup build system

Clean up pom.xmls, to eliminate dupliciate aggragator
functions, properly import upstream artifacts and correct
relativePath declarations.

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

Remove logback related stuff

as discussed on https://lists.opendaylight.org/pipermail/odlparent-dev/2017-July/001262.html

tools/netconf-testtool doesn't seem to really require logback-config

netconf-it appears to be completely broken anyway, and is not in build

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

Cleanup unused config subsystem dependencies

Change-Id: I25816dd6ed2b0dd11b40d6dd68549a061380a6ae
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

More migration to odlparent 2.0.2

Some odlparent artifacts were missed in the original migration to
odlparent 2, and were still on 1.9.0. This patch fixes that.

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

Add 'features/protocol-framework/' from commit 'cb42405784db97d0ce2c5991d12a89b46d185949'

Change-Id: I60f397e609aa0f26dbb39c4c5bab17a6b0d8b447
git-subtree-dir: features/protocol-framework
git-subtree-mainline: 6d7e12bf3ef64e5004703a1d540e7e26f30a9595
git-subtree-split: cb42405784db97d0ce2c5991d12a89b46d185949
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Add 'protocol-framework/' from commit '5d015c2c5f800d136406c15fcb64fd531d4ffc26'

Change-Id: I206dca66b860abaa9bc25615d42703c638bf5b3f
git-subtree-dir: protocol-framework
git-subtree-mainline: 21588aadb96699b6f4f87e2cbbbb08fe1e38ed78
git-subtree-split: 5d015c2c5f800d136406c15fcb64fd531d4ffc26
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge changes I6575ddd8,I4b78f51d,I0a0441e6

* changes:
  Cleanup Precondtions in singleton
  Cleanup Throwable catches
  Cleanup NetconfDevice

Cleanup Precondtions in singleton

Use Verify when appropirate.
Use Assert in UT.

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

Cleanup Throwable catches

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

Cleanup NetconfDevice

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

Merge "Bug 5824: Migrate netconf to the new XML parser"

Merge "Bug 8745: Migrate EditConfig to the new XML parser"

Bug 5825: Migrate restconf to the new XML parser

Migrate restconf components to the new XML parser from YANG tools

Change-Id: I87aa8df157ca5cba953852415c49806f2c61ff7b
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>

Bug 5824: Migrate netconf to the new XML parser

Migrate netconf components to the new XML parser from YANG tools.

Change-Id: I42899d4d07cb2f917c50fa862cd6453bb0a8764d
Signed-off-by: Igor Foltin <igor.foltin@pantheon.tech>