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>

Update .gitreview to stable/carbon

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

Merge "Fix formatting in callhome-provider"

Merge "BUG-7529: Karaf 4 Migration"

Fix formatting in callhome-provider

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

Bug 8095: Add operational devices to call home collection.

Original call home feature has config presence but not operational.
Operational devices are more convenient for monitoring
populations of call home devices (and will be richer model
in future).

Change includes adding a call home status (connected, disconnected, ...)
to operational devices.

Address "functional" checkstyle issues. Turn on checkstyle to list all
issues and fix all that have a non-cosmetic impact (unused imports,
catching exceptions, ...).

Reenable @Ignored unit tests

Change-Id: I3382e67414a9208c65b967c1a2ebaa3247cd4fba
Signed-off-by: Allan Clarke <clarkea@brocade.com>

BUG-7529: Karaf 4 Migration

Add missing karaf 4 features to artifacts.

Change-Id: Ie259e875aa8f2c6cf743ee7b821fa25b144bf255
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

Merge "Bug 8086 - Cannot mount honeycomb in clustered odl on CentOS"

Merge "Option to receive only leaf nodes in websocket notifs"

Bug 8086 - Cannot mount honeycomb in clustered odl on CentOS

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

Merge "BUG-1422 Introduce Call-Home functionality for the NETCONF Topology."

BUG-1422 Introduce Call-Home functionality for the NETCONF Topology.

Introduce Call-Home reversed SSH support.

Basic support for host key based white-list and authorization by username/password.
Rebased with master; some code flaws fixed
Addressed minor code fixes
Addressed most of Tomas' review issues
Addressed Jakub's review issues
Addressed more or Tomas' review issues
Removed mistaken hoisting of odl-netconf-callhome-ssh to be topmost feature
Reenabled "ignored" unit tests

Change-Id: If3e3c9fdd0af0c97043f8930b7f922586863cabf
Signed-off-by: Balaji <bvaradar@brocade.com>
Signed-off-by: Mike Arsenault <mike@mike-arsenault.com>
Signed-off-by: Allan Clarke <clarkea@brocade.com>
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

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>

Merge "Handle shcema without namespace."

Handle shcema without namespace.

currently when the ODL connects to a device and recieves a schema
without a namespace it disconnects.
This fix allows the ODL to ignore schemas without namespaces, but still
maintains the connection and handles the rest of the schemas which do
have a namespace.

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

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

Change-Id: Ic53b0649b52529e9bf244b82d85582d343aa95e9
Signed-off-by: Wenbo Hu <huwenbo1988@gmail.com>

Fix autorelease by using correct prerequisite version

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

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>

Option to receive only leaf nodes in websocket notifs

Change for Bug-7718 in the netvirt project.

The notifications sent over websockets contain the
entire subtree, even if just one node changed. This is
very inefficient for monitoring subtrees that may have
large numbers of nodes. Point in case, we need to monitor
changes to the port statuses under /neutron/ports/ but we
do not want to have to IO and parse the entire list of
ports for each change. This change proposes a new query
parameter to the "data-change-event-subscription" REST call
that specifies that the subscriber is only interested in
receiving the leaf nodes that have changed. Each leaf node
is (already) accompanied by its path.

Here are examples of the notifications sent with the query
param and without. The notification is generated by one of
two port's status transitioning from DOWN to ACTIVE.

Here it is without the query param. Note how all existing
ports are sent. Note how the status is sent three times
and the port twice (the notification send each level of the
tree in it's entirety):
{
    "notification": {
        "data-changed-notification": {
            "data-change-event": [
                {
                    "data": {
                        "status": {
                            "content": "ACTIVE",
                            "xmlns": "urn:opendaylight:neutron"
                        }
                    },
                    "operation": "updated",
                    "path": "/neutron:neutron/neutron:ports/neutron:port[neutron:uuid='97a251dc-3bbe-49b1-a08f-b164922bb0c3']/neutron:status"
                },
                {
                    "data": {
                        "port": {
                            "status": "ACTIVE",
                            "uuid": "97a251dc-3bbe-49b1-a08f-b164922bb0c3",
                            "xmlns": "urn:opendaylight:neutron"
                        }
                    },
                    "operation": "updated",
                    "path": "/neutron:neutron/neutron:ports/neutron:port[neutron:uuid='97a251dc-3bbe-49b1-a08f-b164922bb0c3']"
                },
                {
                    "data": {
                        "ports": {
                            "port": [
                                {
                                    "status": "ACTIVE",
                                    "uuid": "97a251dc-3bbe-49b1-a08f-b164922bb0c3"
                                },
                                {
                                    "status": "ACTIVE",
                                    "uuid": "2278122f-1e21-4557-8631-c9cef5299ba4"
                                }
                            ],
                            "xmlns": "urn:opendaylight:neutron"
                        }
                    },
                    "operation": "updated",
                    "path": "/neutron:neutron/neutron:ports"
                }
            ],
            "xmlns": "urn:opendaylight:params:xml:ns:yang:controller:md:sal:remote"
        },
        "eventTime": "2017-02-22T02:42:08+02:00",
        "xmlns": "urn:ietf:params:xml:ns:netconf:notification:1.0"
    }
}

Here is the same exact notification but generated with a registration
using the new query param:
{
    "notification": {
        "data-changed-notification": {
            "data-change-event": {
                "data": {
                    "status": {
                        "content": "ACTIVE",
                        "xmlns": "urn:opendaylight:neutron"
                    }
                },
                "operation": "updated",
                "path": "/neutron:neutron/neutron:ports/neutron:port[neutron:uuid='d6e6335d-9568-4949-aef1-4107e34c5f28']/neutron:status"
            },
            "xmlns": "urn:opendaylight:params:xml:ns:yang:controller:md:sal:remote"
        },
        "eventTime": "2017-02-22T02:27:32+02:00",
        "xmlns": "urn:ietf:params:xml:ns:netconf:notification:1.0"
    }
}

Here is an example of the REST call with the new query param:
curl -u admin:admin http://10.9.8.1:8181/restconf/streams/stream/data-change-event-subscription/neutron:neutron/neutron:ports/datastore=OPERATIONAL/scope=SUBTREE?odlLeafNodesOnly=true

Also fixed the existing unit test and added one. However,
there are some issues with jersey where sporadically the
InputStream it passes in to read the post data is EoF.

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

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 7987: Json HTTP PATCH: Problem parsing simple leaf value

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

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>

Switch time keeping to java.time interfaces

Date-based interfaces are not thread-safe and ambiguous, Java 8
gives us java.time, which has proper domain model.

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

Eliminate ListenerConstants

These constatns are used only in AbstractNotificationData, inline them
in that class.

lso convert to using Java 8 Time API to fix thread-safety issues with
SimpleDateFormat. Since we need to choose an appropriate API, we choose
to report instants in UTC, so we do not have a dependency on the concept
of the local time zone.

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

Clean up revision formatting

Remove open-coded revision formats and use cached formatted
revisions where available.

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

Simplify data encoding in AbstractNotificationsData

Rather than performing a two-step conversion via a writer,
output stream directly into bytes and then convert the result
into a String via StandardCharsets.UTF_8.

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

Improve AbstractQueryParams XML parsing

Use a shared, hardened, DocumentBuilderFactory and do not pass
the XML via a field but rather as an argument.

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

Mass conversion to static methods

This is mechanic fixup of methods which Eclipse points out as
being potentially-static.

Secondary effect is use of lambdas where the conversion can be made
automatic.

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

Use UntrustedXML for interfacing with Documents

Eliminate duplicated factory setup call sites in favor of using
centralized UntrustedXML.

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

Remove unneded semicolon

This is an eclipse warning, fix it.

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

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 7741 - "Please try again" should not result in http code 500

Change ErrorType to TRANSPORT
Change ErrorTag of DocumentedException to RESOURCE_DENIED
Change ErrorSeverity to ERROR
Add mapping in BrokerFacade to 503
Edit RPCError in NetconfReadOnlyTransaction
Add JUnit test for BrokerFacade

Change-Id: I039886d80a89e0ef4d376229617b3b4b7520a04c
Signed-off-by: matus.kubica <matus.kubica@pantheon.tech>

Merge "Bug 6645 - 2 digits milliseconds can not be parsed in notification eventTime"

Bug 6645 - 2 digits milliseconds can not be parsed in notification eventTime

Create a Parser and a Formater based of Java 8 DateTimeFormater API, using
the ISO_DATE_TIME format, the UTC time-zone offset, and the local time-zone.

As we're moving away from the pattern approach to parse/format those values,
we have to handle the leap second corner case; it is possible to encounter
time value being 23:59:60, see https://en.wikipedia.org/wiki/Leap_second.
For that reason, if the parsing fails, we catch the exception and check whether
the time has a leap second, and if it has, we replace the second-of-minute of
60 with 59.

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

Bug 6581 - Make timeout for ask configurable

Change-Id: Ie430aa39cc6a7b30e86ef3c73d27b35a10e97dac
Signed-off-by: miroslav.kovac <miroslav.kovac@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>

BUG-6859 - Binding generator v1 refactoring

Fix compilation failure after binding generator move.

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

Merge "Bug-6346: Allow over-ride of non-module capabilities"

Bug-6346: Allow over-ride of non-module capabilities

This feature will allow user to override non-module based capability like
:writable-running or :candidate capabilities of the device. It can used
for devices that do not report or incorrectly report non-module based
capabilities in their hello message.

Change-Id: I1ba4ac1ca84a528434658e8af7c3db981146edb6
Signed-off-by: Om Prakash <op317q@att.com>
Signed-off-by: Jakub Morvay <jmorvay@cisco.com>

Bug 6996: Set correct module name to error message

When module doesn't exist in mountpoint schema context,
error always contained only ""yang-ext" module does not exist in mount
point." Now correct module name is set to error message.

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

Merge "Null write transaction when cancel is called"

Merge "Bug 5581: Optimize subtree filtering"

Merge changes from topic 'bug7850'

* changes:
  BUG-7850: SourceIdentifier.getRevision() can return null
  BUG-7850: YangLibProvider needs to accept null revisions
  BUG-7850: do not open-code non-existent revision
  BUG-7850: fixup Swagger generator

Merge "Refactor netconf-util's configuration service"

BUG-7850: SourceIdentifier.getRevision() can return null

Fix assumptions which are not in-line with API documentation,
i.e. that getRevision() will return a magic string and never null.

Once the users are fixed, getRevision() will return null as per
its defined contract.

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

BUG-7850: YangLibProvider needs to accept null revisions

The API contract indicates that SourceIdentifier.getRevision()
may legally return null. The implementation is currently broken,
so make sure we account for both possibilities.

Also share empty revision object for efficiency, as it is constant.

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

BUG-7850: do not open-code non-existent revision

Revision 0000-00-00 is reserved and should not be used anywhere
directly.

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

BUG-7850: fixup Swagger generator

Remove open-coded "0000-00-00" string and refer to its original
definition. Also use SimpleDateFormatUtil instead of open-coded
formatter and fix methods which can be made static.

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

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>

Null write transaction when cancel is called

Write transaction is finished and can't be used anymore.

Change-Id: Id9efcab3ca0161d5dba4cef3b56e6e568c1b1e6c
Signed-off-by: Andrej Mak <andrej.mak@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 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>

Merge "Fix error reporting for PUT/POST"

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>

Merge "Remove unused imports in the whole NETCONF project"

Remove unused imports in the whole NETCONF project

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

Merge "Bug 7295 - Incorrect handling of device transactions in clustered setting"

Pull in feat-jetty instead of feat-war

feat-war pulls in jetty 9.2, we need 8.1 for the transition.

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

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>

Merge "BUG-7183: do not hard-code scala version"

Merge "Compile-time dependencies matter"

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>

Merge "Remove blocking get in read transaction"

Remove blocking get in read transaction

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

Compile-time dependencies matter

This is an analogy of https://git.opendaylight.org/gerrit/#/c/51921/1

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

BUG-7183: do not hard-code scala version

Akka artifacts should be coded against scala.version, not to a specific
version. Also drop scope=test, as it is not needed.

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

Refactor netconf-util's configuration service

- log netconf configuration service lifecycle events
- NetconfConfigUtil's getNetconfConfigurationService method fails if
  netconf configuration service is not present
- make properties for netconf configuration service constant
- netconf configuration service does not have to be singleton

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

Refactor NetconfDocumentedExceptionTest class

Conform to API changes introduced in
https://git.opendaylight.org/gerrit/#/c/48322/.

Also move test to correct package.

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

Merge "Bug 5581: Merge empty map instead of exist check"

Bug 5581: Merge empty map instead of exist check

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

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>

Declare -util/-tcp/-ssh prerequisites

This is imperfect since we drop the relationships from the POMs, which
breaks -Psft (running SFT on the features individually). We need to
improve the plugin to handle this more generically.

Change-Id: Ic7ddc583450b3e207eb4a13f837e8994cb3d5ff0
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-6875: adjust for yangtools requiring getYangVersion working

ModuleDependencySort requires a correct YangVersion, fake version 1.

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

Pull in odl-karaf-feat-war

This acts as a proxy feature for karaf's war, pull that in.

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

Merge "Clean ControllerContext up"

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>

Clean ControllerContext up

ControllerContext is supposed to be an immutable context, which has always
a global instance, which is updated as needed.

Current code mutates the context in an uncoordinated fashion, which means
it can easily end up being inconsistent if a SchemaContext change occurs
while a request is being executed.

Add FIXME markers to issues to be solved in future and clean up various
aspects of the file:
- make sure the URI encoding charset is available
- use lambdas wherever possible
- fix keyword order to match JLS
- trim long lines

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

BUG-7529: provide Karaf 4 features

Change-Id: I649dcae1d60155d5dd656eb3416183889a8f6a14
Signed-off-by: Stephen Kitt <skitt@redhat.com>
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>

Remove unused reference to DOMMountPointService

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

Merge "Use yangtools ParserStreamUtils.findSchemaNodeByNameAndNamespace() method"

Replace FindBugs :jsr305 by full :annotation (Bug 7663)

Change-Id: I17e6f6f75ca4972f433b19a1c518898941bab3e4
Signed-off-by: Michael Vorburger <vorburger@redhat.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>

Merge "Prepare upgrade to Netty 4.1"

Replace mockito-all by mockito-core (see Bug 7662)

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

Prepare upgrade to Netty 4.1

We need to widen the range of allowed versions for odl-netty.

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

Merge "Bug 7491 - Ignore exceptions during invoking notification listeners"

Bug 7491 - Ignore exceptions during invoking notification listeners

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

Close missing resources when netconf-topolgy goes down

When the connection goes down, we are closing the NetconfDeviceCommunicator
instance but we aren't closing the RemoteDeviceHandler. I believe this is an
oversight, so implementing the Autocloseable interface in NetconfConnectorDTO
that will close both the resources. The consumer of this DTO can then call #close()
to release the resources.

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

Bug 5407 - Deprecate Netconf Loopback Mount Through Config Subsystem

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

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

Merge "Bug 7432 - eliminate use of yang-parser-impl internals"

Bug 7433 - Remove use of YangSchemaSourceImpl from restconf tests

Change-Id: I3a5901618cbe162eed07cf0301626e6228df4360
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-4219: use upstream karaf.version

Rather than hard-coding a specific version, pick it up
from odlparent.

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

BUG-4219: use scope=provided for karaf dependencies

We should not be pulling in the dependencies, but rather
require them to be provided.

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

Merge "Bug 7206 - Chinese improper code problem."