====================================== #opendaylight-meeting: kernel projects ====================================== Meeting started by rgoulding at 17:00:25 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-meeting/2018/kernel_projects/opendaylight-meeting-kernel_projects.2018-03-06-17.00.log.html . Meeting summary --------------- * agenda bashing (rgoulding, 17:00:29) * LINK: https://git.opendaylight.org/gerrit/#/c/67283/ (rgoulding, 17:00:41) * LINK: https://git.opendaylight.org/gerrit/#/q/topic:simple-dist_web (rgoulding, 17:00:56) * Cluster isolation scenario (rgoulding, 17:01:11) * blockers (rgoulding, 17:01:57) * LINK: https://lists.opendaylight.org/pipermail/mdsal-dev/2018-March/001517.html (rgoulding, 17:03:01) * Oxygen Blocker Review (rgoulding, 17:04:01) * ODLPARENT-139: likely to be downgraded to major since there is a workaround (rgoulding, 17:05:24) * NETCONF-520: reduced severity and will probably close (api change only) (rgoulding, 17:06:52) * NETCONF-521: may be working but we need to test before we can reduce severity or close (rgoulding, 17:07:29) * no other blockers for oxygen were identified by meeting participants (rgoulding, 17:08:36) * Josh's Websocket Uri should have scheme wss if registration is https (rgoulding, 17:09:01) * LINK: https://git.opendaylight.org/gerrit/#/c/67283/ (rgoulding, 17:09:12) * there are build failures currently happening with the aforementioned patch (rgoulding, 17:10:01) * some master netconf patches are successfully building, and it is unsure at this time whether the failures are related to this patch (rgoulding, 17:11:19) * ACTION: vorburger to start a thread on mdsal-dev and netconf-dev related to broken master (rgoulding, 17:15:10) * LINK: https://git.opendaylight.org/gerrit/#/q/topic:simple-dist_web (rgoulding, 17:15:34) * package name will be left as is in the current patch set, with “aaa” in the package name (rgoulding, 17:18:42) * the "mature" netconf project actually violates the separate namespace rule already (vorburger, 17:20:13) * ACTION: netconf-aaa-plugin violates this aforementioned namespace rule, so rgoulding to file an issue (rgoulding, 17:20:20) * tpantelis points out that restconf has the same sort of issue too (rgoulding, 17:20:40) * rovarga believes it should be okay to leave this “test” dependency for the time being (rgoulding, 17:23:38) * ACTION: rgoulding to investigate getting rid of embed-dependency of h2 (rgoulding, 17:28:53) * ACTION: rgoulding to pull down Web API changes and then test RESTCONF to see if it works with these changes (rgoulding, 17:35:18) * Binding V1 codegen in Fluorine (rgoulding, 17:37:20) * LINK: https://lists.opendaylight.org/pipermail/mdsal-dev/2018-March/001517.html (rgoulding, 17:37:21) * code generated by fluorine codecs would not load in the oxygen container, basically (rgoulding, 17:38:08) * rovarga wanted to raise awareness since no one really said anything on the mailing list (rgoulding, 17:38:25) * this one doesn’t change any of the API contracts so consumers will not need fix-up patches (rgoulding, 17:39:14) * (just changes class internals) (rgoulding, 17:39:20) * LINK: https://lists.opendaylight.org/pipermail/tsc/2018-March/009053.html (rovarga, 17:39:22) * this one would require changes to existing users (rgoulding, 17:39:48) * this is a change to md-sal apis (rgoulding, 17:40:33) * “2) Replace org.opendaylight.yangtools.yang.binding.Identifiable.getKey()” —> this will break a lot of code, but it is a mechanical change (rgoulding, 17:41:24) * this will break projects outside of our ODL ecosystem that rely on MD-SAL java apis (rgoulding, 17:42:21) * none of these changes should be super intrusive, and will extend the life of the binding spec v1 (rgoulding, 17:43:59) * the fallout is obvious and easy to fix (rgoulding, 17:44:17) * there is a draft calendar for fluorine (rgoulding, 17:53:04) * LINK: https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/docs-rtd-verify-fluorine/27/html/release-process/managed-release.html (klou, 17:53:06) * Cluster isolation scenario (rgoulding, 17:55:56) * LINK: https://jira.opendaylight.org/browse/CONTROLLER-1817 (rgoulding, 17:56:32) * quarantine should restart bundle 0 (rgoulding, 17:57:50) * original testing was done by Gary Wu with a much older version of akka (rgoulding, 17:58:09) * there were some issues regarding how to restart karaf container found (when we were supposed to restart bundle 0 we weren’t really) (rgoulding, 17:58:57) * this wasn’t an issue in Nitrogen (rgoulding, 17:59:06) * to clear akka state (restart JVM) you need to kill bin/karaf, then restart, which was not possible in karaf 3.0.X (rgoulding, 18:00:46) * there was a property set to restart karaf, which did not actually clear everything out (rgoulding, 18:01:06) * not sure why sometimes the event is sent only sometimes (rgoulding, 18:01:46) * advice is to dig into what akka is doing and why it is not working with our implementation (rgoulding, 18:02:36) * akka has a debug mode in the akka configuration to help create more verbosity in the log (rgoulding, 18:03:23) Meeting ended at 18:06:14 UTC. Action items, by person ----------------------- * rgoulding * netconf-aaa-plugin violates this aforementioned namespace rule, so rgoulding to file an issue * rgoulding to investigate getting rid of embed-dependency of h2 * rgoulding to pull down Web API changes and then test RESTCONF to see if it works with these changes * vorburger * vorburger to start a thread on mdsal-dev and netconf-dev related to broken master People present (lines said) --------------------------- * rgoulding (50) * odl_meetbot (3) * klou (1) * rovarga (1) * vorburger (1) Generated by `MeetBot`_ 0.1.4