=== Ursinha is now known as Ursinha-afk === mrevell-dinner is now known as mrevell === danilo-afk is now known as danilos === cprov-afk is now known as cprov === mrevell is now known as mrevell-lunch === Ursinha-afk is now known as Ursinha === mrevell-lunch is now known as mrevell [15:01] #startmeeting [15:01] Meeting started at 09:01. The chair is barry. [15:01] Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE] [15:01] hi everyone and welcome to this week's ameu reviewers meeting. who's here today? [15:01] me [15:01] me [15:02] wow [15:02] everyone must be busy with something else, can't think what [15:03] me [15:03] intellectronica, gary_poster cprov salgado sinzui noodles775 jml allenap EdwinGrubbs rockstar bac ping [15:03] me [15:03] hi [15:03] me [15:03] me [15:03] barry: apologies [15:03] me [15:03] no worries [15:03] [TOPIC] agenda [15:03] New Topic: agenda [15:04] * Roll call [15:04] * Action items [15:04] * UI review call update [15:04] * Linked artifacts (e.g. screenshots) from bugs and merge proposals should not disappear [bac] [15:04] * Peanut gallery (anything not on the agenda) [15:04] [15:04] [15:04] [TOPIC] * Action items [15:04] New Topic: * Action items [15:04] * gary_poster and barry will transfer review guidelines from the old wiki and old old wiki to the new wiki [15:04] postponed until after 3.0 [15:04] bah :-) [15:04] me [15:04] * cprov to update guidelines to clarify how code sensitive to env changes should be written [15:04] barry: shhh, sorry again [15:05] cprov: no worries [15:05] [TOPIC] * UI review call update [15:05] New Topic: * UI review call update [15:05] beuno: hi! would you like to say a few words here? [15:06] me [15:06] doesn't look like it :) [15:07] it doesn't ;) [15:07] anyway... [15:07] me ;) [15:07] [TOPIC] * Linked artifacts (e.g. screenshots) from bugs and merge proposals should not disappear [bac] [15:07] New Topic: * Linked artifacts (e.g. screenshots) from bugs and merge proposals should not disappear [bac] [15:07] hi [15:07] barry, hi [15:07] me [15:07] doing QA i found screenshot links where the item was no longer there. very frustrating. that is all. [15:07] I love you guys [15:08] beuno: :) [15:08] and you've done an amazing job at 3.0 [15:08] bac: best to always use an upload, then? [15:08] UI reviews went so well, I want them to be done everywhere else [15:08] intellectronica: probably. or just don't prematurely purge on rookery. [15:09] bac: agreed! i would also strongly urge people to file a bug, link that bug to your branch, and include demo/qa plan in your cover letter. as i'm qa'ing things this week i find it very difficult when those things are missing [15:09] beuno: things are looking good, are they not? :) [15:09] also, we should allow attachments on merge proposals. [15:10] barry, yes. There are a few "critical" issues on major pages [15:10] jml: +1 [15:10] like the bug page, mps and branch index [15:10] some of them have been addressed [15:10] and some of them, we won't have time [15:10] lesson of the day: don't levae the biggest pages last [15:10] which, of course, means that the merge proposals page should use the same attachment infrastructure as bugs. [15:11] beuno: we're going to have to cp them after the release [15:11] bac has basically said "keep qa'ing but if you find something, it's too late now" [15:11] jml: there are various opportunities to share infrastructure that we should consider now [15:11] barry, right, it's fixable [15:11] in general, it's an awesome launchpad [15:11] all tempaltes have been converted [15:11] * barry is just amazed we actually converted all 375 templates [15:12] even with me sneaking away for 2 weeks [15:12] barry, me too! [15:12] you guys rocked the house [15:12] intellectronica, very much so. MPs already use bugs comment infrastructure, largely. [15:13] beuno: can we do a post-mortem after 3.0 is released, and maybe when you're done sprinting, etc. to evaluate the 3.0 process? and also to think about what needs to be done post-3.0 and what's on the plate for 4.0? [15:13] beuno: it would be great to evaluate what we just did before we leap headfirst into 4.0 [15:14] barry, absolutely [15:15] great! [15:15] [TOPIC] peanut gallery [15:15] New Topic: peanut gallery [15:15] that's everything on the agenda, does anybody else have anything for today? [15:16] barry, beuno: we are putting a 3.0 retrospective at the TL meeting [15:16] barry, nope. [15:16] flacoste, perfect [15:16] jml: i think you're right, so... [15:16] #endmeeting [15:16] Meeting finished at 09:16. [15:16] barry: ideally, TL would be able to do a retrospective with their team [15:16] before then [15:16] thanks everyone and have a happy qa day [15:16] but i'm not sure it's practical [15:17] flacoste: isn't the tl next week? [15:17] barry: it is [15:17] but sinzui is sprinting this week for example [15:18] the southern hemisphere has basically a day less in this week [15:18] etc. [15:18] we are releasing [15:18] flacoste: exactly :) [15:18] anyway... see you back at the ranch === ursula_ is now known as Ursinha === salgado is now known as salgado-lunch === salgado-lunch is now known as salgado === mrevell is now known as mrevell-dinner === Ursinha is now known as Ursinha-brb === danilos is now known as danilos-afk === mrevell-dinner is now known as mrevell === Ursinha-brb is now known as Ursinha [23:30] #startmeeting [23:30] Meeting started at 17:30. The chair is barry. [23:30] Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE] [23:30] hello and welcome to this week's asiapac reviewers meeting. who's here today? [23:30] i am! [23:30] hey [23:30] moi [23:30] barry: rockstar may be lying down [23:31] excellent [23:31] thumper: cool [23:31] thumper, mwhudson y'know your evil plan to increase the number of antipodeans is backfiring [23:31] why? [23:31] jml [23:32] ;) [23:32] anyway... [23:32] not much happened in ameu [23:32] the plan hadn't really got far enough that you could call it evil... [23:33] bac requested that folks keep their screen shots around long enough to do qa. some screen shots were not attached to the bug, but instead were on rookery, and got deleted [23:33] I'd like to be able to add attachments to a code review [23:33] it's recommended to attach them to the bug, or at least keep them around until the end of the cycle [23:33] thumper: that came up too [23:33] * thumper adds it to the wish list [23:33] barry: you can attach them to the mail [23:34] barry: and we could link them on the UI [23:34] barry: you woke me up for that? lies back down to monitor the release [23:34] thumper: yep! [23:34] barry: the images are stored in the librarian (I think) [23:34] barry: I'll test [23:34] cool, thanks [23:34] perhaps not today [23:35] thumper: in the original mp request is probably the most important [23:35] the only other thing from ameu is: lp 3.0 is awesome and beuno loves you [23:35] that's all i have. what's up with you? [23:36] I can no longer complain about the lack of public ec2test. [23:37] wgrant: we have that now? [23:37] well, he does [23:37] it's not actually 'public' yet [23:37] mwhudson: cool [23:37] because i need to land ec2test changes [23:37] I don't have anything review specific to bring up [23:38] i guess we could mention that review diffs will be a bit different post rollout? [23:38] perhaps [23:38] mwhudson: how so? [23:38] pushing will cause the diff to be updated [23:38] they'll update on push [23:38] and they're merge --preview diffs, not diff -r ancestor: diffs [23:38] (so you can get conflicts) [23:39] thumper: are the old diffs kept around? [23:39] kinda [23:39] not really [23:39] yet [23:39] updates are nice [23:39] thumper: there was this idea of linking a code review comment to the diff it applied to, i guess that's not done yet? [23:39] there is [23:39] but that is a db patch [23:39] right [23:39] mwhudson: what's the reason behind the merge --preview diffs? to be explicit about conflicts? [23:40] that is the only way we can get reasonable diffs [23:40] barry: because if you merge the target [23:40] barry: conflictyness seems interesting to the reviewer [23:40] barry: you don't want the merged details in the diff [23:41] thanks. how does this affect support for dependent branches? [23:41] another thing, jml has this half-written branch that will pull all the details for an ec2 test run from the merge proposal [23:41] heh [23:41] well [23:41] right now they aren't really supported [23:41] abentley is working on fixing the dependant branch support [23:41] being renamed to "prerequisite branch" [23:41] which i think will be a useful improvement [23:42] mwhudson: including commit message? [23:42] thumper: i presume so [23:42] mwhudson: where useful == awesome [23:42] barry: part of this would be to get LP to somehow work well with pipes [23:42] barry: have you used bzr-pipelines yet? [23:43] barry: IMO if you like looms, you'll love pipelines [23:43] thumper: i keep meaning to. i know they are the official goodness to use instead of looms [23:43] barry: they just work better with LP as they are branch based [23:43] honestly though, i haven't had many branch stacks during these last two cycles [23:43] :) [23:43] including commit message. [23:44] it assembles the [r=...] gunk from the mp [23:44] and takes the actual text of the commit message from the mp too [23:44] (but iirc, you can override it) [23:44] jml: my last commit message is usually meaningless, e.g. 'pick some lint' or 'merge rf' [23:44] barry: the proposal has a commit message field [23:45] barry: not set very often right now. tarmac uses it [23:45] i'll have to start using that. i've basically ignored the Subject: field until now [23:45] but it all sounds cool [23:45] barry: it isn't the subject [23:45] thumper: isn't that last commit message stuffed into the subject in a 'bzr send'? or is that something else? [23:46] barry: we don't do anything with that subject [23:46] thumper: that's why i've started to ignore it [23:46] barry: but the commit message is a different bit [23:47] i have one other thing i'd like to ask y'all [23:48] shoot [23:48] is this time still the best for the asiapac meetings? [23:48] still good for me [23:48] although we change next week [23:48] daylight savings kicks in [23:48] * thumper thinks [23:48] so becomes one hour later [23:48] personally, i wouldn't mind moving it earlier by an hour (or more) but i want to make sure we continue to meet at a good time for you [23:49] I'm fine with this time [23:49] thumper: you just blew my mind [23:49] could be fine an hour earlier too as of next week [23:49] so that means 2130 utc? [23:49] mwhudson: got any preference? [23:49] barry: yes [23:50] * mwhudson blinks his attention back, sorry about that [23:50] 2130 utc next week is 0930 local? [23:50] that's fine with me [23:50] i'll probably bitch and moan again by 01-nov when we fall out of daylight savings, but for now that would be great [23:50] mwhudson: 2130 utc next week is 1030 local [23:51] mwhudson: daylight savings sunday [23:51] oh right [23:51] that's completely fine [23:51] great! 2130 it is [23:51] that's all for me [23:51] another hour earlier would be ok, though risks bumping into our standup [23:51] barry: well, in nov it would be another hour earlier for you [23:51] barry: which would make it what? [23:51] an hour earlier than that would be ok, though that's getting a little early here (0830) [23:52] thumper: i think it would make it 1630 for me which would be perfect [23:52] mwhudson: I was talking about US going out of daylight savings [23:52] thumper: i'm just being general [23:52] thumper: now jml is off, we can go earlier, is the basic summary [23:52] right [23:52] right [23:53] we could do 20:30 utc [23:53] 2130 should be fine now and after 01-nov [23:53] if not, it's good to know we can push it a bit, but probably not necessary [23:53] anyway... [23:53] anything else guys? [23:53] what about 2100 ? [23:54] barry: it used to collide withour standup [23:54] barry: but we moved that [23:54] thumper: ah cool. 2100 is fine too [23:54] mwhudson: 2100? [23:54] thumper: yes, fine [23:54] barry, mwhudson: sold! [23:54] beauty [23:55] thanks guys... i think we're done! [23:55] #endmeeting [23:55] Meeting finished at 17:55. [23:55] thanks barry [23:55] cheers