[00:41] mwhudson, I'll have a look today [00:41] jml: oh thanks === abentley1 is now known as abentley === danilo_ is now known as danilos === wgrant changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [wgrant] || RC candidates take priority || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews === noodles775 changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [wgrant, noodles775] || RC candidates take priority || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews === henninge changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [wgrant, noodles775, henninge] || RC candidates take priority || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [10:47] wgrant: do you have any idea why the config changes you are implementing in your branch were kept on a wiki page? [10:48] wgrant: I mean, as opposed to doing what you are doing now from the start. Did the soyuz team expect problems with these changes? [10:48] noodles775: ^ [10:49] henninge: I just didn't ever submit them. [10:49] wgrant: Oh, you wrote those instructions! [10:50] henninge: I did. [10:50] Well, most of them, at least. Various bits have been added since. [10:50] wgrant: So it is a well tested setup? [10:51] It has been used by only a few people, but it does work. [10:52] I guess what I want to know is if the soyuz team is aware of that, i.e. if you had a pre-implementation chat about them. [10:53] wgrant: ^ [10:53] henninge: Not really, no. [10:54] wgrant: don't get me wrong, the changes look fine to me. [10:55] but I guess since this does not affect production code, I should not be over-paranoid. [10:55] I wasn't aware of them, but am all for them. [10:55] wgrant: r=me ;) [10:55] The process-upload.py invocation is stolen from the production configs, actually. [10:55] henninge: Thanks. Can you land it? [10:55] wgrant: sure [10:55] henninge: Thanks. [10:56] wgrant: can you set a commit message, please? [10:57] noodles775: did you just reserve a spot in the queue or do you actually have an mp ready ... ;-) === henninge changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [noodles775, henninge] || RC candidates take priority || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews === henninge changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [noodles775, henninge] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [10:58] henninge: Done. [10:58] henninge: I've got an MP which was already approved, but I've since added a few revisions: [10:59] https://code.edge.launchpad.net/~michael.nelson/launchpad/475808-unembargoing-oops/+merge/16141 [10:59] And since the revs are now displayed inline, you can see which ones :) (The bug has the information about what failed during QA on dogfood etc., as well as the new QA which went through df fine). [11:00] henninge: are you OCR'ing today? [11:01] or just being very helpful :) [11:01] noodles775: no, I was going to offer to trade reviews again ... ;) [11:03] henninge: sure - although I'll not get to it 'til after lunch, so it might be worth waiting for a better prospect ;) [11:03] noodles775: but I think with your branch I'd feel better if Aaron picked up on it. Or is he on leave already? [11:03] noodles775: yes, mine is not urgent anyway [11:03] henninge: I think he'll be around when it's morning. [11:03] * henninge goes to land wgrant's branch === jelmer_ is now known as jelmer === salgado changed the topic of #launchpad-reviews to: on call: salgado || reviewing: - || queue [noodles775, henninge] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [11:52] noodles775, has your branch been reviewed already? [11:52] salgado: no it hasn't. It's actually a re-review (I should have requested a new one), as I've updated it since it was approved: [11:52] https://code.edge.launchpad.net/~michael.nelson/launchpad/475808-unembargoing-oops/+merge/16141 [11:53] oh, ok. I'll get to it in a minute [11:53] Thanks. [12:09] noodles775, what is the ancestry component of a delayed copy? === mrevell is now known as mrevell-lunch [12:09] salgado: An older version of the package that has been published in the destination archive. [12:10] It's used to determine the correct component to publish to ('main', 'universe', 'multiverse' etc.) [12:10] right [12:10] noodles775, """Return the ancestor of the given parkace [12:10] in getNearestAncestor [12:11] salgado: Right, thanks. I guess my fingers slipped there! [12:15] noodles775, I think the docstring for that method should also state that it can return None [12:15] Yep, I'll update that too. [12:20] 172 + if (is_ppa and binary.component not in [12:20] 173 + distroseries.upload_components): [12:20] 174 # Only complain about non-PPA uploads. [12:21] noodles775, ain't that going to complain only about PPA uploads instead of non-PPA ones, as the comment says? [12:22] salgado: yes - I replaced line 155 with 172 - and missed the not, that saves one ec2 failure - thanks! [12:23] you mean, if we have a test for that. ;) [12:23] if? ;) [12:24] henninge: is there anything I need to do for this MP https://code.edge.launchpad.net/~adiroiban/launchpad/bug-435165/+merge/16084 :D [12:32] salgado: just for the pride of soyuz, distroseriesqueue.txt would have failed ;) [12:36] heh. ok [12:37] adiroiban: yes, keep poking your reviewer about it ... ;-) [12:37] adiroiban: or find a new one [12:38] :) no hurry. I just wanted to make sure the ball is not on my side :D [12:39] noodles775, r=me with those small changes === salgado changed the topic of #launchpad-reviews to: on call: salgado || reviewing: - || queue [henninge] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [12:40] salgado: thanks, already pushed. BTW: I'm planning to land it on devel now, rather than db-devel as the mp says. [12:40] noodles775, yeah, I guessed that from reading your comment on the bug report === mrevell-lunch is now known as mrevell [12:54] henninge, I assume yours is bug-489227-import-status? === salgado changed the topic of #launchpad-reviews to: on call: salgado || reviewing: - || queue [-] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [14:33] salgado: yes, it is [14:36] salgado: thanks for the review. ;) Yes, background would probably be helpful. Sorry about that, I was lazy and just copied the bug description. === EdwinGrubbs_ is now known as EdwinGrubbs [14:52] salgado: can you review https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-482176-add-team-member-ajax-part2/+merge/16325 [14:52] salgado: you will find it very familiar [14:53] that's what I was going to say [14:56] EdwinGrubbs, how did you split it up? [14:56] salgado: all of your original changes are in the first branch plus a tiny bit of cleanup. [14:57] salgado: this branch redid a lot of the javascript because adding teams was more complicated than expected. [14:58] abentley: Ping. [14:58] jpds: pong [14:58] abentley: Hi, would you be able to land https://code.edge.launchpad.net/~jpds/launchpad/fix_176396/+merge/16082 for me now that PQM is opened? [14:58] EdwinGrubbs, because we had to update different sections of the page or something else? [14:59] jpds: I think we still have the re-roll to do, don't we? [14:59] Re-roll? [15:00] We usually do a second rollout on Fridays to fix any issues we discovered after the release. [15:00] Ah, right, did not know that. [15:01] salgado: yes, I added the Recently invited list, since that is the status that new teams should be added to if you admin the super team. If you admin the member team, then it will go into the proposed status, but you have to do that through the "Add my teams" page. [15:08] EdwinGrubbs, right. we completely forgot about the latter case. you still have to use the "Add my teams" page, though, if you want to propose one of your teams as member? or does the new widget handle that as well? [15:09] salgado: the new widget handles that now. The model's addMember() method now checks whether the reviewer is an admin of the owning team and then sets the status to APPROVED or INVITED accordingly. [15:09] oh, nice [15:10] salgado: that's backwards, the reviewer has to be an admin on the new member team. The user has to be an admin of the owning team to even see the "Add member" link. [15:11] right [15:11] EdwinGrubbs, I'll step out now to get some lunch but I'll review your branch as soon as I get back [15:12] and thanks a lot for finishing it === salgado is now known as salgado-lunch === abentley changed the topic of #launchpad-reviews to: on call: salgado || reviewing: - || queue [abentley] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [15:45] salgado-lunch: Could you review my proposal too please? https://code.edge.launchpad.net/~abentley/launchpad/jobstatus/+merge/16309 === EdwinGrubbs is now known as Edwin-afk === salgado-lunch is now known as salgado [16:25] abentley, sure === Edwin-afk is now known as EdwinGrubbs [17:07] salgado: I'm back [17:23] EdwinGrubbs, sorry, was on a call. I'll continue your review now [17:31] EdwinGrubbs, btw, would you mind asking somebody else to have a look at your windmill test? I've never written one myself, so can't even think about reviewing it [17:31] salgado: ok [17:36] rockstar: can you do a UI review and review a windmill test. Salgado is reviewing the rest of the code in the branch. === salgado changed the topic of #launchpad-reviews to: on call: salgado || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews [18:27] salgado: will you be here on Monday if I don't finish the changes today? [18:27] EdwinGrubbs, yep [18:33] intellectronica: ping [18:34] EdwinGrubbs: yo [18:35] mars, when you have some time, can you have a quick look at https://code.edge.launchpad.net/~salgado/lazr-js/bug-482235/+merge/16349 for me? I'd like some feedback (mostly on the XXXs there and the lack of documentation for the plugin) before I polish it and ask for a proper review [18:36] intellectronica: hi [18:40] EdwinGrubbs: ? [18:41] intellectronica: you said "yo" [18:41] EdwinGrubbs: you said "ping" [18:43] intellectronica: oops, sorry, I was going to ask you if you could do a ui review and review a windmill test. Salgado reviewed the rest of the branch. [18:44] https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-482176-add-team-member-ajax-part2/+merge/16325 [18:44] EdwinGrubbs: sure. i can review the test. for the ui review you'll have to find one more reviewer [18:44] intellectronica: thanks, will do [18:45] intellectronica: I have to reboot, so I'll be away for just a sec [19:09] EdwinGrubbs: i need to go now, but if you still want me to review request it in an mp and i'll do it later === salgado changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews === salgado is now known as salgado-afk === mup_ is now known as mup