=================================== #opendaylight-genius: genius-weekly =================================== Meeting started by vorburger at 15:06:02 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-genius/2017/genius_weekly/opendaylight-genius-genius_weekly.2017-11-30-15.06.log.html . Meeting summary --------------- * Everyone please #info in.. (vorburger, 15:06:15) * skitt (skitt, 15:06:20) * vorburger (vorburger, 15:06:20) * edavsua (edavsua, 15:07:25) * faseelak (faseelak, 15:07:43) * ACTION: skitt will bring virtual food and drink to the next weekly meeting ;-) (vorburger, 15:10:13) * faseelak had run netvirt csit on genius stable/nitrogen patches till yesterday (faseelak, 15:10:23) * LINK: https://docs.google.com/spreadsheets/d/1sxjIx5DIvaR4Lix4o8kNTWvIQX_0KqD61UNhy3qB1R0/edit#gid=461297039 (shague, 15:10:48) * ACTION: faseelak to check the csit failure in netvirt to figure out which module is failing (faseelak, 15:11:19) * Broken CSIT (vorburger, 15:13:15) * ACTION: vorburger to check openflowplugin patches to analyse whether they are causing csti failures (faseelak, 15:13:16) * Agenda bashing (vorburger, 15:13:43) * LINK: https://trello.com/c/4z0r4TU8/44-meeting-agenda-2017-11-30 (vorburger, 15:13:50) * David: Listener scale test results? (vorburger, 15:14:39) * ACTION: Hema will finish the review on listeners (edavsua, 15:18:32) * ACTION: Vivek and Edwin will comment on the scale tests by email (edavsua, 15:19:23) * DJC removal (vorburger, 15:19:46) * LINK: https://git.opendaylight.org/gerrit/#/c/66064/ just got merged (vorburger, 15:20:06) * ACTION: vorburger to update email thread and Weather item Wiki page (vorburger, 15:20:20) * Upgrade Race Condition (vorburger, 15:20:42) * LINK: https://git.opendaylight.org/gerrit/#/c/66050/ (vorburger, 15:20:48) * ACTION: faseelak to review it (vorburger, 15:21:13) * SingleTransactionDataBroker with RetryingManagedNewTransactionRunner (vorburger, 15:21:49) * LINK: https://git.opendaylight.org/gerrit/#/c/65812/ (vorburger, 15:21:58) * viveks (viveks, 15:23:58) * just go merged (vorburger, 15:29:19) * keep an eye on CSIT to see if more starts failing (vorburger, 15:29:30) * ACTION: vorburger rebase https://git.opendaylight.org/gerrit/#/c/66072/ and run CSIT there (vorburger, 15:30:23) * InterfaceManagerService API without RpcResult (vorburger, 15:30:48) * LINK: https://git.opendaylight.org/gerrit/#/c/65004/ (vorburger, 15:30:57) * ACTION: edavsua to review https://git.opendaylight.org/gerrit/#/c/65004/ and merge it if is OK (vorburger, 15:31:52) * Id manager: use transaction manager (vorburger, 15:32:09) * LINK: https://git.opendaylight.org/gerrit/#/c/64726/ (vorburger, 15:32:15) * ACTION: faseelak to review https://git.opendaylight.org/gerrit/#/c/64726/ and merge if she's OK with it (vorburger, 15:33:07) * Resource batcher clean-up, phase 1 (vorburger, 15:33:47) * LINK: https://git.opendaylight.org/gerrit/#/c/65675/ (vorburger, 15:33:52) * ACTION: skitt to look at why its causing CSIT failures (vorburger, 15:34:03) * Review other pending Gerrits (vorburger, 15:34:32) * LINK: https://git.opendaylight.org/gerrit/#/q/project:genius+status:open+Verified%253E0+is:mergeable+NOT+label:Code-Review%253C0 (vorburger, 15:34:35) * LINK: https://git.opendaylight.org/gerrit/#/c/64620/ (vorburger, 15:35:24) * viveks says scale testing was succesful (vorburger, 15:35:48) * ACTION: viveks to merge https://git.opendaylight.org/gerrit/#/c/64620/ (vorburger, 15:36:10) * LINK: https://git.opendaylight.org/gerrit/#/c/64762/ (vorburger, 15:36:26) * LINK: https://git.opendaylight.org/gerrit/#/c/58233/ (vorburger, 15:36:50) * ACTION: faseelak to self -1 the WIP https://git.opendaylight.org/gerrit/#/c/58233/ (vorburger, 15:37:46) * LINK: https://git.opendaylight.org/gerrit/#/c/65923/ (vorburger, 15:38:09) * ACTION: vorburger to review and merge https://git.opendaylight.org/gerrit/#/c/65923/ (vorburger, 15:38:33) * LINK: https://git.opendaylight.org/gerrit/#/c/66096/ (vorburger, 15:39:23) * ACTION: faseelak to peer review and merge https://git.opendaylight.org/gerrit/#/c/66096/ (vorburger, 15:40:07) * LINK: https://git.opendaylight.org/gerrit/#/c/66097/ (vorburger, 15:40:14) * ACTION: edavsua to review and merge https://git.opendaylight.org/gerrit/#/c/66097/ (vorburger, 15:41:03) * ACTION: faseelak to review and merge https://git.opendaylight.org/gerrit/#/c/66097/ (vorburger, 15:41:08) * ACTION: Hema to review and merge https://git.opendaylight.org/gerrit/#/c/66097/ (vorburger, 15:41:51) * Question about MNTR and retries (vorburger, 15:42:57) * LINK: https://git.opendaylight.org/gerrit/#/c/66097/2/itm/itm-impl/src/main/java/org/opendaylight/genius/itm/confighelpers/ItmInternalTunnelAddWorker.java (vorburger, 15:43:00) * interface ManagedNewTransactionRunner has 2 different implementations: ManagedNewTransactionRunnerImpl VS RetryingManagedNewTransactionRunner (vorburger, 15:45:05) * OptimisticLockFailedException (vorburger, 15:46:38) * LINK: https://git.opendaylight.org/gerrit/#/c/65726/3 (faseelak, 15:46:40) * LINK: https://jira.opendaylight.org/browse/NETVIRT-1023 (vorburger, 15:46:52) * dimple pointed out that the above patch is causing the bug NETVIRT-1023 (faseelak, 15:49:36) Meeting ended at 15:49:50 UTC. Action items, by person ----------------------- * edavsua * edavsua to review https://git.opendaylight.org/gerrit/#/c/65004/ and merge it if is OK * edavsua to review and merge https://git.opendaylight.org/gerrit/#/c/66097/ * faseelak * faseelak to check the csit failure in netvirt to figure out which module is failing * faseelak to review it * faseelak to review https://git.opendaylight.org/gerrit/#/c/64726/ and merge if she's OK with it * faseelak to self -1 the WIP https://git.opendaylight.org/gerrit/#/c/58233/ * faseelak to peer review and merge https://git.opendaylight.org/gerrit/#/c/66096/ * faseelak to review and merge https://git.opendaylight.org/gerrit/#/c/66097/ * skitt * skitt will bring virtual food and drink to the next weekly meeting ;-) * skitt to look at why its causing CSIT failures * viveks * viveks to merge https://git.opendaylight.org/gerrit/#/c/64620/ * vorburger * vorburger to check openflowplugin patches to analyse whether they are causing csti failures * vorburger to update email thread and Weather item Wiki page * vorburger rebase https://git.opendaylight.org/gerrit/#/c/66072/ and run CSIT there * vorburger to review and merge https://git.opendaylight.org/gerrit/#/c/65923/ * **UNASSIGNED** * Hema will finish the review on listeners * Vivek and Edwin will comment on the scale tests by email * Hema to review and merge https://git.opendaylight.org/gerrit/#/c/66097/ People present (lines said) --------------------------- * vorburger (55) * faseelak (10) * odl_meetbot (7) * skitt (5) * edavsua (3) * shague (1) * viveks (1) Generated by `MeetBot`_ 0.1.4