[NETCONF-949] Convert callhome-provider to OSGi DS Created: 06/Feb/23 Updated: 04/May/23 Resolved: 04/May/23 |
|
| Status: | Resolved |
| Project: | netconf |
| Component/s: | netconf |
| Affects Version/s: | None |
| Fix Version/s: | 6.0.0 |
| Type: | Task | Priority: | Medium |
| Reporter: | Robert Varga | Assignee: | Robert Varga |
| Resolution: | Done | Votes: | 0 |
| Labels: | pt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Epic Link: | Eliminate Blueprint | ||||||||||||
| Description |
|
callhome-provider's blueprint does not really use much in terms of the extension and should be amenable to conversion to OSGi DS. Convert it to reign in dynamic services. |
| Comments |
| Comment by Robert Varga [ 06/Feb/23 ] |
|
We need AAAEncryptService properly provided. |
| Comment by Robert Varga [ 08/Feb/23 ] |
|
This needs a ton more work, as the internals have been severely damaged in https://git.opendaylight.org/gerrit/c/netconf/+/54002 . Most notably there are components being created multiple times and not cleaned up (i.e. CallHomeAuthProviderImpl) as well as there are direct implementation-dependent callouts of questionable quality – i.e. CallHomeAuthProviderImpl is making calls into CallhomeStatusReporter, for which it creates its own instance, even when IetfZeroTouchCallHomeServerProvider These need to be cleaned up before we push for OSGi DS, as otherwise the wiring is ... funky. |