======================================
#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