=================================== #opendaylight-genius: genius-weekly =================================== Meeting started by vorburger at 14:05:23 UTC. The full logs are available at http://meetings.opendaylight.org/opendaylight-genius/2017/genius_weekly/opendaylight-genius-genius_weekly.2017-09-21-14.05.log.html . Meeting summary --------------- * info in? (vorburger, 14:05:30) * Vishal (vthapar, 14:05:31) * edavsua (edavsua, 14:05:44) * viveks (viveks, 14:06:13) * faseelak (faseelak, 14:06:39) * Agenda (vorburger, 14:06:58) * LINK: https://trello.com/c/AFS8LqP6/35-meeting-agenda-2017-09-21 (vorburger, 14:07:00) * Deprecation/Removal of any unused projects? (vorburger, 14:07:13) * viveks explains that arputil used to be used only by alivenessmonitor, but shague_ clarified on list it's now used by netvirt, so can't remove (vorburger, 14:08:18) * shague (shague_, 14:08:53) * fcapsmanager, fcapsapplication, alivenessmonitor, resourcemanager are all not actively used currently (vorburger, 14:09:11) * there is no "burning problem" to remove them, it's more just a general dont-keep-unused-code (vorburger, 14:09:51) * fcapsmanager / fcapsapplication may be remove once statusdiag by faseelak in infrautils works (vorburger, 14:12:20) * https://git.opendaylight.org/gerrit/#/q/topic:genius.DJC-to-infra.JC finally hopefully ready (vorburger, 14:12:44) * ACTION: vorburger will request CSIT on genius and netvirt once https://git.opendaylight.org/gerrit/#/c/60303/ (vorburger, 14:16:09) * Better share the load of reviewing "trivial" clean up changes (vorburger, 14:16:27) * edavsua says that due to the rule to get x2 +1 to merge, it's sometimes difficult to get for simpler clean up changes (vorburger, 14:17:57) * ACTION: viveks vthapar Hema will help to review the low risk "mechanical" changes often proposed by skitt and edavsua (vorburger, 14:20:21) * ACTION: ALL use "Mechanical:" commit message prefix for changes which are clean up type changes done automatically by IDE like tools (vorburger, 14:22:34) * ACTION: skitt edavsua vorburger ALL to include Hema as Reviewer on more changes (vorburger, 14:23:20) * ready changes (vorburger, 14:26:15) * LINK: https://git.opendaylight.org/gerrit/#/q/project:genius+status:open+Verified%253E0+is:mergeable+NOT+label:Code-Review%253C0 (vorburger, 14:26:22) * LINK: https://git.opendaylight.org/gerrit/#/q/project:genius+status:open+Verified%253E0+is:mergeable+NOT+label:Code-Review%253C0+branch:master (vorburger, 14:26:42) * ACTION: faseelak https://git.opendaylight.org/gerrit/#/c/61931/ (vorburger, 14:28:03) * ACTION: viveks faseelak https://git.opendaylight.org/gerrit/#/c/63121/ (vorburger, 14:29:01) * ACTION: vthapar https://git.opendaylight.org/gerrit/#/c/59177/ +1 and merge (vorburger, 14:30:37) * LINK: https://git.opendaylight.org/gerrit/#/c/63381/ (vorburger, 14:31:06) * LINK: https://git.opendaylight.org/gerrit/#/c/63372/5 (vorburger, 14:32:43) * LINK: https://git.opendaylight.org/gerrit/#/q/topic:transaction-helper (vorburger, 14:32:56) * ACTION: vorburger skitt to further discuss https://git.opendaylight.org/gerrit/#/q/topic:transaction-helper (vorburger, 14:34:42) * LINK: https://git.opendaylight.org/gerrit/#/c/63120/ (vorburger, 14:35:50) * ACTION: faseelak edavsua review https://git.opendaylight.org/gerrit/#/c/63120/ (vorburger, 14:37:44) * LINK: https://git.opendaylight.org/gerrit/#/c/53998/ (vorburger, 14:38:01) * ACTION: vorburger add vthapar to upcoming change generalizing the idea from https://git.opendaylight.org/gerrit/#/c/53998/ (vorburger, 14:42:57) * Sonar Coverage (vorburger, 14:50:10) * ACTION: vorburger maybe together with skitt to look at re-enabling InterfaceManagerConfigurationTest (vorburger, 14:51:03) * ACTION: ALL look for projects that can be removed, because that will help to drive coverage up ;-) (vorburger, 14:51:35) * Cookies (vorburger, 14:51:42) * No cookies today, only samosas (vorburger, 14:51:52) Meeting ended at 14:51:54 UTC. Action items, by person ----------------------- * edavsua * viveks vthapar Hema will help to review the low risk "mechanical" changes often proposed by skitt and edavsua * skitt edavsua vorburger ALL to include Hema as Reviewer on more changes * faseelak edavsua review https://git.opendaylight.org/gerrit/#/c/63120/ * faseelak * faseelak https://git.opendaylight.org/gerrit/#/c/61931/ * viveks faseelak https://git.opendaylight.org/gerrit/#/c/63121/ * faseelak edavsua review https://git.opendaylight.org/gerrit/#/c/63120/ * viveks * viveks vthapar Hema will help to review the low risk "mechanical" changes often proposed by skitt and edavsua * viveks faseelak https://git.opendaylight.org/gerrit/#/c/63121/ * vorburger * vorburger will request CSIT on genius and netvirt once https://git.opendaylight.org/gerrit/#/c/60303/ * skitt edavsua vorburger ALL to include Hema as Reviewer on more changes * vorburger skitt to further discuss https://git.opendaylight.org/gerrit/#/q/topic:transaction-helper * vorburger add vthapar to upcoming change generalizing the idea from https://git.opendaylight.org/gerrit/#/c/53998/ * vorburger maybe together with skitt to look at re-enabling InterfaceManagerConfigurationTest * vthapar * viveks vthapar Hema will help to review the low risk "mechanical" changes often proposed by skitt and edavsua * vthapar https://git.opendaylight.org/gerrit/#/c/59177/ +1 and merge * vorburger add vthapar to upcoming change generalizing the idea from https://git.opendaylight.org/gerrit/#/c/53998/ * **UNASSIGNED** * ALL use "Mechanical:" commit message prefix for changes which are clean up type changes done automatically by IDE like tools * ALL look for projects that can be removed, because that will help to drive coverage up ;-) People present (lines said) --------------------------- * vorburger (39) * odl_meetbot (6) * vthapar (1) * faseelak (1) * viveks (1) * edavsua (1) * shague_ (1) Generated by `MeetBot`_ 0.1.4