[15:00] <barry> #startmeeting
[15:00] <MootBot> Meeting started at 09:00. The chair is barry.
[15:00] <MootBot> Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE]
[15:00] <barry> welcome to this week's ameu reviewer's meeting.  who's here today?
[15:00] <BjornT> me
[15:00] <EdwinGrubbs> me
[15:00] <bac> me
[15:00] <barry> deryck sends his apologies
[15:01] <salgado> me
[15:01] <abentley> me
[15:01] <adeuring> me
[15:01] <intellectronica> me
[15:02] <sinzui> me
[15:02] <barry> jtv: ping
[15:02] <flacoste> me
[15:02] <jtv> me
[15:02] <barry> leonardr: ping
[15:02] <jtv> barry: thanks :)
[15:02] <barry> :)
[15:02] <bigjools> me
[15:02] <barry> cprov, gary_poster, gmb, noodles775 ping
[15:03] <noodles775> me
[15:03] <gary_poster> me, sorry
[15:03] <cprov> me
[15:03] <leonardr> me
[15:03] <gmb> barry: OTP at the moment; apologies are on the schedule.
[15:03] <gary_poster> :-)
[15:03]  * barry reloads the page
[15:03] <gary_poster> lol
[15:03] <barry> gmb: are you sure you committed your draft? :)
[15:04] <barry> danilos, allenap, rockstar ping
[15:04] <danilos> me
[15:04] <allenap> me
[15:05] <barry> [TOPIC] agenda
[15:05] <MootBot> New Topic:  agenda
[15:05] <barry>  * Roll call
[15:05] <barry>  * Action items
[15:05] <barry>  * Mentoring update
[15:05] <barry>  * Peanut gallery (anything not on the agenda)
[15:05] <barry>  
[15:06] <barry> [TOPIC] action items
[15:06] <MootBot> New Topic:  action items
[15:06] <barry>  * intellectronica to report on relevant summary of ui reviewers meeting here
[15:06] <barry> intellectronica: tell us some good news!
[15:07] <intellectronica> i think there has been some kind of misunderstanding
[15:07] <intellectronica> there aren't really any ui reviewers meetings
[15:07] <barry> intellectronica: ui phone call maybe?
[15:08] <intellectronica> i suppose the most interesting thing i can tell you is that martin is planning on beefing up the guidelines and in particular, start collecting case reports for interesting branches that went through ui review
[15:08] <intellectronica> so if you're working on a ui branch from either side and think the process was interesting and other can learn from it, consider compiling a case report
[15:09] <beuno> yes
[15:09] <intellectronica> oh, we should have talked about a million dollars
[15:09] <barry> :)
[15:09] <barry> beuno: intellectronica was giving us a summary of the ui call
 i suppose the most interesting thing i can tell you is that
[15:09] <barry>                  martin is planning on beefing up the guidelines and in
[15:09] <barry>                  particular, start collecting case reports for interesting
[15:09] <barry>                  branches that went through ui review  [10:08]
 so if you're working on a ui branch from either side and
[15:10] <barry>                  think the process was interesting and other can learn from
[15:10] <barry>                  it, consider compiling a case report
[15:10] <barry>  
[15:10] <barry> intellectronica: thanks; beuno do you have anything you want to add?
[15:10] <beuno> well
[15:10] <beuno> my goal is for the team to do it's own reviews
[15:11] <beuno> and have teh UX team help with the long-term changes
[15:11] <beuno> and I'm not sure how to get there, but I'm starting to
[15:11] <barry> one other thing we talked about was adding a 'ui' specialty to ReviewerSchedule for people who have graduated from ui mentat
[15:12] <beuno> so if anyone has suggestions, ideas, comments, anything, I'd be very grateful
[15:12] <beuno> yes, it's on my ToDo
[15:12] <barry> beuno: cool, thanks
[15:12] <barry> beuno: do you want to join this meeting regularly?  i could add 'ui reviewer report' to the agenda
[15:13] <beuno> barry, I would like to try, yes
[15:13]  * barry will be pingmaster :)
[15:13] <barry> beuno: awesome, thanks.  i'll remove intellectronica's action item then
[15:14] <barry> anything else on ui reviewing, from anybody?
[15:14] <barry> cool, moving on
[15:14] <barry> [TOPIC] mentoring update
[15:14] <MootBot> New Topic:  mentoring update
[15:15] <barry> i think i'll remove this topic for now.  when wgrant wants to become a reviewer we can add it back
[15:15] <intellectronica> aren't we all reviewing now? i think this item can be skipped until some community members start reviewing
[15:15] <barry> [TOPIC] peanut gallery
[15:15] <MootBot> New Topic:  peanut gallery
[15:15] <barry> intellectronica: yes, we are all reviewers!
[15:15] <barry> so, does anybody have anything not on the agenda?
[15:16] <intellectronica> yes
[15:16] <barry> i guess that's it then!
[15:16] <intellectronica> almost :)
[15:17] <barry> intellectronica: go4it
[15:17] <gary_poster> barry: (I'm not going to be OCR on Wednesday because of team lead duties)
[15:17] <gary_poster> 6 mo. :-)
[15:17] <intellectronica> a reminder to everyone that python branches with a diff > 800 lines and javascript branches with a diff > 1600 lines are exceptional, and need to have a review sorted in advance. OCRs will often not be able to take them just like that
[15:18] <barry> gary_poster: gotcha.  EdwinGrubbs will have to carry the west
[15:18] <gary_poster> k
[15:18] <barry> intellectronica: thanks, and remember as a reviewer you always have a right to kick the branch back
[15:19] <barry> anything else today?
[15:19] <barry> 5
[15:19]  * cprov feels guilty and hides.
[15:19] <barry> 4
[15:19] <barry> 3
[15:19] <barry> 2
[15:19] <barry> 1
[15:19] <barry> #endmeeting
[15:19] <MootBot> Meeting finished at 09:19.
[15:19] <barry> thanks everyone!
[15:20] <jtv> thanks barry
[15:20] <gary_poster> thanks barry
[15:20] <intellectronica> thanks barry
[23:30] <barry> #startmeeting
[23:30] <MootBot> Meeting started at 17:30. The chair is barry.
[23:30] <MootBot> Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE]
[23:30] <barry> hello and welcome to this week's asiapac meeting, who's here today?
[23:31] <barry> thumper, rockstar ping
[23:31] <thumper> hey
[23:32] <barry> thumper: hi.  i think it's just the three of us today, right?
[23:32] <barry> should be a quick meeting i think
[23:32] <thumper> yep, although rockstar is out for a walk right now, so maybe just me
[23:32] <thumper> did rockstar go to the earlier one?
[23:32] <barry> nope, he's joining us now
[23:33] <barry> s'okay, we can start (and maybe finish without him :)
[23:33] <thumper> ok
[23:33] <barry> not much to report from ameu; gary_poster will suspend ocr duties while he fills in for flacoste, who's filling in for kiko
[23:33]  * thumper nods
[23:33] <barry> that's really about it.  got anything on your mind today?
[23:34]  * thumper thinks
[23:34] <thumper> mainly just 3.0 redesign
[23:34] <thumper> although...
[23:34] <thumper> there was a question earlier by gmb
[23:34] <thumper> about "do mechanical 3.0 changes need a ui review"
[23:34] <thumper> I was under the impression that mechanical ones don't
[23:35] <thumper> new pages/designs do
[23:35] <barry> those are up to the reviewer+dev.  if they feel it doesn't need another ui review, they're fine
[23:35] <thumper> ok
[23:35] <thumper> that's it for me
[23:35] <wgrant> One thing...
[23:35] <wgrant> Not sure where is appropriate, though.
[23:35] <thumper> shoot
[23:36] <thumper> ask away
[23:37] <wgrant> It seems to me that contributors should always get a reviewer from the team that their branch targets, because otherwise the team has no chance to object.
[23:37] <wgrant> The dev wiki just says to poke an OCR, but that probably often isn't right.
[23:37] <thumper> wgrant: I think it depends on the change
[23:37] <barry> wgrant: that should be okayedin a pre-imp call, likely with a team member
[23:38] <thumper> wgrant: if it is something trivial, then I'd be happy to OK a bugs or soyuz change
[23:38] <barry> we're all /supposed/ to be be able to review anything
[23:38] <thumper> but if it is not trivial, or requires more specialised knowledge
[23:38] <thumper> I'll pass it on, like I have done
[23:38] <rockstar> oops.
[23:38] <rockstar> Hi
[23:38] <barry> rockstar: hi
[23:38] <barry> thumper: yep
[23:38] <barry> wgrant: does that make sense?
[23:39] <wgrant> barry: It says to do the pre-imp chat with the OCR, but I suppose in most cases they'll just defer to a member of the right team, in which case that's fine.
[23:39] <wgrant> It does.
[23:39] <barry> wgrant: cool, yep
[23:39] <wgrant> Thanks.
[23:40] <barry> wgrant: it's great to see you here, and fantastic to see your branches landing.  how is the processing going for you?
[23:40] <barry> er, process
[23:40] <wgrant> barry: Very well, apart from the opacity of PQM and ec2test.
[23:40] <barry> wgrant: well, at least we feel your pain there too :)
[23:41] <thumper> wgrant: we are wanting to move to LP queues
[23:41] <wgrant> The actual review stuff is fine.
[23:41] <thumper> wgrant: which will make at least that part more visible
[23:41] <wgrant> But after the review, it goes into a black hole and sometimes doesn't appear again.
[23:41] <rockstar> And once we're on queues, tarmac.
[23:41] <barry> rockstar: ftw
[23:41] <thumper> rockstar: maybe ;-)
[23:41] <wgrant> There's also the problem of security branches, but I believe you're working that out elsewhere.
[23:41] <wgrant> That hit me today.
[23:41] <rockstar> thumper, don't toy with my emotions like that...  :)
[23:41] <thumper> I'm supposed to come up with a plan
[23:41] <barry> wgrant: because you can't make private branches?
[23:42] <thumper> but I've not had time right now with the 3.0 stuff
[23:42] <wgrant> barry: Correct.
[23:42] <thumper> we are planning the ability for anyone to be able to have a private branch
[23:42] <thumper> to fix security bugs
[23:42] <wgrant> barry: I got spm to make it private, but then had to go back to him to get it visible to ~launchpad.
[23:42] <thumper> but there are issues about who gets emailed when
[23:42] <wgrant> I'm not a member of ~launchpad, so I can't subscribe them to my private branch.
[23:42] <barry> thumper: would they have to be tied to security bugs?
[23:42] <thumper> barry: possibly
[23:42] <thumper> barry: we are lacking some ui ability
[23:42] <barry> wgrant: yep, that's losa territory
[23:43] <thumper> barry: it may just work, but I'm not convinced, and will have to look at the tests
[23:43] <barry> cool.  rockstar anything on your mind for today?
[23:44] <rockstar> barry, nope, not really.  I think the UI review for mechanical changes thing was the only thing.
[23:44] <barry> if there's nothing else from thumper, wgrant...
[23:44] <wgrant> Is devpad.info going to be public at some point?
[23:44] <thumper> nope
[23:45] <thumper> buildbot is moving
[23:45] <wgrant> It's annoying getting buildbot failures with me in the blamelist, and not being able to actually see what went wrong.
[23:45] <wgrant> Ah.
[23:45] <thumper> wgrant: lpbuildbot.canonical.com
[23:45] <thumper> wgrant: but I'm not sure on the visibility of it
[23:45] <rockstar> It should be public.
[23:46] <wgrant> thumper: Aha!
[23:46] <wgrant> But no, it's not public.
[23:47] <barry> i think we're done...
[23:47] <barry> #endmeeting
[23:47] <MootBot> Meeting finished at 17:47.
[23:47] <thumper> thanks barry
[23:47] <barry> thanks everyone