Return correct code 503 when a mountpoint is not mounted.

In case of mount point not mounted status code should be 503 and error
tag 'resource-denied-transport'

JIRA: NETCONF-341
Change-Id: I290b29e23952d30196c6d83d7984d5ebee4690a1
Signed-off-by: Iaroslav Kholiavko <iaroslav.kholiavko@pantheon.tech>

Add fields-aware NETCONF DOM API

- Added option to specify fields that are read from NETCONF device.
  Currently it is possible to only read one whole subtree
  using get/get-config NETCONF RPC. However, NETCONF RFC allows
  more complex subtree filtering including multiple subtrees
  in single RPC.
- It is implemented as:
  1) DOMDataBroker extension - option to create customized
     read-only, read-write transactions, and transaction chains.
  2) NetconfDataTreeService methods - using get(..) and
     getConfig(..) calls.

JIRA: NETCONF-735
Change-Id: I584bb9740f3ba6756ce283067198f9b344c75f9c
Signed-off-by: Jaroslav Tóth <jtoth@frinx.io>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor NetconfDataTreeServiceImpl

Rather than using boolean dispatch, create three specific
implementations to drive the logic behind the calls.

JIRA: NETCONF-705
Change-Id: I5c394ca78f7a98853fd99b8060a36d0db86d443d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Deprecate messagebus-netconf

messagebus-netconf is not used by anyone and has a good number
of deficiencies -- in its implementation as well as in the
framework (from controller).

Deprecate it for removal in the next release.

JIRA: NETCONF-761
Change-Id: I852b23fc30362c4ae609f04da9145580d887eca9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Unwanted commit to non candidate device fix

- netconf is sending commits to non candidate devices
  while editing their running config which results into
  NetconfDocumentedException: Unknown element "commit".

JIRA: NETCONF-755
Signed-off-by: mhurban <mar.hurban@gmail.com>
Change-Id: I536b43394269c4dd5223e4322ae830b813016ff0
(cherry picked from commit bf199d0da1729e4e68ad4059cfa8089d70d1932f)

Fix unstable test for NetconfDeviceCommunicator

The root cause of the issue is that this test requires more time for
verification on Windows-type OS, and sometimes that was a reason for
test failure.

JIRA: NETCONF-760
Change-Id: I7aab0f89fe4c466e2cdf772afffe3d54eb0f19df
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>

Generate new jax-rs Response for every /.well-known/* request

Generate new javax.ws.rs.core.Response every single time when
 request on `/.well-known/*` is processed, so the new output-stream
 will be used for every response.

javax.ws.rs.core.Response.status(Status.OK) creates new
 OutboundMessageContext with CommittingOutputStream, which is
 closed during the process of sending the response to client.
 Closed output-stream caused
 `java.lang.IllegalStateException: The output stream has already
 been closed.` when `/.well-known/*` endpoint was requesting
 multiple times, which made the endpoint working only once.

JIRA: NETCONF-757
Change-Id: Id8759602261333976337b11aa2c0fbfd1d3b04b9
Signed-off-by: Michal Banik <michal.banik@pantheon.tech>

Adopt akka-2.6.x

Adopt following upstreams to enable akka-2.6.x:
- controller-3.0.6
- aaa-0.13.1

Change-Id: Ib0ab4cb2f631559714383d4a4bae0ad672bf4e42
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>

Fix AAA feature references

We should be depending on version ranges for AAA dependencies,
add the corresponding overrides. Also remove a mistake in
features-netconf: it should not be depending on any AAA features
directly.

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

Depend on AAA 0.13.0 release

Reflect the fact upstream has a release, dropping the -SNAPSHOT
suffix.

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

Require odlparent-8.1 features

We really want to make sure we are using 8.1.x series features,
express that in feature.xmls.

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

Bump upstream versions

Adopt following upstreams:
- odlparent-8.1.0
- infrautils-1.9.5
- yangtools-6.0.4
- mdsal-7.0.5
- controller-3.0.5

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

Fix authentication mechanism in RESTconf(rfc8040 northbound)

Issue raised after https://git.opendaylight.org/gerrit/c/netconf/+/89419

JIRA: NETCONF-753
Change-Id: I50c13c3bf7d23f97f569fb53f45c315d0a3b72fe
Signed-off-by: Kostiantyn Nosach <kostiantyn.nosach@pantheon.tech>

Fixing errors check in RestconfDocumentedExceptionMapper

- Added check for number of errors that should not equal to 0;
  otherwise warning logs are generated.

Change-Id: I5e173d6389c0b17b278aae8abb0d585cb45e21d1
Signed-off-by: Jaroslav Tóth <jtoth@frinx.io>

Remove netconf-topology-singleton's NetconfConnectorDTO

This DTO is readily available from netconf.topology.spi, remove
code duplication and fix source registration leak -- allowing
modernizer to be enabled.

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

Fix a raw type warning

Add proper wildcards to silence warnings.

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

Enable modernizer in restconf-nb-bierman02

Clean up violations and flip the switch.

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

Fix restconf-nb-rfc8040 mocking

The structure of SchemaNode lookup has changed, migrate mocking
to the proper method.

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

Bump MRI upstreams

Adopt following versions:
- odlparent-8.0.3
- infrautils-1.9.4
- yangtools-6.0.3
- mdsal-7.0.4
- controller-3.0.4

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

Remove commented-out code

This got past review, fix it up.

JIRA: NETCONF-471
Change-Id: Ie9643a7765f267bdee5492a6d6e4a7a6677c765b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use java.util.Optional

This is the last real place where we use Guava's Optional, migrate
it.

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

Enable modernizer in where no errors are present

There are no violations present, flip enforcement on ensuring
it stays that way.

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

Cleanup warnings

Migrate isFoo() callers to use getFoo() and add @NonNull annotations
to constants so as to reduce Eclipse warnings.

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

Lower org.json exposure

osg.json package is now used only by bierman02 and unit tests,
adjust dependencies accordingly.

JIRA: NETCONF-471
Change-Id: Ifcefd3728ff349acfcbe52c47d1563d6f682ccbd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump MRI upstreams

Adopt following versions:
- odlparent-8.0.2
- infrautils-1.9.3
- yangtools-6.0.2
- mdsal-7.0.3
- controller-3.0.3

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

Integrate NetconfTopologyRPCProvider

This splits off NetconfTopologyRPCProvider into baseline topolocy
providets, hence it gets stood up automatically when the component
starts up.

JIRA: NETCONF-750
Change-Id: I01855370b397b172ae7bec9e07d2d8be5cdf1c71
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fixed domain-name usage in NETCONF southbound

- 'host' in the NETCONF mount request is described by union
  type - it accepts both IP address and domain-name.
- This domain-name wasn't considered as option during creation
  of connector to remote device - it ended with NPE.

Change-Id: I0f064aced76de3465da88376fcb6c6b589ed545f
Signed-off-by: Maros Marsalek <mmarsalek@frinx.io>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Implement RESTCONF Extensibility (discovery)

Added Root Resource Discovery mechanism according to RFC8040,
serving host-meta documents.

The copyright headers are the best we could back-engineer, as it
seems original code used copy&paste templating.

JIRA: NETCONF-499
Change-Id: I414ebda7e34db7cd468fb0e488bcac457d7cf7ff
Signed-off-by: Wsx25289 <10200860@zte.com.cn>
Signed-off-by: Iaroslav <iaroslav.kholiavko@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Hide NetconfTopologyRPCProvider.encryptPassword()

This method is exposed for testing, move its correspoding test
to the same package, reducing the method's visibility.

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

Fix NETCONF session(channel) activation for the CallHome(over TLS)

This patch fixes an issue when activation always fails with a message
saying that "Session is already activated". The root cause is the behavior
of the compareAndExchange method(returns the witness value, which will
be the same as the expected value if successful).

JIRA: NETCONF-5
Change-Id: Ib9b0acead1091c8e881330c22b99c9138605e1f1
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>

Fix checkstyle

Upgraded checkstyle is catching a few errors, fix them up.

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

Remove jaxb-api dependency

We need just a simple java.util.Base64 encoder, not the entire API.

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

Refactor KeepaliveSalFacade time tracking

This is very simple state tracking, but unfortunately it is utterly
convoluted. Essentially want to be sending keepalives as long as there
is not something outstanding already.

JIRA: NETCONF-717
Change-Id: I4ee045b4d3395572a59167948ea8843bcac584ce
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ditch blueprint from netconf-util

We do not need manual blueprint, just use OSGi DS.

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

Split transaction lifecycle

Change-Id: I0ec27e3720e23639b379f8ae594cb84dfb814516
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Switch default namespace for data-changed-notifications

Oversight in the original patch, this namespace needs to be present
on the data-changed-notification element.
Close xml writer at the end.
Rename "data-changed-event" to "data-change-event".

JIRA: NETCONF-743
Change-Id: Ia53c04bd0ef9f608804d0f884dac1f2180240343
Signed-off-by: Tomas Cere <tomas.cere@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Lower NotificationListenerAdapter

We do not want to flood logs here, drop the message to debug.

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

Zero-out canceled transactions

We may end up picking the transaction in read path, make sure we
null it away when closing.

JIRA: NETCONF-743
Change-Id: I537bace43a407a8a5f1adf5d6a7e8bbcb78e7da4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert aaa-authn-odl-plugin to OSGi DS

There is no point in keeping blueprint here, remove it.

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

Fixed handling of identityref type in list key

- There was not covered a case when key was leafref to
  the node which has type identityref.
- Handling failed parsing of identityref value - throwing
  RestconfDocumentedException with better description.

Change-Id: I4aa296cbb1a62b6f607323cb5e944d70e67ba3ff
Signed-off-by: Martin Sunal <msunal@frinx.io>
Signed-off-by: Jaroslav Tóth <jtoth@frinx.io>

Correct operations input/output media types

We do not produce/consume raw yang-data, add the missing XML
qualifier.

JIRA: NETCONF-741
Change-Id: I2978527125c80ae142af1f0b307c3b99adbe3ba9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Make ListenerAdapter serialize JSON directly

Change websocket notification serialization to serialize
to json directly instead of first serializing to xml and
subsequently converting to json.
While we are here fixup the websocket notification format
so it actually conforms to the sal-remote.yang model and
encode namespaces according to the json-ietf standard
instead of bundling an xmlns node in the json ouptut.

JIRA: NETCONF-471
Change-Id: I1b90c99980c0692b217523640bd6f67050f21e14
Signed-off-by: Tomas Cere <tomas.cere@pantheon.tech>

Eliminate unnecessary blocking checks

According to RFC6241:
1) "delete" operation must first check if the configuration data
   currently exists in the configuration datastore.
2) "create" operation must first check if the configuration data does
   not already exist in the configuration datastore.
3) "replace" operation replaces any related configuration in the
   configuration datastore. If no such configuration data exists in
   the configuration datastore, it is created.

If request goes via NetconfRestconfStrategy, we do not need to check
existence on the controller side, as the implied checks are part of
the underlyig behavior.

If request goes via MdsalRestconfStrategy, first we need to do
"if exists" check. Read requests in this case are collected in a batch
and then collecting the results.

JIRA: NETCONF-403
Change-Id: I6354c345d099017c1e82878af36ac37bbf5662c9
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

fix example string generation for types with isBasicLatin pattern

Generex generator fails to create example if pattern contains
{isBasicLatin} condition. The issue is fixed by replacing original
condition by range of allowed basic latin characters (0x00 - 0x7F).

JIRA: NETCONF-737
Signed-off-by: Cmarada, Michal <michal.cmarada@pantheon.tech>
Change-Id: I7bc5bf3621a40804122acd4604cfe8c132ffff24

Add new revision for odl-netconf-callhome-server.yang

JIRA: NETCONF-5
Change-Id: If5c9ca2045f7a13074da839b0c55d7484ac6a650
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Identify Call-Home device connected over TLS by the provided certificate

JIRA: NETCONF-5
Change-Id: Ie5008ac806e875902e1b28bc3aa94f6f7d3d466b
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Test JSONRestconfServiceModule

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

Split error messages

If there are more than one error message, we will split them by space

JIRA: NETCONF-403
Change-Id: I0136c8579916ccdf9fe06dc031f371ec17d8ee11
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Add Call-Home implementation for the TLS secure-transport

This is a quick implementation of TLS transport. There are a few
wiring issues, which will need a follow up.

JIRA: NETCONF-5
Change-Id: I666444b7c187d2afbc8869a32331cba4a1393add
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not use org.opendaylight.controller.sal.common.util.NoopAutoCloseable

NoopAutoCloseable hards back to Java 7 days and is an anachronism
with advent of lambdas. Replace its use with no-op lambdas.

As this is the last class used from sal-common-util in both components,
remove that dependency as well.

A quick follow-up audit shows we can remove the dependency from two
other artifacts, leaving rest-nb-bierman02 the only artifact still
needing it for AbstractMXBean.

Change-Id: Id97d3cdf110047778248e21021db39a6da2c3a5b
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Use @RunWith(MockitoJUnitRunner.StrictStubs.class)

MockitoAnnotations.initMocks(this) is replaced with @RunWith(MockitoJUnitRunner.StrictStubs.class)
Remove unnecessary Mockito stubbings

Change-Id: If775073f721a312e1126084aaabbf6ee22d2a77b
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Update infrautils to 1.9.1

We have 1.9.0 integrated, but downstreams are using 1.9.1. Align
versions.

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

Add workaround for SSH connection issue related to SSHD-1028

Provide custom version of Nio2Session which overrides the
bugged method.

JIRA: NETCONF-736
Change-Id: I0ef091680a9bdbe6bcab889335bb5ff48e91b703
Signed-off-by: Tibor Král <tibor.kral@pantheon.tech>
(cherry picked from commit 28925cdfe32f72ebcd034c9c7c9c107f176ca452)

Test SchemaExportContentYangBodyWriter

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

Remove Jakub Morvay from committers

Jakub's last contribution was on 29.3.2019, i.e. more than
18 months ago. Remove him from committers due to inactivity.

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

Add an explicit constants for WithDefaults parameter

RFC8040 explicitly enumerates allowed values, make sure we validate
them accordingly. Also fix the bug of ignoring unknown values.

JIRA: NETCONF-733
Change-Id: I5e4a19061c2c082563385a32ba574ddf97a8c967
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fixes from modernizer maven plugin in rfc8040 module

Remove some legacy APIs

Change-Id: I689190caf9be2357b2ffb2d2197783ebaba694d9
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Fix for enum constant

Now correct method name is calling

JIRA: NETCONF-403
Change-Id: Ia7b7531ab937fa346562fc4367c5e01aecbbb598
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Bump to mdsal-7.0.1/controller-3.0.1

Pick up mdsal fixes and improvements.

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

Remove dependency on MavenProjectAware

We will be deprecating this interface, so that everyone codegen
can work in other build systems. This plugin is not using, hence
the removal.

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

Add TLS data to call-home yang-models

The configuration model for callhome-server does not allow TLS to
be configured -- i.e. we are blindly assuming the devices will be
connecting via SSH.

Rectify this by deprecating the old configuration leaf and adding
the option to configure authentication on a per-transport basis.

JIRA: NETCONF-5
Change-Id: I56f14ad9472b5e87836be326a02c6663c229fb32
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>

Use SchemaPath->Absolute bridge methods

SchemaPath allows for easy transformation to ease migration, use
the methods as they improve readability.

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

Bump upstreams for Silicon

Adopt the following upstream versions:
- odlparent-8.0.0
- yangtools-6.0.0
- mdsal-7.0.0
- controller-3.0.0
- infrautils-1.9.0

Since mdsal's testing infrastructure now operates on binding packaging,
we ditch multiple testing copies of models, as we can now reuse already
existing packaging.

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

Remove Tom Pantelis from committers

Tom has been inactive in this project since 28.1.2019, i.e. more
than 21 months. Remove him from committers on grounds of inactivity.

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

Move ReadData.READ_TX_TYPE

This is a private constant, which ends up being misused in tests,
move it to its proper place and update tests accordingly. This
ends up simplifying callers.

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

Remove RestconfDataServiceConstant.PatchData

This is a useless holder class for a constant which should be private.
Move the string constant and remove the class.

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

Remove RestconfDataServiceConstant.PostData

This is a useless holder of a single String, move it towards its
majority user, noting a FIXME for reuse.

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

Remove RestconfDataServiceConstant.PutData

This is a useless holder class for a single constant, move the String
it its sole user and remove it.

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

Remove RestconfDataServiceConstant.DeleteData

This is a useless holder class for a private string, move the string
to its sole user and ditch the holder.

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

Inline checkItemExistsWithinTransaction()

This is a very simplistic method, inline it so we can properly
evolve it.

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

Fix insert/before logic error

Previous patch has made a logic mistake around negation and complements,
leaving Insert.AFTER broken. Fix it up.

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

Fold PlainPatchDataTransactionUtil.mergeDataWithinTransaction()

This is a very simple method, fold it into its sole caller.

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

Add explicit enumeration of allowed "insert" parameters

Using an enum allows us to simplify things and in future make sure
we have covered all possibilities.

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

Shortcut PutDataTransactionUtil.readData()

Simplify the dispatch between utils, making it clear we do not need
a SchemaContext in this path. It also clarifies the contract to the
point it is clear PutDataTransactionUtil.readList() is an utter
misnomer and the wrong place to do things.

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

Eliminate a ReadDataTransactionUtil.readData() alternative

We have a slew of readData() methods, eliminate one of them, which
is used only from tests and PutDataTransactionUtil.

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

Centralize RestconfStrategy allocation

Move dispatch logic for various implementations to RestconfStrategy,
so we have explicit control over it.

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

Mis a whitespace problem

We have a slight mis-alignment here, fix it up.

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

Binding QName constants to YANG Library model

We are depending on the model enum, hence we might as well bind
QNames to it, so that we have a consistent view.

JIRA: NETCONF-725
Change-Id: Ia04e7c9369c1f48f154d4694bc254ff126d6f35e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Simplify RestconfStrategy instantiation

Do not check mount point nullness twice.

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

Turn RestconfStrategy into an abstract class

The interactions between RestconfStrategy and FooBarUtil.bazXyzzy()
show that strategies may have some common logic. We do want to
properly specialize these and have them hosted somewhere, where
they can be decomposed.

Turn RestconfStrategy into an abstract class, so that we gain full
control of it and mark an earmark
TransactionUtil.ensureParentsByMerge() for migration.

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

Eliminate ParametersUtil

Methods here are only used by ReadDataTransactionUtil, rehost
them to eliminate some of the spaghetti.

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

Move TransactionUtil.checkItemDoesNotExists()

This method has callers only in PostDataTransactionUtil. Move it
there and hide it.

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

Move TransactionUtil.checkItemExists()

This method has a single user, move it there. Also mark
FutureCallbackTx.addCallback() for elimination, as it is obviously
blocking.

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

Refactor RestconfStrategy

This is part one of refactoring: a strategy implementation is only
dependent on the backing services, not on data being accessed.

Every caller should supply a strategy and the path separately, as
the strategy is an invariant.

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

Remove an unneeded dependency

restconf-nb-bierman02 does not use yang-library, remove this
dependency.

JIRA: NETCONF-725
Change-Id: I7f24f8cf3e4347a0c5f64d517c74db5b69afd0be
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Upgrade yang-library schema in sal-netconf-connector

Upgrade the schema used for determining the model set present
on the device. We will end up interpreting RFC7895 as RFC8525, which
is just fine.

JIRA: NETCONF-725
Change-Id: If74baa5476f84d511b7c28278cfc54749fb698b1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Squash registrations

Use a minimally-sized List instead of default allocation.

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

Add SchemaSourceRegistration list to NetconfConnectorDTO

When a NetconfDevice is initialized, we keep track of it via
NetconfConnectorDTO. As a device actually can have some YANG Library
source attached to it, we are registering them as well, but we are
failing to unregister them.

Keep the registration list in the DTO and route close requests
towards it -- allowing us to also remove the registrations.

JIRA: NETCONF-675
Change-Id: Id1d5fb2ff23aad766273e22d5f6f6639300fb3a5
Signed-off-by: PeterSuna <peter.suna@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Update NETCONF user documentation

- Remove old CSS configuration
- Add JSON payloads

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

Extend Websocket streams for data-less notifications

A new filter named "odl-skip-notification-data", similar to "odl-leaf-nodes-only", is added in the subscription API.
Using this filter, Client can get notification without data.

JIRA: NETCONF-689
Change-Id: I0cec77f69cb141fabc9f839c9a91626d3c667655
Signed-off-by: Nikhil Soni <nsoni@luminanetworks.com>

Bump MRI projects

This patch adopts:
- odlparent-7.0.6
- infrautils-1.8.1
- yangtools-5.0.6
- mdsal-6.0.5
- controller-2.0.4

Also adjust schema managers to work with IRSchemaSource instead
of ASTSchemaSource.

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

If there is only one revision is present, return it.

JIRA: NETCONF-720
Signed-off-by: Vladyslav Marchenko <vladyslav.marchenko@pantheon.tech>
Change-Id: I9103ef41af9bda1681baddf853fb3f739c20b620

Use YangConstants.RFC6020_YIN_MEDIA_TYPE

application/yin+xml media type is an RFC6020 definition, and hence
its common definition lives in YangConstants. Use it for there
instead of brewing our own.

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

Revert "Revert "Upgrade mina-sshd to 2.4.0""

This reverts commit f25f45ff27c8a7c7df780df609ec33f6662ea61e,
re-bumping to 2.5.1.

JIRA: NETCONF-716
Change-Id: I520d807c50a8107c0189745cf56e0fbc75871230
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Enable server heartbeats

When we configure the server to have any timeout, make sure we give
initiate a heartbeat at least three times before timing out. Also, if
we do not specify the idle timer, we need to disable NIO2 read timeout.

JIRA: NETCONF-716
Change-Id: I2c6ed07b5d3b29f6524b0ed99067895c4dfdc020
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Disable NIO read timeout by default

Rework which timeouts are disabled, as we are hitting timeout
after 615 seconds.

JIRA: NETCONF-716
Change-Id: I98ef51b4346c2aee0c9c87d58035bc08ef2c77d1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Revert "Upgrade mina-sshd to 2.4.0"

This reverts commit 560e49a0c5d879e763eb82f737db92bc78a276d3
as well as c2e07b76734c34a2fe88262530b4ac4a387b8ee7, as these
are suspected to cause a regression.

JIRA: NETCONF-716
Change-Id: Ie73f3a7b0b4f41e690b368035f212938916b40dd
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>

Revert "Expand rsa-ssh2 signatures"

This reverts commit 3453c508f27c64feb7220c84160a34df732b9738.

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: Ib3d442178877c82c760a625ff5acaa546ba797d4

Extra superfluous edit-config RPC sent

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.

JIRA: NETCONF-482

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I718bda377245d884864e160f2566e7c3d8e62e47

Bump conf.yaml versions to Silicon

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: I010ef27ac4307e4b2b2c3a501ffb7561ece5ddaa