Bump versions by x.y.(z+1)

Change-Id: I8d92d69131d32196bf7b2958a7d08b1ca3970b59
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>

Merge "Bump to odlparent 2.0.7" into stable/nitrogen

Bump to odlparent 2.0.7

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

NETCONF-526 : edit-config payload default-operation needs to be set to default-value

For the PUT/POST calls right now default-operation tag in the edit-config is set to
"none". As per RFC with this value devices can reject a config operation when
elements present in the request whose structure is not already present in the tree.
For example a nested parent-child structure where parent structure does not yet exist
yet in the tree may result in an error based on implementation.
We should  leave this field out ( which will default to merge) and let the device make
the right decision.

Change-Id: I8b85ef072785381897a19724dc94003b8f315bd4
Signed-off-by: Balaji Varadaraju <bvaradar@luminanetworks.com>
(cherry picked from commit 011fbb4e78b64748c4be16c43b3deb46a5c3b23c)

Merge "Add local MultivaluedHashMap implementation" into stable/nitrogen

Restconf 8040 should return 204 for delete and put

restconf was returning 200 Ok but according to RCF8040, for put,
"If an existing resource is modified, a "204 No Content"
status-line is returned."" and for delete, "If the DELETE request
succeeds, a "204 No Content" status-line is returned.".

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

NETCONF-514: Use exists instead of read for PUT

The PUT operation needs to check if the path already exists
in order to return the proper response code according to the RFC
however calling exists instead of read is more efficient on the
back end.

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

NETCONF-512: Add cases for missing NormalizedNode types

ReadDataTransactionUtil.prepareData was missing cases for
LeafSetNode, OrderedLeafSetNode, OrderedMapNode and UnkeyedListNode.

Change-Id: Iaf44ce9b24044584e37d524835ec4a51843acd32
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit c7e371eb1e23ba3b86747a27172acca78a449706)

Don't include stacktraces in REST responses

OWASP best practices recommend user facing error messages should not
contain stack traces or line numbers. The full stack trace should still be available in karaf.log.

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

Merge "NETCONF-509: no-input RPCs should use RestconfDocumentedException" into stable/nitrogen

Fix root data node QName equality check

This fixes the http error status 500 when trying to read
the whole mounted datastore using the new restconf
implementation based on RFC8040.

When serializing a NormalizedNode that represents
the data root we need to serialize only its children.

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

Add local MultivaluedHashMap implementation

Corresponding Carbon patch - https://git.opendaylight.org/gerrit/#/c/63054/

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 one.

Change-Id: I4f260f27684659fd734581617f6fa2002a6cd78c
Author:     Sai MarapaReddy <sai.marapareddy@gmail.com>
Signed-off-by: Sai MarapaReddy <sai.marapareddy@gmail.com>

NETCONF-509: no-input RPCs should use RestconfDocumentedException

When invoked with an "input" value currently the implementation throws
IllegalStateException up to the servlet container which results in the
ugly jetty default error page and reveals stack trace information.
Provide a clear message that the invoked RPC should not take any input values.

Change-Id: Id13499b80bed9a2ea7666a3aa6d775da341055ab
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>
(cherry picked from commit 914a50cc793909ffa813e60f9ac6c73b0cfbb5bc)

Bump versions by x.y.(z+1)

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

Merge "Close nested JSON writers" into stable/nitrogen

NETCONF-494: use RFC7951 JSON codecs

RFC8040 requires us to use a different codec for encoding instance
identifiers. Update the RFC8040 northbound to select this codec.

Also update the bierman02 northbound to explicitly bind to
the legacy LHOTKA codec.

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

Close nested JSON writers

With I9c597fe96937867cffdcbeeddd4b1507de20f2ee,
JSONNormalizedNodeStreamWriter.createNestedWriter() leaves it up to
the caller to close the provided JsonWriter. This patch ensures that
that is the case.

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

NETCONF-506: fix YII deserialization in FilterContentValidator

Treat leafref keys in the same way as indentityref keys.
Follows https://git.opendaylight.org/gerrit/#/c/53996/

See also NETCONF-384.

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

NETCONF-505: fix decoding of URLs with external leafref

Use mount point schema rather than global.

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

Set schemas to global context

- onGlobalContextUpdated schemas need to be set to the global context

Change-Id: I86c898d8b70c0f1a7fcc61be1cb0b542c8f05096
Signed-off-by: Miroslav Macko <miroslav.macko@pantheon.tech>

Add missing 'Path' annotation in jax-rs implementations for REST
services

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

Manual cherrypicks from oxygen

  * Fix hardcoded port
    * https://git.opendaylight.org/gerrit/#/c/65872/2
  * Add update method of handlers for rest services
    * https://git.opendaylight.org/gerrit/#/c/65458/
  * NPE when try get schema as yang
    * https://git.opendaylight.org/gerrit/#/c/65804/

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

Merge "Add missing flush() on buffered writer." into stable/nitrogen

Fix KeyPairProvider initialization in NetconfSshServerProvider

As a side offect of https://git.opendaylight.org/gerrit/#/c/60138,
RSA private key is stored unencrypted on disk.

Without additional countermeasures this might not be a good idea.

This patch sets DEFAULT_PRIVATE_KEY_PATH to null which disables
key pair serialization.

Change-Id: Ibbf51f702bc47d768db16d6a3f406a1b2ec906fb
Signed-off-by: Marek Gradzki <mgradzki@cisco.com>
(cherry picked from commit f01da36f40cdcc21a88c40330b43334d3eb06a84)

Bump versions by x.y.(z+1)

Change-Id: Ia4aa58a813a19e8891e279edf38e612a37c2d3c3
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>

Extra superfluous edit-config RPC sent - Netconf-482

Extra superfluous edit-config RPC sent just prior to correct edit-config RPC for top-level lists
POST & PUT to certain top-level lists in a mounted device's model returns a 500 error via Restconf,
although in some cases (device-dependent) the config appears to have committed.
This problem first appeared in Boron-SR3, and continues in Carbon SR-1.

Change-Id: I473e6e1f34d6f2bb9aa63f12b90fc479e4a8e3cd
Signed-off-by: Atul Gosain <agosain@luminanetworks.com>

Add missing flush() on buffered writer.

Missing flush caused, that backing ByteArrayOutputStream was empty
and thus mounted resources was not showing up in apidoc.

Change-Id: I14445b9f85cea1918321e59519129e21cb37dd81
Signed-off-by: Andrej Záň <andrej.zan@pantheon.tech>
(cherry picked from commit a2c0fda6b15dbf6717e2405bab868cda2fc209a3)

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>

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 9112: NPE in karaf cli when device is still connecting

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

Remove aaa version declarations

These are now properly published, no need to repeat them here.

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

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>

Merge "BUG-9132: don't provide a value for restconf/streams/events" into stable/nitrogen

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

Merge "Minor cleanup of blueprint config" into stable/nitrogen

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

Bump versions by x.y.(z+1)

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

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>

Merge "Bug 8988 - Check for empty payload properly" into stable/nitrogen

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>

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>

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>

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

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>

Update .gitreview to stable/nitrogen

Change-Id: Id226e9a7947e3502b256c6a605bacfd1402b2db8
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>

Bug 8745: Migrate EditConfig to the new XML parser

Switch EditConfig class to use the new XML parser from YANG tools.

Introduce EditOperationNormalizedNodeStreamWriter as a replacement
of the old XML parser extension EditOperationStrategyProvider which
has been used for tracking edit-config data tree operations defined
in the parsed XML source.

Remove EditOperationStrategyProvider as it is not needed anymore.

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

Bug 8824 - NETCONF request hangs when rpc-rply has invalid xml

The decoder handler is blindly throwing the SAXException but
nobody is there to intercept it. Also, as we're in the netty
world, to be able to propagate the exception, we're using the
NetconfMessage POJO.
That latest has been modified to accept either the decoded message
or an exception, if exception is thrown.

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

Bug 8832 - rpc-error in keepalive rpc-reply shouldn't bounce the session

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

Do not pull in yang-maven-plugin

This plugin is not used and pollutes the build environment with
incorrect version.

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

Remove JSON restconf-service config yang and Module classes

The module provides a service but it isn't widely used and most
likely any consumers have been converted to blueprint. Otherwise
a consumer still using CSS would need to be converted to
blueprint which is the future direction anyway.

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

Merge "Remove rest-connector config yang and Module classes"

Merge "Add JSONRestconfService impl for the restconf Draft18 impl"

Merge "Remove <Embed-Dependency> in netconf-config pom"

Remove rest-connector config yang and Module classes

The rest-connector has been converted to blueprint so the config
yang is no longer needed.

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

Remove <Embed-Dependency> in netconf-config pom

The pom embeds the threadpool-config-impl dependency. Not sure exactly
why this was done but I suspect as a work around for the java imports from
the blueprint XML not being included in <Import-Package>. We should not
embed the dependency. The maven-bundle-plugin can scan blueprint XML files
but there is an issue with using it so we can instead add the necessary
packages to <Import-Package>.

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

Upgrade to odlparent 2.0.2

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

Simplify RC4 removal

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

Use RSA for ssh server

The bouncy-castle bump increased the default size of DSA to 2048 which
some clients can choke on. We shouldn't use DSA anyhow so switch it up
to 4096 RSA everywhere.

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

Upgrade to odlparent 2.0.2

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

Upgrade to odlparent 2.0.2

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

Merge "Bug 8725: Remove netconf device mounting via the config system"

Merge "Skip SFT in features-netconf-connector"