15:05:04 #startmeeting Marconi 15:05:05 Meeting started Tue Jun 17 15:05:04 2014 UTC and is due to finish in 60 minutes. The chair is flaper87. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:05:06 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:05:09 The meeting name has been set to 'marconi' 15:05:22 #topic roll call 15:05:32 o/ 15:05:34 o/ 15:05:34 o/ 15:05:42 o/ 15:05:46 \o 15:05:51 \o/ 15:05:59 o/ 15:06:00 :D 15:06:18 #link https://wiki.openstack.org/wiki/Meetings/Marconi#Agenda 15:06:35 #topic action items 15:06:43 #chair kgriffs 15:06:44 Current chairs: flaper87 kgriffs 15:06:52 o/ 15:06:54 flaper87: carry on with the chair 15:06:56 :) 15:07:12 flaper87 to do the plumbing for specs 15:07:23 I haven't started, I was actually planing to do that today 15:07:27 #action flaper87 to do the plumbing for spec 15:07:34 it is flooding all over the place now :( 15:07:35 no more action items 15:07:59 yeah 15:08:19 spec-tacular 15:08:30 #topic lets be friendly with flwang 15:08:30 lol 15:08:48 Turns out that our meeting is at 3am flwang TZ 15:09:00 this is our predicament 15:09:01 http://goo.gl/X6A3ki 15:09:11 kgriffs: thanks 15:10:00 * flaper87 is staring at the timetable 15:10:02 :P 15:10:10 to make things easier I am in Seattle (GMT-8) 15:10:10 should we start rotating meeting times? 15:10:24 malini1: I was about to say that 15:10:28 we do that in glance 15:10:39 seems viable. 15:11:14 that could work 15:11:28 flaper87: does glance alternate every other week? 15:11:29 Ok, lets try to find another time that works for flwang and announc meeting time rotations on the mailing list 15:11:33 kgriffs: yes 15:12:07 moving on 15:12:13 #topic Juno-2 15:12:13 OK, do you want to take that action? 15:12:40 #action flaper87 talk to flwang and find a meeting time that works for him 15:12:52 cool, thanks 15:12:54 #action kgriffs announc the new meeting times in the mailing-list 15:12:58 :P 15:13:06 kgriffs: Juno-2, the floor is yours 15:13:07 heh 15:13:12 #link https://launchpad.net/marconi/+milestone/juno-2 15:13:31 I spent some time putting together a list of bps and bugs for j-2 15:13:47 holy molly 15:13:52 Flavio is looking through python client bugs and I've still got some server bugs to triage, but this is pretty close 15:14:29 kgriffs: I think we can move amqp out of Juno 15:14:56 what's the status of the benchmarking tool? 15:15:04 its in review. 15:15:06 Any chance we can increase the priority there? 15:15:11 sriram: ohreally ? 15:15:13 I have a patch out there. 15:15:14 what is the status of AMQP in general? 15:15:15 * flaper87 slaps himself 15:15:32 I saw some notes in the regular channel about dropping the storage driver and investigating AMQP as transport? 15:15:41 flaper87: I believe malini has an agenda item about our review backlog. ;) 15:15:42 tjanczuk: as of now, it's a complete no-go 15:15:55 for amqp 1.0 as well? 15:15:59 tjanczuk: yes 15:16:23 Is there a write up of reasons somewhere so that we can compare notes? 15:16:32 tjanczuk: working on that 15:16:38 ok, thanks 15:16:50 ok, please, everyone. Go through blueprints and update status 15:16:53 same for bugs 15:17:07 feel free to triage bugs, that's a task for everyone ;) 15:17:18 sure, ok. 15:17:27 moving on 15:17:30 #topic Proposal to define a set of review comment prefixes to better communicate intent 15:17:32 kgriffs: ? 15:18:30 kgriffs: knock knock ? 15:18:50 ok, lets skip until he's back 15:18:55 back 15:18:59 ok 15:19:01 kgriffs: go ahead 15:19:01 sorry, trying to figure out how to present this 15:19:27 there was swift session all about what is basically their code of conduct and quality standards 15:19:45 they said something about reviews being easy to take the wrong way 15:20:11 and they said it was good to specifically say something what a nitpick or whatever 15:20:13 brb 15:20:42 so, this is something I'm still playing with, but I thought we might cross-pollinate that idea with what we do for inline code comments 15:20:51 you mean to use things like: (nit:) remove the dot at the end 15:20:55 ? 15:21:00 yeah, stuff like that 15:21:08 oh, that sounds good to me 15:21:15 it states the intent of the comment 15:21:16 I've been experimenting with it lately in my comments (some of you no doubt noticed) 15:21:20 works for me 15:21:25 for reference 15:21:33 are the swift CoC and quality docs available? 15:21:34 guess the idea is to be nice when doing the critiques, rt? 15:21:35 link? 15:21:35 Lets work on a list of tags on the wiki 15:21:45 nit, perf, imo, etc. - I think it will take some time to figure out a good list 15:21:53 nice idea. 15:22:02 it will definitely help. 15:22:15 malini1: the idea is to more clearly communicate the intent and context for the comment 15:22:18 #action Kurt to write a wiki page with possible review comment tags 15:22:25 to minimize misunderstandings 15:22:31 #undo 15:22:32 Removing item from minutes: 15:22:36 Here is some prior art: https://www.google.com/search?q=emoticons+cheat+sheet&espv=2&tbm=isch&imgil=Ruan9Cq-NET-6M%253A%253Bhttps%253A%252F%252Fencrypted-tbn3.gstatic.com%252Fimages%253Fq%253Dtbn%253AANd9GcQpDv1DqbWnszg8HJpTdnA3yKGFTQZTLb_XDUvK3MOANaGoVl4b%253B530%253B713%253By33FowaEZmlvMM%253Bhttp%25253A%25252F%25252Fwww.phonearena.com%25252Fnews%25252FWindows-Phone-emoticon-cheat-sheet-gets-posted_id24438&source=iu& 15:22:36 #action kgriffs to write a wiki page with possible review comment tags 15:23:06 emoticons... mmm. Interesting idea. 15:23:12 (and I meant it with a ;) 15:23:16 OK, I will play around with this and we can see where it leads 15:23:21 I prefer the tags 15:23:25 me too 15:23:28 because it allows other reviewers to decide on that 15:23:47 if I see a "(nit) Remove the dot" comment, I may choose to approve the patch anyway 15:24:00 because the other reviewer stated it's just a nit 15:24:13 if it's tagged with a PERF flag, I definitely won't do that 15:24:14 makes sense 15:24:14 etc 15:24:33 s/flag/tag/ 15:24:34 lol 15:24:37 o___0 15:24:44 * flaper87 was thinking about red flags 15:24:46 anyway 15:24:51 kgriffs: good idea 15:25:09 kgriffs: btw, i gotta step out, mind leading the meeting from here? 15:25:16 sure 15:25:43 you're chair already, sorry about that 15:25:48 kgriffs: https://wiki.openstack.org/wiki/Meetings/Marconi#Agenda 15:26:14 OK, I'm going to go out of order a little on the agenda to timebox the bigger discussion 15:26:22 #topic Introduce the Marconi c# client 15:26:25 abettadapur: ^^^ 15:26:36 hi 15:26:38 so yes 15:26:45 we have a c# marconi client that you can view 15:26:54 yayy!! 15:26:59 awesome! 15:27:04 https://github.com/abettadapur/marconiclient 15:27:19 it covers both v1 and v1.1 of the api 15:27:31 abettadapur: do you know if there is a repo for non-python openstack bindings? 15:27:51 balajiiyer and i were talking about this and i dont think there is...? 15:27:59 neat! 15:28:01 his idea was to post a link to it on the wiki somewhere 15:28:06 and direct people to use it 15:28:16 "we have clients for the following platforms..." 15:28:17 https://wiki.openstack.org/wiki/SDKs#OpenStack_Toolkits 15:28:20 err 15:28:22 #link https://wiki.openstack.org/wiki/SDKs#OpenStack_Toolkits 15:28:33 that's the cannonical home, as far as I can tell 15:28:41 for other clients/sdks 15:28:47 oh there you go 15:29:03 looks like all of them point to personal repos 15:29:13 right 15:29:16 So probably thts the way to go, for now 15:29:43 FWIW, it is also linked from the marconi home page on the wiki 15:30:36 abettadapur: nice work, thanks for taking this on. It is helpful to have more ways for devs to use Marconi, and also gives us more feedback on API design 15:30:39 updated the sdks page 15:30:50 kgriffs: of course! :) 15:31:05 #topic review queue 15:31:23 malini1: ^^^ 15:31:34 Our review queue is at the longest it has ever been https://review.openstack.org/#/q/status:open+project:openstack/marconi,n,z 15:31:51 It is a good thing because we are getting a lot of new contributors 15:32:11 On the flip side, I dont think we are doing justice to all the great work 15:32:30 It is a tirn off for new contributors, if the patches take too long to land 15:32:34 turn* 15:33:04 I personally dread contributing to certain projects because it takes forever to get reviews 15:33:37 We need a way to make sure Marconi doesnt go the same route 15:33:42 thoughts on how to do this? 15:34:11 continue to work to promote marconi champions to core 15:34:15 that's going to be critical 15:34:33 I was also thinking of setting SLA's for each patchset 15:35:07 what kind of SLA's? 15:35:15 if a patchset sits idle with no reviews for x days, do something 15:35:23 shout, scream, glow red whatever 15:35:28 ah 15:35:44 SLAs confused me. I was thinking review quality and/or code quality 15:35:49 alcabrera: +1 on ur idea as well 15:35:55 thanks for clarifying, malini1! 15:36:08 so a TTL of sorts 15:36:15 I see, makes sense. 15:36:19 alcabrera: +1 15:36:22 yeah..except we dont garbage collect ;) 15:36:28 hahaha 15:36:31 or maybe we do...! ;) 15:36:36 :P 15:36:38 but yes 15:36:39 tht wud be scary! 15:36:45 that's a lively approach 15:36:53 in the sense that things will get serviced more routinely 15:36:53 probabilistic garbage collection :P 15:36:56 here's the catch 15:37:00 can we automate this? 15:37:14 get gerrit/bot-things to ping us when things reach the TTL 15:37:22 so we don't have to manually track patches 15:37:23 we probably need to investigate & review at next meeting? 15:37:30 sounds good to me 15:37:40 a good question for *-infra 15:37:42 gerrit has some sort of API 15:38:10 we could write a bot that annoys us or at least creates a report that highlights reviews in red that need attention 15:38:14 sometimes just raising visibility of who is doing reviews helps. you could do a weekly leaderboard. 15:38:29 game mechanics - not a bad idea 15:39:01 tjanczuk: tht wud mean we are looking at the problem once a week.I think e need a faster turnaround than tht 15:39:18 &most of us don't like meetings :-P 15:39:40 I can investigate options & update next week 15:39:53 it's something that worths discussing with infra guys 15:40:00 No I meant somethign along the lines of https://github.com/joyent/node/graphs/contributors 15:40:09 it could help another projects as well 15:40:10 one thing to note is that we had some people missing in action for a few weeks after the summit 15:40:16 now we are playing catch up 15:41:05 FWIW, there is this: http://stackalytics.com/?release=juno&metric=marks&module=marconi 15:41:32 malini1: were you thinking something more proactive, like a bot that would ping specific people in IRC? 15:41:53 kgriffs: tht wud be real cool 15:42:15 maybe we shud just clean out the queue now? 15:42:36 If this becomes a problem again, lets think abt the bot stuff 15:42:41 ok 15:43:10 the next 2-3 days, let's all focus on reviewing 15:43:13 maybe a welcome bot that provides a list of patches ordered by lowest ttl? 15:43:34 vkmc: you mean in the review page? 15:43:42 maybe all we need is a filter :D 15:44:20 malini1, in IRC I thought... otherwise we would have to make it part of Gerrit and I'm not sure if that is possible from our side 15:44:20 yep, let's focus on reviewing... 15:44:43 and ongoing, everyone should spend part of each day reviewing patches 15:45:06 vkmc: lets take this offline to #marconi to figure out how.if to automate 15:45:13 malini1, +1 15:45:19 kgriffs: tht's all I have 15:45:39 #action malini1 and vkmc to experiment with tools/bots for improving review velocity 15:46:01 #topic implementing the project rename - make a plan 15:46:24 malini1: you found this, iirc? https://blueprints.launchpad.net/sahara/+spec/savanna-renaming 15:46:34 kgriffs: yes 15:46:36 #openstack-marconi -> #openstack-naav -- that'll be fun 15:47:16 alcabrera: tht sounds sarcastic :( 15:47:20 whoa look at that dependency tree. 15:47:31 malini1: ah, my apologies. I meant it sincerely. :) 15:47:36 wow 15:47:40 I am dreading this 15:48:20 I think there will less work for us than savanna, but by no means a trivial amount 15:48:21 alcabrera, we should add to the topic 'HTTP/1.1 301 Moved Permanently => #openstack-naav' 15:48:42 hehe, vkmc. :) 15:48:53 one question I had was what happens to the pending gerrit patches? 15:49:03 hmmm 15:50:07 I remember seeing something abt this somewhere...grr 15:50:19 btw, there is a blueprint for this that I registered 15:50:21 #link https://blueprints.launchpad.net/marconi/+spec/project-rename 15:51:07 well..I saw tht as the question in Sahara etherpad :D 15:51:25 iirc infra chaps schedule a downtime for renaming 15:51:47 I was thinking first step would be to make a list of work items. The order them so that things like renaming the repo is last 15:51:49 vkmc: could be. They probably have to lock gerrit and run some scripts and stuff 15:52:06 kgriffs, yes... that happened with Quantum => Neutron 15:52:24 aah..Neutron also went thru renaming? 15:52:26 it's good that there's lots of prior art here 15:52:56 it's bad tht it is a known problem, & every project goes thru this :( 15:53:16 malini1, yeah... in Grizzly :/ that probably was a pain in the neck 15:53:50 malini1: would you mind taking a first pass at making the list of work items (adding to the bp)? Someone in #openstack-infra could help us make sure the list is complete from their side 15:53:56 there may be a wiki page about this 15:54:00 sure 15:54:08 I will start updating the bp 15:55:10 #action malini1 to make a list of work items for the rename and ask infra what things we need to include outside of code changes 15:55:17 thanks! 15:55:21 http://ci.openstack.org/gerrit.html#renaming-a-project 15:55:33 #link http://ci.openstack.org/gerrit.html#renaming-a-project 15:55:43 thanks vkmc 15:55:44 #link https://wiki.openstack.org/wiki/Network/neutron-renaming 15:55:52 np malini1 15:56:04 #link https://etherpad.openstack.org/p/savanna-renaming-process 15:56:13 all the links 15:56:20 to preserve prior art 15:56:37 #topic open discussion 15:57:26 tiny update - code of conduct changes are working their way through the pipeline 15:57:34 I was really happy to see my concerns taken seriously 15:57:39 cool 15:57:43 :) 15:57:49 nice 15:57:52 #link https://review.openstack.org/#/c/98205/ 15:58:49 alcabrera: :) 15:58:58 alcabrera: nice :) 15:59:02 :) 15:59:02 L22: "participanting" typo 15:59:23 kgriffs is really serious on ramping up reviews ;) 15:59:27 participanting. <3 15:59:28 if that isn't a real word, it *should* be. :D 15:59:34 agreed, kgriffs 15:59:38 let's coin it 15:59:51 heh 16:00:13 anyway, I don't mean to make light of the topic... this is really important stuff. thanks for working on it! 16:00:43 no worries. I'm happy to see it moving. :) 16:01:05 healthy community -> happy people, better for everyone 16:01:09 anyway 16:01:12 I think we're timed out 16:01:17 over time 16:01:18 ok people. let's get those patches merged! 16:01:19 yes 16:01:24 #endmeeting