============================ #opendaylight-genius: genius ============================ Meeting started by vorburger at 14:13:16 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-genius/2017/genius/opendaylight-genius-genius.2017-05-11-14.13.log.html . Meeting summary --------------- * info in please (vorburger, 14:13:27) * faseelak (faseelak, 14:13:40) * vorburger (vorburger, 14:13:42) * skitt (skitt, 14:13:45) * edavsua (edavsua, 14:13:48) * Gerrit Reviews (vorburger, 14:14:00) * LINK: https://git.opendaylight.org/gerrit/#/q/project:genius+status:open+Verified%253E0+is:mergeable+NOT+label:Code-Review%253C0 (faseelak, 14:14:36) * ACTION: Peri to review https://git.opendaylight.org/gerrit/#/c/55860/ (vorburger, 14:15:06) * ACTION: faseelak will review https://git.opendaylight.org/gerrit/#/c/56760/ (vorburger, 14:15:28) * https://git.opendaylight.org/gerrit/#/c/56570/ is ready for merge to stable/carbon, once it re-opens (vorburger, 14:15:59) * ACTION: vorburger to re-review https://git.opendaylight.org/gerrit/#/c/53544/ (vorburger, 14:16:18) * https://git.opendaylight.org/gerrit/#/c/56031/ to be considered in 2 weeks after DDF (master) (vorburger, 14:19:49) * ACTION: edavsua wil review https://git.opendaylight.org/gerrit/#/c/56031/ (vorburger, 14:20:03) * ACTION: faseelak vorburger will write a spec documenting proposal in https://git.opendaylight.org/gerrit/#/c/56031/ (vorburger, 14:22:09) * Test Code Coverage (vorburger, 14:25:03) * according to faseelak it's now at 42% up from 30% (vorburger, 14:25:32) * biggest gaps are in mdsalutil, alivenessmonitor and fcaps-application (vorburger, 14:25:56) * LINK: https://sonar.opendaylight.org/ (faseelak, 14:26:45) * mdsalutil has a lot of @Deprecated. If we just cleaned up and got rid of all of that, the coverage would already look better (vorburger, 14:29:40) * ACTION: vivek to table work at DDF for nitrogen to get rid of all @Deprecated code in genius mdsalutil (vorburger, 14:31:37) * LINK: https://git.opendaylight.org/gerrit/#/q/project:genius+status:open,25 (vorburger, 14:34:14) * Reviewing very old stuck Gerrits, abandon? (vorburger, 14:34:41) * LINK: https://git.opendaylight.org/gerrit/#/c/52334/ (vorburger, 14:35:57) * that's waiting for openflow plugin merge https://git.opendaylight.org/gerrit/#/c/51498/ (vorburger, 14:36:39) * ACTION: Hema_ will ping people having very old pending changes stuck with Merge Conflict (vorburger, 14:38:08) * ACTION: Michael to rebase / review / merge https://git.opendaylight.org/gerrit/#/c/50192/ (vorburger, 14:38:27) Meeting ended at 14:40:36 UTC. Action items, by person ----------------------- * edavsua * edavsua wil review https://git.opendaylight.org/gerrit/#/c/56031/ * faseelak * faseelak will review https://git.opendaylight.org/gerrit/#/c/56760/ * faseelak vorburger will write a spec documenting proposal in https://git.opendaylight.org/gerrit/#/c/56031/ * vorburger * vorburger to re-review https://git.opendaylight.org/gerrit/#/c/53544/ * faseelak vorburger will write a spec documenting proposal in https://git.opendaylight.org/gerrit/#/c/56031/ * **UNASSIGNED** * Peri to review https://git.opendaylight.org/gerrit/#/c/55860/ * vivek to table work at DDF for nitrogen to get rid of all @Deprecated code in genius mdsalutil * Hema_ will ping people having very old pending changes stuck with Merge Conflict * Michael to rebase / review / merge https://git.opendaylight.org/gerrit/#/c/50192/ People present (lines said) --------------------------- * vorburger (24) * odl_meetbot (4) * faseelak (3) * skitt (1) * edavsua (1) Generated by `MeetBot`_ 0.1.4