[03:40] <wallyworld> can anyone tell me, given a BranchRevision object, the correct way to figure out if the revision was from merging in another branch and what that other branch is? Do you look for some attribute of any revision parents?
[03:42] <StevenK> wallyworld: wgrant says "lol"
[03:42] <wallyworld> why lol?
[03:42] <wallyworld> did i say something funny?
[03:43] <wgrant> You may be able to use look at the revision's parents' getBranch method.
[03:43] <wgrant> s/look at //
[03:43] <wallyworld> that's what i'm doing now - i get the first parent and use getBranch() on that
[03:43] <wallyworld> i wasn't sure if that was the correct thing to do or not
[03:44] <wallyworld> i don't have any good data to check
[03:44] <wallyworld> that my theory is correct that it is the right thing to do
[03:44] <lifeless> I suggest discussing this with tim / a bazaar dev
[03:44] <lifeless> I would, but its too much like work
[03:45] <wallyworld> lifeless: and you are on HOLIDAYS
[03:45] <lifeless> exactly
[03:45] <wgrant>  /abr lifeless
[03:45] <lifeless> abr
[03:45] <lifeless> ?
[03:45] <wallyworld> lifeless: so shut down your irc client :-)
[03:45] <StevenK> OUT, DAMN SPOT
[03:46] <wgrant> lifeless: auto op, ban, remove, de-op.
[03:47] <lifeless> wgrant: thats an odd thing to suggest
[03:48] <lifeless> wallyworld: On leave != uninterested :)
[03:48] <lifeless> wallyworld: just relaxed :>
[03:49] <wallyworld> lifeless: well i hope you have a beer in hand or something :-)
[03:49] <lifeless> I'm playing wow :)
[03:50] <lifeless> wallyworld: while you are here
[03:50] <lifeless> bug 691343
[03:50] <_mup_> Bug #691343: OOPS page should reflect new bug filing location <Launchpad itself:New> < https://launchpad.net/bugs/691343 >
[03:50] <lifeless> would be easy and extremely useful to fix :)
[03:50] <wallyworld> lifeless: ok. i'll take a look
[03:52]  * wallyworld assigns bug ^^^^ to himself
[09:08] <mrevell> Hello
[09:20] <danilos> wgrant, hi, are you perhaps still around?
[10:05] <wgrant> danilos: Hi.
[10:05] <wgrant> danilos: You're wondering about TTBJ.build?
[10:06]  * StevenK looks for a reviewer in here as well ...
[10:06] <StevenK> wgrant: Doesn't that just link to the Build table?
[10:07] <wgrant> danilos: I think it should link to the TTB.
[10:07] <wgrant> danilos: But that whole part of the model is being ripped out when I get bored.
[10:07] <wgrant> Which may end up being next week.
[10:09] <wgrant> StevenK: Down to 9.1EB yet?
[10:09] <StevenK> 9223372036695 MB
[10:10] <wgrant> :(
[10:10] <StevenK> wgrant: You have 23 hours 55 minutes to get here and download the entire Internet
[10:11] <lifeless> wtf
[10:12] <StevenK> lifeless: So, internet here said I had 500MB download limit. I logged in again after it disconnected me for hitting the limit, and it seems my limit is now 9.2EB
[10:12] <wgrant> The limit also decreased as we used it.
[10:13] <wgrant> It seems to be counting properly.
[10:13] <lifeless> wheeee
[10:13] <wgrant> Despite being 9.2EB of 500MB.
[10:13] <jpds> How odd.
[10:13] <StevenK> I suspect an overflow bug of some kind
[10:13] <lifeless> [10:13] <lifeless>     Hard / Soft  Page ID
[10:13] <lifeless>       59 /  242  BugTask:+index
[10:13] <lifeless>       31 /  267  ProjectGroupSet:CollectionResource:#project_groups
[10:13] <lifeless>       25 / 3459  Archive:+index
[10:13] <lifeless>       25 /  289  Distribution:+bugs
[10:13] <lifeless>       18 /  237  POFile:+translate
[10:13] <wgrant> jpds: Yet remarkably convenient.
[10:13] <danilos> wgrant, that's right (sorry, phone and emails :)
[10:13] <lifeless>       12 /    1  Bug:EntryResource
[10:13] <lifeless>        8 /  120  Question:+index
[10:13] <wgrant> lifeless: Nooooooo.
[10:13] <lifeless>        6 /   18  DistroSeriesLanguage:+index
[10:13] <lifeless>        6 /    0  Person:+leave
[10:13] <lifeless>        5 /    2  Person:+bugs
[10:13] <lifeless> StevenK: overflow would count down :P
[10:14] <StevenK> Haha, Person:+leave
[10:14] <StevenK> lifeless: It does count down as it use it ...
[10:14] <lifeless> sorry, I meant up
[10:14] <lifeless> overflows would be in 2's complement
[10:14] <StevenK> Yes. It's very strange
[10:14] <danilos> wgrant, also, do you think we could just switch getCurrentBuildFarmJob to get the new-style jobs directly thus not needing to add it for the TranslationTemplateBuildJob?
[10:14] <lifeless> I suspect an unsigned where it should be signed ;)
[10:15] <StevenK> And I don't want to tell them, since then they might fix it
[10:16] <lifeless> meep
[10:16] <wgrant> danilos: If you can find a way to do that, sure.
[10:16] <lifeless> filebug is borked in prod
[10:16] <lifeless> on chromium
[10:16] <wgrant> danilos: I'm not sure it's possible yet.
[10:16] <lifeless> apparently if you click 'just so', worked when  retried.
[10:17] <wgrant> I forget if BuildFarmJob.builder is set during the build.
[10:17] <wgrant> And at 9:20pm after just getting home from a sprint, I'm not entirely sure that I want to find out.
[10:18] <StevenK> Like the sprint was that hard, you slacker
[10:19] <lifeless> wgrant: 900ms queries
[10:19] <wgrant> danilos: It looks like BuildFarmJob.builder is not set during the build, so you probably can't do it.
[10:19] <lifeless> wgrant: *, well, 15
[10:19] <wgrant> danilos: You have to go through the BuildFarmJobOld
[10:20] <wgrant> lifeless: I fail at interpreting OOPS reports. How do I find an OOPS for that pageid?
[10:20] <lifeless> wgrant: you start on the oops summary, sent to canonical-launchpad
[10:20] <lifeless> wgrant: then, you look for bugs tagged timeout, and you find (just filed) bug 691478
[10:20] <_mup_> Bug #691478: Archive:+index timeouts <timeout> <Launchpad itself:Triaged> < https://launchpad.net/bugs/691478 >
[10:20] <lifeless> wgrant: and in there, click on the link :)
[10:20] <danilos> wgrant, tbh, I don't really understand most of this, so I am just seeing if there's anything that we really need to do to make it possible for you guys to finish the migration
[10:21] <wgrant> danilos: I don't know exactly what is necessary, sorry. I need to swap that all back in.
[10:21] <wgrant> It's been a few months.
[10:21] <lifeless> wgrant: oh, to find the entry in the report, search for the pageid in the summary report.
[10:21] <wgrant> lifeless: Hm, OK. Thanks.
[10:21] <danilos> wgrant, right, so perhaps just adding the build property to the TTBJ should solve what we can do on our side, and I'll let you worry about the rest
[10:22] <wgrant> lifeless: Ow.
[10:22] <danilos> wgrant, though, I am not sure how do we switch buildmaster to use new TTBs instead of TTBJs for translation jobs, but I guess that's easy enough
[10:23] <danilos> wgrant, also, what happens with all the BFJOld info like score and such, I don't see that in BFJs anymore
[10:23] <wgrant> danilos: The new classes (TTB, BPB, SPRB, etc.) are mostly just used for storing data for histories.
[10:23] <wgrant> Most control is still performed by the BFJOld.
[10:24] <wgrant> The score is on BuildQueue.
[10:24] <wgrant> I want to merge BuildQueue into BuildFarmJob, but some others do not.
[10:24] <wgrant> Score will remain there for now.
[10:24] <wgrant> And BuildQueue will link to BuildFarmJob.
[10:24] <danilos> wgrant, ah, ok, so we are still pretty far off from being able to completely switch to the new model if I got that correctly
[10:24] <wgrant> danilos: Not *really*.
[10:25] <wgrant> Just needs some code moved around, basically.
[10:25] <danilos> wgrant, heh, ok, then I didn't get that correctly :)
[10:25] <wgrant> But I need to work out what.
[10:25] <wgrant> The job-specific schema changes are done.
[10:25] <wgrant> Apart from deleting the old tables.
[10:25] <danilos> wgrant, right, so do you think it makes the most sense for you to deal with that? would it help if TTBJ.build was provided and pointed at TTB?
[10:26] <wgrant> danilos: Do that for now, since it makes the buildd-manager log sad.
[10:26] <danilos> wgrant, ok, I hope I'll be able to come up with a nice simple query to get that (i.e. I am hoping a TTBJ has enough information to do it)
[10:27] <danilos> wgrant, thanks for the input
[10:27] <wgrant> danilos: Sorry I can't be more help.
[10:27] <wgrant> But uni pushed all that stuff way down.
[10:28] <danilos> wgrant, yeah, don't worry, it's just that _that_ bigjools guys is pushing us to implement stuff that I believe we mostly implemented :)
[10:28] <wgrant> danilos: It's all a bit of a mess, yes.
[10:28] <wgrant> We are really close now, though.
[10:29] <danilos> wgrant, and I don't know enough to tell him off, so I am trying to learn all about this in a few hours which turns out is not the simplest thing around ;)
[10:29] <danilos> wgrant, not to mention that TTBJ is at the same time a BFJOld and a BranchJob, just to make it more interesting :)
[10:29] <wgrant> danilos: Yeah, I like to blissfully forget that detail.
[10:30] <danilos> hey bigjools, I was just gossiping about you, but if I tell you what it was, it won't be proper talk-behind-the-back stuff
[10:30] <lifeless> hmm, we should probably log returned rows in the oops query history
[10:30] <wgrant> lifeless: The number of returned rows?
[10:30] <lifeless> yeah
[10:30] <wgrant> Indeed.
[10:30] <lifeless> so we can see slow but fast per row
[10:31] <lifeless> vs fast but few rows
[10:31] <lifeless> vs slow and few rows
[10:31] <danilos> wgrant, yeah, I hate that, and I am hoping we can soon just make TTBJ a BranchJob and let any buildd stuff live in TTB
[10:31] <wgrant> danilos: I will hopefully be deleting TTBJ.
[10:32] <wgrant> Although it may end up staying, but not as part of the build farm model.
[10:32] <danilos> wgrant, well, we can't delete the branchjob stuff, that's what schedules the build
[10:32] <wgrant> Right.
[10:32] <danilos> wgrant, right, we should just rename it
[10:32] <wgrant> TTB will probably know about it, but nothing else.
[10:32] <lifeless> wgrant: there wqere some dups
[10:32] <lifeless> wgrant: but only 2.5 seconds worth
[10:33] <wgrant> lifeless: yeah, max of 3 reps.
[10:33] <wgrant> Not 100.
[10:33] <lifeless> wgrant: max of 26 perhaps
[10:33] <lifeless> row 4 of duplicates
[10:33] <danilos> wgrant, right, though perhaps not even that (i.e. we use it to add a TTB to the queue, but that should be it)
[10:33] <lifeless> ah nope, wrong column
[10:34] <lifeless> wgrant: so I think we've still got a per-row-in-ui just-in-time query being triggered
[10:34]  * bigjools is not here
[10:34] <wgrant> lifeless: Possibly. Can I see recent OOPSes for Archive:+packages somehow?
[10:35] <wgrant> I want to compare them to Archive:+index.
[10:35] <lifeless> wgrant: log into devpad, grep.
[10:35] <wgrant> Yay.
[10:35] <wgrant> I will just trigger new ones instead.
[10:35] <wgrant> On DF.
[10:35] <lifeless> wgrant: or
[10:35] <lifeless> OOPS-1811H1896
[10:35] <lifeless> 25 seconds!
[10:35] <lifeless> (top 10 durations in the lpnet summary)
[10:36] <wgrant> Minimal reps.
[10:36] <wgrant> Just... very slow.
[10:36] <wgrant> Hm.
[10:36] <wgrant> That OOPS must be wrong.
[10:36]  * wgrant blames the GIL.
[10:36] <wgrant> 5s gap after a 200ms query.
[10:37] <lifeless> there's a bug for getting per thread times
[10:37] <lifeless> just needs tuits
[10:37] <wgrant> And a couple of other similar ones.
[10:37] <lifeless> wgrant: large row counts coming back through storm can legitimately show up as that, fwiw
[10:37] <wgrant> Oh, that +index OOPS is the same.
[10:37] <wgrant> Multi-second gaps after ~200ms queries that wouldn't return that many rows.
[10:38] <lifeless> heh
[10:38] <wgrant> Like <100.
[10:38] <lifeless> well, someone should merge my single threaded appserver branch :)
[11:38] <henninge> danilos: do you know about LaunchpadWriteTarFile?
[11:38] <danilos> henninge, now I do :)
[11:39] <henninge> danilos: it's in lp.translations.utilites.translation_export
[11:39] <henninge> but I think it belongs under lp.testing.
[11:39] <danilos> henninge, right, and it might, though testing does sound weird, let me look
[11:40] <henninge> danilos: oh, right. I thought it was only used in tests
[11:40] <danilos> henninge, it might make sense to move it to lp.services though
[11:40] <henninge> but the language pack exporter uses it, too.
[11:40] <henninge> danilos: yes, I agree.
[11:41] <henninge> danilos: I'll put it in lp.services.tarfile
[11:42] <danilos> henninge, sure, though you might want to avoid using the same name as the global tarfile module imho :)
[11:42] <henninge> ok
[11:42] <henninge> lptarfile
[11:42] <henninge> ?
[11:42] <danilos> henninge, i.e. you can't do "from lp.services import tarfile" and "import tarfile" then
[11:43] <danilos> henninge, I don't know, it's also only for writing
[11:43] <henninge> I would not recommend taht anyway
[11:43] <henninge> importing the module, I mean.
[11:43] <henninge> tarfile_helpers
[11:43] <danilos> henninge, heh, I am not saying I'd recommend it, it's just that it'd be unobvious to a reader of the code which tarfile module is being used in that case
[11:44] <henninge> true
[11:44] <henninge> danilos: how is tarfile_helpers?
[11:44] <danilos> henninge, probably good enough, go with that and see if your reviewer thinks otherwise
[11:44] <henninge> cool, thanks
[11:59] <deryck> Morning, all.
[12:31] <maxb> *blink*   Is PPA +copy-packages supposed to offer o-series and p-series as destinations?
[12:33] <jelmer> maxb, there's a bug open about that
[12:34] <jelmer> maxb, we previously didn't really support distro series that were present but not initialized
[14:12] <gmb> gary_poster: Are you the right person to talk to about making lxml a dependency for Launchpad?
[14:12] <gary_poster> gmb, I guess
[14:13] <gmb> gary_poster: I ask because I've got a branch in review that depends on it; bac has pointed out that we don't necessarily have it on our appservers.
[14:14] <gary_poster> gmb, lxml is notoriously annoying to build as a Python egg.  Is the Deb-packaged version acceptable?  Also...
[14:14] <gmb> gary_poster: There's now some discussion in launchpad-reviews about this...
[14:14] <gary_poster> ISTR that we have had an lxml dependency in the past
[14:14] <gmb> ORLY.
[14:14] <gary_poster> yeah, I saw, gmb, I'm not entirely sure where to write :-)
[14:14] <gmb> Over there. There are more people paying attention :)
[14:59] <maxb> gmb: Remember to tag the launchpad-dependencies branch :-)
[15:03] <gmb> maxb: Done, thanks for the reminder. There are two sets of instructions for maintaining that .deb and on of htem is wrong.
[15:03] <gmb> I'll update it now...
[15:03] <maxb> gmb: where is the second?
[15:05] <gmb> maxb: Well, the first is here: https://dev.launchpad.net/Hacking#Changing%20the%20launchpad%20dependency%20debs (and was wrong). The second (correct; or at least more up-to-date) version is here: https://dev.launchpad.net/LaunchpadPpa#launchpad-dependencies
[15:05] <gmb> I'd got all the way through the process before finding hte second.
[15:06] <gmb> I've updated /Hacking to point at the LaunchpadPPA page.
[15:07] <maxb> hah, nice. I never knew of the first, when I wrote the second
[15:07] <maxb> Neither did anyone who was asking me if there were any instructions for doing it :-)
[15:08] <gmb> Heh.
[15:08] <gmb> Well, they were similar enough that I didn't screw things up. The first just used the old, non-bzr-builddeb way.
[16:08] <henninge> danilos: fxing bug 619121
[16:08] <_mup_> Bug #619121: Inactive template in rosetta: Confusing wording <bugjam2010> <lp-translations> <trivial> <ui> <Launchpad itself:In Progress by henninge> < https://launchpad.net/bugs/619121 >
[16:08] <henninge> danilos: do you agree with "Template active?" instead of "Accept translations?"
[16:18] <danilos> henninge, yes
[16:18] <danilos> henninge, or "Template is active"
[16:19] <henninge> without the '?'
[16:19] <henninge> I think I like that even better
[16:19] <danilos> henninge, I don't, I think it should just be "???"
[16:20] <henninge> ;-)
[16:20] <henninge> danilos: bug 670821
[16:20] <_mup_> Bug #670821: translator comment formating lost <lp-translations> <trivial> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/670821 >
[16:20] <henninge> do you know of any reason not to use a <pre> here?
[16:21] <danilos> henninge, well, one: comments in PO files commonly have some whitespace padding as in:
[16:21] <danilos> #.   This is all so cool
[16:21] <danilos> #.     that it doesn't make sense at all
[16:22] <danilos> (imagine those two were aligned: basically, we'd need equivalents for "rectangle whitespace" handling)
[16:22] <danilos> henninge, but, it's probably an improvement to just use <pre>, which is why it's 'trivial' :)
[16:22] <henninge> danilos: but doesn't <pre> give us that?
[16:23] <danilos> henninge, well, my point was that it'd be nice to strip initial spaces (same amount from every line) and then put it into <pre>
[16:24] <danilos> henninge, i.e. think of a case like this as well:
[16:24] <danilos> #.    numbered items:
[16:24] <danilos> #.      1. one
[16:24] <danilos> #.      2. two
[16:24] <danilos> henninge, it'd be nice if we stripped those 4 leading spaces from each line, but not a big deal imo
[16:25] <henninge> no, not really. ;)
[16:25] <danilos> henninge, yeah, so go for the trivial :)
[16:25] <henninge> we can do that when we redesign the +translate page ... ;)
[16:25] <danilos> henninge, heh, right, "when" :)
[16:25] <henninge> meaning 'if'?
[16:26] <danilos> henninge, well, meaning "we" :)
[16:26] <danilos> or rather, "we?" :)
[16:31] <henninge> they!
[17:27] <danilos> MutatedCelebrityError :(
[17:35] <maxb> hah
[17:35] <maxb> that is an AMAZING classname :-)
[17:41] <danilos> :)
[17:41] <danilos> I'd probably enjoy it more if we didn't get ~800 OOPSes with that :)
[17:45] <benji> I'm reminded of the TemporalParadox raised when you tried to edit a past version of an object in the Zope 2 history tab.
[20:19] <LPCIBot> Project db-devel build (231): FAILURE in 3 hr 38 min: https://hudson.wedontsleep.org/job/db-devel/231/
[21:59] <LPCIBot> Project devel build (313): FAILURE in 3 hr 13 min: https://hudson.wedontsleep.org/job/devel/313/
[21:59] <LPCIBot> Launchpad Patch Queue Manager: [r=bac][ui=none][bug=685624] Provide
[21:59] <LPCIBot> TranslationTemplatesBuildJob.build property linking to
[21:59] <LPCIBot> TranslationTemplatesBuild objects to stop polluting
[21:59] <LPCIBot> build-manager logs.