===================================
#opendaylight-genius: genius weekly
===================================


Meeting started by vorburger at 14:03:24 UTC.  The full logs are
available at
http://meetings.opendaylight.org/opendaylight-genius/2017/genius_weekly/opendaylight-genius-genius_weekly.2017-08-17-14.03.log.html
.



Meeting summary
---------------

* info in  (vorburger, 14:03:34)
  * Vishal  (vthapar, 14:03:45)
  * skitt  (skitt, 14:03:46)
  * faseelak_  (faseelak_, 14:04:05)
  * viveks  (viveks, 14:04:12)
  * LINK:
    https://lists.opendaylight.org/pipermail/genius-dev/2017-August/001797.html
    (vorburger, 14:04:37)
  * netvirt already uses "Rebase if Necessary"  (vorburger, 14:05:41)
  * infrautils used to start with "Fast Forward only", but then changed
    it to "Rebase if Necessary"  (vorburger, 14:06:55)
  * edavsua  (edavsua, 14:09:38)
  * VOTE: Voted on "Shall the genius project change its Gerrit Submit
    Strategy from Merge if neccessary to Rebase if neccessary?" Results
    are  (vorburger, 14:09:50)
  * ACTION: vorburger will raise LF ticket for Gerrit submit change
    (vorburger, 14:11:14)

* idmanager Daexim integration OK?  (vorburger, 14:11:22)
  * LINK: https://git.opendaylight.org/gerrit/#/c/61769/  (vorburger,
    14:11:26)
  * LINK:
    https://git.opendaylight.org/gerrit/#/c/61769/4/idmanager/idmanager-impl/src/main/java/org/opendaylight/genius/idmanager/IdManager.java
    (vorburger, 14:11:44)
  * ACTION: Faseela to prompt Muthu to review it as well  (vorburger,
    14:13:51)
  * nobody seem to have any objections  (vorburger, 14:14:02)

* https://git.opendaylight.org/gerrit/#/c/60265/  (vorburger, 14:14:28)
  * LINK: https://git.opendaylight.org/gerrit/#/c/60265/ was the only
    use of this method in netvirt, which has been changed  (vorburger,
    14:15:58)
  * ACTION: Faseela to +2 and merge when build passed  (vorburger,
    14:17:44)

* Next steps re. deprecating AsyncDataTreeChangeListener?  (vorburger,
  14:18:29)
  * LINK: https://git.opendaylight.org/gerrit/#/c/59177/  (vorburger,
    14:18:40)
  * there was the idea to add clustered version as well, but that could
    come into a follow up change also  (vorburger, 14:20:31)
  * ACTION: vorburger to re-review it and +1 as David has taken his
    feedback into account  (vorburger, 14:21:28)
  * ACTION: Vishal to re-review and +2 and merge it if no other
    objections  (vorburger, 14:22:31)
  * ACTION: Vishal to see with David if he or him will do the follow-up
    to add the async part  (vorburger, 14:25:41)

* FYI mdsal ordering problem  (vorburger, 14:26:10)
  * LINK: https://git.opendaylight.org/gerrit/#/c/60313/  (vorburger,
    14:26:22)
  * ACTION: vorburger cherry pick to Nitrogen  (vorburger, 14:28:25)
  * This (c/60313) is the patch to look at how to work around case of
    weird arbitrary looking comparison failures between actual YANG
    DataObject read from DS and expected objects from Xtend  (vorburger,
    14:30:53)

* review changes  (vorburger, 14:31:12)
  * LINK:
    https://git.opendaylight.org/gerrit/#/q/project:genius+status:open+Verified%253E0+is:mergeable+NOT+label:Code-Review%253C0
    (vorburger, 14:31:19)
  * LINK: FYI
    https://wiki.opendaylight.org/view/BestPractices/Coding_Guidelines#Utility_classes_with_only_static_methods
    (vorburger, 14:41:14)



Meeting ended at 14:55:11 UTC.



Action items, by person
-----------------------

* vorburger
  * vorburger will raise LF ticket for Gerrit submit change
  * vorburger to re-review it and +1 as David has taken his feedback
    into account
  * vorburger cherry pick to Nitrogen
* **UNASSIGNED**
  * Faseela to prompt Muthu to review it as well
  * Faseela to +2 and merge when build passed
  * Vishal to re-review and +2 and merge it if no other objections
  * Vishal to see with David if he or him will do the follow-up to add
    the async part



People present (lines said)
---------------------------

* vorburger (32)
* odl_meetbot (6)
* edavsua (6)
* viveks (3)
* vthapar (2)
* skitt (2)
* faseelak_ (2)



Generated by `MeetBot`_ 0.1.4