Bug 7933: NPE when posting using XML

PUT operation:
- fix issue when putting malformed XML body did not
fail with appropriate status.

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

Fix PatchEditOperation

Enums support methods, so instead of a switch dispatch use normal
property. Also simplify users so they do not pass strings around.

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

Optimize JsonToPatchBodyReader

Use a StringBuilder instead of a StringBuffer, use switch statements
and du not use string concat when using StringBuilder.append().

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

Fix various warnings

The code is full of eclipse warnings around static methods, raw types
and similar. Fix that up.

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

Bug 8153: enforce check-style rules for netconf

- enforce checkstyle in all modules + cleanup

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

Bug 8153: enforce check-style rules for netconf

- call-home

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

Bug 8153: enforce check-style rules for netconf

- netconf-testool

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

Bug 8351: enforce check-style rules for restconf

- enforce checkstyle in all modules + cleanup

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

Bug 8351: enforce check-style rules for restconf

- sal-rest-docgen

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

Bug 8351: Enforce check-style rules for restconf - sal-rest-connector

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

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 8153: enforce check-style rules for netconf

- netconf-cli module

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

Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-connector

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Merge "Bug 8351: Enforce check-style rules for restconf - sal-rest-docgen"

Merge "Bug 8153: Enforce check-style rules for netconf - yanglib"

Bug 8074 - actor name [test-device] is not unique!

When creating a new actor with same name as an old one, it might fail
becuase old actor might not be completely gone at this point.

This creates a slaveActor which will never be closed, but instead
every time that mountpoint is closed or master is changed, it would
update NetconfNodeActor data and keep the same actor.

Only time that actor is closed is when node was deleted. Or slave
became a master

The name of the actor is removed - letting akka to handle that -
log the akka created name.

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

Merge "Resolve exception from registerClusterSingletonService"

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>

Resolve exception from registerClusterSingletonService

registerClusterSingletonService return RuntimeException in
case of problems with registration and client should implement
strategy to resolve this issue

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

Bug 8153: Enforce check-style rules for netconf - yanglib

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Merge "Bug 7891: Add ssh feature to netconf-netty-util"

Merge "Bug 8364: Set bundle context when module reloads"

Merge "Bug 8361 - NetconfConnector cannot be created due to NullPointerException"

Bug 8153: Enforce check-style rules for netconf - netconf-topology-singleton

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8364: Set bundle context when module reloads

Fix for bug Bug 8214 handles waiting for AAA, when Restconf
is installed for the first time. Override handleChangedClass
to do the same thing for reinstall case. Also close old
module to release bound ports and started threads.

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

Bug 8361 - NetconfConnector cannot be created due to NullPointerException

- Adds Nullpointer checks in RemoteDeviceConnecterImpl
- If Yang module capabilities are missing its override flag is set to false
- If Non module capabilities are missing its override flag is set to false

Change-Id: I25d172e94a1013d79c0c7db37c930c89ad2ed94f
Signed-off-by: Martin Mihálek <mamihale@cisco.com>

Bug 7891: Add ssh feature to netconf-netty-util

Odl-netconf-netty-util needs sshd-core bunde to work.
It is provided by karaf. However, if odl-netconf-netty-util
is placed to featuresBoot, ssh provided by karaf isn't
started, so ResolutionException is thrown. To prevent this
add ssh feature to odl-netconf-netty-util.

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

Merge "Bug 8153: Enforce check-style rules for netconf - sal-netconf-connector"

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-topology"

Bug 8153: Enforce check-style rules for netconf - netconf-monitoring

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Merge "Bug 8153: Enforce check-style on messagebus-netconf"

Bug 8351: Enforce check-style rules for restconf - sal-rest-docgen

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Merge "Do not use InetAddress.getByName()"

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

Merge "Bug 8084 - FilterContentValidator.getKeyValues creates invalid YII key values"

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>

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

Bug 8153: Enforce check-style rules for netconf - netconf-topology

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style rules for netconf - sal-netconf-connector

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Do not use InetAddress.getByName()

We have better alternatives for these, use those.

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

Bug 8153: Enforce check-style on messagebus-netconf

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.

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

Merge "Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-notification"

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

TypeDefinition returns a namespace and revision of original module
if it is coming from grouping. We need a DataSchemaNode which
returns namespace and revision of module in which node is situated.

Change-Id: I48246f86d55414c58226f7b523800e847bac5ee4
Signed-off-by: miroslav.kovac <miroslav.kovac@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: I7a76f6b4d9e6e348ec0d58abe24e56dfeae66f24
Signed-off-by: miroslav.kovac <miroslav.kovac@pantheon.tech>

Bug 8084 - FilterContentValidator.getKeyValues creates invalid YII key values

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

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-tcp"

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-notification-impl"

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-notification-api"

Bug 8153: Enforce check-style rules for netconf - config-netconf-connector

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8032 - Initialization in sal failed, disconnecting from device

MountpointService is initialized with blueprint and therefore it
should never throw null pointer exception anymore.

All the binding and dom broker are deleted because they are
not needed any more.

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

Merge "Bug 8153: Enforce check-style on netconf-impl"

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-console"

Merge "Bug 8153: Enforce check-style rules for netconf-api"

Merge "Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-yang-library"

Merge "Refactor netconf clustered topology tests"

Merge "Bug 8153: Enforce check-style rules for netconf - aaa-authn-odl-plugin"

Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-notification

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style rules for netconf - netconf-notification-impl

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style rules for netconf - netconf-tcp

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style rules for netconf - netconf-notification-api

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style on netconf-impl

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add checkstyle for test suites.
Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style rules for netconf - netconf-console

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Merge "Bug 8197: Deregister schema sources on actor stop"

Bug 8153: Enforce check-style rules for netconf-api

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-yang-library

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Merge "Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-monitoring     Organize Imports for Checkstyle compliance.     Checkstyle compliance: line length.     Checkstyle compliance: various types of small changes.     Checkstyle compliant Exception handling.     Checkstyle final clean up & enforcement."

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-netty-util"

Merge "Bug 8153: Enforce check-style rules for netconf - client."

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-ssh"

Merge "Bug 8153: Enforce check-style rules for netconf - netconf-util"

Merge "Bug 8152: Add way to configure idle timeout"

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>

Bug 8153: Enforce check-style rules for netconf - aaa-authn-odl-plugin

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8153: Enforce check-style rules for netconf - netconf-util

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8181 - Add karaf4 feat to artifacts

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

Bug 8153: Enforce check-style rules for netconf - mdsal-netconf-monitoring
    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.

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

Bug 8153: Enforce check-style rules for netconf - netconf-netty-util

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

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>

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>

Bug 8153: Enforce check-style rules for netconf - client.

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Make EditConfig methods static

Some of these do not refer to object state and hence we can
make them static.

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

Eliminate ReadWriteTransaction proliferation

Many of the internal methods really need only a WriteTransaction,
hence adjust the type to make it more clear what a method can
actually do.

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

Bug 8153: Enforce check-style rules for netconf - netconf-ssh

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.
    Add the fail on violation flag into the pom.xml .

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

Bug 8214: Consume AAAService

Consume AAAService ensuring the bundles are available from the BundleContext
prior to trying to load the rest of RESTCONF.

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

Fix odl-netconf-ssh single feature test

Add dependency on odl-netconf-tcp and odl-netconf-util modules so we
will see their features repositories generated in odl-netconf-ssh
feature.xml file.

Also set dependency="true" atribute for both dependency features when
overriding generated feature.xml file.

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

Merge "BUG-5581: batch read check"

Merge "Bug 8152: Transaction is already opened"

BUG-5581: batch read check

When posting data we are executing potentially a lot of reads,
which can hurt if the backend is not residing on the node which
is executing the operations.

Fix this by issuing read requests in a batch and then collecting
the results, which means we will amortize request latency.

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

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>

Bug 8153: Enforce check-style rules for netconf - netconf-mapping-api

    Organize Imports for Checkstyle compliance.
    Checkstyle compliance: line length.
    Checkstyle compliance: various types of small changes.
    Checkstyle compliant Exception handling.
    Checkstyle final clean up & enforcement.

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

Merge "Change handling of netconf cluster transactions"

Merge "Migrate to karaf4 for the local karaf distribution"

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>

Merge "Close read-only transactions"

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>

Migrate to karaf4 for the local karaf distribution

Several folks have put in incredible effort to make karaf4 a reality in ODL.
Kudos to those folks for this great effort.  This patch officially converts
netconf's local karaf distribution to use odlparent's karaf4-parent artifact.

The karaf4 implementation will still be accessible at the following well-known
path:

> karaf/target/assembly

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

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

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-8187: call home throwing exception on startup"

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

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