====================================== #opendaylight-meeting: kernel projects ====================================== Meeting started by rgoulding at 15:55:15 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-meeting/2018/kernel_projects/opendaylight-meeting-kernel_projects.2018-08-28-15.55.log.html . Meeting summary --------------- * agenda bashing (rgoulding, 15:55:17) * DataBroker (or DOMStore) test suite (“TCK” ?) for all impls (rgoulding, 15:55:38) * DOMStore base impl to share beetween InMemoryDOMDataStore and alt impl? (rgoulding, 15:55:47) * LINK: https://jira.opendaylight.org/browse/NETCONF-101 (rgoulding, 15:55:54) * LINK: https://jira.opendaylight.org/browse/NETCONF-213 (rgoulding, 15:55:57) * remove blueprint plugin generatedDir (rgoulding, 16:05:14) * blueprint plugin generatedDir removal (rgoulding, 16:06:12) * LINK: https://git.opendaylight.org/gerrit/#/c/75180/ (rgoulding, 16:06:21) * the basic premise is to get rid of src/main/resource/org/opendaylight/blueprint in favor of standard OSGI-INF (rgoulding, 16:07:43) * also gets rid of Import-Service and Export-Service from the bundle manifest (rgoulding, 16:09:33) * during maven-bundle-plugin (rgoulding, 16:09:45) * our custom generatedDir will be honored for backwards compatibility for now (rgoulding, 16:10:52) * this will come in 4.0.0 of odlparent (rgoulding, 16:12:02) * vorburger says if it works he is for pushing it in (rgoulding, 16:14:19) * the backwards compatibility mentioned above is only at runtime; when we migrate to odlparent 4.0.0 then we will have to move everything to OSGI-INF (rgoulding, 16:18:59) * clustering update (rgoulding, 16:20:10) * NETVIRT-1324 (rgoulding, 16:21:49) * LINK: https://jira.opendaylight.org/projects/NETVIRT/issues/NETVIRT-1324?filter=allopenissues (rgoulding, 16:21:53) * shague updated and said it still has WARN in controller (rgoulding, 16:22:21) * the application is doing retries, but there are still WARN logs (rgoulding, 16:22:43) * emitted from LocalThreePhaseCommitCohort (rgoulding, 16:23:08) * the issue is that the application handles this, but the log is then misleading (rgoulding, 16:24:12) * since the application actually looks at the returned Future (rgoulding, 16:24:23) * should we close this bug since the logging is considered normal (rgoulding, 16:24:48) * shague says the WARN is okay. Wants to leave open since he wants to work through the test surrounding this. The controller emitting warn is fine (rgoulding, 16:25:32) * vorburger to put explanation as to why the WARN from opendaylight-cluster-data-akka.actor is fine (rgoulding, 16:26:54) * where was this discussion originally had? we should link that in to the bug (rgoulding, 16:27:08) * LINK: https://jira.opendaylight.org/browse/CONTROLLER-1855 (vorburger, 16:27:53) * we can get back to whether this should be WARN or DEBUG in a later discussion. (rgoulding, 16:28:25) * DataBroker (or DOMStore) test suite ("TCK" ?) for all impls (rgoulding, 16:29:12) * vorburger doesn’t care about htis one as much (rgoulding, 16:29:22) * DOMStore base impl to share beetween InMemoryDOMDataStore and alt impl? (rgoulding, 16:29:30) * the question vorburger has is that he has toyed with implementing a new DOMDataStore, and found out he was copying/pasting a lot of code from InMemoryDomDataStore (rgoulding, 16:30:03) * he couldn’t extend it so he copied/pasted (rgoulding, 16:30:16) * should there be an SPI for implementing DOMDataStore where this code could be reused (rgoulding, 16:30:32) * DOMStore not DOMDataStore above ^^^ (rgoulding, 16:31:00) * should there be something common for implementors of DOMStore (rgoulding, 16:31:26) * LINK: https://github.com/opendaylight/mdsal/blob/master/dom/mdsal-dom-inmemory-datastore/src/main/java/org/opendaylight/mdsal/dom/store/inmemory/InMemoryDOMDataStore.java (rgoulding, 16:32:45) * vorburger to follow up with this offline (rgoulding, 16:32:55) * reasons why he had to copy/paste and why he couldn’t extend (rgoulding, 16:33:24) * most people would require digging back in to answer anyway (rgoulding, 16:33:34) * NETCONF-101 (rgoulding, 16:33:56) * LINK: https://jira.opendaylight.org/browse/NETCONF-101 (rgoulding, 16:34:21) * NETCONF-213 (rgoulding, 16:35:22) * ryan to update oltu patch (rgoulding, 16:37:43) Meeting ended at 16:37:46 UTC. People present (lines said) --------------------------- * rgoulding (48) * odl_meetbot (4) * vorburger (2) Generated by `MeetBot`_ 0.1.4