[00:03] wgrant: can you hear me? [00:03] wallyworld_: No. [00:03] bugger [00:04] sidnei: here but my mike is broken [00:04] sinzui: ^^^^ [00:05] wallyworld_: I had to mute/unmute from sound preferences this morning to make my mic really work [00:09] sinzui: wgrant: skype works, but not mumble :-( [00:19] sigh, what i really want is just to be able to attach comments ajaxily to the wiki page [00:24] mwhudson: that would rock [00:25] maybe we can make thumper add that to wikkid :) [00:25] sup? [00:25] attach comments? [00:28] I think WYSIWYG editing without bring up the full edit widget [00:28] e.g. change one paragraph only [00:28] or add an annotation to the text [00:31] wallyworld_: Hi. [00:32] wgrant: hello [00:32] why does my brain always pop up 'cucumber' when i'm really trying to remember 'celery' [00:32] wallyworld_: Can you QA your recipe build emails thing on staging? [00:34] wgrant: yes. just going through my emails now and saw that one is still todo [00:34] I'm doing the gpghandler thing. [00:34] wgrant: not sure how to qa it though [00:35] mwhudson: because they are both terrible project names [00:35] lifeless: +1 [00:35] wallyworld_: Create one successful build, one depwait build, see what happens? [00:36] wgrant: that's the point. i'm not sure how to create a buidl with a dep issue [00:36] wallyworld_: Branch an existing package, add 'fewfwefwefwew' to the Build-Depends field, build. [00:37] wgrant: ok. i'm a total packaging noob you realise :-) [00:37] wgrant: on qastaging, i assume there are logs we can look at to see what email would have been sent [00:37] wallyworld_: You can check the staging mailbox. [00:38] Have you accessed that before? [00:38] nope :-( [00:38] Let's see if I can find the wiki page. [00:39] if it's on the wiki, i'll find it [00:39] https://wiki.canonical.com/InformationInfrastructure/OSA/LPHowTo/ConnectToStagingMailbox [00:39] But the password says to ask a LOSA. [00:39] That's false. [00:41] excellent thanks [00:42] You will need a reasonable mail client. [00:42] The mailbox can be several hundred thousand messages. [00:47] wgrant: can you send me the password? what do you consider to be a reasonable client? [00:48] wallyworld_: I PM'd it to you internally. [00:48] wallyworld_: I don't know how Kmail is these days. [00:48] Last time I tried it with IMAP it was rather segfaulty. [00:49] wgrant: ah thanks. didn;t see the pm, sorry [00:53] lifeless: dammit email half-written and i need to go [00:54] will finish later :) [00:54] mwhudson: no worries [00:56] wgrant: well i'm trying kmail now. thunderbird refuses to connect (could be because i'm using a recent (buggy) snapshot to get better unity integration) [01:00] qa-ok! [01:00] It is poppy-fixing time. [01:00] Well, once asuka gets its act together. [01:02] Oh, the mailbox is tiny now. [01:02] I guess because question emails aren't being sent. [01:14] bugger, qastaging is down [01:15] It'll be back in a couple of minutes. [01:16] There we go. [01:16] Just took a while to update WADL. [01:16] cool [01:16] Hm. [01:16] Not actually back yet. [01:16] But the appserver is started. [01:17] There. [01:25] How interesting... [01:26] you can be logged into launchpad.net as one user and login.launchpad.net as another [01:26] which for those not familiar with login.launchpad.net is very confusing when trying to access branches via codebrowse [01:27] an openid feature [01:27] :> [01:27] hmm... but logging out of login.launchpad.net and logging in as a new user doesn't fix it [01:27] how do I 'logout' of codebrowse? [01:28] there is a logout button on the launchpad site [01:28] use it [01:28] but I'm logged in as the correct user on launchpad already? I have to relogin again? [01:29] The launchpad.net logout button will you log you out of both launchpad.net and bazaar.launchpad.net. [01:29] (by a hideous redirect via bazaar.launchpad.net) [01:30] ah [01:30] k [01:34] #@%!@%@!&^ qastaging. every url i navigate to times out :-( [01:36] Probably bug heat. [01:40] wgrant: i'm having trouble figuring out how to get to a page that let's me see (and edit) the Build-Depends field for any particular package. can you point me in the right direction? [01:41] wallyworld_: It's in the branch. [01:42] wallyworld_: Edit debian/control. [01:42] .me looks [01:43] wallyworld_: Hm, nevermind, staging is too old anyway. [01:43] We'll have to do it on mawson. [01:44] wgrant: by "the branch", do you mean code.qas.lp.net/project or code.qas.lp.net/ubuntu/+source/xxxx or ?? [01:44] wallyworld_: It needs to be a branch with packaging metadata. lp:ubuntu/hello, for example. [01:48] wgrant: so i go to lp.net/ubuntu, search for a project eg firefox, and then go to lp.net/+source/firefox. is that correct? [01:49] wallyworld_: Easiest way to get the branch is just 'bzr branch lp:ubuntu/hello' [01:50] wgrant: fair enough. i was curious though how to find it via the gui. [01:50] I don't know how to search for packages effectively. [01:50] I always go straight to https://launchpad.net/ubuntu/+source/SOMESOURCE [01:52] wgrant: right. i got to that by seaching from lp.net/ubuntu. it seems to work well if you know the project you are looking for [01:53] wgrant: so mawsom = dogfood right? but do we have access to it's mailbox? [01:54] wallyworld_: It sends mail to the staging mailbox. [01:54] ah ok [02:07] wallyworld_: I've scheduled a few builds of various sorts. [02:07] Will probably take like half an hour to build them. [02:07] At least. [02:07] rubidium is slow :( [02:08] wgrant: ok. i was just messing with a local branch of ubuntu/hello [02:08] i'll see what turns up in the inbox [02:18] Subject: [DOGFOOD] [recipe build #36909] of ~wgrant hello-lololol-daily in natty: Dependency wait [02:18] <_mup_> Bug #36909: clicking on bug filters does nothing < https://launchpad.net/bugs/36909 > [02:18] wallyworld_: ^^ [02:19] wgrant: looks like it worked :-) [02:19] thanks [02:20] And there was no notification from the succesful build. [02:20] So we are good. === StevenK changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: StevenK | https://code.launchpad.net/launchpad-project/+activereviews [04:03] wgrant: When you're back, can you look at https://code.launchpad.net/~stevenk/launchpad/copies-use-overrides/+merge/61195 again? I think I've addressed almost all of your concerns. [04:05] ಠ_ಠ massive commits again [04:06] StevenK: have a moment to look at a couple of my merge proposals? [04:07] mwhudson: thanks for that feedback [04:07] good questions [04:09] oh good, i was worrying i rambled a bit :)( [04:12] You rambled a bit, but it was still excellent :) [04:13] Anything like this is bound to be rambly. [04:16] cjohnston: Sure, link me up, and I'll review them after I have reviewed my lunch. [04:17] StevenK: Reviewed. [04:18] wgrant: ZCML does not like inheritence [04:19] I played with it last night [04:19] StevenK: Upsetting. [04:20] Still, IBaseOverridePolicy -> IOverridePolicy. [04:20] wgrant: No argument from me about that [04:25] StevenK: https://code.launchpad.net/~chrisjohnston/launchpad/728192/+merge/61046 https://code.launchpad.net/~chrisjohnston/launchpad/627628/+merge/61055 [04:27] https://code.launchpad.net/~chrisjohnston/launchpad/197793/+merge/61053 === jtv changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: StevenK, jtv | https://code.launchpad.net/launchpad-project/+activereviews [04:31] hi wallyworld_, I'm in [04:34] 196 Critical bugs [04:34] Yay [04:39] wgrant: less talking, more fixing or you don't get fed [04:39] But archiveuploader killed my cat. [04:41] Okay, you've got me. How? [04:43] jtv: hello [04:43] jtv: hi [04:43] hi [04:43] jtv: I saw your comment yesterday in irc you were offline by the time I saw it [04:43] cool, thanks—I was just reading the email thread [04:48] cjohnston: All of those MPs are Approved by Henning? [04:51] StevenK: thats correct [04:51] cjohnston: So why do you need me? :-) [04:52] He hasn't done anything with them since approving them, so I wasn't sure if he wasn't able to merge or something [04:53] cjohnston: Ah. I'll approve all 3 and toss them at ec2 [04:53] cjohnston: If you want to set a commit message on all three? [04:56] done [04:57] cjohnston: The commit message on 728192 isn't really clear. Perhaps "IArchive:+index no longer has a form in a heading." ? [04:58] ok [05:02] statik: thats quite an endorsement for the proposal.. thanks [05:02] oh no, my commit had a build failure :( [05:02] nigelb: An ec2 failure? [05:03] That's a "Test results: FAILURE" email. [05:03] wgrant: Test results: 645825-ui-example => devel: FAILURE [05:03] cjohnston: Will you be around in roughly four hours? [05:03] wgrant: yeah, test failure [05:04] StevenK: no.. I need to go to bed soon. :-/ [05:04] nigelb: How many errors? [05:04] 6 errors [05:04] Is the problem clear? [05:05] cjohnston: Fair enough -- all three of those branches are currently having ec2 instances created for them, you'll get e-mails from them in about four hours telling you what happened. [05:05] cool... t [05:05] t [05:05] y [05:05] Haha [05:05] wgrant: looks like "AttributeError: 'TestProductSeriesHelp' object has no attribute 'oopses'"\ [05:06] I've seen that before ... [05:06] Er [05:06] that could be a setUp method failing to upcall [05:06] I've *not* seen that before [05:06] want a full paste of the entire mail? [05:06] I didn't have a setUp method. [05:06] nigelb: That would be helpful. [05:06] wgrant: Can you also glance at https://code.launchpad.net/~rvb/launchpad/distro-overlay2-bug-758908-dependencies/+merge/61110 please? [05:07] wgrant: http://paste.ubuntu.com/609335/ [05:08] Suspcious [05:08] I have seen this before. It was something fairly obscure. [05:08] But I cannot recall the details. [05:09] Right, running ec2 land & in a for loop was a stupid idea. [05:10] a for loop? [05:10] Oh. [05:10] A shell for loop. for i in foo bar ; do echo $i ; done [05:10] nigelb: You call your superclass' setUp method in your test method. [05:10] That's *not* going to go well. [05:10] 28+ super(TestProductSeriesHelp, self).setUp() [05:10] Delete that line. [05:11] wgrant: ah [05:12] mwhudson: ah, a setUp method failing to upcall. Now I get what you meant. [05:12] wgrant: commit and push again? [05:12] nigelb: Right. [05:12] In this case it was setUp doubly upcalling, so the OOPS handler was registered twice. [05:12] Then tearDown unregistered it only once. [05:13] ah so my crystal ball was only a little cloudy [05:16] lifeless: a concern about service fakes... you mention running the real service's tests against the fake. But the gains have to come from somewhere; in the FakeLibrarian for instance we had to sacrifice multi-process access. Won't a single canonical fake for a service imply something very close to the real thing? [05:20] lifeless: (or someone else smarter than /me): there's a bug report with an Object NotFound oops for urls of the form "translations.lp.net/inactiveproject" and "bugs.lp.net/inactiveproject" yet when I test that on lp.net the page renders fine. any idea why? [05:20] wallyworld_: ~registry can see disabled projects. [05:21] wgrant: pushed again --> https://code.launchpad.net/~nigelbabu/launchpad/645825-ui-example/+merge/61175 [05:21] wgrant: oh ok. so on the page where such links are rendered, we may want to see if the user is a member of ~registry before we decide not to print the links? [05:22] of do we just want to not print them in all cases? [05:22] wallyworld_: I'm not sure that there's value in showing them to ~registry. [05:22] easier to do the latte [05:22] latter [05:22] * jtv goes for a latte [05:22] cool. so i'll take the easy road :-) [05:23] jtv: 2 sugars for mine please [05:25] wallyworld_: here you go → ☕ [05:25] lol [05:25] but where's the sugar? [05:28] hehe [05:33] wgrant: will you be landing it in ec2? [05:33] nigelb: I will. [05:33] wgrant: Thanks :) [05:41] nigelb: The instance is running now. [05:41] wgrant: great, thanks :) [05:43] aha, UDS photos are up. There's a lot of photos of that guy with blue hair :p [05:43] I haven't been able to find one yet. [05:43] Do you have a link? [05:43] I went looking last week. [05:43] But failed. [05:44] http://photos.pixoulphotography.com/Events/UDS-Oneiric/17103699_kzzLF6/1/1296058796_HdrLgVm#1296058440_N4fmvNv [05:44] wgrant: I have a bunch in my fb album, but these are much better [05:45] Subtle. [05:45] * mwhudson goes looking for the pictures of beuno harassing charlieS [05:45] haha [05:47] it was more like stalking [05:47] Oh? [05:48] beuno: once you'd pinned in the corner with your rapidly waving hands, it wasn' [05:48] beuno: once you'd pinned in the corner with your rapidly waving hands, it wasn't stalking any more :) [05:48] true [05:48] pinned HIM [05:48] i should give up on typing for today [05:49] can't find any photos of that anyway [05:49] it started off as stalking, then came the beer [05:49] * StevenK grumbles at publishBinaries [05:49] StevenK: Hm? [05:49] It fit its original purpose very well. [05:50] It's ignoring the component I'm passing it [05:50] It probably overrides it to main if it's a PPA. [05:50] (Which it isn't) [05:50] That is probably no longer appropriate. [05:50] It probably is. [05:50] What's it doing? [05:51] The copied binaries continue to keep the original binaries component. [05:51] I doubt that's publishBinaries' fault. [05:51] Unless you've touched it in bad ways. [05:52] * StevenK dumps the query it runs [05:52] Bleh, which doesn't help [05:53] It's a very pretty query :) [05:54] Project windmill-db-devel build #288: STILL FAILING in 1 hr 10 min: https://lpci.wedontsleep.org/job/windmill-db-devel/288/ [05:57] jtv: much of the cost of real services from integrity or encryption [05:57] jtv: e.g. fsync, transactions, multiuser syncronisation [05:57] jtv: a network fake doesn't need to do any of those, can pretend to do gpg etc [05:57] lifeless: but then we'd still have to exempt the fakes from those aspects of testing, right? Maybe separate "interface" tests from "quality of implementation" tests? [05:57] It's so comforting that windmill continues to utterly fail on Jenkins [05:58] StevenK: They're all bitrot failures. [05:58] 7 or so are easy to fix. [05:58] yes, but then how can you tell if a db fsyncs something *when you're talking over the network to it* ? [05:59] so the external contract really can't talk about those aspects anyway [05:59] Ah, so that's already not a part of the service's tests as such. [05:59] we'd want 3 sets of tests of a service [05:59] external contract [05:59] details-of-the-fake-test-support-api [05:59] wgrant: :-( [05:59] details/internals-of-the-real-version [05:59] this is what bzr has [06:00] and it works very very well [06:00] wgrant: And publishBinaries() was doing the right thing after all [06:00] StevenK: Of course. [06:00] the external contract tests run against both implementations [06:00] the other two sets against only one [06:00] * StevenK bites back a barb [06:05] wgrant: And I didn't address the arch-indep bit because I have no idea how to [06:08] Hm, perhaps that will work [06:15] wgrant: I can't have cBO just return the archtag passed in? [06:16] StevenK: You can. [06:16] If you can reliably map back to it. [06:16] The bother is returning None if it's arch-indep [06:17] When I can't trust the destination, and all I have is the BPN [06:17] I'm able to deal in copyBinariesTo(), since I have the source BPPH, and that is trust-worthy [06:17] Right. You may recall that I suggested a strategy last week to overcome that. [06:18] Joining against DAS. [06:19] wgrant: Right, and that works fine for returning the archtag, but it won't return None if it's arch-indep. [06:19] StevenK: It will if you make it return None. [06:20] wgrant: Uh? How can I make Storm do that? [06:20] StevenK: If it's arch indep, add a DistroArchSeries.id == None constraint. [06:21] Hmm, although that may need to be in a join condition. Difficult. [06:21] And calculate_target_das() already tosses back the NAI [06:23] So? [06:24] It isn't making any sense how to implement this. [06:24] You constrain BPPH.distroarchseries to NAI. [06:24] But you don't actually join against DistroArchSeries yet. [06:25] I suggest that you do a left join against DistroArchSeries such that you get the archtag back for arch-dep, and None back for arch-indep. [06:26] make_package_condition() already does BPPH.distroseries == das [06:26] Yes, but it doesn't join against DAS. [06:26] It just compares the fk with a precalculated value. [06:33] lifeless: Could you please try it on another browser, then? [06:34] It is not a bug unless it occurs on browsers that don't have unstable daily build warnings on them. [06:42] FROM LEFT JOIN ... WTF, Storm?! [06:45] jtv: translations question concerning recent activities on a person's translation page (getTranslationHistory). i'm trying to find a test for this. no luck so far, just about finished running all tests under translations/*. i want to find a test which calls into ActivityDescriptor. But more to the point, i need to find a way to filter getTranslationHistory so that it ignores POFiles associated with inactive projects. if i can [06:45] navigate to the pofile sourcepackage->product attribute i may be able to check for is_active but the product attribute desc says "The best guess we have as to the Launchpad Project...". Best guess? Is there a definitive association? [06:51] wgrant: I'm clearly missing something vital: http://pastebin.ubuntu.com/609354/ [06:53] StevenK: You have a left join there. [06:53] But you're not joining onto anything. [06:53] You're just joining DistroArchSeries onto nothing. [06:54] So I need BPPH, DAS, condition? [06:54] Roughly. [07:25] wallyworld_: use POFile.potemplate to get to the POTemplate. From there it's either POTemplate.product, or POTemplate.{distroseries, sourcepackagename}. [07:29] jtv: thanks. that should be doable in storm. any idea why there appears to be no test coverage for the recent activies stuff on the view? there's a doc test for person.translation_history but nothing for the rendering of that info onto the page. or am i missing something? [07:29] wallyworld_: I was almost certain there was. [07:30] jtv: yeah, i would have thought so. but a complete run of all translations/* tests didn't hit my breakpoint :-( [07:31] wallyworld_: let me just look at the code you're talking about... [07:32] jtv: thanks. ideally i wanted to write or extend an existing test and then do the code [07:33] Naturally. [07:34] wgrant: ajax log? [07:36] lifeless: Check what the vocab AJAX request returns. [07:36] wgrant: I think I have the join working. But how to get it to return None? [07:36] StevenK: That's where my plan falls apart slightly. You may need to have it in the join condition, not the where clause. [07:36] 'cause you don't get a NULL row if there are any matches. [07:37] Always disliked that inconsistency. [07:37] wgrant: But the DAS itself is in *candidates? [07:37] wallyworld_: can't find it either—and I'm pretty sure I spent quite some time writing up tests for this in Barcelona. [07:37] And is already the NAI by that time anyway [07:37] jtv: well that sucks balls :-( [07:37] wallyworld_: hope I didn't end up forgetting to "bzr add" the tests or something... [07:38] jtv: so you're off to check your hard drive? :-) [07:38] Yes, it sucks balls and circumstances were difficult due to time pressure etc. [07:38] for uncommitted branches? [07:39] Nope, it's not on this hard drive, the backup drive this would be on is probably one that died, or maybe it's on the predecessor which also died, and the machine itself is behind a UPS that had a nervous breakdown and I still have to figure out whether it's really the power or the UPS itself. [07:39] So not much chance of that. :( [07:41] wallyworld_: we could probably dig up the MP, but it's not likely to be worth the trouble. I fear this will take new tests. [07:42] * wallyworld_ sobs siliently [07:43] Can you sob _re_siliently? That'll help. [07:43] * wallyworld_ can't type [07:46] jtv: there's a request from danilo up on lps, is that still relevant do you know? [07:46] It is. [07:46] He said was going to get to that when he was on maintenance. [07:46] Which is next week. [07:46] ah k, we'll let it ride for now then; ta [07:47] wgrant: I still think I'm missing something. http://pastebin.ubuntu.com/609367/ [07:50] gmb: nigelb bought up the failures here, and wgrant helped him fix it. [07:50] gmb: It's currently running through ec2 again. [07:57] gmb: I did remove the super().setUp and wgrant landed it in ec2 again [07:57] It's probably got another 3 hours to go [07:58] * StevenK prods wgrant more [07:58] how many hours does it take anyway? o.O [07:58] nigelb-work: About 4 [07:58] ~4h [07:58] so, its about half way through by now I guess [07:58] $ utilities/ec2 list [07:58] 645825-ui-example => devel [OK] (up for 2:21:13.892062) [07:58] Summary: running: 1 [07:58] Yup. [07:59] btw, there were one or two easy bugs I thought could be just closed off [08:00] like there was a bug which said, If LP is linking to bugs not existing and showing a tool tip that it does not exist, then it shouldn't link the bug at all [08:00] But I don't think we show any such tooltip anymore, at least I didn't see it. [08:03] that's bug 4595 (yeah, OLD) [08:03] <_mup_> Bug #4595: Don't auto-linkify non-existent bug reports < https://launchpad.net/bugs/4595 > [08:05] and another one, bug 337848, I can't find examples of what's mentioned in the bug report [08:05] <_mup_> Bug #337848: IRC links on profile page are asked the wrong way < https://launchpad.net/bugs/337848 > [08:07] lifeless: Hah. [08:07] Have you considered that 123 might be a few too many teams? [08:07] I wonder what they all are. [08:07] wgrant: no [08:07] I was only in 95. [08:07] Greated 25 new ones, fine. [08:07] poor form [08:07] Created one more. [08:07] Boom. [08:07] undefined. [08:07] Does not like more than 20 pages, apparently. [08:08] ha ha ha ha ha ha ha [08:08] wgrant: browser version my shiny metal .... [08:08] * StevenK gives up on arch-indep [08:08] Too hard [08:08] lifeless: Nobody else is insane enough to have that many teams :) [08:08] wgrant: many folk are [08:08] And it hadn't been seen outside a daily build of a browser... [08:09] wgrant: Are you going to push back on the arch-indep thing? [08:09] StevenK: Not if you can prove to me that it's OK. [08:09] And the tests don't do that? [08:09] No, those tests don't cover much at all. [08:10] They're the 3 use cases we discussed? [08:10] They are the three basic cases, yse. [08:11] * StevenK pushes up 5 revs [08:11] !! [08:11] lifeless: There are 42 people with 120 or more teamparticipations. [08:11] Close enough to nobody else :P [08:12] wgrant: I'm not even top 10 [08:12] Yes, but your Person.id is 2, so all suggestions of exceptionality are valid and reasonable. [08:13] who's 1? [08:13] sabdfl? [08:13] Yes. [08:13] yes [08:13] sabdfl is #14 [08:14] lifeless #24 [08:14] Not good enough. [08:14] wgrant: #9 [08:14] for sabdfl [08:14] "SELECT person.name, COUNT(*) FROM teamparticipation JOIN person ON person.id = teamparticipation.person GROUP BY person.name HAVING COUNT(*) > 120 ORDER BY COUNT(*) DESC;" was what I did. [08:15] yeah [08:15] 149 [08:15] is what I see [08:15] wgrant: There, 5 more revisions on the MP. Be shocked. [08:15] StevenK: I am shocked and saddened. [08:15] Saddened? [08:15] Saddened. [08:16] Why? [08:16] sabdfl | 152 [08:16] StevenK: I will need to find something else to harrass you about. [08:16] If you've started committing regularly. [08:16] Haven't you harrassed me enough about this branch? [08:16] It's a very dangerous branch. [08:16] So no. [08:16] My frustration is peaking [08:17] In its current state it will break pocket copies in at least two ways. [08:17] Your frustration may peak, but it still won't land :) [08:18] I'm willing to state that it needs more tests, but if you're just going to say "It's dangerous" and not point out which particular cases need more care, then I'm just going to get more frustrated. [08:18] I pointed out some of the cases in my first comment. [08:18] And reviewing a branch isn't supposed to be a battle of wills. [08:19] It's not a battle of wills. [08:19] So, my worries are: [08:19] 1) This is now used in copies, forrealz. [08:19] At the moment it feels like it. [08:20] 2) It only finds overrides within the pocket, which means that pocket-copies eg. from -proposed to -updates will end up in the wrong place. [08:21] # When we copy source/binaries from one pocket to another, their [08:21] # overrides are respected. [08:21] 3) I don't know how well it's going to handle partner and copy archives and stuff. [08:21] ^ That is the case you're concerned about? [08:22] 4) This changes a lot and could have severe consequences on several untested processes, so no caution is undue. [08:22] "their overrides are respected" is unclear, but that could be it. [08:24] wgrant: I don't care about landing, I care about personally feeling that I'm making progress. At the moment it feels like The Immovable Object versus The Unstoppable Force [08:25] You are making progress. But this is a big change with a lot of cases that need to be covered before it's safe. [08:26] # When we copy source/binaries from one pocket to another, the [08:26] # overrides are unchanged from the source publication overrides. [08:26] I have made specific statements about the things that concern me. [08:26] StevenK: It may be better to phrase (and code) that as just picking the latest overrides, regardless of pocket. [08:27] Project devel build #728: FAILURE in 14 min: https://lpci.wedontsleep.org/job/devel/728/ [08:27] Uh oh [08:28] bzrlib.errors.InvalidHttpResponse: Invalid http response for https://xmlrpc.launchpad.net/bazaar/: Unable to handle http code 502: Bad Gateway [08:28] That's not meant to happen when there's no rollout going on. [08:28] its not meant to happen period [08:28] What does it mean if a derived distro is an overlay over a parent distro ? [08:29] lifeless: Wasn't it an accepted downside of the current nodowntime process? [08:29] stub: It inherents all of the parent's packages, except for those are explicity published in it. [08:29] stub: Think PPA, except for Distributions [08:29] stub: The derived series doesn't have the parent's packages in its own archive. Its builds' sources.lists refer to the parent series' archive. [08:29] Like PPAs, right. [08:30] wgrant: interrupted requests yes, bad gateways no [08:30] lifeless: Isn't that a bad gateway? [08:30] StevenK: So what does it means if it doesn't have that set? I had assumed that behaviour for derived distros. [08:30] lifeless: It's not a 503 or a 504, so it's probably a 502. [08:30] stub: If it isn't an overlay, it is just a derived series [08:30] Why would you have a derived distro that doesn't inherit the parents packages? [08:30] stub: Because you only want a subset [08:31] For instance [08:31] So you would have to manually specify all the packages to pull in? Whitelist rather than blacklist? [08:31] Yes [08:31] ok. [08:31] stub: debian -> ubuntu is non-overlay [08:31] stub: ubuntu -> oem is overlay [08:31] I see. [08:32] with overlays clients have both archives in their sources.list [08:32] with non-overlay they have only the derived distro [08:33] Will a PPA become a non-overlay derived distro? [08:34] An overlay derived distro. [08:34] With multiple parents too, whee [08:34] a single parent overlay derived distro is roughly equivalent to a PPA [08:35] but with better management of packages [08:35] Cool. So we could, say make our own personal Ubuntu variant overlay ubuntu, overlay the Launchpad PPA, Bazaar PPA etc. [08:35] it should probably nuke all ppas in favour of such overlays eventually [08:35] lifeless: That's Julian's intention. [08:35] It's also a long-term goal [08:36] YouBuntu :-) [08:36] stubuntu === StevenK changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: jtv | https://code.launchpad.net/launchpad-project/+activereviews [08:39] good morning [08:39] Hi adeuring! ;)+ [08:40] s/+// [08:40] moin henninge [08:43] So I just approved a patch to the DistroSeriesParent table, which is not yet in use I think. The new columns - did we not realize they were needed when we initially created the table, or do we need to tweak something to avoid unnecessary landings on db-devel ? [08:44] stub: Julian rushed the DistroSeriesParent creation [08:44] Effectively [08:45] stub: http://www.readwriteweb.com/cloud/2011/04/5-graph-databases-to-consider.php [08:45] stub: I think the whole parent/child overlay thing was not completely thought through when the DSP table was created. [08:45] k. As long as everyone is aware we need to do the db stuff upfront when possible. Was wondering if two people got tasked with different parts of the same feature and we ended up with two patches in different cycles because of that. [08:46] Right. [08:46] The requirements for multiple parents and all the rest weren't discovered until we'd already started developing. [08:46] The feature's design got a bit hectic as a result, and we have some grand plans for simplifying it but we can't afford to do that now. [08:46] Yeah, I was expecting we learnt they were needed part way through. [08:46] I don't think we can avoid that entirely. [08:48] lifeless: Might want that sooner than later - losas raising disk space warnings again and we can't rely on the SAN arriving in time for that. [08:48] stub: what service are they raising that on ? [08:48] wildcherry, but all [08:48] all three db nodes [08:48] ah [08:49] where are we at on the branchrevision shrinkage [08:49] chokecherry has space, but two copies of the database. If we ran out now, I'd need to switch to single master running on chokecherry. [08:50] How can I get bzr shelve to show what is stored in an id? [08:50] bzr unshelve --preview ID [08:51] lifeless, jtv: I think we are down to one remaining glitch in hacking the internal tables to drop the id column. There might be a redundant index we can drop right now. [08:51] That'd be nice. [08:51] What's the glitch? [08:52] lifeless, jtv: If we want, we might be able to punt this to 3rd party support and let them chase through the hidden dependencies in the catalog. [08:52] how much space are we expecting to save [08:52] stub: sounds good... may send them into a panic though. [08:52] lifeless: order of 20GB last I looked. [08:52] if its enough to save us, then we should escalate it, yes. [08:52] jtv: IIRC the index we are using as the new primary key is still linked to a unique constraint on the table. That needs to be broken. [08:53] jtv: We end up with a database schema that can be dumped, but the restore fails. [08:53] win [08:54] jtv: https://code.launchpad.net/~stub/launchpad/drop-branchrevision-id/+merge/55480 [08:54] lifeless: I think I'd get them to go over it anyway - more eyes, less chance of blowing off our foot. [08:54] in this case, entire leg [08:54] :) [08:55] stub: any particular part you'd like me to look at? [08:56] jtv: If you have time, run that patch, do a dump restore, and see if the problem that demonstrates is obvious to fix. I think you are more familiar with these internals as you have talked it over more with the core devs. Maybe it is just a silly mistake I introduced working from your notes? [08:56] stub: Your patch number is 62, but you insert 99 into the DB [08:56] stub: I don't have that much time right now :( Especially with the holidays I'm way behind on Feature stuff. [08:57] StevenK: There is also deliberate syntax error in there - I don't want this to accidentally land anywhere :) [08:58] stub: Heh, I missed that [09:01] lifeless: We can also consider switching the data partition to RAID 5 for the extra space. Need to benchmark on similar hardware to see if that copes with our current write load (I suspect it would). [09:02] I seem to remember a few GB we can save by repacking some indices [09:03] I'll also talk with elmo about options on thursday [09:05] neo4j.org does look interesting [09:36] Project windmill-devel build #99: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill-devel/99/ === almaisan-away is now known as al-maisan [10:14] gmb, hi [10:14] i liked your post [10:14] could i persuade you to put it on the lp blog? [10:15] poolie: Thanks. I've already got that on my todo list for today. [10:16] :) [10:18] Morning [10:20] gmb it's so great to see new non-canonical contributors [10:20] and such an advantage for launchpad compared to the other leading brands [10:21] \o/ [10:21] It worked :) [10:21] Definitely [10:21] * nigelb hugs gmb, wgrant, and StevenK :) [10:21] :) [10:24] nigelb: You're welcome! Wait until your change is on production. :-) [10:24] With a bit of luck that will be tomorrow morning. [10:24] \o/ [10:24] Now, on to find another candidate bug to fix. [10:25] Faster! [10:27] soren: sort of. [10:27] soren: we generally try very hard to help GMail users, they make up a huge percentage of our users [10:28] soren: in this case, a lot of the web-side filtering is a work-around for GMail brokenness [10:28] Same with all the textual cruft in the emails :/ [10:28] I see. [10:45] err, what is this? [10:45] description = IBug['description'] [10:45] never seen an Interface used like that. [10:48] henninge: Interfaces are not classes. Their members appear as items, not attributes. [10:48] wgrant: but what does this give me? [10:48] There is no instance involved [10:49] I didn't know that, btw. [10:49] henninge: It should give you the interface field. [10:50] ah, obviously [10:51] so that is written confusingly. It should not call the value "description" but "description_field" [10:51] wgrant: thanks [10:51] Hmm? Why? [10:51] Oh, right. [10:51] Yeah, I guess. [10:54] Project windmill-db-devel build #289: STILL FAILING in 1 hr 11 min: https://lpci.wedontsleep.org/job/windmill-db-devel/289/ [11:02] rvba: Q about the +localpackagediffs page === al-maisan is now known as almaisan-away [11:02] rvba: for various reasons we'll want to disable or replace the checkboxes for some of the DSDs we display. [11:02] jtv: yes? [11:03] Do you have any quick ideas of where in the code to look for those? [11:03] (This is where advanced frameworks always get to me... you can't grep because it's all implicit) [11:04] jtv: right now this is controlled by canPerformSync (grep for that) [11:05] rvba: excéllent, thank you! [11:05] but of course it's a global thing. [11:05] How global? [11:05] I may affect Inuits when I mess with it? [11:06] ;) it either all the checkboxes are displayed ... or none. [11:06] Oh! [11:06] _That_ global. [11:06] Then I can't use that right now. [11:06] But canPerformSync will probably still get me closer to where I need to be. [11:06] And perhaps save a few Inuits. === jam1 is now known as jam [11:07] Perhpas you could compute the ids of the DSD for which the checkbox should not be displayed in the view with a method to query that, then use the method in the template. [11:07] Just chatted with J and we both had examples where the checkbox shouldn't be displayed: child version > parent version, and package is already being sync'ed. [11:08] Right. [11:08] I'd rather annotate the DSDs. There may be more than just a boolean: when there's a sync job pending, I'd like to replace the checkbox with a "please wait" clock. [11:08] jtv: I know I don't have to tell you this ... but beware of the dreaded one query per DSD thing ;). [11:09] Oh yes, thank you, this database stuff is all very new to me. :-) [11:09] jtv: ;) [11:10] (This morning I found myself wondering about prefetching Zope navigations in SQL, which should tell you something about how much I care!) [11:10] jtv: anyway, greping for canPerformSync will give you all the right spots. [11:10] You know: https://launchpad.net/ubuntu/natty/ could fetch Ubuntu and Natty in 1 query. [11:10] That's a great help. Thanks again! [11:11] jtv: Glad to help. [11:11] allenap: while I'm at it, maybe I should just fix IPlainPackageCopyJob.getActiveJobs to check for job status? [11:12] (jtv: I think allenap is not here today.) [11:12] oh [11:55] Vexed. Going to get food. === almaisan-away is now known as al-maisan [12:00] jtv , hi [12:00] jtv, i was wondering if you could pilot james's long-outstanding approved mp [12:01] poolie: what MP is that? [12:02] it' the top one on activeeviews [12:02] https://code.launchpad.net/~james-w/launchpad/more-matchers/+merge/32057 [12:02] thanks [12:02] nearly 6m old [12:02] I hope it'll still work then! === jtv changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews [12:03] poolie: I'm a bit under the weather and not thinking too clearly... all it seems to need now is to be landed, right? [12:03] apparently so [12:04] tested and landed that is [12:04] Project windmill-devel build #100: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill-devel/100/ [12:04] OK === henninge is now known as henninge-lunch [12:09] thanks [12:19] poolie: riddled with conflicts now. :( [12:19] Thanks for picking that up though. We need to get used to checking for this. [12:28] yes [12:28] if nly there was a page that showed you all the outstanding reviews :) [12:28] actually the +mps page could be better about showing ones that need help [12:41] poolie: this is weird... when I try to merge that branch into a fresh devel, I get conflicts on files that the branch (going by the MP diff against devel) did not touch. [12:42] that is strange [12:42] i'll try it when my meeting's over [12:42] there's also one from cjwatson that may now be able to proceed [12:42] would be worth pinging it [12:42] poolie: That's waiting on python-apt fixes from Platform. [12:43] And jelmer's has a few test failures that I was going to look at this week. [12:43] yeah, I need to check with mvo about that [12:43] thanks poolie... there's nothing more I can do today I'm afraid. [12:43] just wasn't sure if it was still blocked [12:44] i guess the conflicts may be that he newly added lib/lp/soyuz/testing? [12:44] i'll merge it and see what i can do [12:44] Probably. [12:44] Maybe I _should_ however take the time to file bugs for suspend, hibernate, battery time estimation, and Tomboy not saving my data for hours on end. :( [12:45] They combine so neatly into total data loss. [12:49] lifeless or flacoste this page times out for me 100% of the time now: [12:49] https://bugs.launchpad.net/ubuntu/+bugs?field.tag=python27 [12:49] james_w: do you think you'll come back to this or do you want it piloted? [12:49] please, is there anything you can do to help me? [12:50] it would be great if someone could land it for me [12:51] barry: Hm, worked second load for me. [12:52] wgrant: dang [12:52] * barry rides the reload button [12:52] barry: Annoyingly for you, works first time for me. [12:52] Maybe you have more teams than me or something. [12:52] could be, yeah [12:54] 8.49s, so it's very close. [12:58] wgrant: i'll keep hammering it. maybe i'll get lucky and hit a warm cache ;) [12:58] barry: Do you have an OOPS ID? [12:58] We may work out it's just over the threshold and bump that page by a couple of seconds. [12:59] lifeless suggested it was far worse. [12:59] But the fact that it's a few hundred ms under for me suggests he may have ben mistaken. [12:59] (Error ID: OOPS-1964EA274) [12:59] and hey, a reload just got me the page! [12:59] Thanks. [13:00] wgrant: thanks [13:01] Hm, we should just increase the limit. [13:01] The main query was only 7.5s. [13:02] It was just doing the final rendering when it was killed :( [13:03] wgrant: related, if i do an advanced search to get also fixed released and tagged python27 i get another timeout: [13:03] (Error ID: OOPS-1964DV295) [13:03] https://bugs.launchpad.net/ubuntu/+bugs?field.searchtext=&orderby=-importance&field.status%3Alist=NEW&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=FIXCOMMITTED&field.status%3Alist=FIXRELEASED&assignee_option=any&field.assignee=&field.bug_reporter=&field.bug_supervisor=&field.bug_commenter [13:03] er, http://tinyurl.com/6xyzrf3 [13:03] our urls rock [13:04] 9s query :( [13:18] bigjools: [13:18] https://bugs.launchpad.net/launchpad/+bug/325982 [13:18] <_mup_> Bug #325982: Search URL is long and hard to paste < https://launchpad.net/bugs/325982 > [13:19] yeah I think I've seen that before [13:22] sinzui: i should know this but don't remember, did stevenk graduate as a reviewer? [13:23] oh, i guess i could ask StevenK directly... === danilos changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: danilos | https://code.launchpad.net/launchpad-project/+activereviews [13:30] is the LP.cache meant to be included on anonymous pages? === henninge-lunch is now known as henninge [13:30] I am not sure what this is trying to achieve: [13:30] That doesn't seem to make very much sense. [13:31] It would seem reasonable to include it sometimes, but I can't think of anything that needs it now. [13:31] I think it is supposed to mean "Don't complain about missing user attribute, just make the condition False." [13:31] wgrant: would this be correct: [13:32] ? [13:32] I don't think so. [13:32] hm [13:32] 'nothing' is the correct thing to use there. [13:32] If view/user might not exist. [13:32] but it seems to evaluate to true always [13:33] * henninge checks that [13:33] It should be None for anonymous requests... [13:47] hi mrevell [13:47] Hey bac [13:53] anyone know anything about this failure on buildbot: https://lpbuildbot.canonical.com/builders/lucid_db_lp/builds/958/steps/shell_6/logs/summary? [13:53] rabbitmq failed. [13:53] Spurious. [13:53] Already forced. [13:54] I will talk to lifeless tomorrow and probably disable it. [13:54] Since it has been happening a lot. [13:56] wgrant: I read that wrong. The LP.cache["context"] is unconditional. The tal:condition above works correctly. [13:57] wgrant, ok, cool, thanks [14:02] henninge, ping for standup [14:02] yup [14:04] deryck: trouble ... [14:04] henninge, ok, we'll start and join us when you can === matsubara-afk is now known as matsubara [14:12] bigjools: Sadly, security updates sometimes do add new packages (thing Firefox and co.) [14:13] bigjools: Also, I suggested this yesterday or so: [14:13] 15:26 < wgrant> StevenK: I evisage that primary archives will use a chain of UnknownComponentOverridePolicy and FromExistingDestinationOverridesPolicy, while PPAs might use MainOnlyOverridePolicy and FromSourceOverridesPolicy, and copy archives might use FromSourceOverridesPolicy and FromExistingPrimaryOverridesPolicy [14:13] 15:27 < wgrant> Er, last case is the wrong way around. FromExistingPrimaryOverridesPolicy and FromSourceOverridesPolicy [14:14] Which seems to be a similar sort of idea to your PPA main overriding thingy. [14:14] I am glad we are thinking along similar lines. === abentley changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: danilos, abentley | https://code.launchpad.net/launchpad-project/+activereviews [14:38] bac: Indeed I did [14:38] bac: thumper sent out a mail about it [14:39] jcsackett: ping [14:41] bigjools: At the last comment of "Do our tests suck so much that made no difference?!" yes [14:42] bigjools: I was attempting to nail down a bug when I noticed that. [14:44] Eventually the copier will be split up enough that it's unittestable. [14:48] StevenK: *sadface* [14:52] abentley: Do you have time to review https://code.launchpad.net/~sinzui/launchpad/answers-api-3/+merge/61346 [14:53] sinzui: yep. I was already reading it, in fact. :-) [14:58] sinzui: I would prefer to write the new tests as unit tests. r=me. [15:00] abentley: I understand. My intent is to document how the webservice works. I have full faith in the existing unit tests are correct [15:00] Yippie, build fixed! [15:00] Project devel build #729: FIXED in 5 hr 23 min: https://lpci.wedontsleep.org/job/devel/729/ [15:01] sinzui: ISTM that the web service you just added could not possibly be already unit-tested. [15:02] abentley: I only exposed the methods. So I wrote a documentation that verified the they were exposed, and explain to a user how to make a request to get the data [15:02] woot! sub 200 crit bugs in launchpad itself (22 in stuff like launchpad-buildd, lazr.delegates, qa-tagger etc) [15:02] sinzui: hi [15:03] abentley: What the methods do are already tested [15:03] sinzui: we have a call now [15:03] Does https://launchpad.net/~launchpad-hackers do anything these days? [15:03] sinzui: Right. The fact that they are exposed, and exposed correctly is not already tested, and I would unit-test that if it were me. [15:04] wgrant: don't know. [15:04] abentley: How are they not exposed correctly? [15:04] wgrant: also, I wonder how we could change Launchpad so you wouldn't have to ask. [15:04] sinzui: I'm not asserting that they're not exposed correctly. [15:04] sinzui: I'm saying that I would add unit tests to ensure that they were exposed correctly. [15:05] jml: I'm pretty sure that's just about in scope for next week. [15:05] Let's try deleting it in a transaction on dogfood and see what breaks. [15:05] wgrant: if you squint hard enough, yes. [15:05] abentley: I think that doc test does that. Since I changed an interface, I wanted that documented. I do not like writing duplicate tests, which I think a unittest would be in this case. I did not change the model. [15:06] sinzui: I don't like writing duplicate tests, either, so I would not write the doctests. [15:07] wgrant: you've checked celebrities, I guess. [15:07] abentley: Interfaces are the only thing that needs documentation in Lp, And since I actually want users to read how to use answers of the API, doctests are the correct way to demonstrate what was exposed [15:07] Project windmill-db-devel build #290: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill-db-devel/290/ [15:07] sinzui: I'll be back in a bit. am very keen to talk today. [15:08] jml: Yeah. It's all old pre-opensourceing private branch subscriptions. [15:08] jml: I am on mumble [15:08] didn't you hear me [15:09] And branch visibility policies for lp-dev-utils and bzrbuildbot... both of which should probably be canonical-launchpad-branches now. [15:10] sinzui: I don't think interfaces need this kind of documentation. I don't think it is really helpful. [15:12] I disagree. In fact. the first unit-test only tests of answers did not demonstrate how to use it. We had to land multiple fixes because we actually want to know how to write scripts a particular way. [15:14] sinzui: oh, on line 59, you misspelled "IQuestionTarget". [15:15] Oh, yes, there is a good chance I that. I wrote this in bed last night. [15:15] I will spell check the changes [15:16] sinzui: How does a team merge handle memberships in the team that is to be destroyed? [15:17] wgrant: all membership are removed. Nothing is transferred because there is a high probability of hitting a CyclicTeamMembershipError [15:17] sinzui: That's what I hoped. Thanks. [15:17] sinzui: pong. [15:18] jcsackett: I wanted to catchup in bugs, I am, or will be, in a meeting now. Maybe later [15:18] sinzui: no, I didn't. I'm not on mumble. [15:18] nothing again? [15:18] sinzui: ok. i'll try to be more responsive on your next ping. :-) [15:18] sinzui: it's been having trouble connecting today. [15:18] * jml will try rebooting. [15:24] jml: I do not see you speaking. Did you hear me? [15:24] sinzui: I am not connected to mumble [15:24] sinzui: you see merely a figment. [15:24] My mumble sees you [15:24] sinzui: can I skype you or call your phone? [15:24] I will start skype [15:25] sinzui: thank you [15:42] deryck: could you please run the two queries in https://pastebin.canonical.com/47578/ on staging? [15:48] Project windmill-devel build #101: STILL FAILING in 47 min: https://lpci.wedontsleep.org/job/windmill-devel/101/ [15:50] adeuring, sure, I'll get them now. Was on a call, sorry. [15:50] deryck: thanks! [15:50] adeuring, also, I can take IRC on #launchpad now. Sorry about being late. [15:51] no problem :) === al-maisan is now known as almaisan-away [16:01] adeuring, https://pastebin.canonical.com/47580/ twice for each query. once cold, once hot. [16:01] deryck: thanks! [16:01] adeuring, np! [16:21] sinzui: ready when you are [16:56] \o/ [16:56] So, buildbot succeeded with my changes [16:57] does that mean that rev lands into production soonish? [16:58] Project windmill-db-devel build #291: STILL FAILING in 1 hr 10 min: https://lpci.wedontsleep.org/job/windmill-db-devel/291/ === beuno is now known as beuno-lunch [17:00] cjohnston's still got some unlanded branches. what's happening there? [17:00] nigelb: yes. [17:01] nigelb: it needs to be verified as not going to break production [17:01] jml: *whee* Now, I have to find more to work on. [17:01] tbh, I don't know how to do that, or what page to look at. [17:02] I'm just scrolling through the easy bugs and trying to figure out if there's something I can work on. [17:02] hmm. http://lpqateam.canonical.com/ [17:02] but it's out of date [17:37] interesting, I just got a chrome "ERR_SOCKET_NOT_CONNECTED" while spinning waiting on this page: https://launchpad.net/ubuntu/+source/munin [17:38] not the first time I've had this issue.. refresh works very fast [17:38] It seemed to spin for about 9 - 10 seconds .. I wonder.. is something cutting off the connection before the OOPS message appears? === deryck is now known as deryck[lunch] [17:59] jml: one I still need to work on, the other I believe is ready [18:00] cjohnston: cool. I'm just worried about it getting stuck in "Approved" while we wait for a core dev to land it. [18:00] So should I ping one of the reviewers? [18:01] cjohnston: a good idea. henninge is generally around in the .de day [18:01] He is the one who approved it iirc.. Does he have commit access? [18:02] cjohnston: indeed. [18:03] cjohnston: although, huh, I guess it's a bit sub-optimal that you can't tell that easily from the website. [18:03] Why would he approve it but not land it? [18:03] cjohnston: I honestly don't know. [18:03] ok [18:03] I pinged somone yesterday and got them to land 3. [18:04] cjohnston: one possibility is that he ran it through tests and they failed. [18:04] so I'm up to 4 landed I think [18:04] ok [18:04] cjohnston: however, the normal way of doing that would also result with you receiving an email [18:04] I didn't get any hate mail from it.. but I dunno [18:04] right. === beuno-lunch is now known as beuno [18:05] cjohnston: anyway, I'm being dragged away from my computer. will keep hassling to get them landed from my side, you should do the same. [18:05] * jml is off [18:05] Okie.. ty [18:27] Why the F do users keep linking packages to /launchpad when the package does not contain Lp code? Don't these users know I can track them down and club them like a baby seal? === deryck[lunch] is now known as deryck [18:37] deryck: I'd like to do a pre-imp call on bug 648075 [18:37] <_mup_> Bug #648075: Automatic translations export fails intermittently < https://launchpad.net/bugs/648075 > [18:37] abentley, sure. I have TL call in 20 minutes. can we do it before, or should we catch up after? [18:38] deryck: I think "before" works. [18:38] ok, firing up mumble now [18:46] I see that since blueprints were re-enabled, users are using it to request features. I do not think any of the 286 blueprints I see are work we are undertaking. [19:00] deryck: did you consider registering NotFound as a 404 error? [19:02] abentley, it was already handled as a 404. I just decorated it to not OOPS. see: https://code.launchpad.net/~deryck/launchpad/404-bug-api-generates-oops-701246/+merge/61407 [19:02] well, not literally decorated it. [19:13] deryck: I mean that you can specify an error status for a particular exception class, and that will always be used when that exception is raised. [19:16] deryck: e.g. lp.soyuz.interfaces.archive.CannotCopy [19:25] deryck: so there was a bug in lib/canonical/launchpad/webapp/errorlog.py; good catch! [19:33] gary_poster: sinzui: flacoste: deryck: http://webnumbr.com/.join(launchpad-oops-bugs.all,launchpad-timeout-bugs.all,launchpad-critical-bugs.all) === bac changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: abentley | https://code.launchpad.net/launchpad-project/+activereviews [19:48] benji, thanks! Yeah, that was a bit tricky to spot, but once I saw it was sooo clear. ;) [19:48] abentley, sorry, was on TL call [19:48] deryck: no worries, I thought so. [19:48] :) [19:50] abentley, so NotFound is not an exception. it's imported from zope.publisher and we have a view hooked up to it, which sets the status to 404. [19:51] deryck: heh, so we oops on 404 deliberately [19:52] deryck: but we filter it on the oops server if the url was non-local [19:52] deryck: so we see bad links that we generate (in principle) by depending on referer [19:52] deryck: NotFound is an exception. It derives from BaseException. [19:53] I've fixed the stable -> db-devel conflict (Julian -> me conflict ;)). Should I submit it to pqm or someone wants to take a look at it first? [19:53] abentley, are we both talking about zope.publisher.interfaces.NotFound? [19:53] deryck: Yes. We don't define it, but it is an exception. [19:54] abentley: can you have a look at a very simple branch: https://code.launchpad.net/~bac/launchpad/bug-781987/+merge/61459 [19:54] bac: sure. [19:55] abentley, ok, sure. if you follow it back far enough. my point though is that it really is already registered in a sense as 404, via zcml and the view it's hooked up to, no? [19:55] my little fix has generated so much interest ;) [19:57] sinzui: did you want to chat? [19:57] deryck: Perhaps it's registered as a 404 in some sense, but I don't think the class is registered with lazr.restful as a 404. If it was, you wouldn't need to call expose on its instances. [19:57] lifeless, yeah, I meant "not raising an OOPS for any resource" when dealing with the webservice only. I wasn't meaning for any request on launchpad itself. sorry for the bad choice of language. [19:57] deryck: de nada, I should have read the diff more closelhy [19:58] bac: r=me [19:58] thanks aaron [19:59] abentley, ah, ok. I see what you're suggesting now. I arrived at this fix after chatting with benji, and thought this was the best way to fix this for the webservice. [19:59] hmmm, I do think it's right. [19:59] deryck: I think you could call expose on the class instead, and that would mean that all instances would be treated as 404s. [20:00] jcsackett: do you have time to mumble? [20:00] abentley, what do you mean by client? launchpadlib, lazr.restfulclient? [20:00] something else? [20:01] you can declare that a class of exception is always the client's "fault" (using a different mechanism) and such it won't generate an OOPS, or you can decorate a particualar instance [20:01] deryck: I don't think I said anything about clients. Did I? [20:01] abentley, doh, sorry. misread class as client [20:02] that's why it didn't make sense ;) [20:02] deryck: ah. [20:02] abentley, now which class do you mean? NotFound ? [20:02] deryck: yes. [20:03] deryck: e.g. expose(NotFound, 404) [20:05] abentley, so expose may be misleading. it doesn't expose this as 404. it sets a variable on the exception to 404, which the error handler uses to determine if we should log an OOPS. [20:05] maybe IRC isn't the best for this discussion. [20:06] abentley, should we mumble to save time? [20:06] deryck: sure. [20:11] I'm going to submit my fix to the pqm then. I hope you guys can cover up for me in case anything goes wrong. [20:11] (it's a tiny fix so I'm not really worried) [20:39] Project devel build #730: FAILURE in 5 hr 31 min: https://lpci.wedontsleep.org/job/devel/730/ === almaisan-away is now known as al-maisan === al-maisan is now known as almaisan-away [21:39] jam, ping: re Loggerhead CSS? [21:43] abentley, that branch is done, if you're up for giving me a proper review. Though I'm EOD'ing very soon. [21:44] deryck: sure [21:44] abentley, thanks! [21:47] deryck: r=me. Your change at line 8/9 is probably not needed, but I think it's correct anyway. [21:48] abentley, I'm fairly certain it is. I can confirm. [21:48] Project windmill-devel build #102: STILL FAILING in 1 hr 9 min: https://lpci.wedontsleep.org/job/windmill-devel/102/ [21:49] deryck: the fact that expose exists implies that you should be able to attach a status to an instance, so I think it makes sense to honour that. [21:51] abentley, ah, I see what you mean. I'm patching the class, not the instance now. and yeah, we should probably leave the change I added, so it is possible in the future. [21:53] Project windmill-db-devel build #292: STILL FAILING in 1 hr 10 min: https://lpci.wedontsleep.org/job/windmill-db-devel/292/ [21:54] we should kill those windmill notices here. no one is doing anything about it. [22:05] Later on, everyone === abentley changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews [22:31] wallyworld: define POPO pls? [22:32] mwhudson: plain old python objects. ie ones with no infrastructure attributes or extending storm classes etc [22:32] ah ok [22:32] sorry for the confusion [22:34] np [22:34] i that case i think i agree with you [22:36] (and am not going to spend half my day on that thread today given that i no longer work on launchpad :-p) [22:37] :) [22:39] Project windmill-db-devel build #293: STILL FAILING in 46 min: https://lpci.wedontsleep.org/job/windmill-db-devel/293/ === lifeless changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews | Criticals: 210 [######=_] === lifeless changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews | Critical bugs:210 - 0:[######=_]:256 [22:41] sinzui: after our discussion and some technical problems i have a test that replicates the problem. bad news is the simple fix we talked about doesn't seem to address it. http://paste.ubuntu.com/609782/ is what i've done. [22:42] sinzui: any thoughts? === matsubara is now known as matsubara-afk === salgado is now known as salgado-afk [22:51] jcsackett: what are you trying to reproduce? [22:51] lifeless: it's a permission bug on flag-membership-expired on relation account. [22:52] lifeless: bug 783476 [22:52] <_mup_> Bug #783476: Scripts failed to run: loganberry:flag-expired-memberships < https://launchpad.net/bugs/783476 > [22:52] jcsackett: have you run security.py to apply those permissions to the test db ? [22:53] (silly question I know, but still...) [22:53] not so silly--i was under the impression that was done automatically either via make or the testharness. [22:54] its only done via 'make schema' [22:54] or manually [22:54] database/schema/security.py -d launchpad_ftest_template [22:54] is the invocation, IIRC [22:55] i thought i did do make schema (what i meant when i said make). i'll try the direct invocation. [22:59] lifeless: that worked. thanks a ton. [22:59] \o/ [23:13] jcsackett: is all okay? [23:13] sinzui: all is excellent. just wrote the MP. [23:14] jcsackett: sorry about not telling you about the security,py. I did that a lot when working on the job fixes. I know it is an obscure thing [23:15] jcsackett: You will need to remind the LOSA to run that to qa the work, and there will be warning about no permission assigned to out db functions. That is a non-issue [23:26] sinzui: good to know. thanks. :-) [23:32] jcsackett: I approved your branch with a comment [23:35] sinzui: i am making the change you suggest. [23:36] I am certain that bug 730393 can be fixed in less then an hour, but I have been away from the code for so long I do not remember how we suppress oopses for error pages we show users [23:36] <_mup_> Bug #730393: when a user urls hacks we can get InvalidBatchSize oopses < https://launchpad.net/bugs/730393 > [23:39] Maybe I should just update the oops reporting util. InvalidBatchSize, NotFoundError, and GoneError should only be reported as an oops when Lp is the referer [23:41] yes yes yes [23:42] yes yes [23:42] and finally yes [23:43] lifeless: was that to add an conditional list error classes to exempt without a referer, similar to what we do with Unauthorized? [23:44] without a referer or with a non .*launchpad\.net/.* referer [23:45] jcsackett: wgrant, wallyworld. I just got a message that I need rescue a child. I am going to miss the meeting in 15 minutes. [23:45] sinzui: jcsackett: wgrant: i am ok to delay? or will it take too long to do what you need to do? [23:46] sinzui: okay. i hope all is well. i can also wait for you. [23:46] wallyworld: maybe 15 minutes [23:46] late [23:46] sinzui: ok. why don't you pings us when you are back [23:47] lifeless: i think only report oopses with .*launchpad\.net/.* as referer...those we can fix [23:47] wallyworld: okay I will [23:47] * sinzui find car [23:47] sinzui: [of the NFE/DE cases - yes] [23:47] sinzui: IBS could be self inflicted [23:47] sinzui: perhaps we need to think about it more [23:48] sinzui: oh, IBS with referers that are not .*launchpad\.net/.* are not self inflicted and we can discard them too [23:49] sinzui: so yes, I agree