====================================== #opendaylight-meeting: kernel projects ====================================== Meeting started by rgoulding at 16:01:44 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-meeting/2018/kernel_projects/opendaylight-meeting-kernel_projects.2018-05-29-16.01.log.html . Meeting summary --------------- * agenda bashing (rgoulding, 16:02:00) * LINK: https://jira.opendaylight.org/browse/INFRAUTILS-41 (rgoulding, 16:02:09) * LINK: https://jira.opendaylight.org/browse/TSC-113 (rgoulding, 16:02:18) * LINK: https://lists.opendaylight.org/pipermail/netvirt-dev/2018-May/006878.html (rgoulding, 16:02:28) * LINK: https://lists.opendaylight.org/pipermail/tsc/2018-May/009739.html (rgoulding, 16:02:37) * Removal of some deprecated APIS in NETCONF (rgoulding, 16:04:24) * jersey2 update (rgoulding, 16:05:04) * swagger UI bug (rgoulding, 16:05:43) * INFRAUTILS-41 (rgoulding, 16:06:13) * LINK: https://jira.opendaylight.org/browse/INFRAUTILS-41 (rgoulding, 16:06:26) * vorburger wants to know plan/next steps (rgoulding, 16:06:45) * this is for upgrading dropwizard.metrics from 3.2.5 to 4.0.2 (rgoulding, 16:07:00) * rovarga asks if this is provided by other JVMs (rgoulding, 16:08:08) * vorburger says he has found that it exists in OpenJDK, but hasn’t tested for IBM’s JDK9 (rgoulding, 16:08:33) * definitely good on OpenJDK and Oracle (rgoulding, 16:08:58) * provides GC stats and other JVM things (rgoulding, 16:09:06) * skitt brings up the fact that dropwizard should have an optional dependency on this (rgoulding, 16:10:26) * the manifest should make it optional? (rgoulding, 16:10:35) * rovarga says that if it isn’t optional then ODL will fail to load if the JVM does not have it (rgoulding, 16:11:29) * the question becomes whether the metrics are mission critical; rovarga claims they are not (rgoulding, 16:11:54) * TSC-113 (rgoulding, 16:15:53) * LINK: https://jira.opendaylight.org/browse/TSC-113 (rgoulding, 16:15:57) * upgrade webapps to use jersey 2 (rgoulding, 16:16:20) * tpantelis is driving this forward (rgoulding, 16:16:32) * tpantelis is running the multi-patch job to test if this works (rgoulding, 16:16:56) * neutron patch was already merged (rgoulding, 16:17:01) * tpantelis ran into same problem with dual dependency chain via javax.annotation in the aaa / netconf patches (rgoulding, 16:17:41) * the two patches need each other (i.e., this will break temporarily) (rgoulding, 16:18:05) * we are waiting on a successful multi-patch job (rgoulding, 16:19:57) * jmorvay to check after the meeting and add +1, then tpantelis can self-merge (rgoulding, 16:20:29) * for netconf (rgoulding, 16:20:34) * tpantelis and rgoulding can do it for AAA (rgoulding, 16:20:45) * passing multi-patch is a gating requirement (rgoulding, 16:20:56) * AGREED: (rgoulding, 16:21:01) * should we run netvirt csit? (rgoulding, 16:21:46) * skitt claims it is in a good state right now, so if failures occur there is probably a problem (rgoulding, 16:22:00) * to run netvirt CIST, use test-netvirt-current (skitt, 16:22:30) * to run on a multi-patch job there is a different keyword required (rgoulding, 16:22:51) * you need to supply the url of your multi-patch-job build and pass it to CSIT (rgoulding, 16:23:02) * Autorelease oxygen failed to build odl-netvirt-openstack from netvirt (rgoulding, 16:24:41) * LINK: https://lists.opendaylight.org/pipermail/netvirt-dev/2018-May/006878.html (rgoulding, 16:24:48) * this has moved on on list discussions (rgoulding, 16:24:54) * Request to split genius repo to enhance usability by other projects (rgoulding, 16:25:11) * LINK: https://lists.opendaylight.org/pipermail/tsc/2018-May/009739.html (rgoulding, 16:25:29) * this one is panning out on mailing lists (rgoulding, 16:25:38) * Removal of some deprecated APIS in NETCONF (rgoulding, 16:25:58) * netconf is using many deprecated yangtools, mdsal and controller APIs (rgoulding, 16:26:46) * we need to adapt to new counterparts (rgoulding, 16:26:53) * also, there are many places where NETCONF code exposes deprecated APIs, such as protocol-framework, that we should have a plan to remove such abstractions (rgoulding, 16:27:44) * removal of the former (usage of deprecated APIs) is blocking upstreams from removing the APIs (rgoulding, 16:28:35) * we want to garner support to help with removal of such concepts (rgoulding, 16:28:43) * the question arrises, how do we get people to work on such things? (rgoulding, 16:29:07) * rovarga suggests making a task list that we can prioritize (rgoulding, 16:29:43) * one big thing is to get rid of the protocol-framework (rgoulding, 16:29:53) * and migration to plain netty or mina (rgoulding, 16:30:03) * ACTION: jmorvay to create JIRA tasks to showcase some of the places we can clean up (rgoulding, 16:31:18) * DCL Removal Discussion (rgoulding, 16:36:55) * there is one last patch to remove DCL (rgoulding, 16:37:10) * LINK: https://lists.opendaylight.org/pipermail/release/2018-May/015217.html (rgoulding, 16:37:47) * should we agree to merge this? (rgoulding, 16:37:53) * AGREED: (rgoulding, 16:37:55) * rovarga merged. (rgoulding, 16:38:00) * LINK: https://git.opendaylight.org/gerrit/#/c/71801/ (rgoulding, 16:38:24) * config subsystem removal (rgoulding, 16:38:34) * tpantelis claims no upstream projects have used for a while (rgoulding, 16:38:44) * downstreams should be migrated by now (we hope) (rgoulding, 16:38:55) * archetype was changed a while ago to utilize blueprint (rgoulding, 16:39:07) * ACTION: tpantelis to send out final removal notice for when it is timed (rgoulding, 16:39:19) * tpantelis claims that he removed the CSS northbound loopback (rgoulding, 16:41:18) * NetconfMonitoringActivator shouldn’t be there (rgoulding, 16:43:35) * LINK: https://github.com/opendaylight/netconf/blob/master/netconf/netconf-monitoring/src/main/java/org/opendaylight/netconf/monitoring/osgi/NetconfMonitoringActivator.java (rgoulding, 16:43:37) * so we should work to remove (rgoulding, 16:43:44) * tpantelis already migrated XML stuff to NETCONF; it was only like 3 classes (rgoulding, 16:44:53) * SUBMIT removal (rgoulding, 16:47:16) * ACTION: AsyncWriteTransaction.submit() is only used in controller, so once we switch that we can remove it there (rgoulding, 16:48:13) * removal of controller mdsal apis (rgoulding, 16:56:42) * tpantelis is looking into a plan for removing these (rgoulding, 16:56:52) * this could also solve CheckedFuture removal (rgoulding, 16:56:58) * vorbuger brings up the point that a demo patch of this would help other projects adapt (rgoulding, 16:57:45) * ACTION: tpantelis to look at what other API(s) include CheckedFuture in the signature (rgoulding, 16:59:35) * this is made easier since only Controller is consuming many of these (rgoulding, 16:59:52) Meeting ended at 17:01:22 UTC. People present (lines said) --------------------------- * rgoulding (84) * odl_meetbot (4) * skitt (1) Generated by `MeetBot`_ 0.1.4