=================================== #opendaylight-genius: genius-weekly =================================== Meeting started by vorburger at 14:03:05 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-genius/2017/genius_weekly/opendaylight-genius-genius_weekly.2017-11-02-14.03.log.html . Meeting summary --------------- * Please #info in ... (vorburger, 14:03:17) * vorburger (vorburger, 14:03:24) * faseelak (faseelak, 14:03:29) * shague (shague, 14:03:53) * agenda bashing (vorburger, 14:04:47) * LINK: https://trello.com/c/dQKgWchC/40-meeting-agenda-2017-11-02 (vorburger, 14:04:49) * CSIT before merge (vorburger, 14:04:58) * bug scrub (vorburger, 14:05:03) * Gerrit scrub (vorburger, 14:05:14) * Vishal (vthapar, 14:05:18) * CSiT failures following merge of Tom's recent change (vorburger, 14:07:46) * ACTION: faseelak is looking through CSIT log to try understand failure (vorburger, 14:08:04) * LINK: https://git.opendaylight.org/gerrit/#/c/64544/ (vorburger, 14:08:34) * LINK: https://git.opendaylight.org/gerrit/#/c/65016/ (vorburger, 14:08:48) * LINK: https://git.opendaylight.org/gerrit/#/c/64544/ patch causing CSIT problems (shague, 14:16:27) * LINK: https://git.opendaylight.org/gerrit/#/c/65016/ revert patch looks to fix csit tests (shague, 14:16:48) * LINK: https://jenkins.opendaylight.org/releng/view/netvirt/job/netvirt-csit-1node-openstack-ocata-gate-stateful-oxygen/226/ csit showing revert passing (shague, 14:17:12) * tom finds line 77 was accidentally removed (shague, 14:18:12) * tpantelis just saw the problem, live while in the meeting (vorburger, 14:18:14) * ACTION: Tom Pantelis will raise a new Gerrit ASAP, we won't merge the revert (vorburger, 14:18:53) * Tom thanks Faseela for having pointed him in the right direction to see what was wrong in the clean up patch that broke CSIT (vorburger, 14:19:18) * What patches shall we run CSIT on? (vorburger, 14:19:36) * Clean-up patches to be split as dependant reviews and CSIT to be run on final patch (faseelak, 14:23:08) * Listener patches (vorburger, 14:23:31) * Hema had some questions, David couldn't make it, but Tom is with us for the first time (welcome!!), and was involved in the new listeners (vorburger, 14:24:05) * Gerrit Reviews (vorburger, 14:36:12) * LINK: https://git.opendaylight.org/gerrit/#/q/project:genius+status:open+Verified%253E0+is:mergeable+NOT+label:Code-Review%253C0 (vorburger, 14:36:22) * LINK: https://git.opendaylight.org/gerrit/#/c/63323/ (vorburger, 14:36:54) * ACTION: Hema will rebase https://git.opendaylight.org/gerrit/#/c/63324/ and we'll merge it and the c/63323/ for stable/nitrogen together (vorburger, 14:39:04) * LINK: https://git.opendaylight.org/gerrit/#/c/64623/ (vorburger, 14:39:57) * ACTION: Hema is -1 this one based on the previous discussion with Tom and will follow-up with David and Tom (vorburger, 14:40:48) * LINK: https://git.opendaylight.org/gerrit/#/c/64625/ (vorburger, 14:41:10) * ACTION: Hema will do that -1 this one based on the previous discussion with Tom and will follow-up with David and Tom (vorburger, 14:42:06) * LINK: https://git.opendaylight.org/gerrit/#/c/65008/ (vorburger, 14:42:50) * ACTION: faseelak will run genius CSIT and merge it when that passes (vorburger, 14:45:12) * LINK: https://git.opendaylight.org/gerrit/#/c/64844/ (vorburger, 14:45:32) * ACTION: vorburger will self merge https://git.opendaylight.org/gerrit/#/c/64844/ once Tom's patch has gone in (vorburger, 14:46:39) * fix for CSIT failure https://git.opendaylight.org/gerrit/#/c/65049/1 (faseelak, 14:46:57) * LINK: https://git.opendaylight.org/gerrit/#/c/64762/ (vorburger, 14:47:06) * ACTION: Hema will do also -1 this 3rd one based on the previous discussion with Tom and will follow-up with David and Tom (vorburger, 14:47:23) * JIRA bug scrubbing (vorburger, 14:47:47) * LINK: https://jira.opendaylight.org/issues/?filter=10136 (vorburger, 14:47:59) * LINK: https://jira.opendaylight.org/browse/GENIUS-56 (vorburger, 14:48:48) * lets review the JIRAs int he other order... ;-) (vorburger, 14:49:51) * LINK: https://jira.opendaylight.org/browse/GENIUS-77 (vorburger, 14:49:53) * awaiting mor information, logs from original issue were wiped out (vthapar, 14:51:36) * ACTION: vthapar to follow up on this (vthapar, 14:51:46) * LINK: https://jira.opendaylight.org/browse/GENIUS-85 (vorburger, 14:52:13) * Faseela says there was a gerrit for this already? (vorburger, 14:53:39) * ACTION: vthapar will update bug with latest status and next step (vorburger, 14:53:51) * LINK: (vorburger, 14:54:21) * LINK: https://jira.opendaylight.org/browse/GENIUS-32 (vorburger, 14:54:26) * ACTION: faseela to update (vorburger, 14:57:58) * LINK: https://jira.opendaylight.org/browse/GENIUS-86 (vorburger, 14:58:19) * ACTION: faseelak will get in touch with Kency to get this one moving (vorburger, 14:58:48) * LINK: https://jira.opendaylight.org/browse/GENIUS-94 (vorburger, 14:58:59) * closed (vorburger, 14:59:02) * LINK: https://jira.opendaylight.org/browse/GENIUS-90 (vorburger, 14:59:36) * ACTION: vthapar will eventually look at this we said last week (vorburger, 14:59:50) * ACTION: edwin will provide input on failures seen in the related patch (vorburger, 15:00:35) Meeting ended at 15:01:22 UTC. Action items, by person ----------------------- * faseelak * faseelak is looking through CSIT log to try understand failure * faseelak will run genius CSIT and merge it when that passes * faseelak will get in touch with Kency to get this one moving * vorburger * vorburger will self merge https://git.opendaylight.org/gerrit/#/c/64844/ once Tom's patch has gone in * vthapar * vthapar to follow up on this * vthapar will update bug with latest status and next step * vthapar will eventually look at this we said last week * **UNASSIGNED** * Tom Pantelis will raise a new Gerrit ASAP, we won't merge the revert * Hema will rebase https://git.opendaylight.org/gerrit/#/c/63324/ and we'll merge it and the c/63323/ for stable/nitrogen together * Hema is -1 this one based on the previous discussion with Tom and will follow-up with David and Tom * Hema will do that -1 this one based on the previous discussion with Tom and will follow-up with David and Tom * Hema will do also -1 this 3rd one based on the previous discussion with Tom and will follow-up with David and Tom * faseela to update * edwin will provide input on failures seen in the related patch People present (lines said) --------------------------- * vorburger (57) * odl_meetbot (8) * shague (5) * vthapar (3) * faseelak (3) * skitt (1) * Vishal (0) Generated by `MeetBot`_ 0.1.4