=== Ursinha is now known as Ursinha-afk | ||
jtv | say henninge, aren't you ocr today? | 08:34 |
---|---|---|
StevenK | No, it's gmb | 08:35 |
* StevenK already checked | 08:35 | |
=== henninge_ is now known as henninge | ||
jtv | henninge: I just put that API simplification branch up for review. | 08:57 |
henninge | lemme look | 08:58 |
henninge | jtv: looks good. Shouldn't it have a bug? | 09:03 |
jtv | henninge: well, it's a feature-branch task so I'm not sure we need to go to those lengths. And it's just split off from, depending on your point of view, yesterday's branch or the job I'm working on now. | 09:04 |
henninge | ok | 09:04 |
StevenK | gmb: O hai, Mr OCR! | 09:14 |
gmb | StevenK, Morning. If you've got something that needs review bung it in the queue and I'll take a look in about an hour's time. Still catching up on emails. | 09:24 |
lifeless | gmb: hey | 09:24 |
lifeless | gmb: I have an _easy_ fix for you | 09:24 |
lifeless | https://bugs.edge.launchpad.net/malone/+bug/615644 | 09:25 |
_mup_ | Bug #615644: BugTask:+distrotask timeout on HEAT lookup <timeout> <Launchpad Bugs:Triaged> <https://launchpad.net/bugs/615644> | 09:25 |
gmb | lifeless, Easy fix for me to create a branch for, or is there already a branch that needs reviewing? | 09:26 |
lifeless | gmb: easy fix for a sparkling young fella like you to make a branch for | 09:27 |
gmb | lifeless, Righto, I shall take care of it presently. | 09:29 |
StevenK | gmb: Maybe you should suggest lifeless reviews the branch :-) | 09:29 |
lifeless | gmb: just needs a ordered.first() turned into a .max() | 09:31 |
gmb | lifeless, Ah, even better. | 09:31 |
gmb | lifeless, Do you know, or can you point me to a way of finding out, whereabouts that query is getting issued? I'm struggling to find it (though this may be a case of E_NOT_ENOUGH_CAFFEINE). | 09:53 |
lifeless | gmb: well, have a look at the linked oops | 09:54 |
lifeless | gmb: as for trapping it, uhm, I've been strugglying with that too | 09:54 |
lifeless | putting a hook in the thing tha tappaends to get_request_statements might be best | 09:54 |
gmb | lifeless, Ah, I think I may have found it. I was looking for Storm but maybe the query is pure SQL. | 09:54 |
gmb | lifeless, Yep, it's HasBugHeatMixin.recalculateBugHeatCache() that's the culprit. lib/lp/bugs/model/bugtarget.py:239 | 09:56 |
gmb | lifeless, https://code.edge.launchpad.net/~gmb/launchpad/bug-615644/+merge/32175 if you're interested. | 10:12 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
=== StevenK changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: - || queue: [StevenK] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
StevenK | gmb: *cough* :-) | 10:18 |
gmb | StevenK, Righto. Good job you put it in the queue otherwise (/me looks at +activereviews) jelmer was first up :) | 10:19 |
gmb | StevenK, That could have done with a more detailed cover letter. See https://dev.launchpad.net/CoverLetters for templates. So, expect questions. | 10:22 |
gmb | (Although I'm kidding a bit because of the nature of the branch) | 10:23 |
gmb | \0/ for killing doctests | 10:23 |
StevenK | gmb: I seriously tried three times to write a better cover letter and come up with writers block. | 10:23 |
gmb | StevenK, Understandable. | 10:24 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: StevenK || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
StevenK | gmb: I'm starting to cook dinner, so I'll be afk and such like | 10:31 |
gmb | StevenK, Okidoke. | 10:33 |
henninge | jtv1: you mp is not emitting emails again. Could you investigate that or have it investigated, please? | 10:34 |
jtv1 | henninge: grrr. OK. | 10:35 |
StevenK | gmb: Are you open to bribes of pasta bake? :-) | 10:35 |
jtv1 | henninge: btw I did get the email where I requested the review | 10:36 |
henninge | jtv1: but there is none on the list. | 10:36 |
henninge | jtv1: also, when I claim it, I should get one... let me check. | 10:36 |
gmb | StevenK, Pasta bake may or may not be sufficient for your branch to land without oversight (see extended reviewing guidlines, subsection 1, "Bribes of food, money or sundry electronic items") | 10:37 |
StevenK | Bwahaha | 10:38 |
jtv1 | henninge: do you normally get them when you propose for merging into the recife branch? I mean, the emails to the list aren't tied to specific LP branches? | 10:38 |
henninge | oh, I did not think about that ... | 10:39 |
henninge | jtv1: I just remeber OCRs picking up on recife branches so I always thought they had been notified. | 10:40 |
henninge | but maybe it's just because they appear in +activereviews. | 10:40 |
jtv1 | henninge: sounds more likely… I don't think anyone follows the lp-reviews list for this anymore. | 10:40 |
henninge | jtv1: just replying to it by makes it much more easier to comment on the diff, though ... | 10:41 |
jtv1 | henninge: yes, it does… but for now I think I'll just say that I've investigated and found this as the most likely cause. :-) | 10:41 |
henninge | jtv1: well done. | 10:55 |
henninge | ;-) | 10:55 |
* jtv1 salutes to the sound of heroic trumpets | 10:55 | |
henninge | jtv1: review about to be sent, r=me with a few suggestions. | 10:55 |
jtv1 | henninge: great, thanks! | 10:55 |
* henninge lunches | 10:58 | |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
StevenK | gmb: All of the checks in test_initialise() are ripped straight from the doctest, and are checking that source and binary publications were copied correctly. | 11:14 |
gmb | StevenK, Okay, that's fine. Please add comments explaining that (you don't need to comment the individual asserts, though) | 11:25 |
StevenK | gmb: Yup, was going to do so tomorrow morning | 11:25 |
gmb | Cool. | 11:26 |
=== noodles775 changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: - || queue: [noodles775] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
noodles775 | Hi gmb, when you've time: https://code.edge.launchpad.net/~michael.nelson/launchpad/561586-move-js-to-lp-app/+merge/32180 | 11:58 |
noodles775 | I'll take a look at StevenK's branch now too. | 11:58 |
gmb | noodles775, Thanks muchly. Looking at your MP now. | 12:00 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: noodles775 || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
wgrant | gmb: Hi. https://code.edge.launchpad.net/~wgrant/launchpad/bug-590708-getBuildsByArchIds-timeout/+merge/32181 is a nice simple RC branch, when you have time. | 12:03 |
=== wgrant changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: noodles775 || queue: [wgrant] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
noodles775 | ^^^ can take priority over mine which is not RC. | 12:03 |
gmb | noodles775, r=me. | 12:11 |
gmb | wgrant, Looking now | 12:11 |
noodles775 | Thanks gmb | 12:11 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: wgrant || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
wgrant | gmb, noodles775: Thanks. | 12:12 |
gmb | wgrant, Nice solution :) | 12:13 |
gmb | r=me | 12:13 |
* gmb lunches before anything else comes up | 12:14 | |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: lunch || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
wgrant | gmb: Ahem. | 12:14 |
wgrant | Thanks. | 12:14 |
wgrant | Do I request an 'rc', or a 'release-critical'? | 12:15 |
StevenK | The latter | 12:15 |
wgrant | D: There are two Julian Edwardses, both with hidden email addresses. | 12:16 |
bigjools | I am julian-edwards | 12:17 |
bigjools | I guess I should expose my canonical one :) | 12:17 |
wgrant | bigjools: I know, but the UI doesn't tell me the username. | 12:17 |
wgrant | It just lists two 'Julian Edwards <email address hidden>' | 12:17 |
wgrant | So I have to use Firebug to work out who is who, I guess. | 12:18 |
bigjools | hmmm that's bad | 12:18 |
wgrant | It should probably show the username, yeah. | 12:18 |
bigjools | +editemails doesn't let me change privacy. WTF do I go | 12:18 |
wgrant | The might be on +edit, IIRC. But you can't just expose your primary; you have to expose all :( | 12:19 |
bigjools | :/ | 12:19 |
wgrant | Yes. | 12:20 |
* StevenK subsribes bigjools to 1,000,000 mailing lists | 12:20 | |
StevenK | subscribes, even | 12:20 |
* wgrant gives up and uses the non-AJAX form. | 12:21 | |
bigjools | so "Hide my email addresses from other Launchpad users" really ought to be on the +editemails form | 12:21 |
bigjools | sigh | 12:21 |
bigjools | bug 387774 it seems | 12:22 |
_mup_ | Bug #387774: Hiding email addresses should be done from the email settings page <Launchpad Registry:Triaged> <https://launchpad.net/bugs/387774> | 12:22 |
wgrant | Hah, I reported it. | 12:22 |
bigjools | lol | 12:22 |
StevenK | Then fix it? | 12:22 |
* StevenK hides | 12:23 | |
bigjools | Curtis Hovey wrote on 2009-06-16: "We can fix this in July and August." | 12:23 |
bigjools | lmao | 12:23 |
wgrant | It was more than a month before the code was released! I am in the clear. | 12:23 |
wgrant | bigjools: He didn't state a year... | 12:23 |
bigjools | exactly :) | 12:23 |
StevenK | Bwahaha | 12:23 |
StevenK | wgrant: Fix it now? | 12:23 |
wgrant | Silence! | 12:23 |
StevenK | You have the code ... | 12:23 |
* StevenK baits wgrant via phone | 12:24 | |
wgrant | Uhoh. | 12:24 |
bigjools | I filed bug 615809 | 12:25 |
_mup_ | Bug #615809: I should be able to selectively hide email addresses <Launchpad Registry:New> <https://launchpad.net/bugs/615809> | 12:25 |
* wgrant me-toos. | 12:26 | |
bigjools | I don't expect it to get fixed soon, Curtis has even more bugs than me | 12:28 |
wgrant | bigjools: Anyway, can you RC-stamp that branch? | 12:35 |
bigjools | wgrant: done. let's ask gmb nicely to land it :) | 12:41 |
wgrant | bigjools: Thanks. | 12:47 |
wgrant | gmb: Can you please land it? | 12:47 |
=== jtv1 is now known as jtv | ||
=== matsubara-afk is now known as matsubara | ||
=== Ursinha-afk is now known as Ursinha | ||
gmb | wgrant, Yes, sure. | 13:22 |
wgrant | gmb: Thanks. | 13:23 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
gmb | bigjools, On a similar note, can I get an RC for this: https://code.edge.launchpad.net/~gmb/launchpad/bug-615644/+merge/32175 | 13:24 |
gmb | Timeout fix. | 13:24 |
* bigjools looks | 13:24 | |
bigjools | gmb: nice change, I'm glad we have stub :) | 13:25 |
gmb | bigjools, Aren't we all :) | 13:26 |
bigjools | gmb: FWIW your bug is unassigned and not in-progress | 13:27 |
bigjools | rc=me anyway | 13:27 |
gmb | bigjools, Ooops. And there's no Kanban card for it either. This is what I get for accepting on-the-fly assignments. Thanks. | 13:27 |
bigjools | :) | 13:27 |
leonardr | gmb, can you take a look at https://code.edge.launchpad.net/~leonardr/lazr.restful/benji-updated-size-link/+merge/32187 ? | 13:36 |
gmb | Sure | 13:36 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: leonardr || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
gmb | leonardr, Woah, 1233 lines? Oh, wait. I see. nm | 13:39 |
gmb | leonardr, r=me. | 13:50 |
leonardr | gmb, thanks | 13:50 |
gmb | np | 13:50 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
gmb | james_w, Is https://code.edge.launchpad.net/~james-w/launchpad/archive-collection/+merge/31499 in a reviewable state? | 14:48 |
gmb | Or is it WIP after your conversation with jml? | 14:48 |
james_w | gmb: WIP, sorry, just changed the status | 14:49 |
gmb | james_w, Cool, thanks. | 14:49 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: jelmer || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
=== leonardr changed the topic of #launchpad-reviews to: Today's trained review ape: gmb, leonardr || reviewing: jelmer || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
jelmer | gmb: Thanks! | 15:10 |
=== gmb changed the topic of #launchpad-reviews to: Today's trained review ape: gmb || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
gmb | jelmer, np | 15:19 |
* gmb -> bbiab | 15:19 | |
=== gmb changed the topic of #launchpad-reviews to: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
=== deryck is now known as deryck[lunch] | ||
=== salgado is now known as salgado-lunch | ||
=== Ursinha is now known as Ursinha-afk | ||
=== deryck[lunch] is now known as deryck | ||
=== Ursinha-afk is now known as Ursinha | ||
=== leonardr changed the topic of #launchpad-reviews to: leonardr || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
=== salgado-lunch is now known as salgado | ||
=== matsubara is now known as matsubara-lunch | ||
jelmer | Hi leonardr | 18:45 |
jelmer | Could I add a merge proposal to your queue? | 18:45 |
leonardr | jelmer, sure | 18:46 |
jelmer | leonardr: Thanks! The mp is https://code.edge.launchpad.net/~jelmer/launchpad/55288-publisher-unknown-distroseries/+merge/32232 | 18:46 |
=== leonardr changed the topic of #launchpad-reviews to: leonardr || reviewing: jelmer || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
rockstar | leonardr, us Tarmac guys wants yous to look at https://code.edge.launchpad.net/~jkakar/launchpadlib/fake-launchpad/+merge/26391 please. | 18:57 |
leonardr | rockstar, ok | 18:57 |
=== leonardr changed the topic of #launchpad-reviews to: leonardr || reviewing: jelmer, rockstar || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
rockstar | leonardr, I realize that it's a HUGE diff. | 18:58 |
rockstar | leonardr, technically it's jkakar's branch, but I'm willing to help facilitate where needed. | 18:58 |
jelmer | rockstar: Ooh, a fakelaunchpad, nice! | 18:59 |
mars | leonardr, +1 from Ursinha, lifeless, Diogo and I on having that as well. The QA tools use launchpadlib a lot | 18:59 |
leonardr | rockstar: this branch has not had any code revisions since i made some sweeping comments back in may | 19:00 |
leonardr | i can take another look but a lot of it is going to boil down to "you should really make those changes" | 19:00 |
rockstar | leonardr, okay, great. | 19:00 |
rockstar | What's more, I think jkakar is on holiday for a bit. | 19:02 |
EdwinGrubbs | jelmer: do you want to finish reviewing this feature? https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-602385-register-project-from-sourcepackage-page-part2/+merge/32234 | 19:08 |
EdwinGrubbs | jelmer: oops, I saw your name at the top, and I thought you were oncall. I'll ask leonardr. | 19:08 |
EdwinGrubbs | leonardr: can you review https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-602385-register-project-from-sourcepackage-page-part2/+merge/32234 | 19:09 |
=== EdwinGrubbs changed the topic of #launchpad-reviews to: leonardr || reviewing: jelmer, rockstar || queue: [Edwin] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
=== leonardr changed the topic of #launchpad-reviews to: leonardr || reviewing: jelmer || queue: [Edwin] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
=== matsubara-lunch is now known as matsubara | ||
salgado | leonardr, can I add myself to the queue with https://code.edge.launchpad.net/~salgado/lazr.restful/fix-exporting-empty-entries/+merge/32239 ? | 19:51 |
leonardr | salgado: yeah, but i think you're the limit | 19:51 |
=== salgado changed the topic of #launchpad-reviews to: leonardr || reviewing: jelmer || queue: [Edwin,salgado] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
leonardr | jelmer, r=me | 20:06 |
=== leonardr changed the topic of #launchpad-reviews to: leonardr || reviewing: Edwin || queue: [salgado] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
jelmer | leonardr: Thanks! | 20:09 |
leonardr | edwin, some stupid questions | 20:15 |
leonardr | what does the 'count' attribute do in the img tag within the copyright-expand link? | 20:15 |
leonardr | edwin: is source_package_name really a name? it looks like a source_package object that _has_ a name | 20:18 |
EdwinGrubbs | leonardr: the count can be removed. I copied the link from elsewhere in the widget template, although I don't understand what it does for those links. I'll investigate and see if it is just cruft for those links also. | 20:23 |
leonardr | ok | 20:23 |
EdwinGrubbs | leonardr: confusingly, the sourcepackage object is just a combination of a distroseries and a sourcepackagename object, and the sourcepackagename table in the db just has a name column. | 20:24 |
leonardr | ok, so it is a sourcepackagename object that has a name | 20:25 |
EdwinGrubbs | leonardr: yes, the sourcepackage.name actually just returns the sourcepackagename.name | 20:26 |
leonardr | edwingrubbs, does link_source_package need to check that self.product is not None, or is that a given? | 20:29 |
EdwinGrubbs | leonardr: hmm, it won't be None because of where it is called, but I should turn that into an argument to link_source_package() so that it does surprise someone when they reorganize the main_action(). | 20:31 |
leonardr | ok | 20:32 |
bac | leonardr: do you have time for another branch to review? | 20:33 |
jelmer | I was about to ask the same :-) | 20:33 |
bac | jelmer: trade? | 20:34 |
bac | jelmer: meaning, i'll review your branch if you'll do mine. | 20:35 |
leonardr | edwingrubbs: "If that is your situation we u= rge" really? is that an email thing? | 20:35 |
leonardr | if so, it might make more sense to change it back, to make it clear where the line break is in the email | 20:35 |
EdwinGrubbs | leonardr: oh, I was fixing a lint error. I'll try to make it look less odd without making lint complain. | 20:36 |
jelmer | bac: Sure! What's your branch? | 20:38 |
bac | jelmer: https://code.edge.launchpad.net/~bac/launchpad/bug-590813/+merge/32248 | 20:39 |
leonardr | edwin: r=me with the changes we discussed | 20:43 |
=== leonardr changed the topic of #launchpad-reviews to: leonardr || reviewing: salgado || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
jelmer | bac: Mine is at https://code.edge.launchpad.net/~jelmer/launchpad/publisher-use-debian-1/+merge/32245 | 20:45 |
leonardr | salgado, your changes to test_webservice, are they just delinting? | 20:53 |
salgado | leonardr, yes, removal of unused imports | 20:54 |
leonardr | ok | 20:54 |
jelmer | bac: Thanks for the review. | 20:54 |
leonardr | salgado, r=me | 20:54 |
salgado | leonardr, cool, thanks! | 20:55 |
bac | jelmer: np | 20:55 |
salgado | leonardr, no need to bump the version or add anything to NEWS.txt as there was no release where that bug was present, right? | 20:59 |
leonardr | salgado: right, i'll do a release tomorrow | 20:59 |
jelmer | bac: I'm still reading up on snapshots, but a review of your branch is in progress as well. | 21:00 |
bac | jelmer: take your time, no rush | 21:00 |
=== leonardr changed the topic of #launchpad-reviews to: On call: - || reviewing: salgado || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
=== leonardr changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews | ||
jelmer | bac: It seems like this could be an issue in a lot of situations - every sql based collection field I guess? | 21:07 |
bac | jelmer: yes, that is true. | 21:07 |
jelmer | bac: r=me, though I wonder if it would make sense to have a more structural solution for this kind of thing as well. | 21:08 |
jelmer | bac: Something like a shortlist() implementation that would give up on trying to collect the full sequence contents rather than errorring out perhaps. | 21:09 |
jelmer | It seems like a pity to give up on snapshotting product series (which would be access quite often I imagine) just because of a very rare case. | 21:10 |
=== Ursinha is now known as Ursinha-lunch | ||
bac | jelmer: my understanding of 'shortlist' is that we want it to complain loudly b/c it indicates our original assumptions were wrong | 21:14 |
bac | jelmer: lifeless started an email thread about changing Snapshot to ignore CollectionFields by default. it seems like a good idea which may get implemented in the future. | 21:15 |
jelmer | bac: Ah, that would make sense indeed. | 21:16 |
=== Ursinha-lunch is now known as Ursinha-afk | ||
=== matsubara is now known as matsubara-afk | ||
=== salgado is now known as salgado-afk | ||
=== Ursinha-afk is now known as Ursinha |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!