Code Review / mdsal.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: e32efe4) | patch
Fix ActionProviderService(Adapter) 37/97137/2
authorPeterSuna <Peter.Suna@pantheon.tech>
Mon, 26 Jul 2021 11:19:32 +0000 (13:19 +0200)
committerRobert Varga <nite@hq.sk>
Wed, 11 Aug 2021 13:43:27 +0000 (13:43 +0000)
commit62fc1bafb3a0d5687865e6084251125e84ee863c
treea641dad236679d7b1ff2b26d3bd37145546d58c6tree | snapshot
parente32efe4c29c3c45c670c30d48a6589ffd5ed9884commit | diff
Fix ActionProviderService(Adapter)

ActionProviderServiceAdapter is always passing an empty set of
instances to DOMActionProviderService, which is a direct violation of
API contract. Unfortunately the implementation interprets empty set as
a no-op, resulting in the violation being ignored silently and routing
not working.

JIRA: MDSAL-679
Change-Id: I53a8977365166228ed30130519a2024372d89365
Signed-off-by: PeterSuna <Peter.Suna@pantheon.tech>
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit cbf88a5bd758d3cab9652b02b4d91b43f3cc939b)
binding/mdsal-binding-api/src/main/java/org/opendaylight/mdsal/binding/api/ActionProviderService.java diff | blob | history
binding/mdsal-binding-dom-adapter/src/main/java/org/opendaylight/mdsal/binding/dom/adapter/ActionProviderServiceAdapter.java diff | blob | history
binding/mdsal-binding-dom-adapter/src/main/java/org/opendaylight/mdsal/binding/dom/adapter/osgi/OSGiActionProviderService.java diff | blob | history
binding/mdsal-binding-dom-adapter/src/test/java/org/opendaylight/mdsal/binding/dom/adapter/AbstractActionAdapterTest.java [new file with mode: 0644] blob
binding/mdsal-binding-dom-adapter/src/test/java/org/opendaylight/mdsal/binding/dom/adapter/ActionProviderServiceAdapterTest.java [new file with mode: 0644] blob
binding/mdsal-binding-dom-adapter/src/test/java/org/opendaylight/mdsal/binding/dom/adapter/ActionServiceAdapterTest.java diff | blob | history
MD-SAL: Model-Driven Service Adaptation Layer