16:03:47 #startmeeting md_sal_interest_call 16:03:47 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 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 16:03:47 The meeting name has been set to 'md_sal_interest_call' 16:03:50 #chair catohornet 16:03:50 Current chairs: catohornet tbachman 16:03:59 #topic bug scrub 16:07:19 #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 #info moizer asks if we already have takers for all of the bugs from last week’s “easy” list 16:08:18 #info ttkacik1 says yes 16:09:56 #info tbachman asks if the bugs list is for all projects, or across all projects 16:10:23 #info ttkacik1 says the cross-project bugs are tracked in the Wednesday morning IRC meeting 16:10:57 #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 #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 #info catohornet asks if moizer’s volunteer could take this 16:11:53 #info moizer says he’d have to check with abhijitkumbare, as it’s technically his volunteer 16:12:12 #info debalina says she can take 2155 16:12:25 #action debalina to take on bug 2155 16:13:07 #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 #action maros to reach out to edwarnicke to ask him what the issue is — provide more details 16:13:58 tbachman: 1135 is *simply* complaining about the crypticness of the message 16:14:08 #info icbts says he can also look into that bug — might be something like a log4j setting 16:14:18 tbachman: not complaining that there is a bug in functionality 16:14:34 edwarnicke: thx! 16:14:57 #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 #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 #info maros says the main issue is tracked in bug 1110 16:18:49 #info catohornet says it looks like bug 2320 is difficult to reproduce 16:19:27 #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 #info catohornet asks if we can verify, resolve, close, etc.? 16:20:15 #info ttkacik1 says 2320 is an incorrect use of the data store. Should be closed and marked as invalid 16:22:10 ouch 16:23:48 #action catohornet to mark 2320 as resolved/invalid 16:25:06 #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 #action debalina to take 2375 16:27:55 #info moizer says bug 2523 might be difficult to reproduce 16:28:08 #info moizer says lets keep it around, but take it out of bugs to review 16:30:25 #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 #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 #info catohornet asks ttkacik1 what the status is with Bug 3051 16:35:04 #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 #info ttkacik1 says the bugfix for 3051 in yangtools uncovered bugs in other projects 16:36:58 #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 #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 #info ghall says kot-begemot might be able to help with this 16:40:53 #info ghall asks what other projects are dependent on this 16:41:02 #info ttkacik1 says it’s OVSDB, openflowplugin, and groupbasedpolicy 16:42:14 #info tbachman asks if we can clone yangtolls and cherry-pick the patch to stable/lithium and test 16:42:27 #info ttkacik says yes — you should get indications about the invalid values 16:42:53 #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 #info catohornet asks what the next step is for 3134 — try to reproduce? 16:45:13 #info maros says yes — someone should try to reproduce the issue and debug 16:45:57 #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 #info bug 3138 is the same as 3134 with just a different result 16:48:12 #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 #info ttkacik1 says 3051 is blocking the openflowplugin from version-bumping to beryllium yangtools, due to incorrect use of IpPrefixes 16:49:21 #info abhijitk_ asks if kot-begemot’s changes are the best way to get things unblocked 16:50:38 #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 #info kot-begemot says this used to work, but you need to change the conversion code 16:51:31 #info ttkacik1 says he’s seen patches for the openflowplugin, but just needs to review them 16:51:37 ttkacik1: Can I ask a question about 3051 16:52:11 #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 alagalah: we’ll ask for you 16:52:37 (relay it here) 16:52:42 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 Joining webexd 16:52:55 (joy... morez meetings) 16:52:58 yes 16:53:08 ttkacik1: Ok :) 16:53:12 #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 ttkacik1: Just making sure I'm not crazy 16:53:17 #info ttkacik1 says yes 16:53:46 #info ttkacik1 says if we merged the bug fix to yangtools now would break the downstream projects 16:55:38 #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 #action tbachman to make the fixes for this in groupbasedpolicy 16:58:39 going once.... 16:58:54 going twice…. 16:59:12 #endmeeting