[00:29] Project db-devel build (62): STILL FAILING in 4 hr 4 min: https://hudson.wedontsleep.org/job/db-devel/62/ [00:40] thumper: found problem :-) [00:40] * wallyworld does happy dance [01:25] wallyworld: awesome [01:27] thumper: how's your stuff? [01:31] wallyworld: well, I've figured a way to at least stop the worst error [01:31] but thinking about better error handling [01:31] rob had an idea but I had to cut him short as I had someone arrive [01:32] but I think it is more of a future idea [01:32] definitely [01:32] way out there stuff [01:32] perhaps error handling patterns need to be looked at across the codebase, maybe unify any disparate implementations to use "best practice" [01:33] so... [01:33] I'm going to go for the nicer simple fix that avoids the crappy error message [01:34] possibly get that RCed into the branch we want to release [01:34] my thoughts were blue sky, future stuff [01:34] and move on to looking at the BranchRevision use cases [01:34] just thinking out loud [01:34] suer [01:34] sure [01:36] wallyworld: refactoring +1; best practice mmm I prefer to say 'this works well, and let folk adopt it' - the poppendiecks had a quote about best practice - that it stops improvement & experimentation [01:36] "best practice" to me is defined to be whatever a particular project says it is [01:37] what whatever error handling patterns are deemed to work best for lp/bzr, they should be adopted more or less uniformly IMHO [01:37] "best practice" to me, is often a fallacy ;-) http://www.nizkor.org/features/fallacies/appeal-to-common-practice.html [01:38] ok, so i hit a nerve with the bullshit bingo, overused cliche "-) [01:38] :-P [01:38] all i meant was, i think we need, if not already there, a set of well defined patterns for various "things" that people are encouraged top adopt [01:39] :-P right back at ya [01:39] heh [01:42] wallyworld: we don't have uniform situations, so uniform handling doesn't make sense. [01:43] wallyworld: documenting things that work well so that folk can use them is a good idea. [01:43] hmmm, not sure i agree about the lack of uniformity [01:44] wallyworld: I'd like to reduce some of the variation we have because there is cognitive overhead there - I mean things like some bits being in twisted, some zope, some pop, some wsgi [01:44] +1 [01:45] also, as a new person, i've found several ways to do stuff, i pick one, and people say "oh don't do it that way, that's bad" [01:46] i've need to get some of these new starter ramp up issues documented. it's on my todo list [01:47] wallyworld: when someone says that, it may come from a couple of places [01:47] * stylistic thing - our code review looks at a lot of irrelevant stuff we should just ditch, IMO. [01:47] * functional issues - in which case the place you copied from needs fixing / and XXX would be appropriate. [01:47] * maintenance issues - ditto [01:49] one that comes to mind is design patterns used in "answers" - so not just simple stylistic stuff, but more substantial, legacy implementation issues [01:51] sure, that would fit under 'maintenance issues' above, do you think? [01:52] yeah, suppose so, a form of technical debt [01:53] spm: Can you please kick off a staging update? I've swapped the database we built on the weekend into place. [01:53] stub: sure [02:11] stub: so, cassandra, you know if you're going to stay for the extra days or not, yet ? [02:12] lifeless: I don't think I can make it at all with the travel time :-( [02:12] ok [02:13] I'll book my tickets today/tomorrow then [02:13] stub: perhaps we can get a video feed of the training for you [02:13] or a DVD or something [02:13] If we can, that would be cool for more people than just me [02:13] indeed [02:37] We are now on 2.6 everywhere? I have a 2.5 compatibility fix I'd rather not land. [02:38] I'm not sure of the status of our last buildbots [02:39] spm: ^ [02:39] stub: the db's are [02:39] stub: like you I'm unsure of buildbot status; there was a zomg rt ticket about it [05:33] is devel shut off for landings? === almaisan-away is now known as al-maisan [06:31] jml: I've fixed loading/running of story tests in testr. mp coming soon [06:31] lifeless: i put up a testscenarios mp [06:33] thanks [06:36] thumper: Any reason the revision karma allocater is a separate script rather than part of garbo? [06:36] stub: no good reason [06:36] stub: garbo was underadvertised [06:42] jml: https://code.edge.launchpad.net/~lifeless/launchpad/doctest/+merge/38182 [06:43] lifeless, thumper: https://code.edge.launchpad.net/~stub/launchpad/bug-658124-revision-karma/+merge/38181 as it is, or I can move it to garbo after lunch. Might prefer as it is for rc? [06:45] as is for rc [06:48] stub: rc-reviewed. please let edwin know. [06:48] stub: I've code reviewed too, though ec2land won't find it because I can only have one vote. [06:49] * thumper looks [06:50] stub: I'm curious why 2 queries are needed though, seems like a pg regression ? [06:51] Its one query assembled in two parts [06:52] http://paste.ubuntu.com/511378/ is the query I wanted, but this was the closest we could get with storm (and performs identically) [06:54] stub: ah, I see. [06:55] still seems weird to need to move the limit inside the exists; I would have expected pg to optimise ;) [06:55] By putting the limit inside, I'm requesting <= limit rows. If I put it outside, I get == limit rows. [06:56] limit then filter dupes vs. filter dupes then limit [06:58] anyhow, its faster. woo. === jtv is now known as jtv-afk [07:41] The Debian BTS sync is working again? [07:57] wgrant: I thought I heard a mumbling about it being turned on again, but I'm not sure === jtv-afk is now known as jtv === jtv is now known as jtv-eat [09:08] lifeless: cool. [09:08] lifeless: I'll check it once my inbox is empty. (soon now) [09:33] Project db-devel build (63): STILL FAILING in 4 hr 8 min: https://hudson.wedontsleep.org/job/db-devel/63/ [09:38] Morning [09:41] mrevell: hi [09:52] wedontsleep.org - ha! === jtv-eat is now known as jtv [09:53] StevenK: the caffine link is a nice touch :p [10:06] StevenK, And you've two dead links on the page... [10:06] Indeed [10:08] "Way cheaper than a trip to Starbucks, Foosh Energy Mints are heavily caffeinated for an energy boost that takes effect quicker than monkeys chomp cheese. And they taste great! Try some now! Each mint contains more than 100 milligrams of caffeine...Wow" - RIGHT === al-maisan is now known as almaisan-away [10:11] Clearly the authors of that slogan have never seen monkeys trying to explain to the stupid humans involved that cheese smells rotten (which humans call "having sophisticated tastes") [11:17] does anyone know if it's possible to test that some code did a commit()? [11:20] bzr or db commit? [11:20] db [11:20] the commit is seen by the storm tracer [11:20] do we have any examples that you know of? [11:21] I think so, possibly using the test tracer (rather than the webapp.adapter tracer) [11:21] but 2321, I'm not going to try and find the example right now ;P [11:21] see you in 6 hours then :) [11:22] hah :P [11:22] are you at UDS > [11:22] ? [11:23] I will be [11:23] its 2 weeks from now [11:23] yeah I know, I'm going too [11:23] we'll have to have a beer ;) [11:23] the magic word [11:26] if I run "bin/test test_builder" I get circular import errors. If I run "bin/test -t test_builder" I don't. WTF. [11:28] bigjools: I see that sometimes too, eg. -m lp.archivepublisher fails with an SPPH circular import. [11:28] yes [11:29] I think it's because lp.bugs is importing SPPHG [11:29] SPPH [11:29] which is wrong on many levels [11:29] Well, I suspect c.l.d is involved too. [11:30] which is wrong on many levels [11:30] Heh. [11:31] True. [11:31] the Code guys solved this problem but I can't remember how - to get rid of c.l.{i,d} we need to make the API stuff see all the interfaces [11:31] It's easy enough. [11:31] Add a webservice module somewhere in your tree, import stuff into there. [11:31] Then add the ZCML pointing to that module. [11:32] that's the badger [11:32] That's more c.l.i than c.l.d, though, I think. [11:32] btw the new buildd-manager is on DF [11:32] seems to work :) [11:32] The OMFG-5KLOC-DIFF one? [11:32] 6k, yes [11:32] Ah, it was only 4K when I last looked. [11:33] I didn't extrapolate far enough :P [11:33] bzr di -r submit:|wc -l [11:33] 6286 [11:34] Owww. [11:34] Hmm. [11:34] as jml pointed out we should have started at the top instead of the bottom (wrapping calls in maybeDeferred) and we could have landed stuff sooner [11:34] 2 hour downtime with a day's notice? :( [11:39] bigjools: DistroSeries.checkLegalPocket refuses to publish binaries in non-Release pockets for a Development series (but Frozen is allowed). This seems fairly strange. [11:39] I can't see a good reason for it. [11:39] why would you need to do that? [11:40] Well, I hit it in tests, but Ubuntu frequently uses -proposed and -security before release. [11:40] It just happens to work now because the series is usually frozen beforehand. [11:40] why would they do that when they can upload to -release? [11:41] bigjools: It is desired that the Release pocket matches the images. [11:41] then it should be frozen, no? [11:41] And it's handy to get a head-start on SRUs by uploading to -proposed beforehand. [11:41] True. [11:42] But it seems odd to allow uploads to more when a series is frozen than when it's open. [11:42] seems perfectly cromulent to me [11:42] Still, the whole Frozen-as-a-status thing is crackful. [11:43] why? [11:46] I'm not entirely sure. It just doesn't seem right to have the series bouncing between Development and Frozen, particularly since we have other statuses (Future, Experimental) which don't have obvious freeze semantics. [11:46] I think it would be nicer if there was a separate field for that. [11:46] But anyway. [11:46] My tests now work around the crazy restriction -- just thought it was a bit strange. [11:47] (I'm fixing that hack I introduced last week, and making the related publisher tests more thorough and less sucky) [11:47] (and removing the whole architecture[:7] madness) [11:50] great [12:02] Morning, all. === almaisan-away is now known as al-maisan [12:48] How is RT 41738 ("lpbuildbot needs a Lucid production buildslave") going? [12:49] I was wondering if launchpad-database-dependencies could require strictly pg8.4 yet [12:49] That seems like it sort of has to happen before Thursday... [13:07] bac discovered that Storm's sqlobject gets is_empty completely wrong. Bug 659078. [13:07] <_mup_> Bug #659078: sqlobject is_empty and __nonzero__ are incorrect [13:11] allenap, hi. The kanban board seems to indicate Bug #650991 is qa-ok. Is that right? [13:11] <_mup_> Bug #650991: Add getSubscriptionsForBug to IStructuralSubscriptionTarget

[13:12] deryck: There's one card remaining which I should probably QA on staging. [13:12] allenap, ah, ok. Thanks. [13:14] Are pop-ups (like to assign a bug or request a review) broken for other people on edge? I consistently get 'Loading results failed.' and it's been this way for a few days. [13:24] I agree w/ wgrant, I think. It would be good to have the control for what package uploads are allowed separate from status [13:36] jml: we need to enumerate the pros and cons for that. I'm neutral at the moment. [13:39] jkakar: Everything seems to be working fine here. Is anything else dumped to the js console? [13:40] allenap: Let me check. [13:40] Several people have seen timeouts in the person picker since the 8.4 migration. [13:42] I wish a-f would return an error code upon encountering an error. [13:42] Rather than just continuing and indicating success... [13:43] allenap: Getting '503 Service Unavailable' when I try to search for 'jtv', to assign a bug, at https://bugs.edge.launchpad.net/storm/+bug/659078 [13:43] <_mup_> Bug #659078: sqlobject is_empty and __nonzero__ are incorrect [13:44] jkakar: Ah, see wgrant's comment above. Time-outs it seems. [13:45] allenap: It's really bad right now, I haven't been able to assign a bug or request a review for days. :/ [13:45] I've been using /+request-review on merge proposals. Is there an equivalent for assigning bugs? [13:46] jkakar: Expand the expandy thing with the button on the left of the task. [13:46] wgrant: Ah, right, thanks. [13:47] jkakar: That's really bad. I haven't experienced anything nearly that bad. Are the problems consistent with other browsers? [13:48] allenap: Dunno, just been trying in Firefox. [13:48] * jkakar tries with Chromium [13:50] allenap: Same in Chrome. I wouldn't expect the browser to make a difference though, given that it's a timeout on the server side. [13:50] jkakar: Oh, of course. Do you get an OOPS? [13:51] allenap: Not obviously, just a message that says 'Loading results failed.' in red, in the person or branch picker dialog. [13:52] jkakar: Firebug might be helpful. [13:52] Since the JS isn't. [13:52] jkakar: Gah, bloody javascript! [13:53] (*Our* javascript; I don't mean to malign javascript in general.) [13:53] wgrant: Yeah, I tried Firebug (how I discovered it was returning 503). Didn't see anything obvious about OOPSes. [13:53] jkakar: There's no X-Lazr-Oops response header? [13:53] allenap: Javascript deserves some amount of general maligning. It's too bad it's taking over the world and that, in 5 years, we'll be writing server code in it. :/ [13:53] It's sometimes there. [13:53] * jkakar looks more closely [13:54] wgrant: Hah, now that I try again it's working. :/ [13:54] Awesome. [14:58] sinzui: Have you heard of any situations where a deactivated mailing list can't be reactivated? This is with regard to the last three comments in https://answers.edge.launchpad.net/malone/+question/58912 [14:59] allenap, yes, when the team is renamed, the list and it's email address are invalidated. [15:00] sinzui: Is there any way to know if a team has been renamed, or shall I just ask the user? [15:00] allenap, I am not certain if a losa needs to delete mailman/monharc files or not but It would be mentioned in lphowto on the canonical wiki [15:00] we would need to ask the user [15:00] sinzui: I'll do that. Thanks. [15:02] allenap, actually, this team cannot be renamed because the existing list has not been purged [15:03] sinzui: Ah. Okay, I'll leave this one in the hands of the LOSAs then. [15:11] Project devel build (103): STILL FAILING in 3 hr 46 min: https://hudson.wedontsleep.org/job/devel/103/ === deryck is now known as deryck[lunch] [16:10] EdwinGrubbs, bug 645702 is qa-ok. I think the registry is ready for release [16:10] <_mup_> Bug #645702: oops in holdMessage storing large message

[16:12] sinzui: I'm still waiting on getting the cronscript run on staging for the job queue I added, but the web page didn't timeout when approving a bunch of proposed members, so it's looking good. [16:14] Edwin and keep in mind that staging is super slow. Loading /gdp timed out 3 times. [16:20] jtv: ping [16:20] EdwinGrubbs: yes? [16:21] EdwinGrubbs: you called? [16:21] jtv: are you able to QA bug 650877? [16:21] <_mup_> Bug #650877: Not generating TranslationTemplatesBuilds on staging [16:22] EdwinGrubbs: if staging is finally running… it's not been possible yesterday, or today during working hours. [16:22] jtv: staging is up [16:22] Then I'll try it. [16:23] allenap: ping [16:23] EdwinGrubbs: pong [16:23] Ah, okay, staging. [16:23] allenap: can you QA bug 650991? [16:23] <_mup_> Bug #650991: Add getSubscriptionsForBug to IStructuralSubscriptionTarget

[16:23] EdwinGrubbs: Sure. [16:25] lifeless: ping [16:30] gary_poster, do we now know when we will officially be done with Python 2.5? [16:31] gary_poster, and we can start writing Python 2.6 code? :) [16:32] mars, yes, earlier this morning. I'm going to submit a branch to make sure that the setup is working, but everything here is Lucid: https://lpbuildbot.canonical.com/waterfall [16:32] \o/ [16:32] * mars cues the marching band! [16:32] :-) [16:35] and we are 100% lucid on prod? [16:36] yes, jml. that's what losas tell me. :-) [16:36] and stub [16:36] allenap, is https://bugs.edge.launchpad.net/malone/+bug/551848 fix released or in progress? [16:36] <_mup_> Bug #551848: X-Launchpad-Bug link on +subscribe pages links to wrong page [16:36] gary_poster: wonderful :) [16:36] definitely [16:37] what are we doing on production for the psycopg thing? [16:38] an older version is the only option AFAIK. stub will be working on the proper fix as one of his upcoming tasks. Would you like me to verify with losas, jml? [16:38] gary_poster: no, that's fine, just curious. [16:38] cool [16:39] I guess the versioning/"can't install lp-deps" problems really only hit maverick. [16:50] sinzui: It says Fix Released, but why is there doubt? [16:50] s/Released/Committed/ [16:51] allenap, look at the date? The code from that date was released last month [16:51] So did we release the fix, or should it be targeted to 10.10 === al-maisan is now known as almaisan-away [16:52] * allenap looks. [16:54] sinzui: It's released. I've marked it as such and put it in the 10.09 milestone. [16:54] thank allenap [16:57] james_w: I understand there was a bug related to nest-part/manifests in bzr-builder recently. Is the fixed version in the PPA? [16:58] abentley, the bzr-builder ppa? yes. [16:58] james_w: great, thanks. [16:58] 0.6 === deryck[lunch] is now known as deryck [17:07] james_w: What would the ETA be on there being a working nest-part instruction in *-cat-lpbuildd ? [17:07] maxb, no idea, sorry [17:07] abentley would probably know better than I [17:09] maxb: I've just requested it. You'd have to ask lamont for an actual ETA. === benji is now known as benji-lunch === beuno is now known as beuno-lunch === benji-lunch is now known as benji [18:29] Project db-devel build (64): STILL FAILING in 2 hr 19 min: https://hudson.wedontsleep.org/job/db-devel/64/ === almaisan-away is now known as al-maisan === beuno-lunch is now known as beuno === gary_poster is now known as gary-lunch [18:51] g'night [18:53] james_w: do you know if there is way to find out from a bug task if some one is allowed to upload the associated source package? [18:57] bdmurray, yes there will be a way, but it won't be particularly direct I think [18:58] james_w: I'd want to end up at isSourceUploadAllowed()? [18:59] bdmurray, ubuntu-archive-tools has an implementation of this, but I think IArchive.checkUpload might do it all now [18:59] bdmurray, I think, confusingly, that only takes in to account packagesets [19:00] bdmurray, http://bazaar.launchpad.net/~ubuntu-archive/ubuntu-archive-tools/trunk/annotate/head:/sync-helper.py#L67 [19:02] james_w: okay, I'm trying to allow people to set milestones in Launchpad [19:02] bdmurray, ah, you are talking about inside the LP codebase? [19:02] sinzui: hi! [19:03] james_w: right [19:03] hi fjlacoste === fjlacoste is now known as flacoste [19:03] bdmurray, in that case then checkUpload or verifyUpload or something is probably where you want to start looking [19:04] bdmurray, also look for lp.code's permission checking on package branches, as they have a similar need [19:04] Can anyone else push to staging codehosting? [19:04] james_w: okay, thanks! [19:04] bdmurray, it's complicated by having to deal with components etc., but you should be able to get close to "if this person uploaded a source package of that name to the current distroseries would it be rejected?" [19:05] bdmurray, I don't know if you want to deal with released series differently there [19:42] losa ping [19:44] I want to put a new feature rule on staging (for QA) [19:45] hard_timeout pageid:BugTask:+index 1 5000 === gary-lunch is now known as gary_poster === al-maisan is now known as almaisan-away === Ursinha is now known as Ursinha-afk [20:30] gary_poster: ping [20:30] hey lifeless [20:30] up for a brief call ? [20:31] lifeless, in 45 min ok with you? [20:31] won't you be on a call with flacoste then ? [20:31] (I mean, it would be ok for me) [20:31] * gary_poster though flacoste was out today [20:31] * gary_poster checks calendar [20:31] gary_poster: i'm in :-) [20:32] :-) ok [20:32] lifeless: ok, we'll try again later; I'm stepping out in about 3 minutes though, and don't want our call to be *that* short :-) [20:32] ok [20:47] lifeless: can you QA bug 627701? [20:47] <_mup_> Bug #627701: Make it possible to use feature flags to override the global timeout for specific pages [20:47] EdwinGrubbs: see my losa ping above [20:48] EdwinGrubbs: (no, *I* can't, needs a losa) [20:48] ok, thanks for getting the ball rolling. [20:48] lifeless: hello. just got back from snacking [20:49] lifeless: so, new feature rule.. [20:51] mbarnett: yeah [20:51] mbarnett: https://staging.launchpad.net/+feature-rules [20:52] lifeless: added [21:09] gary_poster: hey, so whenever you want to chat, just ping me. I'm doing food now, then will be looking at parallel testing more. [21:09] cool thanks lifeless [21:10] Woo! Four more windmill tests before a new lazr-js is ready! [21:10] mbarnett: thanks was otp [21:11] I'm trying now :) [21:11] lifeless: cool. [21:11] great, the timeout is way low. [21:11] please remove that rule now, or staging will be useless ;) [21:22] abentley rockstar thumper: standup? [21:23] wallyworld: aye [21:23] wallyworld: isn't it 6:30 where you are? [21:23] yeah, but i have to go to a breakfast at the kid's school [21:24] heh [21:24] breakfast at school? [21:24] there's a middle school orientation thing - he's going into grade 5 next year [21:24] wallyworld, thumper: sure. [21:25] flacoste: oh, there was one thing [21:25] flacoste: this ppa/open teams thing. [21:25] lifeless: yep [21:28] there seems to be a knee-jerk reaction happening, which is a bit unpleasant [21:28] thats really all I had to say [21:43] abentley, thumper, wallyworld, you guys still can't hear me, can you? [21:43] Shite. [21:43] abentley, that's because I'm messing with the gain. === mwhudson changed the topic of #launchpad-dev to: Launchpad Development Channel | Week 4 of 10.10 | PQM is Release-Critical (Release manager: EdwinGrubbs) | firefighting: stable has test failures | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting [22:09] does stable still have test failures? [22:29] thumper: i'm available [22:29] flacoste: just getting off with jam [22:29] thumper: oh laa [22:30] ... [22:35] * lifeless guesses gary_poster is free now ;P [22:44] Ursinha-afk: ping [22:48] is there an easy way to set a projectgroup for launchpad.dev? [23:11] jcsackett: /projectgroups/+new? [23:14] wgrant: and this is what i get for juggling too many things at once on too little sleep. thanks for pointing out the obvious. [23:16] jcsackett: It's not exactly obvious any more, since it's no longer visible to normal users. [23:16] wgrant: that's what confused me. i saw the help state that you had to get an lp team member to do it, and assumed script. [23:17] didn't think about the fact that an lp member has different permissions on the site and could do it from there. [23:17] Heh. [23:26] bdmurray: Can you share code with the thing that allows uploaders to do release targetting? [23:26] bdmurray: That code is wrong (doesn't take into account packageset permissions for teams, or something like that), but it should probably be shared. [23:28] thumper: I'd like a chat when you have a few minutes [23:28] thumper: not urgent, doesn't need to interrupt anything. [23:29] wgrant: I was looking at BugTask.userCanEditMilestone and _userIsPillarEditor but didn't get very far [23:30] bdmurray: There's a reason I gave up on fixing that myself, I presume. [23:31] * wgrant looks. [23:31] bdmurray: The current magic is BugNomination.canApprove. [23:32] wgrant: Isn't a nomination for a series not a milestone? [23:32] Most of the logic should probably be moved onto a method on BugTask, with canApprove just checking if the user can edit any relevant BugTasks. [23:32] bdmurray: Yes, but duplicating that code would be insane. [23:34] bdmurray: So it looks like you want to call distribution.main_archive.verifyUpload [23:34] Not checkUpload. [23:35] lifeless: I've got time now if you want to talk === spm changed the topic of #launchpad-dev to: Launchpad Development Channel | Week 4 of 10.10 | PQM is Release-Critical; devel is closed (Release manager: EdwinGrubbs) | firefighting: stable has test failures | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting [23:44] Can someone tell me the current status of https://rt.admin.canonical.com/Ticket/Display.html?id=41738 ("lpbuildbot needs a Lucid production buildslave") [23:44] ? [23:45] https://code.launchpad.net/~bzr/bzr-history-db/trunk [23:45] maxb: resolved [23:46] yay [23:46] * maxb unholds the meta-lp-deps pg8.4 MP [23:48] jelmer: If you have a moment, could you nudge https://code.edge.launchpad.net/~maxb/meta-lp-deps/pg8.4/+merge/32200 to Approved now? [23:54] maxb: otp, will have a look when I get back [23:55] Ooh, buildd-slavescanner.txt has finally been killed \o/