Release Silicon

Bump MRI upstreams

Adopt following versions:
- odlparent-8.1.2
- infrautils-1.9.8
- yangtools-6.0.6
- mdsal-7.0.7
- controller-3.0.8
- aaa-0.13.3
- netconf-1.13.2

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

Re-create transaction chain upon failure in TopologyStatsProvider

Transaction failures can occur as a side-effect of various datastore
failure scenarios. In case of TopologyStatsProviderImpl, such a failure
requires manual intervention to restart the component (and thus
re-establish exports).

Rework transaction chain lifecycle, so that we close it down on failures
and establish a new chain when we need it for the periodic updates
and/or unbind operations.

Since we are modifying lifecycle, also make sure we work in terms of a
TimerTask with an externally-provided timer -- we really do not care how
we are scheduled.

JIRA: BGPCEP-920
Change-Id: I6f2be6a5c14ac6e191988c3b7bfd56b78b94abb6
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Instantiate BgpPeer only once

For some reason in clustered configuration BgpPeer might be instantiated
twice and this leads to service failure, this patch introduce additional
check to avoid this.

JIRA: BGPCEP-948
Change-Id: Icb355abf89ba1dee0281c32d6dd60c95fcdfdb61
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>
(cherry picked from commit d73b2dd19bbc0989f1c9fdc44419657d176f26cd)

Rework synchronization of SessionStateImpl

SessionStateImpl is a simple state tracker, but its lifecycle was
needlessly complex due to it not being inherently tied to its session.

Make sure SessionStateImpl is instantiated only when we have a session,
which makes a number of operations clearer: we no longer need to deal
with partial initialization and need to protect only internal state.

While we are at it, simplify all of that by keeping simple longs
instead of LongAdder -- and use saturatedOf(long) to convert these to
Uint objects, preventing possible runtime exceptions when overflows
occur.

JIRA: BGPCEP-920
Change-Id: I57b33c7dc33fe63500f5ad96a80d39889c6c5a67
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit de6df3ea177d609faf693fe5c32479ba66998b7f)

Handle TopologyNodeState release properly

- Close transaction chain when PCEP session goes down
- Remove NodeState from the corresponding map in SessionManager properly

JIRA: BGPCEP-920
Change-Id: Ica9dc0a6f43420b5b57694252cb13d2b5350b103
Signed-off-by: Deepthi V V <dvv@luminanetworks.com>
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 3bf063839d01f6c4b80aa4c02ea85d80feae8bfd)

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

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

Bump MRI upstreams

Adopt following versions:
- odlparent-8.1.1
- infrautils-1.9.6
- yangtools-6.0.5
- mdsal-7.0.6
- controller-3.0.7
- aaa-0.13.2
- netconf-1.13.1

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

Update .gitreview to stable/silicon

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

Bump netconf to 1.13.0

Adjust version for MRI-integrated NETCONF.

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

AbstractPeer.createPeerPath() should not be synchronized

This method depends only on peerID and RIB, let the caller provide
the intended ID -- and we can be unsynchronized, reducing cognitive
load.

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

Remove unneeded AbstractRIBTestSetup mocks

We are not using binding interfaces anymore, do not mock them.

JIRA: BGPCEP-853
Change-Id: I5922f2eaca1a7ed87ea5ca52466191f3f7885014
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove RIBImpl's use of DataBroker

We do not need the binding DataBroker anywhere in the core logic,
do not propagate it there.

JIRA: BGPCEP-853
Change-Id: I283857e7dce16b66aa6d83ab25c87adbb6b75e88
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Adopt akka-2.6.x

Bump upstream to controller-3.0.6 to adopt akka-2.6.x.

Change-Id: I5b8732ff17ee47b6a0acd385eeb8ceaf4f1a1280
Signed-off-by: Oleksii Mozghovyi <oleksii.mozghovyi@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: I3bb880b1ebd3ef23b4cd15a3bcab56fc9fd9f529
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not expose Binding access through rib.impl.spi.RIB

We should not be touching binding layer at all, eliminate one avenue
of access.

JIRA: MDSAL-583
Change-Id: I4a3c7d6f864e4b69fc5d0fdc41e0aa1acd5ae341
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove Peer.getRawIdentifier()

This method is not used anywhere, remove it.

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

Clean up AbstractPeer fields

Minimize fields visible to subclasses and make a note of locking
improvement.

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

Rework ribOut interaction

As it turns out, we are using a binding chain separately from the
main chain. The reason for this is that we use domChain for processing
towards from the peer -- which is common for both. bindingChain is
used exclusively for ribOut writeout.

Rename it and correct the guard/use mismatch we introduced in the
previous patch.

JIRA: BGPCEP-853
Change-Id: Idac28712ae9603f7cbfefe613baf2b47c7daff17
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: Id098b859b916fcf34eac04467e90dd7f38785d87
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix RIB documentation

We have a bad reference here, fix it.

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

Document AbstractPeer.bindingChain visibility

This field accessed by a test, annotate it as such and also make
sure we comply with synchronization rules.

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

Initialize local directly

We are wasting a line and some clarity here, fix that.

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

Invert check

The if-body is only a single line and may easily be missed. Save
us a negation and more readable as well.

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

Drop <R, I> generic from RIBSupport

We do not have anything depending on R, I in the interface contract,
be sure to drop the generic arguments to simplify users. This cascades
to other bgp-rib-spi constructs as well.

JIRA: BGPCEP-853
Change-Id: Ib8212f26946fe02f70a947e684aa199785f974d4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove extractAdjRibInRoutes()

This method is not used anywhere, hence we can safely remove it.

JIRA: BGPCEP-853
Change-Id: I5c623f757e94ad28fc0c8b8e08bacf66f619b929
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert LocRibWriter to DOM APIs

LocRibWriter is not very complicated, but it affects also best path
selection and add-path objects. This patch performs a straight conversion
without cleaning up RIBSupport and related classes. That part will be
done in a follow-up patch.

JIRA: BGPCEP-853
Change-Id: If6b22adf8f9a15c744ed873a5e114a251023c7a8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

fix use of read-only transactions

 in openconfig-rp-statement

-  make sure transactions are used in a proper try-with-resources block
- use Optional.map() handling rather than the explicit isPresent()/get() combo

JIRA: BGPCEP-861
Signed-off-by: Gasparini, Claudio (cg479k) <claudio.gasparini@intl.att.com>
Change-Id: Icb46adc1e4d7be70a2151f26e25229baa64c87c9

Use ImmutableMap.toImmutableMap()

Rather than using a temporary map followed by ImmutableMap.copyOf(),
use ImmutableMap's collector.

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

Be more defensive when checking nodes

We have a couple of failures, where we end up throwing a NPE
rather than an assertion. Clean them up.

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

Cleanup PathComputationImpl

Remove superfluous braces and be smarter about filling out the
address family.

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

Cleanup boolean literals

A number of places are using explicit literals where not needed,
as pointed out by sonar. Fix these sites.

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

Simplify matchAfiSafi

Sonar is pointing out a boolean literal, remove it by using a
smarter expression.

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

Simplify AbstractStatementRegistryTest

All state here can use field initializers, hence remove mucking
with @Before.

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

Cleanup AbstractStatementRegistryConsumerTest

Make sure we properly clean up the activator. While we're here,
also use read-only transactions for reads, close them as needed
and do not read more than we need to.

Also improve CheckUtil to report received messages when an
assertion fails -- and property assert errors.

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

Migrate boolean getters

DataObject.isFoo() is now deprecated, use getFoo() instead. There
is also a bunch of drive-by updates to expressiveness, fixing various
(bogus) Eclipse warnings.

Tests relying on explicit order are documented as such and refactored
for clarity.

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

Trim down AdvertizedRoute size

Use alignment shadow to store the additional boolean, as it Java <15
will not do that for us.

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

Optimize isNonAddPathBestPathTheSame()

This is an Eclipse automatic logic optimization: in case of bestPath
being empty/null, we do not need to evaluate the second part of the
expression.

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

Remove RIBSupport.emptyRoutesContainer()

This method is not called anywhere simply because this container
has no semantics in the datastore. Remove the method and all its
implementations.

This means that extensions do not load the associated Builder and
its implementation class. The second is more important, as it means
that the routes container will have only one implementation (from
mdsal-dom-codec) -- allowing JIT to devirtualuze based on CHA.

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

Switch TerminationReason to use Uint8

As binding has switched to more expressive Uint types, embrace them
to allow smaller number of conversions.

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

Use BindingMap.Builder to fix list-related warnings

BindingMap allows us to keep a list-like builder and result in a
Map -- thus arriving at what we want.

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

Further algo-impl improvements

Cleanup invocations to .toString() as well as its implementations
and declarations.

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

precise OpenConfig models original license

add a NOTICE and an openconfig.LICENSE files in the bgp module

OpenConfig YANG models are published under the Apache License version 2
a.k.a. ALv2 but they do not precise it in their description.

Though Alv2 allows redistribution under other licenses, it includes a
redistribution clause (4) that instructs to give any other recipients
of the work or the derivative works a copy of the original license.

Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Id64f9473ebd43df0c851a0c78533ea593634fe60

Cleanup algo-impl classes

Make sure we are using simple initializers and diamond-notation,
plus the usual autosave actions.

Change-Id: Ieaaa0c7294ee95a64a04f510d95bfb47ed522b91
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: I26a2d923a4509ebf5817878a29e7776acff23d3a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor SimpleMvpnNlriRegistry

This is an internal implementation singleton, populate it statically,
cutting some complexity.

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

Wire SimpleEsiTypeRegistry statically

This is an implementation detail, wire the registry statically.
We can optimize it afterwards.

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

Wire SimpleEvpnNlriRegistry statically

This registry is populated with invariants and is an implementation
detail. Move it to impl.nlri and make it self-populated on creation.

This will allow us to optimize it further.

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

Do not populate SimpleNlriTypeRegistry from BGPActivator

SimpleNlriTypeRegistry has static contents, populate it upon
creation, so that we can treat it as an immutable entity.

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

Populate SimpleBindingSubTlvsRegistry statically

This is a standalone dispatch registry. Wire it statically as a
first step before we optimize it.

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

Migrate Ipv4NlriParserTest

Use MockitoJUnitRunner instead of explicit initMocks().

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

Refactor SimpleRouteTargetConstrainNlriRegistry

This registry is utterly static, populated in various ways. Rather
than using Handlers, fine-tune it for the 4 cases we really care
about. We use an ImmutableMap for serializers and a switch() dispatch
to static methods for parsers.

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

Convert bgp-rib-spi to OSGi DS

This again is a plain whiteboard pattern, make the minimal changes
required to work with OSGi DS and other frameworks. Also makes
RIBExtensionProviderActivator implementations properly stateless,
taking out a chunk of static wiring out of various supports.

JIRA: BGPCEP-944
Change-Id: I5dfaf8562a3fc79ea4a8328e6b9db8b7d5537f86
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove unused blueprint

This blueprint is not used anywhere and references non-existent
classes. Remove it to reduce confusion.

JIRA: BGPCEP-925
Change-Id: I9afa164bba6e9f75b9049b73e6d9d51b7b332280
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Merge if statements

Sonar does not like this nested if, fix that.

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

Fix OSGiBGPTableTypeRegistryConsumer whiteboard

We need to have a greedy policy here, otherwise we might miss some
extensions.

JIRA: BGPCEP-942
Change-Id: I33589f167be78d10139979dc62ab2a17e74e455a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix OSGiRSVPExtensionConsumerContext whiteboard

We need to have a greedy policy here, otherwise we might miss some
extensions.

JIRA: BGPCEP-923
Change-Id: I5111e4710d196bc26cce03eac8ebc5238de68c22
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert openconfig-rp-impl to OSGi DS

Add proper injection of dependencies and remove blueprint. With all
the previous work this is rather trivial.

JIRA: BGPCEP-943
Change-Id: Ia8a6e745e678c7359ec1fea073978d243146bee4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert rsvp-spi to OSGi DS

The SPI wiring here is a bit contrived, this performs minimal
conversion, eliminating AbstractRSVPExtensionProviderActivator.

JIRA: BGPCEP-923
Change-Id: Iff29e1ed7ea3630315553f946286cf40ac0344c9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove blueprint from rsvp-impl

rsvp-impl ends up being very simple in what it does, convert it
to OSGi DS.

JIRA: BGPCEP-924
Change-Id: I365e3daf26edc28ea842e59448144adbacbf4135
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert bgp-flowspec to OSGi DS

As it turns out, FlowspecActivator is just a bunch of static wiring,
which we do not need to inject. Refactor SimpleFlowspecTypeRegistry
into a dedicated immutable FlowspecTypeRegistry, which has provably
static lifecycle and used only for immutable access.

This cuts a ton of complexity and fixes ServiceLoader injection,
plus makes DI conversion a breeze -- eliminating static state in
process of doing so.

JIRA: BGPCEP-941
Change-Id: I0ad4fef346131b44f7f5bca2a637826e821146d7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor BGPTableTypeRegistryProviderActivator

Rather than wiring through static reference extension in blueprint
and keeping stateful singletons, use a proper whiteboard injection
pattern for OSGi, ServiceLoader and Singleton worlds.

This makes bgp-openconfig-spi to completely wire natively in those
environments.

JIRA: BGPCEP-942
Change-Id: I325ab8b3cf09bc71cad217e6c1f26c6c69bc0cc1
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert BGP extensions to OSGi DS

Most of these are straightforward ports.
bgp-linkstate will need a follow-up to deal with BGPActivator.
bgp-flowspace will need a follow-up to deal with BGP/RIBActivator.

JIRA: BGPCEP-941
Change-Id: Ib6ebbb41b3cd1095e1809e91a46fee38ebaf722f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Lower binding-spec-util usage

We do not use reflections in main artifact in most cases, hence
we should not be pulling it as a dependency.

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

Remove use of BindingReflections.getAugmentations()

Augmentations are directly obtainable, simplify our code.

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

Migrate flowspec tests to MockitoJUnitRunner

Do not use initMocks() directly.

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

Eliminate blueprint from bgp-openconfig-rp-spi

The only specialty we are using is the reference list, hence we can
convert to a properly-injected component.

JIRA: BGPCEP-939
Change-Id: Ic9d9e3fa6fb8641bbc9b74e58bbe8338e7491e36
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert bgp-route-target to OSGi DS

This is really a simplistic component, convert it to use OSGi DS
and other frameworks.

JIRA: BGPCEP-940
Change-Id: I0781ce8bc40c1245292e950be939a17d2aa3d5e7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove blueprint from bgp-openconfig-rp-statement

This is a rather straightforward conversion, also add @Singleton
as appropriate.

JIRA: BGPCEP-938
Change-Id: Ibc5d4af8cf88a05f615a7c50a9a8c115b8c2e058
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove hand-written ServiceLoader services

@MetaInfServices provides automatic generation of these files, hence
use it to make bindings to code more explicit.

JIRA: BGPCEP-937
Change-Id: I9a4a5627cddfdd7898b83414404d1cae069795ed
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert bgp-parser-impl to OSGi DS

The blueprint here is very simplistic, just convert it to an
immediate component.

JIRA: BGPCEP-936
Change-Id: I1a4f8d9b8637e5f5dcc18596b44c017d56eb5af2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove blueprint from pcep-base-parser

This is very simplistic component, convert it to an immediate OSGi
DS component.

JIRA: BGPCEP-935
Change-Id: I793716f7404f63b3fa129877b59ebaab219ad6ee
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert data-change-counter to OSGi DS

This is very simple change, convert to OSGi DS and javax.inject.

JIRA: BGPCEP-930
Change-Id: I8a00dc85185c662720db1929aec6d3849f1d13d9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Update infrautils to 1.9.1

This upgrade was missed last time around, catch up.

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

Improve config loader performance

Global context does not change often and hence to is appropriate
to find the root schema node we need for parsing when that happens
and cache it.

This also means we can perform upfront checks and neatly ignore
any registrations which are not satisfied by the model context.

Furthermore OSGiConfigLoader is updated to take mode updates without
restarting, improving overall experience.

Since we are in the area, we also harden XML parsing through use
of UntrustedXML.

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

Switch ConfigFileProcessor to use SchemaNodeIdentifier.Absolute

SchemaPath will be going away and it is a quite bad concept, switch
to its replacement, which is properly founded in RFC7950.

JIRA: BGPCEP-932
Change-Id: I6b4d843130384f50aeeedf866172071566b5ade2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch ConfigLoader to mdsal-binding-runtime

There is no need for the config loader to require Binding/DOM
translation services, hence it is enough when it bindings to
mdsal-binding-runtime instead of mdsal-binding-dom-adapter.

This allows speedier start of services with smaller overall
footprint.

JIRA: BGPCEP-933
Change-Id: I8aa9569591565dd440ac528bccc413b5aea7e358
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Switch ConfigFileProcessors to DOM API

ConfigFileProcessors do not need to interact with Binding API, speed
them up and simplify them by using DOM APIs directly.

JIRA: BGPCEP-933
Change-Id: If20c7f64dd4c708cd69a5c8c3ec6d4358b5b405e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ensure WatchService is cleaned speedily

Rather than registering a shutdown hook, we prefer to use a cleaner.
While this allocates a thread, it is not a privileged action and we
kill the service as soon as we are GC'd.

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

Remove blueprint from topology-config-loader

This is a straight conversion from blueprint to OSGi DS/inject.

JIRA: BGPCEP-929
Change-Id: Ie098c51808c8a4aa11357605f3f802894152f6f3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove blueprint from protocols-config-loader

This is a straight conversion from blueprint to OSGi DS/inject.

JIRA: BGPCEP-927
Change-Id: I3c0c122a21229d481a4cae7ced582308f033d82a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove blueprint from bmp-monitors-config-loader

This is a straight conversion from blueprint to OSGi DS/inject.

JIRA: BGPCEP-926
Change-Id: Ifb83fa309f148a1b72c21561df3410238c637026
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Fix intermittent failures in LlGracefulCapabilityHandlerTest

The test is expecting ordering to be completely kept, hence we need
to use an alternative BindingMap.

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

Remove blueprint from routing-policy-config-loader

This is a straight conversion from blueprint to OSGi DS/inject. The
patch is a bit larger as it introduces some basic infrastructure
for reuse across implementations.

JIRA: BGPCEP-928
Change-Id: Ica5a317cb6f44cf1baec8a9802af85c3b77e1c43
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert config-loader to OSGi DS

The primary task here is getting rid of explicit blueprint while
retaining autowiring functionality.

This forces us to decompose ConfigLoaderImpl into multiple classes,
which ends up being a boon for tests, as they can completely forgo
threading.

First of all FileWatcherImpl was renamed to DefaultFileWatcher and
is a completely separate component, with proper lifecycle without
edge case failures.

Next we split out AbstractConfigLoader with all the loading logic
and AbstractWatchingConfigLoader with the addition of having notion
of a FileWatcher. This allows us to use synchronous loading in
test harness, while also providing the ability to have async
processing.

We then instantiate SimpleConfigLoader and OSGiConfigLoader to
handle the cases of static vs. dynamic codecs, as well as
TestConfigLoader to handle unit testing tasks.

We can therefore drop dependency on odl-controller-blueprint, in
favor of depending on YANG tools codecs instead.

JIRA: BGPCEP-905
Change-Id: I5549bdfeb372b2f9cb15a1690c399401432f4702
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

rename cfg-ldr classes before OSGi DS conversion

this will ease the review of the OSGi DS conversion to come
https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes

Change-Id: I8a4419d45831f11082623ca696c8b92374643e12
Co-authored-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>

Fix List-related warnings in bgp-rib-impl

Use BindingMap to fix warnings around deprecated methods.

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

remove attributes odl:type default from blueprints

Their use is discouraged since Aluminium.
https://docs.opendaylight.org/en/stable-aluminium/release-notes/upgrade-process.html#use-of-odl-type-in-blueprint-is-discouraged

Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I3bb5832b439202d9539338afb71433d34573b74a

remove some useless empty lines

this will ease the review of the OSGi DS conversion to come
https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes

Change-Id: I01aad5992f174e82dd5ea31eed69f33a4cc81167
Co-authored-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>

update wiki URL in poms

Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I1d83a481ff90fe9b1eb30c338bf1704852c4afbc

Clean up List-related warnings in bgp-parser-impl tests

Use BindingMap to fix deprecation warnings.

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

refine path declaration in config-loader FileWatch

Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: I2116c38b0b2a03a24c7492dc2662e6a51e9e2f48

Bump to mdsal-7.0.1/controller-3.0.1

Pick up mdsal fixes and improvements.

Change-Id: I464319a4b2c621100db884e9272544c822585ff6
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-SNAPSHOT

Since MD-SAL has removed some legacy models we are using, we are
re-introducing them here.

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

Fix uint usage warnings

This is another pass through warnings, fixing simple offenses.

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

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

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

Added JSON and XML payloads tabs with RFC 8040 URL

Modified docs/bgp/bgp-user-guide-long-lived-graceful-restart-capability.rst by adding JSON,XML examples with corresponding sphinx tabs

Change-Id: I277e40fb90b1140d9c59ff492024db0361845688
Signed-off-by: Aman <170030039@iitdh.ac.in>

Added JSON and XML payloads tabs with working RFC 8040 URL

Modified docs/bgp/bgp-user-guide-ip-unicast-family.rst by adding JSON,XML examples with corresponding sphinx tabs

Change-Id: I277e40fb90b1140d9c59ff492024db0361845525
Signed-off-by: Aman <170030039@iitdh.ac.in>

Added JSON and XML payloads tabs with RFC 8040 URL

Modified docs/bgp/bgp-user-guide-route-refresh-capability.rst by adding JSON,XML examples with corresponding sphinx tabs

Change-Id: I277e40fb90b1140d9c59ff492024db0361836718
Signed-off-by: Aman <170030039@iitdh.ac.in>

Added JSON and XML payloads tabs with RFC 8040 URL

Modified docs/bgp/bgp-user-guide-linkstate-family.rst by adding JSON,XML examples with corresponding sphinx tab

Change-Id: I277e40fb90b1140d9c59ff492024db0361845468
Signed-off-by: Aman <170030039@iitdh.ac.in>