01:45:19 #startmeeting 01:45:19 Meeting started Wed Jan 15 01:45:19 2014 UTC. The chair is edwarnicke. Information about MeetBot at http://wiki.debian.org/MeetBot. 01:45:19 Useful Commands: #action #agreed #help #info #idea #link #topic. 01:45:37 #topic rollcall: who's here for the Hydrogen Release Sync meeting 01:46:00 #info dbainbri present 01:47:24 #info Hideyuki Tai present 01:47:37 Anyone else? 01:48:18 Christine_: Are you here for the meeting? :) 01:48:28 Hi, yes 01:48:46 Sorry being late~ 01:49:16 No worries :) 01:51:22 #topic Agenda Bashing 01:51:41 Does anyone have anything they'd like to address first before we start walking down topics (or questions, etc) 01:52:07 I don't have anything. 01:52:08 noope 01:52:41 me either 01:52:48 Cool 01:53:00 #topic Review of Logging Recommendations 01:53:31 regXboi was kind enough to prepare a quick draft on logging recommendations: https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings 01:53:54 The idea being... well... we log to much, and we should trim back on how much we log... but it helps to have some recommendations as to when to log at what level 01:54:12 Ok. 01:54:15 Do you guys have any thoughts/feedback/objections to the recommendations on log levels and when to use them? 01:54:45 debug - method entry / exit? 01:54:54 really need that level of debug? 01:55:14 or are those "examples" 01:55:40 No comments from me. 01:55:44 dbainbri: The notion is not to say 'You need to log this' but rather to say 'If you are going to log this, please use debug, not info' 01:55:57 got it. thx. 01:57:03 I'll give this page to our engineer, and let them review it. 01:57:14 hideyuki: Thank you :) 01:57:35 Please feel free to provide any feedback at all, we'd like to get it right, and all of us together will come up with a better solution than one of us :) 01:57:43 Christine_: Any feedback/ 01:57:44 ? 01:58:19 It's fine with me :) 01:58:45 Cool 01:59:09 The idea will be that once we settle on that, we will have each project go back and try to come in line with the recommendation, so we don't have log polution 01:59:24 #topic Dependency Convergance 02:00:00 Have you guys seen the email Madhu sent out about setting up a Jenkins Job to upgrade to the latest versions of dependencies? 02:00:31 Yes, I've seen this mail. 02:00:46 yes 02:00:54 And, VTN Project has already created the job: https://jenkins.opendaylight.org/vtn/job/vtn-version-changes/ 02:00:55 yes. 02:01:22 snmp4sdn also created that. 02:01:51 and fix the dependencies not the latest 02:02:17 #info VTN has created their Dependency Check Job https://jenkins.opendaylight.org/vtn/job/vtn-version-changes/ 02:02:24 Christine_: Do you have the link for yours? 02:02:33 Thank you, edwarnicke . 02:02:44 hideyuki: Has it run yet, and have you guys upgraded your dependencies yet? 02:02:49 yes, here it is: https://jenkins.opendaylight.org/snmp4sdn/job/snmp4sdn-version-changes/ 02:03:10 #info SNMP4SDN has created their Dependency Check Job https://jenkins.opendaylight.org/snmp4sdn/job/snmp4sdn-version-changes/ 02:03:22 dbainbri: What about you guys? 02:03:36 We've run the job, and no error messages appear. 02:03:46 hideyuki: Excellent :) 02:04:06 we are not really doing anything that needs this right now. we are still mostly exploring ;) 02:04:17 #info VTN is reported to be up to date by the Dependency Check Job 02:04:28 dbainbri: Apologies...sometimes you get into a pattern :) 02:04:44 no worries. i am just a fly on the wall. 02:04:50 Christine_: What is the state of SNMP4SDN on getting your dependencies up to date 02:05:50 snmp4sdn-version-changes found two dependencies not the latest, and then I updated them in pom. 02:06:07 So all the dependencies are latest now in snmp4sdn. 02:07:05 #info SNMP4SDN is reported to be up to date by the Dependency Check Job 02:07:20 #topic 02:07:32 #topic Cleaning up System.out.println 02:08:16 So, we need to clean up our use of System.out.println to use proper loggers instead so we don't wind up with unmanagable output to console 02:08:35 I've filed bugs for each project with the results of a quick and dirty grep script I ran 02:09:01 For VTN its here: https://bugs.opendaylight.org/show_bug.cgi?id=304 02:09:22 Thank you, I checked your mail and the bug report. 02:09:24 For SNMP4SDN its here: https://bugs.opendaylight.org/show_bug.cgi?id=303 02:09:51 hideyuki: Thank you, does it look like something you guys could clean up for tomorrow (or do you see any issue with my reporting, as mentioned, it was a quick and dirty script) 02:10:35 Christine_: Does it look like something you guys could clean up for tomorrow (or do you see any issue with my reporting, as mentioned, it was a quick and dirty script) 02:11:20 I think VTN Project can clean up this by tomorrow, edwarnicke . 02:12:23 #info hideyuki has committed the VTN project to cleaning up System.out.printlin issues ( https://bugs.opendaylight.org/show_bug.cgi?id=304 ) by 1/15/2014 02:12:42 Should we delete System.out.pirntln in test code? 02:13:06 hideyuki: To a large extent that's up to you 02:13:22 I'm not quite sure about clean up tomorrow, for snmp4sdn. There are quite many ones. 02:13:28 From my perspective, the project has an interest in requesting that you guys clean up System.out.println in stuff that will run in the controller 02:13:39 Because that will produce console pollution 02:13:55 I see. I'll discuss about this with my colleagues. 02:14:26 I would suggest that using log rather than System.out.println is a good practice in tests, but I do not think my opinion is the deciding one there. I think that's up to the VTN committers to decide. 02:14:53 It seems that we use System.out.println only in test code in VTN Project. 02:14:55 Two days for snmp4sdn to clean up is enough. 02:14:58 #info VTN will decide whether or not to clean up System.out.println statements that are in tests but not production code. 02:15:44 #info Christine_ has committed snmp4sdn to cleaning up System.out.printlin issues ( https://bugs.opendaylight.org/show_bug.cgi?id=303 ) by 1/16/2014 02:16:43 Christine_: I noticed you guys have two third-party projects, expect4j and snmpj for which you've copied the source code into third-party in snmp4sdn, and they contain many many System.out.println statements. 02:16:53 Did those come with the original code, or did you guys add them? 02:18:37 #info regXboi joins the meeting 02:19:14 Christine_: Are you still there? 02:19:20 oh, yes. 02:19:24 wait a moment pls~ 02:19:27 on the topic of updating dependency versions, I believe that opendove is now caught up 02:19:50 Could you provide the link to your job? 02:20:13 #link https://jenkins.opendaylight.org/opendove/job/opendove-version-changes/ 02:20:35 that's verbatim based on Madhu's mail 02:20:49 #info openDove has created their Dependency Check Job https://jenkins.opendaylight.org/opendove/job/opendove-version-changes/ 02:21:17 #info opendove is reported to be up to date by the Dependency Check Job 02:21:20 edwarnicke: Those System.out.println in the third-party projects came with the original code. 02:21:33 OK, thank you for letting me know :) 02:21:37 phooey - on the syslog, what's the link for the minutes again? 02:21:53 The minutes for this meeting? 02:21:59 yes, that wiki page 02:22:20 https://wiki.opendaylight.org/view/CrossProject:Hydrogen_Release_Work 02:22:44 Christine_: I've left out the third-party System.out.println from the bug on the theory that they came from the third-party 02:22:46 #info draft syslog levels and uses are available from https://wiki.opendaylight.org/view/Draft_Syslog_Level_Settings 02:23:33 regXboi: We looked at them earlier in the meeting, folks were fine, but dbainbri expressed some confusion that we were *requiring* debug logs on every function entry and exit, and hideyuki is going to check with his colleagues and come back with feedback 02:23:42 I've added a Feedback section to the page 02:23:57 #topic Copyright/License headers 02:24:19 Did everyone see the email from phrobb about copyright/license headers in files? 02:24:20 have we considered a jenkins job to check code headers? 02:24:31 and I'm here if there are questions 02:24:31 edwarnicke: thanks, we'll notice all system.out.println in snmp4sdn repo :) 02:24:35 regXboi: Sounds like a good idea :) 02:24:38 on the email - I don't think so 02:24:41 Yes, I've checked the mail about copyright. 02:25:17 edwarnicke: I'll talk with you and Madhu tomorrow about how to "make the jenkins job so" 02:25:45 regXboi: Thank you 02:25:52 edwarnicke: welcome 02:26:10 #info regXboi is going to consult with Madhu_offline on 1/15/2014 about getting a Jenkins Job going to check copyright/license headers 02:26:48 We'll check the list in the mail, and will reply to Phil Robb. 02:27:03 Thank you :) 02:27:21 #info VTN has received copyright/license headers email from phrobb and is on it 02:27:37 Christine_: Where do you guys stand with the copyright/license headers? 02:27:49 regXboi: You should have email to opendove-dev from phrobb 02:28:18 I'm pretty sure we have correct copyright headers (thanks to ashaikh), but I'll check the email tomorrow 02:28:32 also the copyright needs to be on all files - not just java... 02:29:09 regXboi: I agree on the all *feasible* files (sadly, not all file types have good commenting mechanisms :( ) 02:29:33 edwarnicke: point accepted 02:29:44 #info regXboi from opendove has been notified of copyright/license headers email from phrobb and will follow up with him tomrrow 02:30:07 sounds good 02:30:11 agreed 02:30:13 Christine_: Still there? 02:30:23 agreed. 02:30:47 Anything else folks feel we need to cover before we end the meeting? 02:30:50 oh, yes. 02:31:07 Christine_: You saw the copyright/license headers email from phrobb ? 02:31:24 we've received mail about Copyright/License Information, will follow it. 02:31:59 #info Christine_ from SNMP4SDN has received copyright/license headers email and will follow it 02:32:02 then I think we need one more info and we can call it 02:32:07 What? 02:32:15 that was what we needed 02:32:19 that last info 02:32:22 #endmeeting