=================================== #opendaylight-genius: genius-weekly =================================== Meeting started by vorburger at 15:04:15 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-genius/2018/genius_weekly/opendaylight-genius-genius_weekly.2018-01-25-15.04.log.html . Meeting summary --------------- * Everyone please #info in.... (vorburger, 15:04:30) * vpickard (vpickard, 15:04:31) * viveks (viveks, 15:04:33) * vorburger (vorburger, 15:04:34) * Agenda Bashing (vorburger, 15:04:45) * vpickard wants to discuss genius auto tunnels (vorburger, 15:05:03) * vorburger is interested in feedback about remaining issues following the "big bump", if there are any? (vorburger, 15:05:40) * Gerrit reviews & are we still holding back for aynthing or can keep going now? (vorburger, 15:06:29) * genius auto tunnels (vorburger, 15:12:08) * ACTION: vpickard to send email to genius-dev regarding bug where br-int added without dpid, update code does not handle case where br-int update comes in where dpid is added. Found this in d/s testing in OOO based deploy (vpickard, 15:12:30) * The Big Bump (vorburger, 15:12:52) * no one aware of any remaining issues, for genius (vorburger, 15:13:19) * TSC call this evening will discuss impact on release timeline (vorburger, 15:14:02) * Gerrit reviews (vorburger, 15:14:16) * LINK: https://git.opendaylight.org/gerrit/#/q/project:genius+status:open+Verified%253E0+is:mergeable+NOT+label:Code-Review%253C0 (vorburger, 15:14:54) * ACTION: viveks will ask Vishal & Hema to please review x5 patches from David re. migrating listeners to new framework (vorburger, 15:16:49) * LINK: https://git.opendaylight.org/gerrit/#/c/66488/ (viveks, 15:18:54) * LINK: https://git.opendaylight.org/gerrit/#/c/66490/ (viveks, 15:19:24) * viveks and vpickard discussed and agreed to get these into stable/carbon AFTER we've them in master (still be reviewed) (vorburger, 15:21:37) * LINK: https://git.opendaylight.org/gerrit/#/c/65290/ (viveks, 15:22:40) * https://git.opendaylight.org/gerrit/#/c/65290/ (vorburger, 15:22:49) * this is an old Gerrit from vorburger with a proposed enhancement to LockManager, for which CSIT passed, back in November... (vorburger, 15:23:49) * ACTION: viveks will ask Faseela to review https://git.opendaylight.org/gerrit/#/c/65290/ (vorburger, 15:24:24) * https://git.opendaylight.org/gerrit/#/c/67231/ is a WIP, viveks will -1 it until it's good to go in (vorburger, 15:25:01) * LINK: https://git.opendaylight.org/gerrit/#/c/67159/ (viveks, 15:25:27) * https://git.opendaylight.org/gerrit/#/c/67159/ (vorburger, 15:26:45) * ACTION: tpantelis to re-review and +1 https://git.opendaylight.org/gerrit/#/c/67159/ if OK for him (vorburger, 15:27:14) * ACTION: vorburger to review https://git.opendaylight.org/gerrit/#/c/67159/ and merge if OK (vorburger, 15:27:51) * ACTION: viveks to check with Tarun why Tom's feedback about removing initial delay why not taken into account? (vorburger, 15:30:40) * ACTION: vorburger to try out https://git.opendaylight.org/gerrit/#/c/67290/ (vorburger, 15:31:59) * ACTION: Faseela to review and if OK merge https://git.opendaylight.org/gerrit/#/c/66502/ (vorburger, 15:32:18) * ACTION: Faseela to review and if OK merge https://git.opendaylight.org/gerrit/#/c/67058/ (vorburger, 15:33:01) * merged https://git.opendaylight.org/gerrit/#/c/66849/ adding portuuid to be retrieved as part of port statistics (vorburger, 15:34:01) * ACTION: vorburger to review https://git.opendaylight.org/gerrit/#/c/67415/ and comment that it should use his new RPC erorr handling util (vorburger, 15:35:34) Meeting ended at 15:36:23 UTC. Action items, by person ----------------------- * viveks * viveks will ask Vishal & Hema to please review x5 patches from David re. migrating listeners to new framework * viveks will ask Faseela to review https://git.opendaylight.org/gerrit/#/c/65290/ * viveks to check with Tarun why Tom's feedback about removing initial delay why not taken into account? * vorburger * vorburger to review https://git.opendaylight.org/gerrit/#/c/67159/ and merge if OK * vorburger to try out https://git.opendaylight.org/gerrit/#/c/67290/ * vorburger to review https://git.opendaylight.org/gerrit/#/c/67415/ and comment that it should use his new RPC erorr handling util * vpickard * vpickard to send email to genius-dev regarding bug where br-int added without dpid, update code does not handle case where br-int update comes in where dpid is added. Found this in d/s testing in OOO based deploy * **UNASSIGNED** * tpantelis to re-review and +1 https://git.opendaylight.org/gerrit/#/c/67159/ if OK for him * Faseela to review and if OK merge https://git.opendaylight.org/gerrit/#/c/66502/ * Faseela to review and if OK merge https://git.opendaylight.org/gerrit/#/c/67058/ People present (lines said) --------------------------- * vorburger (31) * viveks (5) * odl_meetbot (4) * vpickard (2) Generated by `MeetBot`_ 0.1.4