[00:01] sinzui: Sure. [00:03] sinzui: We're on maintenance for a little longer, then? [00:03] Ah, I see the email. [01:04] wgrant: to -dev? or squad only? [01:16] lifeless: Squad only. [01:17] lifeless: What's your opinion on BPB/SPRB URLs? [01:17] Currently I'm reverting BPB to +build, and SPRB to a new +recipebuild. [01:32] wgrant: works for me [01:32] The BPB URL probably isn't ideal, but it's not introducing a new one so I don't care too much. [01:33] yeah [01:33] you could do sourcebuild or something [01:33] but I think recipe build is fine [01:34] Hmm, bigjools didn't start the mawson restore. [02:23] sinzui: Did you want to talk about that bug at some point? [02:38] I'm going to get tired of clicking 'one hour' soon, I can tell. [02:38] lifeless: Hm? [02:38] launchpadlib desktop integration [02:56] wgrant: hi. I do want to talk about that bug. I can now now or any time in the next two hours [03:13] Project windmill-devel build #3: STILL FAILING in 1 hr 4 min: https://lpci.wedontsleep.org/job/windmill-devel/3/ [03:22] anyone know what translationtemplateitem.sequence means? [03:23] lifeless: It's just used to preserve the POT's order, AIUI. [03:23] * StevenK grumbles. I don't want to add DistroSeriesParent to the sampledata [03:29] so don't [03:29] I don't want you to either [03:29] getById or getByID? [03:30] I think the latter. [03:30] But we use both. [03:30] But I need to change one of them to the other. [03:30] So I want more opinions. [03:30] the latter [03:30] Thanks. [03:30] fooID [03:30] is the table convention [03:30] so folk will expect that elsewhere [03:31] Only for SQLObject. [03:31] For Storm we use foo_id. [03:31] which is 90% of our ORM objects [03:31] And that's not camelCase anyway. [03:31] headdesk [03:31] select count(*), potemplate is null from translationmessage group by potemplate is null; [03:31] count | ?column? [03:31] ----------+---------- [03:31] 17773385 | f [03:31] where's jeroen when you need him [03:31] 51852904 | t [03:32] Why? [03:33] to find out why tm.potemplate is set in only some rows [03:33] Bah. Our Storm sugar class defines getById :/ [03:33] I'm looking at bug https://bugs.launchpad.net/launchpad/+bug/534203 [03:33] <_mup_> Bug #534203: Timeouts on POFile:+filter (filter by person) < https://launchpad.net/bugs/534203 > [03:34] lifeless: I think POTemplate is only used for non-shared translations. [03:34] lifeless: Shared ones use POTMsgSet. [03:35] Yeah. [03:35] def shareIfPossible(self): [03:35] """See `ITranslationMessage`.""" [03:35] if self.potemplate is None: [03:35] # Already converged. [03:35] return [03:35] so we'll eventually drop the column? [03:35] I believe so. [03:35] hmm [03:36] this is a good reason not to abuse semantic fields as indicators [03:36] s/reason/example of/ [03:54] Project windmill-devel build #4: STILL FAILING in 40 min: https://lpci.wedontsleep.org/job/windmill-devel/4/ === Ursinha is now known as Ursinha-afk [04:35] Project windmill-devel build #5: STILL FAILING in 41 min: https://lpci.wedontsleep.org/job/windmill-devel/5/ === almaisan-away is now known as al-maisan [07:28] Project devel build #685: FAILURE in 5 hr 32 min: https://lpci.wedontsleep.org/job/devel/685/ [07:28] * StevenK frowns at Jenkins [07:29] Project windmill-db-devel build #231: STILL FAILING in 1 hr 4 min: https://lpci.wedontsleep.org/job/windmill-db-devel/231/ [07:29] Oh, sigh. Jenkins hasn't got a clue how long windmill-devel takes since one build hasn't been sucessful yet. [07:29] Heh. [07:30] However, four slaves up and building makes me happy. [07:30] But not Canonicloud. [07:30] Huh? [07:31] It probably doesn't make the cloud very happy. [07:31] No idea, TBH, but IS haven't chased me. [07:33] StevenK: Linode native IPv6 is handy. My tunneled v6 routing to Fremont is apparently 10ms faster than native v4. [07:33] Mmmmm, I was going to jump when they fix the DNS to handle AAAA. [07:34] You can add AAAA, but the NSes don't have AAAAs themselves yet. [07:34] Which means you can't resolve v6-only anyway :-) [07:34] Right :( === spm changed the topic of #launchpad-dev to: Launchpad Down/ReadOnly 0800-0930 UTC | Merge to devel are open again | https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews === jam2 is now known as jam === al-maisan is now known as almaisan-away [07:50] Project windmill-devel build #6: STILL FAILING in 1 hr 10 min: https://lpci.wedontsleep.org/job/windmill-devel/6/ [08:15] good morning [09:02] morning folks [09:06] Morning bigjools, morning gnuoy, morning all. [09:19] Project windmill-devel build #7: STILL FAILING in 6 min 20 sec: https://lpci.wedontsleep.org/job/windmill-devel/7/ [09:20] Uh oh === almaisan-away is now known as al-maisan [09:20] StevenK: codehosting is down. [09:20] I suspect that's it, but haven't checked the logs. [09:21] Morning [09:21] Yeah, that's it [09:24] Morning, allenap. [09:45] Project windmill-devel build #8: STILL FAILING in 0.53 sec: https://lpci.wedontsleep.org/job/windmill-devel/8/ [09:58] allenap: http://pastebin.ubuntu.com/603166/ [10:00] Project windmill-devel build #9: STILL FAILING in 0.5 sec: https://lpci.wedontsleep.org/job/windmill-devel/9/ [10:01] Bah, stop that, Jenkins! [10:03] StevenK: That looks beautiful :) [10:07] allenap: Thank you :-) [10:15] Project windmill-devel build #10: STILL FAILING in 0.48 sec: https://lpci.wedontsleep.org/job/windmill-devel/10/ === mthaddon changed the topic of #launchpad-dev to: Merge to devel are open again | https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews [10:30] Project windmill-devel build #11: STILL FAILING in 1.6 sec: https://lpci.wedontsleep.org/job/windmill-devel/11/ [10:31] wheee, killed my lp vm doing a full homedir backup with lmirror (OOM killer) [10:31] I think I need to check the memory footprint a little :> === al-maisan is now known as almaisan-away === almaisan-away is now known as al-maisan [10:45] Project windmill-devel build #12: STILL FAILING in 0.5 sec: https://lpci.wedontsleep.org/job/windmill-devel/12/ [10:49] wgrant: I'm working on a branch that will use the StormStatementRecorder to monitor the number of statements used by packagecopier.CopyChecker. [10:49] The goal is the measure how many queries have been added by the recently merged perm check thing ... and also to have a test to keep the number of queries under a tight leash if we add more things in the future. [10:50] Steven told me you might have done something like that already ... but I can't find a test that does this ... any advice? [10:50] rvba: I don't have a test for copychecker itself, but some of the methods it uses are constrained (eg publishBinaries) [10:51] wgrant: ok ... so I guess the work I'm doing makes sense then. [10:51] rvba: Yup. [10:51] rvba: We do need tests for that. [10:51] Just nobody's written them yet. [10:52] And they're pretty easy to write these days. [10:52] right [10:52] I'll take a look at was as been done with publishBinaries. [10:52] wgrant: thanks. [10:52] s/as/has/ [10:52] I may have misremembered. But grep around for StormStatementRecorder to see examples. [10:52] I've done that yeah. [10:53] I think I was going to do it, but something was still not constant enough for it to be practical. [10:55] I see. My plan is to see how many statements it takes on average (after having created a bunch of sources) ... have create a test with that number hardcoded so that if a changes increases the number of queries issued a lot, one will get a warning. [10:58] I can't think of a better way to do this given that checkCopy does quite a lot of things in one call. [11:00] Project windmill-devel build #13: STILL FAILING in 0.51 sec: https://lpci.wedontsleep.org/job/windmill-devel/13/ [11:06] rvba: wgrant: the worker to look for is HasQueryCount [11:07] That too. [11:07] lifeless: yeah, I took a recent test by Gavin that uses that as an inspiration [11:08] cool [11:08] if you're testing a page, RendersWith something or other is a wrapper [11:11] I've only tested a few things ... but it does not seem to be constant (or maybe I have not properly cleaned up the cache) ... with 30 sources I've an avg of 12.23 (.23?) queries, with 300 sources the avg becomes 12.023 [11:12] maybe I should stick to testing small individual methods as opposed to the whole checkCopy method. [11:15] Project windmill-devel build #14: STILL FAILING in 0.49 sec: https://lpci.wedontsleep.org/job/windmill-devel/14/ [11:30] Project windmill-devel build #15: STILL FAILING in 1.3 sec: https://lpci.wedontsleep.org/job/windmill-devel/15/ [11:32] Why is PQM rejecting my landings? [11:33] Command failed! [11:33] running 0 tests... [11:33] ---------------------------------------------------------------------- [11:33] Ran 0 tests in 0.000s [11:33] henninge: What's in the attachments it sends back? [11:33] We are probably in testfix. [11:33] stdout should tell you. [11:33] it does [11:33] Looks like buildbot is a bit unhappy. I've forced it. [11:34] why is that not in the mail itself ... [11:34] No idea :/ [11:34] it used to be [11:34] wgrant: tanks [11:34] thanks [11:35] henninge: It's because PQM is terrible [11:36] how have we been planning to switch to tarmac? [11:37] PQM script success ;-) [11:42] wgrant: we can't land StevenK's changes to use DSP on prod as he's got more schema changes. Can you think of anything that will break in oneiric if we change its parent_series for a month? [11:43] bigjools: Translations might. [11:43] bigjools: Although hopefully only initialisation uses that. [11:43] yeah [11:43] it's been initialised, so ... I am looking for anything else [11:44] Yippie, build fixed! [11:44] Project db-devel build #513: FIXED in 5 hr 32 min: https://lpci.wedontsleep.org/job/db-devel/513/ [11:44] LPCIBot gets a woody [11:44] bigjools: SourcePackage.packaging uses it... [11:44] Which may break bug linking. [11:45] easily fixed [11:45] Project windmill-devel build #16: STILL FAILING in 0.55 sec: https://lpci.wedontsleep.org/job/windmill-devel/16/ [11:47] I'm thinking about getBuildByArch. [11:47] It's *probably* not going to matter, but it's hard to be sure immediately. [11:48] bug 643369 [11:48] <_mup_> Bug #643369: IDistroSeries.deriveDistroSeries() should use the security adapter

< https://launchpad.net/bugs/643369 > [11:52] wgrant: euargh [11:52] that code is gross [11:53] And slow. [11:53] Damn slow. [11:53] wgrant: wtf does it need to traverse over parent_series - it should just go backwards over versions [11:53] bigjools: :/ [11:54] although, we didn't make distroseries.version a debversion did we? [11:54] yeah, that will break horribly if we change parent_series. Damn. [11:57] I'm not sure the breakage will matter. [11:59] wgrant: it might break build uploads [11:59] actually [11:59] wtf [12:00] ah it was only used for security uploads [12:00] the old style [12:00] Project windmill-devel build #17: STILL FAILING in 0.56 sec: https://lpci.wedontsleep.org/job/windmill-devel/17/ [12:01] bigjools: Yes. [12:01] bigjools: Well, not entirely. [12:01] It only creates builds for that case. [12:02] But it will use parent_series when searching for existing builds too. [12:02] eg. in createMissingBuilds. [12:02] But that's probably not relevant here. [12:02] yeah, but it relies on getBuildByArch working [12:02] Since everything should be found in the series itself. [12:02] if it finds nothing it'll start creating new builds [12:15] Project windmill-devel build #18: STILL FAILING in 1.5 sec: https://lpci.wedontsleep.org/job/windmill-devel/18/ [12:18] * StevenK cleans up after that [12:19] Right. Slave deleted, build scheduled. === henninge is now known as henninge-lunch [12:37] Yippie, build fixed! [12:37] Project devel build #686: FIXED in 5 hr 8 min: https://lpci.wedontsleep.org/job/devel/686/ [12:38] lifeless, I see you declined a request to join 'launchpad' team into malone-alpha with "a flag would be better": we do have a flag, but we've set it to team malone-alpha, and we'd like to expand that a step at a time, so if we can't have multiple team rules, can we please add it? (unless other stuff has been arranged in the meantime) [12:39] lifeless, and hi :) [12:40] danilos: You should be able to have multiple team rules. [12:40] danilos: Does it not work? [12:41] wgrant, I don't know, I never knew it was supposed to work [12:42] danilos: It will scan through until it finds a rule that matches. [12:42] From highest priority to lowest. [12:42] There was a bug until last week that meant the 'default' scope took precedence, but multiple team rules should have worked anyway. [12:42] wgrant, cool, I'll try that then [13:19] losa ping: hi, just to check, if we want feature flags changed/added to, I should use LPS "DB query" section or what'd be the preferred way to communicate them? [13:19] danilos: that's fine, yep [13:20] mthaddon, cool === henninge-lunch is now known as henninge [13:37] Project windmill-devel build #19: STILL FAILING in 1 hr 4 min: https://lpci.wedontsleep.org/job/windmill-devel/19/ === Ursinha-afk is now known as ursinha === ursinha is now known as Ursinha === Ursinha-afk is now known as Ursinha [13:51] Morning, all. [13:58] adeuring, abentley, I'll be about 5 minutes late for standup. Sorry. [13:59] deryck: that gives me time to make a coffee :) [13:59] deryck: okay. [14:07] adeuring, henninge -- could be the wifi point I'm using [14:08] adeuring, I'm mooching off the church next door until att arrives :-) [14:09] deryck: have you started? [14:09] abentley, trying to start, connection or mumble not working for me [14:09] deryck: ah. [14:10] abentley, henninge, adeuring -- yeah, just hold the standup and I'll listen [14:14] if only you could hear me, henninge [14:14] abentley, adeuring, henninge -- thanks, guys! === matsubara-afk is now known as matsubara === al-maisan is now known as almaisan-away === almaisan-away is now known as al-maisan === abentley changed the topic of #launchpad-dev to: Merge to devel are open again | https://dev.launchpad.net/ | On call reviewer: abentley | https://code.launchpad.net/launchpad-project/+activereviews === al-maisan is now known as almaisan-away === almaisan-away is now known as al-maisan [15:28] if i'm adding a class to something in YUI to change its display (in this instance, color), do i have to do anything beyond addClass? [15:31] ah, no i do not, but it helps to get the classname right. [15:32] henninge: ping [15:32] Hi sinzui! [15:34] Project windmill-devel build #20: STILL FAILING in 1 hr 6 min: https://lpci.wedontsleep.org/job/windmill-devel/20/ [15:34] henninge: +1 for your fix, but I am not sure you fixed the actual bug since your encoding line is like the original test. [15:35] sinzui: it is testing for multiple lines now, is it now? [15:35] s/now/not/ [15:35] henninge: I think I am running the checkers in the wrong order. The text checker that sets utf_8 must run first, not last [15:35] * henninge thought it did [15:40] sinzui: I'm really happy with the improvements you suggested to the UI review yesterday, it's ready for another review when you have the time: https://code.launchpad.net/~benji/launchpad/click-to-close-boxes/+merge/59818 [15:40] * sinzui looks [15:41] sinzui: ah, now I see. text_check runs last. === al-maisan is now known as almaisan-away [15:43] benji: r=me [15:43] great, thanks [15:43] henninge I can make the ordering change if you do not have time. I should have noticed my stupidity years ago [15:44] sinzui: I can add it, np. [15:44] henninge: ping me when you want me to merge it into trunk [15:45] sinzui: I don't think it need a test, does it? [15:45] It does not [15:48] sinzui: pushed, you can merge now. [15:48] thanks. If the builder at available, this will arrive in the lp ppa in a few hours. [15:49] sinzui: have you seen the queue? :) [15:49] yes [15:49] :( [16:02] Whats the package? [16:02] Oh, and that reminds me [16:02] Would someone like to review my lpreview-body packaging fix? [16:02] mrevell: I added a comment to https://bugs.launchpad.net/launchpad/+bug/771231 that asks you to comment on/approve the fix. [16:02] https://code.launchpad.net/~maxb/lpreview-body/fix-package/+merge/59137 [16:02] <_mup_> Bug #771231: There is no confirmation of what I've done after I create a structural subscription

< https://launchpad.net/bugs/771231 > [16:02] thanks benji, /me looks [16:02] You fail, _mup_ [16:02] sinzui: poke me the build ID and I'll get it building quicker [16:02] Oh, whoops, I fail [16:03] maxb: heh :) [16:04] bigjools: I do not think this is necessary. I think we can wait as long as a week to propagate this fix to all the ppas. [16:04] benji, That's superb. Thanks. Tag changed to qa-ok. [16:04] mrevell: great, thanks [16:05] abentley: Hello. I have a review pending for lpreview-body. It has been missed because lpreview-body is not part of launchpad-project. Could you (as project maintainer) consider amending that? [16:06] maxb: sure. [16:08] maxb: however, I'd say the reason that branch has been missed is because it's for packaging, about which I know little. [16:13] sinzui: I see there's been a fair bit of discussion on bug 80902 recently. Anything I should be sticking my nose into? [16:13] <_mup_> Bug #80902: Can't target bug report from project to distribution, or vice versa

< https://launchpad.net/bugs/80902 > [16:15] jml: I do not know. I am certain the work is harder than most people suppose, and I am certain there is a lot over lap with disclosure [16:15] jml: At the worst, out work on pickers will make that bug easy to fix. at best, we fix that bug [16:15] sinzui: ok. [16:16] At the worst, *our* work on pickers will make that bug easy to fix. at best, we fix that bug [16:16] sinzui: are you planning on starting the picker stuff while on maint? [16:16] Project windmill-devel build #21: STILL FAILING in 41 min: https://lpci.wedontsleep.org/job/windmill-devel/21/ [16:17] I have not committed to it, but I was looking at the timouts and ui issues in the vocabs yesterday. [16:18] sinzui: have you ever encountered in YUI work the method hasClass failing to work in if else statements? [16:19] jcsackett: no. [16:19] jcsackett: can I see a paste of the code [16:21] sinzui: http://paste.ubuntu.com/603311/ [16:22] jcsackett: I think hasClass() is at fault [16:23] jcsackett: isn't it always returning True [16:23] sinzui: no. if the class isn't on the element i see false. [16:23] if i do like {{{ alert(hidden); }}} [16:24] jcsackett: then you are saying addClass() does not work === salgado is now known as salgado-lunch [16:26] jcsackett: have you tried something simpler, like toggleClass() [16:27] sinzui: i didn't know such a thing existed. [16:27] sinzui: still, here's the thing, i can set up conditions so that hasClass comes back false. [16:27] jcsackett: http://developer.yahoo.com/yui/3/api/Node.html [16:27] but the "true" evaulation still fires off. [16:29] jcsackett: See my Javascript Learnings #3. [16:31] abentley, sinzui: i can see how toggleClass does the same thing i'm doing, but it doesn't explain why when hasClass is return 'false' the if clause is evaluting for 'true'. [16:31] i'll move on, but this bugs me. [16:31] clearly there's a gotcha or subtlety i don't get. [16:31] jcsackett: is hidden a boolean all right? (this bugs me too ;)) [16:32] jcsackett: write the if like if (hidden == true) { to determine if you are a victim of coercion. [16:32] jcsackett: what is hidden_class ? You're not defining it in the snippit you posted. [16:33] hidden_class = "adminHiddenComment" [16:35] jcsackett: I wonder if hidden_class is evaluating to undefined? I don't know how hasClass would behave with that input. [16:36] jcsackett: I would make sure 'hidden' is of the right type ... a boolean being 'true' or 'false' always evaluated to 'true' in a test is kinda strange. [16:39] abentley: could you have alook at this (small) MP: https://code.launchpad.net/~adeuring/launchpad/bug-746866/+merge/59954 ? [16:40] adeuring: sure. [16:40] thanks! [16:43] sinzui: hi. i've recently arrived in Budapest. 38 hours with little sleep. running on caffine. gotta stay awake for the welcome dinner :-) when you have a moment, could you please take another look at this mp? i've reworked the implementation to address the issues raised with the first approach. https://code.launchpad.net/~wallyworld/launchpad/poppy-sftp-gpgconf/+merge/59154 [16:44] * sinzui does [16:55] adeuring: r=me, but the spacing change on line 30 of the patch looks wrong. [16:56] abentley: oops... I'll fix it. thanks for the review! [16:56] adeuring: np [17:17] wallyworld: r=me [17:17] sinzui: thanks. [17:28] we really need to move the "What next" at the bottom of +filebug to the top-right menu. [17:30] talking of which, someone will get bug 777777 today [18:03] good night all === salgado-lunch is now known as salgado === matsubara is now known as matsubara-lunch === deryck_ is now known as deryck [19:12] Project windmill-db-devel build #232: STILL FAILING in 1 hr 15 min: https://lpci.wedontsleep.org/job/windmill-db-devel/232/ [19:15] OOPS-1950CQ433 [19:21] deryck: do you have a few minutes to mumble? [19:23] sinzui, I do, sure. I had mumble issues before though. so you can help me test too [19:25] sinzui, I hear you [19:25] sinzui, let me log off and work on sound and come back [19:25] okay [19:26] I assumed it was bad wifi this morning [19:27] deryck: My sound is set to use pulse, I do not see any hacks for the sound driver in /etc/modprobe.d anymore [19:27] maybe your wifi got stolen too, from PSN [19:27] ah, that's it! [19:27] actually that does help.... [19:28] I was hacking sound to get games running under wine with no psn ;) [19:28] I had forgot [19:28] heh [19:28] glad to be so helpful :) [19:28] Project windmill-devel build #22: STILL FAILING in 1 hr 4 min: https://lpci.wedontsleep.org/job/windmill-devel/22/ [19:32] sinzui, right [19:33] sinzui, I'll try again. maybe it wasn't anything gaming related then ;) [19:33] I do not see your icon changing, My computer does not think you are talking [19:33] sinzui, logging off while I futz with settings again [19:33] flacoste: https://bugs.launchpad.net/launchpad/+bug/534203/comments/10 may interest you [19:33] <_mup_> Bug #534203: Timeouts on POFile:+filter (filter by person) < https://launchpad.net/bugs/534203 > [19:33] sinzui, and I've got solid red lips on my end. but the sound settings all seem ok on my end [19:33] flacoste: cold cache we're 60 seconds for the relevant query [19:33] flacoste: tis a miracle it works at all [19:34] flacoste: the most expedient solution would be main memory > db size for all db servers. [19:34] flacoste: this would scale badly because we have complete replicas, no partitioning [19:34] danilos: hi [19:35] danilos: are you still around? [19:36] lifeless: i'll ask charlie/elmo about what's the RAM situation nowadays with our DB servers [19:36] not sure we can get more [19:36] flacoste: I want to hear elmos voice when we ask about a 250GB upgrade. [19:36] flacoste: so it can wait for our biweekly ISP call :> [19:36] lifeless: :-) [19:37] flacoste: for clariry - I'm not recommending this option at this point [19:37] flacoste: just noting that we're starting to hit things which we can't reasonably expect to be hot given our 3:1 db:memory ratio [19:38] sinzui, I take it you can't hear me again [19:38] lifeless: on an unrelated note, we should address bug 297052 at some point [19:38] <_mup_> Bug #297052: Webservice requests never use a slave database because last-write time is unknown

< https://launchpad.net/bugs/297052 > [19:38] given that we have a 2:1 API to web ratio [19:39] flacoste: indeed; or change our scaling tech and obsolete that :>:> [19:39] now that we support read-only API requests, we might trivially send all of them to slaves [19:39] anonymous ones - for sure [19:39] not sure how much of our traffic is anonymous API though [19:40] deryck: https://dev.launchpad.net/Registry/ProjectReview [19:43] benji: thanks [19:43] my pleasure [19:44] jml: btw - subunit trunk -> py3 ok [20:00] Project windmill-db-devel build #233: STILL FAILING in 48 min: https://lpci.wedontsleep.org/job/windmill-db-devel/233/ [20:09] Project windmill-devel build #23: STILL FAILING in 41 min: https://lpci.wedontsleep.org/job/windmill-devel/23/ [20:11] lifeless: i think i'll leave the total time column in, i do use it for sorting to see what page are high-impact (hits * mean) [20:12] flacoste: high impact in what way [20:12] lifeless: free more resources, drop significantly our oops count [20:13] although the later could be get at by sorting on total hits on the timeout candidate report [20:13] and the first one may be dubious [20:13] flacoste: so, for oops count the profile of oopses in the oops summaries is more reliable [20:13] flacoste: e.g. questions rarely oops (though they do) even though they have 50 second long requests [20:14] (i do have the changes made on a local branch, just got doubts when about to submit it) [20:14] flacoste: as a metric of footprint on the system sum of time spent processing isn't unreasonable [20:15] flacoste: but it doesn't tell us much about the fat in the page [20:15] and less is more [20:15] flacoste: something like fat-in-page*hits would be an interesting order [20:15] so let's remove it [20:15] how would you assess fat-in-page? [20:16] I don't know [20:16] :) [20:16] uhm [20:17] high sql *count* would be a reasonable first metric. [20:17] so 99the percentile of sql query count * hits [20:18] as a broad proxy for inefficient + high use [20:19] (not 99th percentil of sql *time*, because things like bug search are not inefficient, they are slow backends - and as such not fat on page but need redesign/new systems [20:19] 99th% by time doesn't need the ht count multiplier to be useful I guess is my point [20:20] ok, i'll remove the total_time column [20:20] not sure about the real usefulness of the 'Fat-in-page' column, i'll add it and let's remove it later if it's doesn't see usage [20:22] ok! [20:23] it will be interesting to see if it lines up with the oops report [20:27] lifeless: actually, looking at it locally, it seems pretty useless :-/ [20:27] all the top-pages are high hits low sql statements [20:27] when I try to view a file on launchpad with a file_id param I get a redirect loop error - is this a launchpad or loggerhead bug? is there a report? [20:28] flacoste: with real data? [20:28]