#opendaylight-meeting: kernel projects

Meeting started by rgoulding at 15:55:15 UTC (full logs).

Meeting summary

  1. agenda bashing (rgoulding, 15:55:17)
    1. DataBroker (or DOMStore) test suite (“TCK” ?) for all impls (rgoulding, 15:55:38)
    2. DOMStore base impl to share beetween InMemoryDOMDataStore and alt impl? (rgoulding, 15:55:47)
    3. https://jira.opendaylight.org/browse/NETCONF-101 (rgoulding, 15:55:54)
    4. https://jira.opendaylight.org/browse/NETCONF-213 (rgoulding, 15:55:57)
    5. remove blueprint plugin generatedDir (rgoulding, 16:05:14)

  2. blueprint plugin generatedDir removal (rgoulding, 16:06:12)
    1. https://git.opendaylight.org/gerrit/#/c/75180/ (rgoulding, 16:06:21)
    2. the basic premise is to get rid of src/main/resource/org/opendaylight/blueprint in favor of standard OSGI-INF (rgoulding, 16:07:43)
    3. also gets rid of Import-Service and Export-Service from the bundle manifest (rgoulding, 16:09:33)
    4. during maven-bundle-plugin (rgoulding, 16:09:45)
    5. our custom generatedDir will be honored for backwards compatibility for now (rgoulding, 16:10:52)
    6. this will come in 4.0.0 of odlparent (rgoulding, 16:12:02)
    7. vorburger says if it works he is for pushing it in (rgoulding, 16:14:19)
    8. 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)

  3. clustering update (rgoulding, 16:20:10)
    1. NETVIRT-1324 (rgoulding, 16:21:49)
    2. https://jira.opendaylight.org/projects/NETVIRT/issues/NETVIRT-1324?filter=allopenissues (rgoulding, 16:21:53)
    3. shague updated and said it still has WARN in controller (rgoulding, 16:22:21)
    4. the application is doing retries, but there are still WARN logs (rgoulding, 16:22:43)
    5. emitted from LocalThreePhaseCommitCohort (rgoulding, 16:23:08)
    6. the issue is that the application handles this, but the log is then misleading (rgoulding, 16:24:12)
    7. since the application actually looks at the returned Future (rgoulding, 16:24:23)
    8. should we close this bug since the logging is considered normal (rgoulding, 16:24:48)
    9. 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)
    10. vorburger to put explanation as to why the WARN from opendaylight-cluster-data-akka.actor is fine (rgoulding, 16:26:54)
    11. where was this discussion originally had? we should link that in to the bug (rgoulding, 16:27:08)
    12. https://jira.opendaylight.org/browse/CONTROLLER-1855 (vorburger, 16:27:53)
    13. we can get back to whether this should be WARN or DEBUG in a later discussion. (rgoulding, 16:28:25)

  4. DataBroker (or DOMStore) test suite ("TCK" ?) for all impls (rgoulding, 16:29:12)
    1. vorburger doesn’t care about htis one as much (rgoulding, 16:29:22)

  5. DOMStore base impl to share beetween InMemoryDOMDataStore and alt impl? (rgoulding, 16:29:30)
    1. 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)
    2. he couldn’t extend it so he copied/pasted (rgoulding, 16:30:16)
    3. should there be an SPI for implementing DOMDataStore where this code could be reused (rgoulding, 16:30:32)
    4. DOMStore not DOMDataStore above ^^^ (rgoulding, 16:31:00)
    5. should there be something common for implementors of DOMStore (rgoulding, 16:31:26)
    6. 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)
    7. vorburger to follow up with this offline (rgoulding, 16:32:55)
    8. reasons why he had to copy/paste and why he couldn’t extend (rgoulding, 16:33:24)
    9. most people would require digging back in to answer anyway (rgoulding, 16:33:34)

  6. NETCONF-101 (rgoulding, 16:33:56)
    1. https://jira.opendaylight.org/browse/NETCONF-101 (rgoulding, 16:34:21)

  7. NETCONF-213 (rgoulding, 16:35:22)
    1. ryan to update oltu patch (rgoulding, 16:37:43)


Meeting ended at 16:37:46 UTC (full logs).

Action items

  1. (none)


People present (lines said)

  1. rgoulding (48)
  2. odl_meetbot (4)
  3. vorburger (2)


Generated by MeetBot 0.1.4.