17:00:08 #startmeeting 17:00:08 Meeting started Wed Jan 15 17:00:08 2014 UTC. The chair is colindixon. Information about MeetBot at http://wiki.debian.org/MeetBot. 17:00:08 Useful Commands: #action #agreed #help #info #idea #link #topic. 17:00:25 ok, I'm going to delay attendance for now since we'll give people a few minutes to come 17:01:18 we're still missing RobDolin and chrisprice 17:02:13 #info copyright/license headers 17:02:33 #info dual_regXboi wants to know about files that don't allow fro comments 17:02:51 #info tykeal believes that phrobb sent out the LICESE file to use in that case 17:02:54 #link https://docs.google.com/spreadsheet/ccc?key=0AoSzir1BfjyWdDQyVElWNG9mcWxhblREckZjbjFxUVE#gid=1 for the release spread sheet / agenda 17:03:07 #action phrobb to clarify this explicitly 17:03:13 thanks tykeal 17:03:16 sure 17:03:35 I always think it's handy to have in the notes ;) 17:03:40 #topic overview of meeting and attendance 17:03:57 #info I'm going to focus on things with a deadline before today that aren't done yet first 17:04:12 also, can everyone pound info with their attendance here and what projects (if any they represent) 17:04:18 #info Ed Warnicke is in attendance 17:04:19 #info tykeal in attendance for infrastructure support 17:04:27 #info abhijitkumbhare is present: Openflowplugin 17:04:28 #info Chris Wright is here 17:04:34 Konstantin for defense4all 17:04:40 #info Madhu for ovsdb 17:04:41 #info notes tykeal == Andrew Grimberg 17:04:44 #info michal_rehak is present: Openflowplugin 17:04:47 #info Ed Warnicke is in attendance for controller 17:04:47 #info dkutenic present for bgpcep 17:04:50 #info Luis for Integration test 17:04:51 #info dual_regXboi opendove 17:04:55 #info Anees Shaikh 17:05:02 #info Konstantin defense4all 17:05:02 Suchi for affinity 17:05:11 #info suchiraman for affinity 17:05:24 #info Konstantin for defense4all 17:05:28 getting those in the minutes 17:05:33 good attendance 17:05:44 #info lori for lispflowmapping 17:06:01 moving to the next topic 17:06:06 #topic Write recommendation for cutting Release Branches and laying Release labels 17:06:15 can we mark this as dead Madhu and LuisGomez 17:06:25 I think we can, but I want to make sure and cross it off 17:06:27 colindixon: yes. should be DONE. 17:06:29 will mark it 17:06:45 #action Madhu to mark this as DONE in the spreadsheet 17:06:48 QQ: What is the link for the recommendation? 17:07:00 * Madhu finding 17:07:10 https://wiki.opendaylight.org/view/Simultaneous_Release:Simultaneous_Release_Plan_2014:DependencyConvergence 17:07:12 is that it? 17:07:19 then pound link that 17:07:27 #info oflibMichal is present for openflowjava 17:07:35 Madhu, could you take an action to make sure the link is in the in the spreadsheet in case folks look for it there 17:07:36 note that pound link has to be the first non-other-command on a line to work I thinkg 17:07:50 colindixon: you're correct on that 17:08:00 #info ttkacik is present for yangtools 17:08:18 all of the #commands work that way 17:08:34 Madhu: is that the right link so we can move on? 17:08:49 #info https://wiki.opendaylight.org/view/Simultaneous_Release:Simultaneous_Release_Plan_2014:ReleaseRecommendations 17:09:10 #info #link https://wiki.opendaylight.org/view/Simultaneous_Release:Simultaneous_Release_Plan_2014:ReleaseRecommendations (same link as above) 17:09:11 i will pretty it up 17:09:12 thanks 17:09:17 lol 17:09:22 Madhu: Can you add the link to the spreadsheet? 17:09:28 edwarnicke: sure. 17:09:36 Madhu: thanks :) 17:09:43 #action Madhu to add the link to spreadsheet 17:09:44 #action Madhu to pretty up those instructions some (he's a glutton for punishment, that one) 17:09:55 Madhu: you can #link and #info. Those are non-chair commands 17:10:05 Madhu: thanks, you did all the work here 17:10:09 tykeal: learning :) thanks 17:10:18 #topic user manual and developer guide 17:10:19 it's ok, I think we're all learning how the bot works 17:10:38 has anyone heard from Chris Price and RobDolin about this? 17:10:48 its a really big task, but it's importand 17:11:21 Yes. Rob sent an email 17:11:27 but didn't get a chance to think it through. 17:11:51 Madhu: Has he asked for more help? 17:12:00 link to the e-mail? 17:12:02 where is it? 17:12:04 yes. 17:12:13 email to the team volunteered to help 17:12:20 ah 17:12:49 ah, so not on any of the lists 17:12:54 #action colindixon to reach out to the doc team and ask them to provide an e-mail to discuss with progress 17:13:02 that's about the best we can do there 17:13:05 or attend one of the meetings :) 17:13:25 edwarnicke: that's like hearding cats! 17:13:45 #topic log level guidelines 17:13:51 dual_regXboi: anything to report? 17:14:04 draft was in place in last meeting minutes 17:14:12 i.e. west coast minutes 17:14:13 sweet 17:14:17 I should have read that 17:14:23 #link https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings 17:14:25 We got some feedback from the 5:45pm PST crowd 17:14:36 and that page was updated accordingly 17:14:39 any other feedback from this crowd would be nice 17:14:46 dual_regXboi: Could we also add to debug block entry and exit (like if statment blocks etc) 17:15:08 um 17:15:09 sure 17:15:19 isn't there a trace level? or am I crazy? 17:15:25 no there isn't 17:15:32 ok, good to know 17:15:35 at least I couldn't find one 17:15:41 I also have some concern about the INFO being used for missing parameter being set to default 17:15:45 Because that happens a lot 17:15:51 But I suspect we don't usually log it 17:15:54 So it may be OK 17:16:03 well... i'd rather log it info than error 17:16:07 which is what i was seeing 17:16:16 ya'll don't set all your parameters... ever ;) 17:16:19 definitely more of an INFO than an ERROR 17:16:29 Cool... lets go with INFO there 17:17:01 this is, missing some general advice on how many lines of logging should be generated in normal operatiojn 17:17:02 QQ: What should our default log level be for the controller overall? 17:17:05 In production 17:17:07 yeah 17:17:08 though maybe more of a DEBUG, dunno ;) 17:17:09 block entrance/exit added 17:17:15 I put that at the bottom 17:17:18 dual_regXboi: Thank you :) 17:17:20 the doc says default should be notice 17:17:27 that was my proposal 17:17:28 Excellent :) 17:17:31 I tend to agree 17:17:32 notice ? 17:17:33 I'm not tied to it though 17:17:43 do we want to say that basically in normal operation bundles should log being loaded and unloaded only? 17:17:58 I think so 17:18:05 I tend to think so as well 17:18:09 and I pound agreed to that? 17:18:10 should the default be warning? 17:18:11 edwarnicke: question 17:18:13 any objections 17:18:21 Madhu: please ask :) 17:18:27 warning? 17:18:27 the more opinions the better the end result :) 17:18:28 I think notice will be a little too chatty 17:18:28 are these recognized level in log back configuration ? 17:18:39 warning? you mean default recorded? 17:18:43 debug info error warn error. 17:18:50 abhijitkumbhare, if they only thing logged in notice is bundle loading and unloading, then I think notice is better than warning 17:18:52 abhijitkumbhare: I don't much mind telling folks that things have started up or shut down by default... those are rare occurences and useful to see 17:18:59 I haven't seen alert emergency notice etc. 17:19:48 ok - looking into ryan's recommendation - notice is only for bundle start/stop - so notice sounds fine 17:19:50 dual_regXboi: can u pls point me the documentation where these log levels are defined ? 17:19:50 Madhu: if notice is only used for bundle startup/shutdown, fixing that should be easy 17:20:01 Madhu: man syslog? 17:20:04 colindixon: my question is not that :) 17:20:15 Apparently not in logback: http://logback.qos.ch/manual/architecture.html 17:20:25 yes. we shud worry only about logback 17:20:27 not syslog 17:20:35 #info there is a lot of useful discussion happening about this topic and if interested people should consult the full logs 17:20:36 well now that's rather annoying 17:20:42 TRACE < DEBUG < INFO < WARN < ERROR 17:20:46 exactly 17:20:52 those are the only 5 levels we have 17:20:56 ok... so give me back the action to remap 17:20:58 so i don't quite understand all others 17:21:14 #info #link https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings is the page being discussed 17:21:26 #action dual_regXboi to remap to logback levels 17:21:36 I should be able to do that this afternoon 17:21:38 Madhu: thanks for bringing that up... could have been ugly to discover late 17:21:40 colindixon: do that as #link and not #info #link 17:21:56 edwarnicke: lol. my mistake that I didn't sync up with dual_regXboi yest 17:22:01 #link https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings is the page being discussed 17:22:02 #link http://logback.qos.ch/manual/architecture.html 17:22:02 crazy work hours. 17:22:04 tykeal: I think both work 17:22:18 Madhu: I feel you :) 17:22:20 but we'll see in these logs :p 17:22:28 ok 17:22:33 have we run this one down? 17:22:34 this has been really useful 17:22:39 a #info never hyperlinks a #link does. but only the first command (as the start of a line) is recognized by the bot 17:23:45 so, did we decide how wanted to structure logging 17:23:59 I think that cdub was about to suggest that we log more to the file than to stdout 17:23:59 how do you mean structure? 17:24:00 I think we decided dual_regXboi would revisit and come back tomorrow 17:24:15 I meant the default log level and what we were expecting to come there 17:24:23 colindixon: will add that to the page 17:24:31 definitely should log to file 17:24:33 there was a suggestion that the log be only bundle startup, bundle teardown, warnings, and errors 17:24:48 and file should be downloadable via web 17:24:52 but i think cdub was going to say that logging more to a logfile would be a good idea, but I'm not sure 17:24:53 default log level (cut off things below) and send them all (>= default) to file 17:24:56 By default, obviously folks can tweak that up 17:25:15 I don't think we should, by default, be logging debug and trace though 17:25:17 so... next question on this 17:25:19 Even to file 17:25:20 Thats a lot 17:25:29 (1) access via opendaylight web? 17:25:33 (2) rotation? 17:25:42 rotation is already there 17:25:42 #info cdub suggests "default log level (cut off things below) and send them all (>= default) to file" 17:25:48 access via web is optional 17:26:05 ok, but would should explain where the file is 17:26:07 #info edwarnicke thinks this may be a lot to log to even file, and we should not do debug and trace there 17:26:15 so that people know how to get to it 17:26:23 #info dual_regXboi suggests providing log file access via the web UI as well 17:26:28 dual_regXboi: it is documentation work :) to say look @ logs folder 17:26:37 lets not add more development work around this. 17:26:44 debug and trace are very volumnious and woudl be expected to adversely effect performance 17:26:49 #info will follow up with this later today and or tomorrow 17:26:59 edwarnicke: I agree 17:27:01 edwarnicke: agreed 17:27:07 even info affects performance as it is right now 17:27:20 lets let dual_regXboi try to capture this and go over it again tonight or tomorrow 17:27:21 true, but we abuse info right now, we need to stop that ;) 17:27:22 yes. info is abused today 17:28:00 I'm going to move on to the next topic unless anyone objects 17:28:03 personally to me, bundle coming up and going down 17:28:03 agreed. we use #info a lot of times when #debug should be used. but we can't clean it all up now 17:28:04 please 17:28:05 #topic Identify differing versions of dependencies on external artifacts and negotiate a recommended version 17:28:07 are not interesting. 17:28:19 cdub and Madhu, is this done 17:28:29 it's not make the version happen 17:28:38 but it's negotiate the version 17:28:44 or recommended version 17:28:48 cdub: u want to take this up ? 17:28:52 Madhu's take is that it's unrealistic 17:29:08 yes :( after spending some time on this. 17:29:14 I'm willing to try, the issue is deps of deps. 17:29:19 Madhu: Could you elaborate, not disagreeing, but curious 17:29:22 so...first level external dpes...maybe 17:29:22 it is not as easy as the internal dependancy 17:29:26 beause 17:29:34 let me give a simple example 17:29:35 2nd level external deps is not really reasible 17:29:45 well, certain dependencies are bad to have multiple version, netty in particular 17:30:00 cdub: If we agree on direct deps, second level deps should follow, correct? 17:30:01 yes, and that's why i'm wiling to try for our first level external deps 17:30:02 open daylight bundle A -> depends on external dependency B depends on external dependency C 17:30:06 could we indentify external dependencies that it's critical to agree on 17:30:11 edwarnicke: not necessary 17:30:13 edwarnicke: no 17:30:19 (except where two first level deps disagree on their own deps) 17:30:29 that is quite common 17:30:36 edwarnicke: 2 projects...each a different direct dep...which in turn have same (differen version) next dep 17:30:49 and many of the dependency convergence is pointing to such 2nd level dep conflicts 17:31:13 Yes 17:31:23 so trying to get a clean dependency convergence for external dependency is not practical 17:31:23 OK, get it :) 17:31:25 we can publish where we diverge...that's easy 17:31:27 thank you for the explanation 17:31:33 and discuss what is improtant to converge on 17:31:35 that work? 17:31:42 Madhu: Would convergence on direct external deps be doable? 17:31:53 #info Madhu believes that converging on external dependencies is likely to be not doable for this release 17:31:53 tony we need also to manage 2level if there are conflicts 17:32:16 edwarnicke: even if possible, am actually starting to question the value of that 17:32:23 i am at a stage now 17:32:26 don't break what is working 17:32:44 #info possible suggestion is to identify external dependencies that might cause problems with different versions and just focus on those 17:32:48 Madhu: are we there now? 17:32:52 if anyone complains the inconsistent external dependency causing issues, then we can address that particular one 17:32:53 I agree with you 17:33:11 that doesn't sound like a horrible suggestion 17:33:25 assume that we're ok and see if things fall out in test? 17:33:35 fix this post-release? 17:33:44 yes. lets consider it for post-release 17:33:46 it's a bit scary but maybe less scary than introducing lots of flux right now 17:33:49 if we all agree on it 17:33:52 cdub, edwarnicke: thought? 17:33:55 ok 17:34:00 thats my biggest concern 17:34:05 i see no harm in publishing what we know now 17:34:17 it's a trying to pick the lesser of two evils for sure 17:34:20 if something jumps out...deal w/ it now, else post-release sounds ok 17:34:20 we should at least converge on OSGI framework... I noticed several different declarations as dependency 17:34:29 ttkacik: osgi.core...yes 17:34:36 that's a good example, as is netty 17:34:38 netty also 17:34:52 converge means ? :) 17:34:54 latest ? 17:34:57 guava and apache commons 17:34:59 it calls for testing effort 17:35:07 all projects using same version of third-party 17:35:29 yeah, latest doesn't matter as much as same version throughout 17:36:00 madhu it seems that it calls for the testing effort 17:36:09 but in distro you have allways highest one 17:36:13 unless there is a project that needs latest (e.g. netty for OF 1.3) 17:36:32 I think we are synced on netty versions, correct? 17:36:51 i think i saw 5.x now 17:36:54 edwarnicke: I don't know that we *need*, but not having them will introduce multiple thread pools and other things 17:36:58 ovsdb at least is in 4.0.10 17:37:05 5.x is not feasible now 17:37:14 I thought there were mails to converge on a 4.x a while back 17:37:19 ttkacik: that is my exact concern 17:37:22 5.x seems like a bad idea 17:37:28 to put it more correctly same version through ODL...not latest one 17:37:52 agree 17:37:52 ttkacik: can someone suggest the absolutely minimum 3rd party bundle that needs convergence ? 17:37:56 lets deal with that only ? 17:38:08 is there consensus that we need to identify a few critical external dependencies that we need to agree on the version for the release? 17:38:12 and then deal with those 17:38:13 yese 17:38:14 to me, this is extremely risky at this point 17:38:20 but if you feel we shud do it... sure :) 17:38:32 I think every agrees on osgi.core 17:38:45 do we have experience with running two version of netty simultaneously? 17:38:49 does it work? 17:38:50 I think the question to me is this: do we have cases where the skew for third party dependencies is causing problems? 17:38:55 colindixon: lets not go there :) 17:38:55 If so, we should fix it 17:39:05 yeah 17:39:19 my suggestion is: osgi.core - API (5.x), netty (4.x), google guava, apache commons, jersey 17:39:19 unless the libraries are embedded into the bundle. 17:39:22 it is a bad idea 17:40:00 guys. if it is not broken, lets not fix it :) 17:40:14 #info the consensus seems to be that we need to try to find external dependencies where different versions can/are causing problems and focus only on those for version convergence 17:40:17 sorry. i have to drop off now. 17:40:20 Madhu: Do we know what's broken? 17:40:26 i dont :) 17:40:32 #info it's not clear which, if any, are breaking anything now 17:40:42 can somebody take the action to figure that out? 17:40:43 and do we know that things that appear to be ok are not broken? 17:40:49 It strikes me as highly likely that osgi.core, and netty could lead to issues 17:40:50 ttkacik? 17:40:54 jersey was. and we fixed that already 17:40:59 could != is 17:41:15 janmedved: testing shud have brought it out by now :) 17:41:23 yeah 17:41:27 if not... enough testing is not done ? 17:41:33 I'm inclined to agree with Madhu here 17:41:49 osgi.core is 5.0 for equinox we are using 17:41:58 but AD-SAL is compiled against 4.2 17:42:07 but is that an issue? 17:42:28 that is what I noticed with AD-SAL 17:43:21 issue is...that 5.0 OSGI has more type-safe APIs, which are hidden when compiling against AD-SAL 17:43:22 ttkacik: but is that noticed skew or noticed problems? 17:43:24 Is it a good idea to be out of sync with our OSGI container? 17:43:53 we do not have great test coverage yet. and my concern is (like with osgo 4.2 vs. 5.0) there will be corner cases that will manifet themselves as soon as we start a havier load 17:43:57 heaiver 17:44:00 edwarnicke: nobody's asking if it's a good idea, that's clearly false, the question is it a *worse* I idea to change things 17:44:14 heavier (more involved) 17:44:17 yeah 17:44:19 colindixon: definitely. Compared to what? is always the question 17:44:34 so, can somebody take an action item to figure out if this is worth doing? 17:44:39 with osgi core 4.2 vs 5.0 is more of technical debt 17:44:48 that would involve seeing if 5.0 breaks things 17:45:23 and doing some due diligence about issues between 4.2 and 5.0 working together? 17:45:33 QQ: Is anyone using a non 4.10 netty that we know of? 17:45:38 5.0 does not break things (we are already using it - equinox is OSGIv5) 17:45:45 ttkacik: do you any issues with modules from md-sal (osgi 5.0) interacting with modules from ad-sal (4.2 based)? loading, API version management, etc... 17:45:53 sorry folks. have to drop off. tty in the evening. bye 17:45:53 changing the AD-SAL to use 5.0 17:45:54 does that work 17:45:56 but will introduce a lot of warnings code coded against 4.2 17:45:59 Madhu: you're all good 17:46:01 thanks for the help 17:46:10 because contracts are type-safe and use generics 17:47:07 janmedved: there is no issue in runtime, but some edge cases in compile time 17:47:11 I really want somebody to take the time to reach out to the projects using 4.2 and help them evaluate if 5.0 is going to break everything, preferably by doing the test on their own before reaching out 17:47:16 and basicly will create technical debt 17:47:53 otherwise, I'm pretty much entirely with Madhu_offline that we should avoid fixing things that aren't broken 17:47:57 if it's not runtime i would tend to argue for not changing it... 17:48:11 and I don't seen anyone jumping up to do that work 17:49:29 #help if somebody was willing to see if moving all projects to osgi.core 5.xo breaks things, then we could be more confident in deciding to enforce using that for the release 17:49:32 ok 17:49:46 colindixon: What's next? 17:49:48 are we done? 17:49:57 we're done with things with deadllines for now 17:49:59 we're way past that iron fist of 30 minutes ;) 17:50:04 I think we follow up with project specific things 17:50:07 tomorrow 17:50:10 then I'm going to motion that we are done 17:50:17 I think we're done 17:50:17 OK 17:50:23 I second that motion 17:50:25 #topic for next meeting 17:50:43 #info we'll start with project specific take up of the recommendations that we've hashed out tomorrow AM 17:50:50 #endmeeting