=== thumper_laptop is now known as thumper === mwhudson__ is now known as mwhudson === mwhudson_ is now known as mwhudson === kiko__ is now known as kiko-afk === mrevell is now known as mrevell-lunch === mrevell-lunch is now known as mrevell === salgado is now known as salgado-brb [15:00] #startmeeting [15:00] Meeting started at 09:02. The chair is barry. [15:00] Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE] [15:00] hello everyone and welcome to this week's ameu reviewer's meeting. who's here today? [15:00] me [15:01] ... [15:01] me [15:01] me [15:01] me [15:01] me [15:01] me [15:01] me [15:01] me [15:02] BjornT: ping [15:02] bigjools: ping [15:02] me [15:02] me [15:02] me [15:02] danilos: ping [15:02] * sinzui passes caffeine to the meeting [15:03] * bigjools has an alarm for the meeting and still misses the start [15:03] [TOPIC] * Next meeting [15:03] New Topic: * Next meeting [15:03] week += 1? [15:03] anybody know you'll be sprinting or offline? [15:04] I hope to be around but it depends on how my son's surgery goes on Monday [15:05] bigjools: hope everything goes well! [15:05] barry: thanks, me too :) [15:05] [TOPIC] * Action items [15:05] New Topic: * Action items [15:05] * barry to get with flacoste to open pqm bugs [15:05] flacoste: we'll chat after the meeting [15:05] hmm [15:06] can't rememberr what this wa about [15:06] flacoste: new item from asiapac meeting [15:06] [TOPIC] * Queue status [15:06] New Topic: * Queue status [15:07] only observations i have is that pqm is HUGE [15:07] and there are a few new pink branches on pending-reviews [15:07] bigjools: i know i have a mentor review for you today [15:08] pqm has 17 branches and it's only week 2 ... oy [15:08] any comments on the queue? [15:09] yeah, it's crazy! [15:09] I think we have a lo of pink because we have been alocating at the end of the on-call review [15:09] yeah I got one today that was already 2/3 days old [15:09] * barry apologizes for forgetting to do that this week [15:09] i just picked up a branch that was put up for review a week ago, and got rejected [15:09] bigjools: That is because I suck. I miss-assigned it [15:10] let's try to get those branches into someone's hands quickly [15:10] sinzui: you purposely giving me soyuz ones? :) [15:10] it may be an idea, to ask that if someone rejects a branch because of workload (rather than a problem with the branch), they try to find someone else so take on it [15:10] bigjools: I did pause for a moment before pass it to you [15:11] intellectronica: good idea [15:11] np, at this busy time it makes sense [15:11] intellectronica: +1 [15:12] * sinzui must bring up the subject of how assigned branches relate to on-call times [15:12] sinzui: ? [15:13] me === salgado-brb is now known as salgado [15:13] sorry [15:13] barry: Do we subtract the time done on an assigned review from the time we would spend doing on-call reviews? [15:13] sinzui: everyone except you [15:13] sinzui: :) [15:14] barry: That is is not entirely true. I take only easy one on saturday [15:14] sinzui: dunno. if we're getting more branches than we can reasonably review, well, we've got other problems [15:15] sinzui: i generally work them (and mentor reviews) around other work [15:15] i usually just go with the flow, and try to only pick up branches out of review days if i've come to a natural pause, or it makes sense in the grand scheme of things [15:15] sinzui: if it becomes too much of a burden, you should probably start rejecting branches [15:16] I see one or more branches are not really need-review too. [15:16] intellectronica: EdwinGrubbs ask me the question after I assigned him a branch [15:17] We don't have an official answer [15:18] half of the pink branchs are really needs-reply [15:18] i think the answer is, reject branches if you overloaded. if lots of branches are piling up rejected or not meeting the sla, then this part of the process is saturated and we need to find other solutions [15:19] sinzui: is that just reviewers not keeping PR up-to-date? [15:19] barry. correct [15:19] pending branches has an update lag though [15:19] * sinzui was reading the launchpad-reviewer archive [15:19] bigjools: but the lag is not measured in days [15:20] ah, that would be bad then [15:20] it would be interesting to see how many of those unchanges ones are from on-call reviews where the reviewer didn't know there was an entry in PR [15:20] unchanged* [15:21] bigjools: i always ask for a PR block for any branch i review [15:21] right, but I don't think all the reviewers do that [15:21] i know [15:21] and some devs stick one in anyway [15:22] hurry up PR-killer :) [15:22] bigjools: that's the real answer! :) [15:22] anyway, moving on [15:22] [TOPIC] * Mentoring update [15:22] New Topic: * Mentoring update [15:22] perhaps we can get time assigned for that post 2.0 [15:23] any comments here either from mentors or mentorees? [15:23] * bigjools still wants a translations branch to review [15:23] bigjools: maybe add a note on your PR queue? [15:23] good idea [15:24] [TOPIC] * Review process [15:24] New Topic: * Review process [15:24] * Module alternatives - do we really want them? [15:24] dunno what this one is about [15:25] who added this one? please take the floor [15:25] intellectronica, I think... [15:25] right [15:26] so, sometimes, we have imports of the form `try: from import cFoo as Foo \n except: import Foo` [15:26] do we really want to do that? [15:26] consider that we have pretty tight control over the environment in which we run [15:26] I have done that twice, both times I had an XXX about when the hack could be removed [15:27] and that falling back silently to an inefficient implementation, is not necessarily a good thing [15:27] intellectronica: The Gutsy to Hardy transition is a good example when we need while all the machines are upgrading [15:28] i mean, what what happens if one day the native-code module disappears from one of the production servers? we'll only notice this when the performance starts degrading [15:28] i don't think we need to do that for things like cStringIO, etc. just use the most appropriate one [15:28] don't we run everything on hardy now anyway? [15:28] not quite [15:29] after today tho, right? [15:29] and specifically, what about elementtree? is the native code version a new thing we're not guaranteed to have on older systems? [15:29] no, next week will see everything upgraded IIRC [15:29] intellectronica: We removed one hack shortly after PQM was upgraded [15:29] that's a bit different because when we /do/ switch to py2.5, we'll have it available there instead of as a separate pkg [15:30] barry: but the import path is different [15:30] from xml.etrees [15:30] or etree [15:30] flacoste: right. isn't that what intellectronica means? [15:31] Is it worth having a canonical/alternatives.py module which does all of this, and we import from there. Then we all use the right modules (and we don't get false lint) [15:31] allenap: i like this idea a lot [15:32] it does introduce a layer of indirection [15:32] but i guess from canoniocal.alternives import ElementTree [15:32] wouldn't be too confusing [15:32] flacoste: sure, but it should be used sparingly. in most cases we should import the most specific version [15:32] yes, very sparingly [15:34] is elementtree the only example? [15:35] well, that one fizzled out :) any other topics for today? [15:35] looks like it [15:36] that's the only example i have [15:36] BjornT: if so, i wouldn't add all the extra machinery just for it [15:36] i agree. is there a bug open on lint for this? [15:36] yeah, i guess not. but i'd recommend adding an XXX if we must do this trick [15:37] BjornT: what do you mean? would like lint warning for such cases suppressed? [15:37] intellectronica: well, why is lint complaining about this? [15:37] intellectronica: i don't want lint to complain at all [15:38] BjornT: the thing is, to make sure that it's correct, the try clause should have only one statement [15:38] i suppose if we handle that correctly, we can have lint not complain [15:38] intellectronica: right. and it has only one statement, hasn't it? [15:39] yes. if we handle this specific case it should be ok [15:39] i'll file a bug on lint (if there isn't one already) [15:39] ACTION: ^^^ [15:40] [ACTION] intellectronica to file bug on lint issue regarding elementtree import [15:40] ACTION received: intellectronica to file bug on lint issue regarding elementtree import [15:40] cool. that's all i have today. [15:40] That special case will require either an ugly sed hack, or a replacement lint script written in python [15:41] python, puh... [15:41] thanks barry [15:41] i take that as a sign. [15:41] #endmeeting [15:41] Meeting finished at 09:43. [15:41] thanks everyone === kiko-afk is now known as kiko === salgado is now known as salgado-lunch === cprov-lunch is now known as cprov === cprov is now known as cprov-out === salgado-lunch is now known as salgado === kiko is now known as kiko-afk === kiko-afk is now known as kiko === salgado is now known as salgado-afk