17:05:42 #startmeeting kernel projects 17:05:42 Meeting started Tue Nov 8 17:05:42 2016 UTC. The chair is colindixon. Information about MeetBot at http://ci.openstack.org/meetbot.html. 17:05:42 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:05:42 The meeting name has been set to 'kernel_projects' 17:05:49 #topic agenda bashing 17:06:43 vorburger: we can hear you 17:07:16 #info no major topics for today 17:07:32 #topic bugs query to do bug scrub going forward 17:07:40 #info ryan did this 17:07:52 #link https://bugs.opendaylight.org/buglist.cgi?list_id=46270&order=product%2Cchangeddate%20DESC%2Cbug_status%2Cpriority%2Cassigned_to%2Cbug_id&product=aaa&product=controller&product=mdsal&product=netconf&product=odlparent&product=yangtools&query_based_on=&query_format=advanced&resolution=--- all bugs in kernel projects 17:08:03 #link https://bugs.opendaylight.org/buglist.cgi?bug_severity=blocker&bug_severity=critical&list_id=46270&order=changeddate%20DESC%2Cbug_status%2Cpriority%2Cassigned_to%2Cbug_id&product=aaa&product=controller&product=mdsal&product=netconf&product=odlparent&product=yangtools&query_based_on=&query_format=advanced&resolution=--- critical/major bugs in kernel projects 17:09:00 https://github.com/vorburger/opendaylight-eclipse-setup 17:10:02 #topic using oomph to manage maven projects 17:10:14 #link https://github.com/vorburger/opendaylight-eclipse-setup#how-to-add-additional-m2e-lifecycle-mapping information about how it can be done and made relatively easy 17:12:35 #info it sounds like the next steps are getting things to work so that eclipse can build in a separate environment with a separate target so eclipse actually works 17:12:58 #info vorburger says that he's done out about 8 gerrits that fix this and it should all work now 17:14:16 #link as far as vorburger knows, this is all done and working as of about 4 hours ago: https://lists.opendaylight.org/pipermail/dev/2016-November/002896.html 17:14:44 #info many, many kudos to vorburger for doing all this hard work 17:16:12 #info colindixon asks why vorburger uses Eclipse over IntelliJ, he answers 1.) it's what he's used to and 2.) philosophically Eclipse is much more open source and community oriented 17:19:04 #topic config filtering parent 17:19:22 #link https://git.opendaylight.org/gerrit/#/c/44561/ vrpolak says that this should be ready to merge it 17:20:07 #info skitt asks why we want to add a new parent to stable/boron, is there some fetures that we need to backport that depend on this 17:21:55 #info vrpolak says that this is really a way to get a better way to specify the version that the distribution reports 17:23:47 https://git.opendaylight.org/gerrit/#/q/project:controller+status:merged+NOT+owner:%22Tom+Pantelis+%253Ctpanteli%2540brocade.com%253E%22 17:25:40 colindixon, check https://git.opendaylight.org/gerrit/#/q/project:controller+status:merged+NOT+owner:%22Tom+Pantelis+%253Ctpanteli%2540brocade.com%253E%22,50 out for an alternative view ;-) 17:26:46 skitt: is that a view you like :p 17:26:50 careful what you wish for 17:26:56 skitt: how do you do that query in gerrit minus the "Mechanical code cleanup" 17:27:05 vorburger, exactly 17:27:27 #topic log messages database 17:27:29 #link https://git.opendaylight.org/gerrit/#/c/39410/ 17:27:30 haha looks like skitt deserve that way more that me 17:27:42 * skitt cheated a bit there 17:27:51 scripted git FTW 17:28:10 * adetalhouet not good with gerrit search tool 17:28:39 #link https://lists.opendaylight.org/pipermail/dev/2016-November/002894.html colindixon's mail picking up this thread 17:28:47 #link https://git.opendaylight.org/gerrit/#/c/39410/ 17:29:01 #undo 17:29:01 Removing item from minutes: 17:30:58 #info vorburger says that first stage is to get this patch to run everywhere so that we can actaully have a catalog we can use going forward 17:31:38 #info vorburger says that we have this extraction checker that isn't running everywhere because it's in the yangtools checkstyle logging and in the odlparent checkstyle we're not allowed to depend on them 17:32:30 #info vorburger had been planning to submit the checkstyle rules to the public checkstyle so that we could deal with it in a sane way 17:34:30 #action vorburger will push the yangtools publicly usable checkstyle things into the upstream checkstyle rules so that it also works for IDEs 17:35:11 #info in the long run, vorburger argues that the "right" way to get stable log messages would be to use some new facade that would give unique IDs to each log message, e.g., JBOSS logging 17:35:36 #info skitt says this hasn't succeeded in his experience except at IBM in a closed environment because it makes logging hard 17:37:23 #info skitt says that using gettext-based methods to use the english text as the log message ID + some lightweight translation to deal with when you edit it or translate it into other languages 17:39:12 #info skitt says that there are cool things he's used where logstash lets you click on surprising error messages in logstash to file a bug, that would be cool here 17:41:07 #link https://github.com/sevntu-checkstyle/sevntu.checkstyle/issues/461 is what I'll have to start on picking up - I had dropped the ball on that. If anyone has capacity to help with that, please contact me! 17:41:35 #info ryan also says that if there are particular log messages that annoy us, we should open bugs for it 17:43:17 #info adetalhouet points out that many of the most annoying log messages come from the config subsystem, which is slowly being phased out 17:44:08 #topic deprecating or removing config subsystem 17:44:44 #info colindixon asks if we can remove config subsystem in carbon, adetalhouet says that ~2/3s of ODL still uses the config subsystem 17:45:24 #info adetalhouet says that his point of view is that it might be bettter to deprecate the config subsystem in nitroget with better messaging now 17:45:27 #undo 17:45:27 Removing item from minutes: 17:47:43 #action colindixon will bring up the issue of maybe deprecating the config subsystem in carbon and/or saying we will do it in the future 17:49:02 #info skitt says that a lot of projects fall into one of two bits: so inactive they don't even merge patches even if you give them for free, and so active that people ignore things until they're actually removed 17:49:18 #info a good example of the second one is the Data Tree Change Listener 17:51:42 * rovarga reads the transcription and does not understand the last info .. 17:52:49 rovarga: basically despite DCLs being deprecated for 2 releases 17:53:03 we haven't seen much progress 17:53:38 #action colindixon to add a topic for next week or maybe an e-mail about how to mechanically potentially allow us to special case true-leaf projects to avoid things 17:53:42 colindixon: part of that is that conversions is a lot of work 17:53:51 #topic patch review 17:54:29 #link https://github.com/awaitility/awaitility 17:54:47 #link https://git.opendaylight.org/gerrit/#/c/48109/ both skitt and vorburger say that awaitility is really nice and it's already introduced and can be used 17:56:46 #chair vorburger 17:56:46 Current chairs: colindixon vorburger 17:56:53 vorburger: cannot find it in NETCONF: https://github.com/opendaylight/netconf/search?utf8=%E2%9C%93&q=awaitility 17:57:38 adetalhouet: I saw it somewhere yesterday; let me check again 17:57:47 greping autorelease 17:58:19 I think it was removed already, along with abstract-topology 17:59:09 adetalhouet: you seem to be right 17:59:21 spectre:autorelease ckd$ grep -r --include=*.xml awaitility . 17:59:22 ./odlparent/odlparent/pom.xml: org.awaitility 17:59:23 ./odlparent/odlparent/pom.xml: awaitility 17:59:24 spectre:autorelease ckd$ 17:59:54 :) 18:00:14 so basically we can claim anything in commit messages ;-) 18:00:28 I see com.jayway.awaitility in netconf 18:00:38 adetalhouet: yep. It was in netconf/netconf-topology/pom.xml in a non-latest master I looked at, but it's gone from there 18:00:58 but that's in the Boron stable branch 18:01:42 #topic lots of people talking about query and depth parameters with patches come in 18:01:46 #undo 18:01:46 Removing item from minutes: 18:01:52 #info lots of people talking about query and depth parameters with patches come in 18:01:55 #endmeeting