Convert sample-bundles to OSGi DS

This is yet another simplistic application, convert it to declarative
services.

JIRA: OPNFLWPLUG-1112
Change-Id: Ifd3feec479149846d97a6c65c98fa46604424b67
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ditch blueprint from drop-test-karaf

Use simple OSGi injection instead of a blueprint. This forces a refactor
in test-common, which ends up working much more nicely than before.

JIRA: OPNFLWPLUG-1112
Change-Id: I9496c9fd41bb6919e205ab354a35c03e1206b697
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ditch blueprint from table-miss-enforcer

This again is a very simple AddFlow-based application, use
annotation-driven activation.

JIRA: OPNFLWPLUG-1112
Change-Id: If994133803cc60020e0fde38f33a15b67c4a5c66
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ditch blueprint from of-switch-config-pusher

The blueprint is again extremely simple, migrate it to OSGi DS.

Also correct artifacts declarations, so that device-ownership-service
has a version declaration.

JIRA: OPNFLWPLUG-1112
Change-Id: Ib916383eb517bde7a4a3c2969ef58ec24aea53da
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ditch blueprint from device-ownership-service

We have an extremely simplistic blueprint with inject annotations
already present. Convert to using Declarative Services.

JIRA: OPNFLWPLUG-1112
Change-Id: I4346ac4d4acd50c354f9d2b2f8296598a8bae6aa
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ditch bluprint from sample-consumer

This sample is extremely simplistic. Turn it into a component.

JIRA: OPNFLWPLUG-1112
Change-Id: I83733f43e6dc965a16e085e7d0fee15206a7ee22
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Ditch blueprint from frm-sync

forwarding-rules-sync has an extremely simplistic blueprint, replace it
with Declarative Services wiring.

JIRA: OPNFLWPLUG-1112
Change-Id: I35d9add3e1b861cdd59000ed9ba0e395f706e6a6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not use RpcService in learning-switch

Use simple RPCs instead of rolled-up RpcServices. Also ditch blueprint
as our dependencies have changed.

JIRA: OPNFLWPLUG-1112
JIRA: OPNFLWPLUG-1125
Change-Id: I76f7a4de09c9eb66cc9d4cd4704485c7a0dac36d
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not use RpcService in bulk-o-matic

Use single RPCs instead of rolled-up RpcServices. We also register all
RPCs as an atomic entity via the SalBulkFlowRpcs component -- ditching
blueprint in the process of doing so.

JIRA: OPNFLWPLUG-1112
JIRA: OPNFLWPLUG-1125
Change-Id: Id7d1a1c8fa330d4f8d798d707fe06a9d8ca9da0e
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not use RpcService in forwardingrules-sync

Use simple Rpcs instead of aggregates, which gives us better
inter-component binding.

JIRA: OPNFLWPLUG-1125
Change-Id: Ie3c914cc7c9ca824a3a7eed6461bb2e657af7489
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Specialize {Add,Remove,Update}Flow implementations

The use of single-layer is a constant when we are instantiating RPCs,
hence we can specialize implementations rather than checking with
DeviceInfo on each invocation.

This leads to lower memory footprint as well as more efficient
execution.

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

Reduce mocking in FlowDirectStatisticsServiceTest

Do not mock things that we can build through generated code.

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

Correct FlowRegistryKeyFactory

@NonNull is a TYPE_USE annotation, place it accordingly. Also clean up
FlowRegistryKeyDto's equals() method.

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

Clean up MatchNormalizationUtil

Multiple things going on here:
- create a secondary weak cache for normalizers
- use ImmutableSet in both normalizer registries
- correct @NonNull placement
- ditch Optional.ofNullable() logic, clarifying what is going on

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

Reuse singleLayer boolean

We have already checked this property, reuse the result.

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

Decompose RPC implementation classes

RpcContextImpl registers a multitude of RPC services when it becomes
active. We really want to make that registration atomic -- which is
something we can do with
RpcProviderService.registerRpcImplementations().

This patch splits up our implementation classes to individual RPCs and
then uses a single call to register them in one go.

For bonus points we get rid of the horrible java.lang.IllegalAccessError
throws and correctly report RPC unavailability (because we do not
register those RPCs).

We also ditch OpendaylightDirectStatisticsServiceProvider and its
associated *LayerDirectStatisticsProviderInitializer, as they can be
cleanly bound without per-request lookups.

JIRA: OPNFLWPLUG-1125
Change-Id: I80ade2f9f51fc765c0d685f544e5b102061d033c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove unused routedRpcRegistration

This field is not referenced, remove it.

JIRA: OPNFLWPLUG-1125
Change-Id: Ia5c3b9c8e701e93268b1ff61e0d99ccb7e0e33b8
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Refactor RpcContext

The public API here does not make sense:
- lookup is only used in tests
- register is only used internally
- unregister is only used internally to lookup already-registered
  service

Eliminate public API methods and inline MdSalRegistrationUtils,
eliminate the need for various lookups.

JIRA: OPNFLWPLUG-1125
Change-Id: I28b4d8e4b8b63e715921cf8e851dcb3bb2f6a13a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Clean up RoleService

Use a switch expression to convert the OFJ enum, resulting in
elimination of the default case.

While we are here, perform overall cleanups by using:
- builders fluently
- adding listener directly
- accessing errors only once

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

Remove AbstractVoidService

RPCs have not been producing Void for a long time, remove
AbstractVoidService, as it has only be useful in that context.

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

Migrate StatisticsManagerImpl

Migrate to not use generated roll-up RPC service.

JIRA: OPNFLWPLUG-1125
Change-Id: I8b2bcf9d113d6d140d96bb64f374d500bea20241
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Do not use RpcService in test-provider module

Removed usage of RpcService from test-provider by registering individual
RPC implementations.

While we are at it, split up activation, so that we have
properly-decomposed components.

JIRA: OPNFLWPLUG-1125
Change-Id: I02bdb491b517bc5898bfa43bab7430fa69c28b6a
Signed-off-by: lubos-cicut <lubos.cicut@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Bump upstreams

Adopt:
- odparent-13.0.10
- infrautils-6.0.5
- yangtools-11.0.5
- mdsal-12.0.4
- controller-8.0.4
- aaa-0.18.4
- netconf-6.0.6

Also fixes up fallout from I35697d2544464d6cdc4ee88d13978eaabcf33353,
which broke unregistration -- this is now found by SpotBugs.

Change-Id: Id58314c7a4157e3c00a1ac0929964b9302be1756
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 0267562f3c2be39373f289008a367776dd325ff8)

Bump upstreams

Adopt:
- odlparent-13.0.7
- infrautils-6.0.4
- yangtools-11.0.4
- mdsal-12.0.3
- controller-8.0.3
- aaa-0.18.3
- netconf-6.0.5

Change-Id: I137ef1ac5d3e68f713badfefd84b47526b090693
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 44ff926b40d9f14f8967bb7392c9eeb24f6eaf09)

Bump upstreams

Adopt
- yangtools-11.0.2
- mdsal-12.0.2
- controller-8.0.2
- aaa-0.18.2
- netconf-6.0.4

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

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

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

Bump netconf to 6.0.3

Adopt latest fixes.

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

Fix odl-openflowplugin-flow-services-rest

We have a odl-restconf reference which is fixed to a single version. We
should be using a range.

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

Bump upstreams

Adopt:
- odparent-13.0.4
- infrautils-6.0.2
- yangtools-11.0.1
- mdsal-12.0.1
- controller-8.0.1
- aaa-0.18.1
- netconf-6.0.2

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

Fixup checkstyle

Upgraded checkstyle is catching new violations, fix them up.

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

Bump netconf to 6.0.1

Pick up latest fixes.

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

Bump upstreams

Adopt:
- odparent-13.0.3
- infrautils-6.0.1
- yangtools-11.0.0
- mdsal-12.0.0
- controller-8.0.0
- aaa-0.18.0
- netconf-6.0.0

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

Modernize codebase

Use following constructs:
- Optional.orElseThrow()
- BindingMap.of()
- List.of()

Also do not mock binding constructs, but rather rely of builders to
provide completely-functioning implementations.

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

Enforce modernize in openflowjava-extension-eric

We have a few violations here -- migrate to using Math.toIntExact() and
Objects.requireNonNull(). Flip enforcement on.

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

Enforce modernizer in openflowplugin-impl

We are using different objects here, which unfortunately is bound to
break at some point in the future. For now change the values to lie
outside of the cached range, use Integer.valueOf() and flip modernizer
to enforcing mode.

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

Modernize openflowplugin-impl

We have a ton of modernizer warnings here, bring them almost completely
under control.

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

Migrate users of Optional.get()

Upgraded modernizer is flagging these users, migrate them to better
alternatives. Since we require Java 17, also convert switches on enum
to switch expressions.

Change-Id: Iffea317aedce4e2e6660217b2a9d4fd277f9c451
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 38f326861531f68cb5aa48cc2b1fbdcc2d75885c)

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

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

Bump upstreams

Adopt:
- aaa-0.17.7
- netconf-5.0.4

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

Bump upstreams

Adopt:
- odlparent-12.0.3
- infrautils-5.0.3
- yangtools-10.0.4
- mdsal-11.0.7
- controller-7.0.4
- aaa-0.17.6
- netconf-5.0.3

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

Correct yangtools bump

Use yangtools-10.0.3.

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

Bump upstreams

Adopt:
- odlparent-12.0.2
- infrautils-5.0.2
- yangtools-10.0.3
- mdsal-11.0.6
- controller-7.0.3
- aaa-0.17.5
- netconf-5.0.2

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

Update docs version

The version on 'master' is 'Argon'. Update the version in the docs
configuration.

Change-Id: I2a092fcfcb1ed582f42edd04d070750f9435f6e1
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Do not guard registration with retries

DTCL registration cannot really fail, do not loop on it needlessly.

Change-Id: Ifd0c1b820ada46cca921e09c3866e0bbc25fa9b4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Clean up DropTest{Committer,RpcSender}

There is no point in catching exceptions during DTCL registration,
those are not allowed to fail transiently. Also do not catch exceptions
around Registration.close(), as those can never fail, either.

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

Adopt 2023.03 Argon upstreams

Adopt:
- odlparent-12.0.1
- infrautils-5.0.1
- yangtools-10.0.2
- mdsal-11.0.4
- controller-7.0.2
- aaa-0.17.3
- netconf-5.0.1

Deal with:
- yang.binding.Enumeration becoming EnumTypeObject
- BitsTypeObject components being primitive booleans
  - this implies that PortState has its fields set to false, unlike
    previously, when they could have been null
- DataBroker not allowing combined transactions
- Unions no longer accepting null member

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

Clean up PortUpdateTranslator/PortTranslatorUtil

Use a simple return with null check and instanceof pattern to reduce
verbosity a bit.

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

Replace slf4j-log4j12

Log4j12 is not developed any more and slf4j-simple should be used
instead of slf4j-log4j12 for routing test outputs.

Also remove unnecessary stubbings detected in some test classes.

JIRA: OPNFLWPLUG-1124
Change-Id: I7b17dfae58125523a3f3c570dd02d0323a3066ab
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Migrate test-provider to CompositeListener

Do not use generated Listener interface and prefer a CompositeListeners,
which allow more flexible dispatch.

Change-Id: I7848c91eb80900ed1bdde70f2f75c13325a41680
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Remove OSGiOpenflowDiagStatusProvider

We have constructor injection, hence we can unify this class with
DefaultOpenflowDiagStatusProvider.

Change-Id: I0a9d826c9d306b29c2325228294ad41dcff61141
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Migrate topology-manager to CompositeListener

Do not use generated Listener interface and prefer a CompositeListeners,
which allow more flexible dispatch. A husk FlowCapableTopologyExporter
is left intact for testing.

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

Migrate topology-lldp-discovery to simplified Listener

Rather than using the composite notification service, use the simpler
NotificationService.Listener.

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

Modernize test-common's use of NotificationService

Do not user generated PacketProcessingServiceListener, but rather
rely on modernized Listener interface.

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

Do not use PacketProcessingListener in learning-switch

We have only a single notification here, do not rely on generated
Listener interface.

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

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

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

Bump MRI upstreams

Adopt the following versions:
- mdsal-10.0.2
- controller-6.0.2
- aaa-0.16.3
- netconf-4.0.2

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

Bump MRI upstreams

Adopt the following versions:
- odlparent-11.0.1
- infrautils-4.0.1
- yangtools-9.0.1
- mdsal-10.0.1
- controller-6.0.1
- aaa-0.16.1
- netconf-4.0.1

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

Remove argparse4j declaration

odlparent is already providing this at a newer version, remove the
downgrade.

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

Bump upstreams for 2022.09 Chlorine

Adopt the following versions:
- odlparent-11.0.0
- infrautils-4.0.0
- yangtools-9.0.0
- mdsal-10.0.0
- controller-6.0.0
- aaa-0.16.0
- netconf-4.0.0

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

Use netconf-3.0.5

Pick up latest fixes from upstream.

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

Bump upstream versions

Adopt:
- yangtools-8.0.6
- mdsal-9.0.4
- controller-5.0.5
- aaa-0.15.5
- netconf-3.0.4

Change-Id: I1e0502f1f3bd27924b4313b0436172fdddf5e3c5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 3beb7b777502c069891a197c1c194e8cf3a13a9c)

Bump MRI upstreams

Adopt the following versions:
- odlparent-10.0.2
- infrautils-3.0.1
- yangtools-8.0.5
- mdsal-9.0.3
- controller-5.0.4
- aaa-0.15.4
- netconf-3.0.3

Change-Id: Id99ba0f08e53663af0771c39e60106d3996d89eb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 99bd25a7bb2305490b0ead613b264a278ca7ebd8)

Bump upstreams

Adopt:
- controller-5.0.3
- aaa-0.15.3
- netconf-3.0.2

Change-Id: I24887574b7003ed21c954e5774d28a83f48632a4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 808a80eac8162ccdbe8db51f34a1fb05fcd2e971)

Fix useless object

There is a leftover object, remove it.

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

Remove commons-codec dependency

This dependency is not used anywhere, make sure we do not get confused.

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

Clean up GetReconciliationStateProvider

Eliminate @SuppressFBWarnings and use Java Base64 encoder.

Change-Id: Ia9e52cd761cc252b7a2ae53cc89cfc55f76677a2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 2b878b3d096edaf41da2e3b4cf62617572f90978)

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

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

Bump upstream versions

Adopt:
- yangtools-8.0.3
- mdsal-9.0.2
- controller-5.0.2
- aaa-0.15.2
- netconf-3.0.1

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

Use explicit checkState() to guard against nulls

Rather than relying on implicit NPE, perform an explicit checkState().
Fixes tests on JDK17.

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

Modernize SwitchConnectionProviderImplTest

There is a ton of things wrong here, modernize the test. We will revisit
it soon for JDK17 compatibility.

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

Fix topology-manager Blueprint XML location

topology-manager container is not created at runtime because
the Blueprint XML file is not in the expected location:
'OSGI-INF/blueprint/' or 'org/opendaylight/blueprint/'.

Move the XML file into the standard Blueprint directory.

Change-Id: I47c166c4f1923299150eecc3053d36c5324fe2f1
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Rename autowire.xml

Rename autowire.xml to use the module name for consistency with
other Blueprint XML files.

Change-Id: Icb69c6cedcaa5abe9bb5559ad102b13bfeba7cbc
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Consolidate blueprint XML files for of-switch-config-pusher

There are two Blueprint XML files cross-referencing between them.
Merge them into a single file for better readability.

Change-Id: I903989a3251ab69c50c36498522c50fc16739414
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Consolidate blueprint XML files for FRM

There are two Blueprint XML files cross-referencing between them.
Merge them into a single file for better readability.

Change-Id: Ic72a7c1a16c131ba479627284defc8f786a06f2c
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Consolidate blueprint XML files for topology-lldp-discovery

There are two Blueprint XML files cross-referencing between them.
Merge them into a single file for better readability.

Change-Id: I0de4d6f80337a83136a4bc90caf496f943cc9108
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Consolidate blueprint XML files for openflowplugin-impl

There are two Blueprint XML files cross-referencing between them.
Merge them into a single file for better readability.

Change-Id: Ib46dd02259658484a278908dc8af46f553a9d4f7
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Fixup duplicate reference

Previous patch introduced a duplicate declaration, remove it.

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

Bump MRI upstreams

Adopt the following versions:
- odlparent-10.0.0
- infrautils-3.0.0
- yangtools-8.0.2
- controller-5.0.1
- aaa-0.15.1
- netconf-3.0.0

Also reconcile leaf-list mapping and correct feature packaging, so that
we do not repackage upstream bundles.

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

Cleanup switch certificate chain handling

SpotBugs is identifying a leak into static state. This is caused by a
rather thorough layering violation. Refactor ConnectionAdapter to always
report switch certificate -- cleaning up the code in process, making it
properly defensive.

JIRA: OPNFLWPLUG-1094
Change-Id: If48b57175347dab056424b8f1b169bd552ea029f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Migrate test-provider to OSGi DS

This is a very simple component, just migrate it over.

Change-Id: I455184dea3c1df406bad8e85e23886f86a4b3595
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Do not use blueprint-maven-plugin in of-switch-config-pusher

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Do not use blueprint-maven-plugin in topology-manager

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Do not use blueprint-maven-plugin in FRM

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Do not use blueprint-maven-plugin in device-ownership-service

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Do not use blueprint-maven-plugin in reconciliation-framework

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Do not use blueprint-maven-plugin in topology-lldp-discovery

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Do not use blueprint-maven-plugin in arb-reconcile-impl

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Correct subject-alternate-names

The ASN.1 definition is 'SEQUENCE', hence the order has significance.
Fix the definition.

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

Do not use blueprint-maven-plugin in openflowplugin-impl

Inline the generated container and dist dependencies on annotations
and the plugin.

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

Replace deprecated initMocks() in openflow-protocol-impl

initMocks has been deprecated. Use MockitoJUnitRunner instead.
Also, remove unnecessary stubbings and fix some typos in
openflow-protocol-impl.

JIRA: OPNFLWPLUG-1122
Change-Id: I9f30868c84b67a69b262044c047b43df1510ff93
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Remove unnecessary initMocks() in openflow-protocol-impl

Some of the initMocks() in openflow-protocol-impl are unnecessary.
Remove them.

JIRA: OPNFLWPLUG-1122
Change-Id: Iad89eccdd1a855b584ace9a79162f39785a3aba2
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Replace deprecated initMocks() in openflowplugin-impl

initMocks has been deprecated. Use MockitoJUnitRunner instead
and remove unnecessary stubbing in openflowplugin-impl.

JIRA: OPNFLWPLUG-1122
Change-Id: Iaf9843a0d310e85964c440f2e82310bc9ad95fd0
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Remove unnecessary initMocks() in openflow-protocol-it

initMocks() used in openflow-protocol-impl is unnecessary. Remove it.

JIRA: OPNFLWPLUG-1122
Change-Id: Ifad9cb5793086d7f573c56fe118b5148eb474ecb
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Replace deprecated initMocks() in topology-manager

initMocks has been deprecated. Use MockitoJUnitRunner instead
and remove unnecessary stubbings in topology-manager.

JIRA: OPNFLWPLUG-1122
Change-Id: I2ebeb322a1feaa78c7f2afbffe2a6fab0d9add52
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Replace Empty.getInstance() with Empty.value()

Empty.getInstance() is deprecated, use Empty.value() instead.

JIRA: OPNFLWPLUG-1123
Change-Id: I7aac7b757867879093b48d5f22788063b196ab18
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Update PTL info

Arunprakash stepped down as PTL:
https://lists.opendaylight.org/g/TSC/message/13998

Change-Id: I779eb9e08810f5f1cfafe42c87d2fa26d91ac4b3
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Remove node-errors.yang

node-errors.yang and its deprecated notifications are only used
in test-provider. Remove the data model and the implementation
in test-provider.

JIRA: OPNFLWPLUG-1119
Change-Id: I43a488d6e86c025837341a16d20e0617d0f4161f
Signed-off-by: Sangwook Ha <sangwook.ha@verizon.com>

Update MRI upstreams

Adopt:
- odlparent-9.0.13
- infrautils-2.0.13
- yangtools-7.0.14
- mdsal-8.0.11
- controller-4.0.10
- aaa-0.14.10
- netconf-2.0.14

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

Migrate to assertThrows()

ExpectedException.none() is deprecated, use assertThrows() to narrow
down the code that is expected to throw instead.

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

Do not instantiate JsonParser

Use parseString() static method instead of using the deprecated instance
method.

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