16:03:47 <tbachman> #startmeeting md_sal_interest_call
16:03:47 <odl_meetbot> Meeting started Tue May 19 16:03:47 2015 UTC.  The chair is tbachman. Information about MeetBot at http://ci.openstack.org/meetbot.html.
16:03:47 <odl_meetbot> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:03:47 <odl_meetbot> The meeting name has been set to 'md_sal_interest_call'
16:03:50 <tbachman> #chair catohornet
16:03:50 <odl_meetbot> Current chairs: catohornet tbachman
16:03:59 <tbachman> #topic bug scrub
16:07:19 <tbachman> #info ttkacik1 says there are no incoming “easy” bugs since last weeks, so maybe we should look into unassigned bugs and prioritize them
16:08:15 <tbachman> #info moizer asks if we already have takers for all of the bugs from last week’s “easy” list
16:08:18 <tbachman> #info ttkacik1 says yes
16:09:56 <tbachman> #info tbachman asks if the bugs list is for all projects, or across all projects
16:10:23 <tbachman> #info ttkacik1 says the cross-project bugs are tracked in the Wednesday morning IRC meeting
16:10:57 <tbachman> #info ttkacik1 says bug 2155 is a medium level bug, and is in RESTCONF, and has to do with how deep the structure is desired by the user.
16:11:17 <tbachman> #info ttkacik1 says he can help anyone who can take the bug with details in how to implement this in the RESTCONF
16:11:39 <tbachman> #info catohornet asks if moizer’s volunteer could take this
16:11:53 <tbachman> #info moizer says he’d have to check with abhijitkumbare, as it’s technically his volunteer
16:12:12 <tbachman> #info debalina says she can take 2155
16:12:25 <tbachman> #action debalina to take on bug 2155
16:13:07 <tbachman> #info maros says that bug 1135 is quite old — we should check with edwarnicke to get a better understanding of what was meant but the bug report
16:13:40 <tbachman> #action maros to reach out to edwarnicke to ask him what the issue is — provide more details
16:13:58 <edwarnicke> tbachman: 1135 is *simply* complaining about the crypticness of the message
16:14:08 <tbachman> #info icbts says he can also look into that bug — might be something like a log4j setting
16:14:18 <edwarnicke> tbachman: not complaining that there is a bug in functionality
16:14:34 <tbachman> edwarnicke: thx!
16:14:57 <tbachman> #info edwarnicke says that bug 1135 is *simply* complaining about the crypticness of the message, and is not complaining that there is a bug in functionality
16:16:18 <tbachman> #info maros says bug 1946 is a complaint about the wrong response code from RESTCONF. It’s an issue with the exception hierarchy — already another bug open on this issue, and this is just a symptom
16:16:41 <tbachman> #info maros says the main issue is tracked in bug 1110
16:18:49 <tbachman> #info catohornet says it looks like bug 2320 is difficult to reproduce
16:19:27 <tbachman> #info moizer says he doesn’t know if this is happening any more, and even if it is, he thinks it’s likely to be more of an issue with the app itself
16:19:35 <tbachman> #info catohornet asks if we can verify, resolve, close, etc.?
16:20:15 <tbachman> #info ttkacik1 says 2320 is an incorrect use of the data store. Should be closed and marked as invalid
16:22:10 <tbachman> ouch
16:23:48 <tbachman> #action catohornet to mark 2320 as resolved/invalid
16:25:06 <tbachman> #info ttkacik1 sasy bug 2375 isn’t a bug — it’s a performance improvement in yangtools. If there are any takers, he can walk them through the bug
16:26:43 <tbachman> #action debalina to take 2375
16:27:55 <tbachman> #info moizer says bug 2523 might be difficult to reproduce
16:28:08 <tbachman> #info moizer says lets keep it around, but take it out of bugs to review
16:30:25 <tbachman> #info maros says that for bug 2679, we need to update the library and remove the workaround — only issue with the workaround is a performance impact
16:32:12 <tbachman> #info maros says if no one picks it up, they’ll try to fix this in time for the Lithium release; only implementations that expose EXI are affected by this
16:34:45 <tbachman> #info catohornet asks ttkacik1 what the status is with Bug 3051
16:35:04 <tbachman> #info ttkacik1 says the bug fix is merged in master, but not stable/lithium, b/c the classes that use this are still putting invalid input to it
16:35:53 <tbachman> #info ttkacik1 says the bugfix  for 3051 in yangtools uncovered bugs in other projects
16:36:58 <tbachman> #info ttkacik1 says there’s a patch for 3051 in at least one branch that’s not yet merged; not merged b/c once it merges, the verify merge jobs will not work for the openflowplugin until they fix their bug related to this
16:37:26 <tbachman> #info ttkacik1 says they’re waiting on the downstream projects to fix their bugs before they put the patch in, so that they don’t break the downstream projects
16:37:53 <tbachman> #info ghall says kot-begemot might be able to help with this
16:40:53 <tbachman> #info ghall asks what other projects are dependent on this
16:41:02 <tbachman> #info ttkacik1 says it’s OVSDB, openflowplugin, and groupbasedpolicy
16:42:14 <tbachman> #info tbachman asks if we can clone yangtolls and cherry-pick the patch to stable/lithium and test
16:42:27 <tbachman> #info ttkacik says yes — you should get indications about the invalid values
16:42:53 <tbachman> #info maros says that they haven’t yet analyzed bug 3134 — can’t tell if it’s an easy fix or something more complicated
16:45:01 <tbachman> #info catohornet asks what the next step is for 3134 — try to reproduce?
16:45:13 <tbachman> #info maros says yes — someone should try to reproduce the issue and debug
16:45:57 <tbachman> #info maros says you just run the feature test for the project — the part where the config pusher is involved is identical for all of the single-feature-tests
16:47:13 <tbachman> #info bug 3138 is the same as 3134 with just a different result
16:48:12 <tbachman> #info abhijitk_ asks about bug 3051 — about the downstream projects needing to be fixed before they can merge the fix for 3051. abhijitk_ asks what the fixes are for the downstream projects
16:48:50 <tbachman> #info ttkacik1 says 3051 is blocking the openflowplugin from version-bumping to beryllium yangtools, due to incorrect use of IpPrefixes
16:49:21 <tbachman> #info abhijitk_ asks if kot-begemot’s changes are the best way to get things unblocked
16:50:38 <tbachman> #info kot-begemot says the changes are to pass prefixes instead of IP address in a few places, but in a few places in the openflowplugin, you can’t fix the test cases b/c the openflowplugin’s match conversion code to openflow semantics makes the assumption that an IP address by itself is an allowed match
16:50:52 <tbachman> #info kot-begemot says this used to work, but you need to change the conversion code
16:51:31 <tbachman> #info ttkacik1 says he’s seen patches for the openflowplugin, but just needs to review them
16:51:37 <alagalah> ttkacik1: Can I ask a question about 3051
16:52:11 <tbachman> #info kot-begemot says that code has grown “organically”, so there’s fixes needed and a minimal amount of cleanup in order to make it comprehensible
16:52:33 <tbachman> alagalah: we’ll ask for you
16:52:37 <tbachman> (relay it here)
16:52:42 <alagalah> ttkacik1: Are you saying that you can't merge it to YangTools StableLi, because once you do, projects like GBP who haven't cut will break (thanks!) but also that you are waiting on OFP, who we are also waiting on ... it seems like a circular dependency ?
16:52:49 <alagalah> Joining webexd
16:52:55 <alagalah> (joy... morez meetings)
16:52:58 <ttkacik1> yes
16:53:08 <alagalah> ttkacik1: Ok :)
16:53:12 <tbachman> #info alagalah asks ttkacik1 if he’s  saying that you can't merge it to YangTools StableLi, because once you do, projects like GBP who haven't cut will break (thanks!) but also that you are waiting on OFP, who we are also waiting on ... it seems like a circular dependency
16:53:13 <alagalah> ttkacik1: Just making sure I'm not crazy
16:53:17 <tbachman> #info ttkacik1 says yes
16:53:46 <tbachman> #info ttkacik1 says if we merged the bug fix to yangtools now would break the downstream projects
16:55:38 <tbachman> #link https://git.opendaylight.org/gerrit/#/c/20704 The gerrit from kot-begemot that provides utility functions for projects to fix this
16:57:25 <tbachman> #action tbachman to make the fixes for this in groupbasedpolicy
16:58:39 <tbachman> going once....
16:58:54 <tbachman> going twice….
16:59:12 <tbachman> #endmeeting