[00:14] Can someone please throw https://code.launchpad.net/~wgrant/launchpad/bug-522800-getFileByName/+merge/43159 at EC2? Julian ran it overnight with one failure, which I've fixed. [00:56] Project devel build (286): STILL FAILING in 3 hr 12 min: https://hudson.wedontsleep.org/job/devel/286/ [00:56] Launchpad Patch Queue Manager: [bug=620458, [00:56] 629804] [r=gmb][ui=none][bug=629804] Back out hack to serve [00:56] restricted files directly from the internal restricted librarian port. === Edwin-afk2 is now known as EdwinGrubbs [02:12] === Top 10 Time Out Counts by Page ID === [02:12] Hard / Soft Page ID [02:12] 68 / 4938 Archive:+index [02:12] 44 / 0 Person:EntryResource:retractTeamMembership [02:12] 42 / 190 BugTask:+index [02:12] 37 / 307 Distribution:+bugs [02:12] 24 / 263 POFile:+translate [02:12] 14 / 18 DistroSeries:+queue [02:12] 11 / 367 Distribution:+bugtarget-portlet-bugfilters-stats [02:12] 11 / 39 Milestone:+index [02:12] 9 / 79 ProjectGroupSet:CollectionResource:#project_groups [02:12] 9 / 33 RootObject:+login [02:30] lifeless: Hmm, that's very odd. [02:59] i hope that's a bot not really robert :) [03:01] you only think there's a difference [03:02] :) [03:03] should I feel complemented or insulted [03:03] wgrant: whats odd abou tit [03:03] lifeless: So few Archive:+index timeouts. [03:04] wgrant: we've had the odd 'good' day [03:04] wgrant: the soft timeouts are pretty consistent - many K [03:04] lifeless: True. [03:04] * wgrant still needs someone to EC2 that regression-fix branch. [03:10] is it all gtg? commit message set etc etc etc [03:13] does "number of affected users" now total up across dupes? === almaisan-away is now known as al-maisan [08:39] good morning [09:19] Hola [09:33] ... and the award for the revision with the most reviewers goes to r10047. === jkakar_ is now known as jkakar === al-maisan is now known as almaisan-away [11:29] jml: hi, I'll be ready for our call in 5 minutes [11:30] bigjools: np [11:30] bigjools: I'm on the strategerie on mumble [11:30] jml: ok, I only just got back from my hospital appt. I now need drugs and coffee. [11:30] bigjools: understood. === Ursinha-afk is now known as Ursinha [12:11] Morning, all. [12:16] Yippie, build fixed! [12:16] Project devel build (287): FIXED in 3 hr 36 min: https://hudson.wedontsleep.org/job/devel/287/ === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha === almaisan-away is now known as al-maisan [14:15] * beuno waves at deryck [14:15] are but attachments on private bugs, private? [14:15] IIRC, there is such a thing as a private librarian === mrevell_ is now known as mrevell [14:17] hi beuno. was on call, sorry. [14:17] beuno, they are indeed truly private now. but that's a source of great pain at the moment. ;) :) [14:18] deryck, pain for whom? :) [14:18] beuno, private attachments aren't available over the api at the moment. accept for an exception made for apport and the retracers. [14:18] except, rather [14:18] deryck, ah, gotcha [14:19] I don't think we need to get them through the api [14:19] beuno, btw, I'm very supportive of the cdn idea and getting a yui loader going. willing to help rockstar as he works on it. [14:19] beuno, we need it desperately on lp [14:20] deryck, awesome, I'm going to push for something canoni-wide if it gets traction, otherwise build something for u1 and slowly push it out for other teams [14:21] beuno, yeah, we'll definitely take it up. the 1.3Mb js file is killing us now. [14:21] deryck, still carrying mochikit? === matsubara is now known as matsubara-lunch [14:23] beuno, yes, we are. but I don't think it adds much. yui itself is 1Mb. lazr-js is the other largest bit. [14:23] deryck, so, FWIW, I manually build a YUI combo file and brought it down to ~500kb [14:24] instead of bringing in all of the -min files in YUI [14:24] beuno, I had a little play at that and didn't have much luck. but I could be including something we really don't need. [14:24] I'm working on this over the next several weeks, so I look at that again. [14:24] deryck, well, what I did, is use their online combo generator [14:25] http://yui.yahooapis.com/combo?3.2.0/build/yui/yui-min.js&3.2.0/build/oop/oop-min.js&3.2.0/build/event-custom/event-custom-min.js&3.2.0/build/attribute/attribute-min.js&3.2.0/build/pluginhost/pluginhost-min.js&3.2.0/build/base/base-min.js&3.2.0/build/classnamemanager/classnamemanager-min.js&3.2.0/build/dom/dom-min.js&3.2.0/build/dom/dom-style-ie-min.js&3.2.0/build/event/event-min.js&3.2.0/build/node/node-min.js&3.2.0/build/widget/widget-min.j [14:25] oooooops! [14:25] sorry [14:25] that covered 95% of yui [14:26] 476kb [14:26] so I use that as the YUI file, and tack on the rest of our js [14:26] I have *no* idea why it differs so much from compressing it from the tree [14:28] you can play with it here: http://developer.yahoo.com/yui/3/configurator/ [14:30] deryck, could you pipe in on the list about the CDN as well then? [14:33] rockstar, I was thinking we could also just proxy yui's cdn [14:33] with https [14:34] beuno, maybe, but then we can't get to lazr-js or our own deps. [14:34] right [14:36] rockstar, yeah, I was planning to. === Ursinha is now known as Ursinha-lunch [14:43] deryck: how do I attach a bug to a series as opposed to a milestone in the UI? [14:43] EdwinGrubbs, target to release link. [14:43] help. I created a new error type and provides a useful message for users, but the page shows the errors docstring. the TALES is just "structure error" [14:43] thanks [14:43] EdwinGrubbs, np === Ursinha-lunch is now known as Ursinha [15:58] salgado: ping. [16:00] * deryck is getting excited about the BugJam next week [16:01] abentley_, hi [16:01] or abentley, hi [16:01] Ursinha, hi. === beuno is now known as beuno-lunch [16:01] jml, mumble? [16:02] sinzui: yes [16:02] abentley, about bug 688092, do you have an example of how qa-tagger failed when you tried running that locally? [16:02] <_mup_> Bug #688092: It's too hard to start hacking on qa-tagger [16:03] Ursinha, not offhand, but I can get one for you. [16:03] thanks abentley, that would be useful [16:03] when you create something it's hard to identify things you should explain to others [16:04] Ursinha, sure. I've been there. [16:06] hi jcsackett [16:06] hi salgado. i'm working on the distribution mirror prober, and it looks like you've been the busiest in it. [16:07] i'm wondering if you can help me figure out some issues i've introduced in trying to fix a bug. [16:07] bug 681034 [16:07] <_mup_> Bug #681034: UnknownURLScheme raised running distributionmirror-prober script [16:07] jcsackett, sure, I can try. although I haven't touched it in a while. can it be in 15'? [16:08] salgado: fifteen works just fine. gives me time to get stuff together so i can ask better questions. :-) [16:08] cool. :) [16:21] hen running a local launchpad instance (for bug import testing), I want to wipe it and start again. make schema has wiped most things, but I can't run the bug import script a second time - it claims the bugs have already been imported [16:21] http://launchpad.dev:58080/93/sB5xaCAHZcRjyaqsZrDH2sV7pCj.txt (the bug has already been imported) [16:23] jcsackett, I'm ready [16:23] salgado, awesome. [16:23] so, the problem in the bug is that on the machine being probed, there is a redirect, which we can't really do anything about. but it's causing an oops because unsupported url's trigger an exception. [16:24] i'm trying to change the prober to simply log the bad redirect so a human can look at it, and move on with triggering the oops. [16:24] salgado: this was my naive attempt: https://pastebin.canonical.com/40796/ [16:24] the problem is that the prober system is more complex than i expected, and i can't seem to find a way to do this that escapes the redirect method without having a timeout across the tests for the prober. :-( [16:25] so: short question: do you have any ideas of how to tackle this? sinzui suggested refactoring the prober to test redirection before actually doing it. i tried that as well, and simply caused further timeouts. [16:25] jcsackett, can I see the diff as well? [16:25] ah, yes, one sec salgado. [16:26] deryck: ^ [16:27] pcjc2, hey, there is a .pickle file somewhere in the tree you need to remove. I think it's called bugmap.pickle. [16:27] jelmer, thanks! :-) [16:27] ok, cool [16:28] bug-map.pickle [16:28] thanks so much [16:28] bzr st should turn it up. [16:28] right, that's it [16:28] salgado: https://pastebin.canonical.com/40800/ [16:28] need to ask some questions at some point about the possibility of munging inline "bug 1423312" from the old SF bug reports to match new LP numbers [16:29] (Would require changes of code on the LP import scripts), or possibly just have the format converter pick up on "Bug 412312", and turn it into a string which won't get auto-linked to a random LP bug) [16:29] <_mup_> Bug #412312: Implement Songbird support [16:29] salgado: my sense tells me the issue i'm running into is the callback stuff and the interaction between the prober and all the twisted machinery behind it. but that may just be suspicion born of ignorance. [16:31] pcjc2, you'll have to get any changes to the importer back upstream to us, since we'll actually run the import. Or sed your export file to change the form of the old link. [16:31] or old text, rather [16:31] I understand [16:32] Any changes which require mapping to new LP bug numbers will have to be within the import scripts run on the LP production servers. I'm just playing locally here to tweak the SF -> LP conversion as best I can without creating too much work for the LP admins [16:34] jcsackett, I haven't touched this code in a while, but I suspect the timeout is because upon an UnknownURLScheme exception you don't do anything that cancels the delayed call that triggers the timeout (e.g. connect() or fail()) [16:35] jcsackett, can you do a self._cancelTimeout(None) before your logger.error() call? [16:36] salgado: i think i tried that, but let me give it another go. [16:39] also, self.redirection_count will always be greater than 0 there [16:39] so you don't need the if/else [16:41] salgado: good to know. the tests, however, still show a timeout error. [16:42] jcsackett, ok, have you pushed your changes? that way I can merge and see what the failure looks like [16:42] salgado: sure, i'll push it up now. [16:47] salgado: https://code.launchpad.net/~jcsackett/launchpad/redirects-681034 [16:47] salgado: the test in question is the test in question is test_redirectawareprober_fail_on_unknown_scheme; right now that asserts that an unknown schema url exception is raised, and instead a timeout is raised. win condition is that fails because NO exception is raised. then i will figure out testing the logging. [16:49] jcsackett, I think I know what's going on [16:50] salgado: lay it on me. do not spare my ego. :-P [16:50] unlike I previously thought, just cancelling the timeout is not enough [16:50] you have to callback() on the deferred (or errback()) [16:51] so you do have to fail(e) on UnknownURLScheme [16:51] and on a layer above that you can catch the exception and not raise an OOPS [16:51] hm. [16:52] jcsackett, but we already have support for that. see ArchiveMirrorProberCallbacks [16:52] it does a failure.trap(*self.expected_failures) [16:52] salgado: ah yes, i see. [16:53] at first I thought you could add UnknownURLScheme to expected_failures [16:53] but that's not a good idea [16:53] no, i would need to create a new excpetion. [16:53] because I think we want an OOPS raised when that exception is raised on the first connect() but not when it's raised in a redirect() [16:54] so, yes, a new exception which you pass on to self.fail() on redirect() [16:54] salgado: it doesn't look like ArchiveMirror uses the RedirectAware prober...the only thing using redirect is MirrorCDImageProberCallbacks [16:54] salgado: that doesn't seem right to me, does it make sense to you? [16:55] that is right [16:55] # According to http://lists.debian.org/deity/2001/10/msg00046.html, [16:55] # apt intentionally handles only '200 OK' responses, so we do the [16:55] # same here. [16:56] I think that has changed recently, but it was intentional that the archive prober doesn't use the redirect-aware factory [16:56] s/that/apt [16:57] salgado: ah. so then the assumption in the bug about it being redirect isn't necessarily accurate, because the RedirectAware prober wouldn't have been involved. [16:57] it was not an archive mirror that caused the OOPS [16:57] https://server4.bremer-it.de/ubuntu///maverick/ubuntu-10.10-server-i386.iso [16:57] that's a CD image mirror [16:58] and the traceback shows it was on redirect() [16:58] * jcsackett didn't understand what CD image mirror was about. i get it now. [16:59] salgado: okay, so basically i can crib from ArchiveMirror to handle expected errors, and add that to CDImageMirror stuff. [16:59] oh, of course. now I see how I mislead you [16:59] yes, that's correct [17:00] MirrorCDImageProberCallbacks is the one you want to change [17:00] * jcsackett nods. [17:00] salgado: thanks so much! [17:01] jcsackett, don't thank me before confirming it actually works. ;) [17:01] salgado: fair. :-) [17:01] flacoste: ping [17:01] hi flacoste [17:02] * flacoste talks to himself [17:02] jml: hi [17:02] flacoste: mumble! === deryck is now known as deryck[lunch] === beuno-lunch is now known as beuno === salgado is now known as salgado-brb [17:19] benji: do you know if there is a way to override devmode without changing the launchpad.conf or creating a new config directory specified by the LPCONFIG env var? [17:20] EdwinGrubbs: hmm, let me see [17:33] EdwinGrubbs: I don't see a way. We could fairly easily add command-line config overrides to bin/run if needed. [17:34] benji: do you want me to open a bug for that? [17:35] EdwinGrubbs: if it's something you need, sure! === benji is now known as benji-lunch === deryck[lunch] is now known as deryck [18:06] g'bye all. See you again on the 20th. [18:08] deryck: ping [18:11] hi EdwinGrubbs [18:15] deryck: I'm working on the milestone +index page for distros, projects, and project groups. These pages list the bugs assigned to the milestone that do not have a conjoined master. For project groups, it's possible that a bug is on a milestone one of the projects and on the devfocus series for another project? Would this count as having a conjoined master? The old code did not count it, but my optimization started doing it as a [18:15] side effect. [18:16] * deryck is reading, processing that.... [18:16] EdwinGrubbs, I honestly don't know if that counts as a conjoined master. It shouldn't as I understand conjoined bugtasks, but there may be a bug there. [18:17] EdwinGrubbs, or do you mean "is it okay if they're conjoined after this work?" [18:18] deryck: I really don't understand the workflow around this to know why we don't want bugs attached to the devfocus series to show up on the milestone page. yes, I mean, "is it ok?" === salgado-brb is now known as salgado [18:18] jcsackett, did it work? [18:19] salgado: seems to. i'm refactoring some code to properly trap things and cleanup some other details, but this seems to be the right path. [18:19] cool! [18:19] sorry i didn't ping you earlier; got wrapped up in pursuit of the solution. :-) [18:19] thanks again! [18:20] np; glad it worked [18:20] EdwinGrubbs, I don't know why we wouldn't want it either. I think the bug should appear if a task has linked the milestone.... [18:21] EdwinGrubbs, as to your issue, are the two projects on different tasks on the same bug? [18:21] deryck: yes, so the bug would show up on the milestone page for one of the projects but not for the other one. [18:22] EdwinGrubbs, ah, that's fine with me. [18:24] deryck: so, you don't think anyone will complain that milestone X on one of the projects has the bug, but milestone X on the project group doesn't because a differ project has it linked to the devfocus series? [18:25] EdwinGrubbs, ah, right. yes, that would be wrong. hard to get my head around it. [18:26] deryck: ok, that's fine. [18:28] * deryck will brb, rebooting === al-maisan is now known as almaisan-away === benji-lunch is now known as benji === abentley_ is now known as abentley === matsubara is now known as matsubara-brb === matsubara-brb is now known as matsubara === matsubara is now known as matsubara-afk === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha [23:26] mbarnett: still around? === Ursinha is now known as Ursinha-afk