[00:18] bigjools, thanks :) [00:19] timrc: Why do you need to? [00:19] timrc: PPA packages by default build against all components. [00:22] wgrant, You won't like it... we want to upload proprietary packages to project PPAs and when we pull them into our archive "downstream" route them into a private component [00:22] wgrant, it would be easy to do this if we could publish packages to different components in the PPA [00:22] timrc: You're not wrong :) [00:22] Perhaps use different PPAs? [00:23] wgrant, that's an option.. was hoping for something more "elegant" [00:23] I campaigned years ago for custom component support in PPAs. [00:23] But ended up satisfied when they relented and implemented multiple PPAs per person [00:23] It works well for most things [00:25] Yeah, there is a certain appeal as it naturally enforces a good separation [00:25] I'll list it as an option for my team to discuss... we basically don't have a good way of handling this scenario right now, though [00:26] people should quit making proprietary software :) [00:28] That sounds like the easiest solution :) [00:36] Hm my 8 mo old daughter is trying to eat the heads off the meerkats on this 10.10 shirt... guess I better go feed her. Thanks for the assistance === timrc is now known as timrc-afk [00:38] it would be a step towards consistency to let ppas do components [00:42] Letting PPAs have Ubuntu's components is not particularly useful [00:42] Letting them have custom components might be [00:44] Except for PPAs intended to be copied to Ubuntu [00:45] I mean, custom components wouldn't hurt if done right, but in that case letting them have Ubuntu's components is useful [00:45] letting them have the same components would permit removing special case code [00:46] components do still need to be per-archive [00:46] (e.g. Debian) [00:46] yes indeed [00:46] Exactly. [00:46] They need to be per-archive [00:48] (DDs had some of this vaguely specced out, even up to dependencies between components and archives) [00:53] one month f changes to devel: [00:53] 586 files changed, 13951 insertions(+), 5928 deletions(-) [00:53] not so much stable on loc [00:56] 3659 files changed, 97472 insertions(+), 95512 deletions(-) [00:56] 2011-12-01 -> 2012-03-31 [00:56] Not so bad [00:56] thats better yes [00:57] 2345 files changed, 58185 insertions(+), 59554 deletions(-) [00:57] 2012-01-01 -> 2012-03-31 [00:58] 1025 files changed, 40735 insertions(+), 35055 deletions(-) [00:58] since new policy (so 2012-02-10 -> now) [00:59] lifeless: PPAs used to have components and we removed them [00:59] it confused the heck of out a lot of users [01:15] lifeless: https://lpstats.canonical.com/graphs/SourcherryCPU/20120314/20120413/ is interesting [01:15] 9.1 stabilised CPU usage and cut it by 40% [01:16] I wonder if it was the new slony [01:19] Nope, not slony [01:20] pg 9.1 perhaps === timrc-afk is now known as timrc [01:20] It was clearly at the time of the upgrade, and slony wasn't upgraded at the same time, so it was probably 9.1 itself, yes. [01:21] did pages get faster tho? [01:21] in theory I could just use a derived distribution overlay with no packages and upload a package to restricted, right :)? [01:22] yes [01:22] timrc: Hahahahaha [01:22] lifeless: We don't have a staging PPR, do we? [01:23] don't think so [01:23] pick a search thats slow hot and try it [01:23] on both prod & qas [01:24] cjwatson, when creating a PPA you could define the PPA as an Ubuntu archive or a Customer archive or something, but seeing as how this will never get developed, probably not worth discussing [01:25] Customer? Custom [01:25] lifeless: Bug searches aren't faster AFAICR [01:25] At any rate we'll figure out some way to horribly abuse LP, jk [02:18] flacoste: http://www.forbes.com/sites/stevedenning/2011/07/08/the-five-big-surprises-of-radical-management/ [02:29] wgrant: do you know where the bug is about lazr.restfulclient not respecting the type of resources in collections? [02:34] lifeless: wadllib [02:35] Bug #340935 [02:35] <_mup_> Bug #340935: Resources are only instantiated using the WADL predicted type, not the actual found type < https://launchpad.net/bugs/340935 > [02:42] * wallyworld__ is sad that his computer has started to spontaneously reboot after the latest updates were applied :-( [02:43] wallyworld__: maybe its building up credit [02:43] wallyworld__: so it won't have to reboot layer [02:43] hah [02:43] like LOC [02:44] indeed [02:44] if i'm lucky i can work for say 10 minutes each time [02:44] anything in syslog? [02:44] you could try putting tail -f /var/log/syslog in a terminal, pinned to top [02:44] might catch a glimpse [02:45] haven't looked yet, just trying to quickly land some branches first [02:47] * bigjools hides the Boris Spike code from wallyworld__ [02:47] who is Boris? [02:47] have you seen Goldeneye? [02:47] a long time ago [02:51] rick_h: around ? [02:52] lifeless: yea, just got home [02:52] hey [02:52] so I'm curious why you used PersonNotification manually [02:52] like [02:52] its a background sending thing [02:52] other uses of it use PersonNotificationSet.add(..) and don't call send [02:53] (I'm also curious why you used it at all vs just sending the message) [02:53] anyone know where traversal stuff like stepthrough is tested? [02:53] or the ++model++ stuff [02:53] lifeless: so for part 1, not sure. I just loaded up the object and that seemed to be the use. I could have misused it I suppose. loading it up now. [02:54] lifeless: for part 2. I used it after discussing ways of sending emails. A background process sounded good so requests weren't hung up on smtp/in band stuff [02:54] lifeless: I forget who pointed it out to be as a pretty light easy to use way to get emails scheduled [02:54] found the ++model++ tests [02:55] lifeless: either deryck or sinzui I think since I chatted with both of them in pre-impl. [02:56] lifeless: but yea, you're right, I shouldn't have called send and I shouldn't have patched send to override the email since it's not backgrounding like I wanted and defeats the purpose [02:56] compleetely missed that [02:56] except spelled better [02:58] so the reason we have background stuff is for teams [02:58] lifeless: ah, I see where I went wrong. So most code uses the Set object, but I wanted to override the To addr so I went after the raw PersonNotificatoin object [02:58] that may notify thousands [02:58] wgrant: LP.cache.context.information_type == undefined :-( [02:58] this is for notifying of changes to individuals [02:58] lifeless: ok, I just assumed it's good practice since things like slow/delayed smtp servers could cause request timeouts/etc [02:59] rick_h: we have local smtp on every server [02:59] rick_h: sending hundreds of mails is a problem, sending a fixed number isn't [02:59] rick_h: thanks, this helps me out [02:59] lifeless: ok, I did notice there seems to be a few ways to send emails out and when I mentioned background this was pointed out to me [02:59] StevenK: Ah, possibly because this is a bugtask, not a bug. [03:00] StevenK: Hopefully the bug is there as well. If not, put it there :) [03:00] lifeless: but yea, I did it wrong because when a user wants to change their preferred email I needed to send to their previous addr and PersonNotification doesn't support it [03:00] which is why I couldn't copy what other uses were doing directly [03:00] wgrant: self_link is the bug. [03:01] wgrant: Just trying to find where is defined [03:02] StevenK: Try LP.cache.bug.information_type [03:03] Ooh, that looks good. [03:03] * StevenK nails wallyworld_ to IRC. [03:04] can you make my pc stop rebooting? [03:04] lifeless: so while I'm fixing that in the morning the goal is to ditch the event as well? [03:04] I'd watch tail -f /var/log/syslog like lifeless suggested [03:04] Or downgrade what you recently upgraded [03:06] it just happens to quickly for me to see what syslog says [03:06] i will try downgrading but it was a large number of packages that were upgraded [03:06] rick_h: well, I'm doing a sketch on it as you're past EOD [03:06] Do you have another machine? [03:06] > 50 or 60 [03:06] nope [03:06] lifeless: yea, I've got to hit the sack soon. Couple of late nights for me [03:06] wallyworld_: No even running Windows? [03:07] my home theatre pc [03:07] lifeless: thanks, shoot me a message on status and I'll pick up/finish whatever needs doing in the morning. Thanks for pointing out my misuse of the notification. [03:07] missed in code review and my own look through it today [03:07] wallyworld_: Install PuTTy on it, ssh into your laptop and tail syslog over it? [03:07] StevenK: you are an ideas man. i may just try that. thanks [03:08] Joustling sticks? [03:08] 'Ideas man' just reminds me of The Castle. [03:09] tell him he's dreamin' [03:09] that's going straight to the pool room [03:09] why would you eat out? [03:11] wallyworld_: Does 'if (some boolean) {' do what I expect in JS, or should I say === true ? [03:12] StevenK: it will evaluate to false if undefined also [03:13] StevenK: so if that's what you want then that's ok [03:14] wow we are so inconsistent about From: [03:15] rick_h: btw, why does this generate multiple emails in those doc tests? [03:15] rick_h: are we also notifying separately to the new address? [03:15] rick_h: nvm, figured it - we're doing a token challenge [03:18] what stops teams getting ssh keys / gpgkeys ? [03:26] if params.distroseries is not None: [03:26] parent_distro_id = params.distroseries.distributionID [03:26] else: [03:26] parent_distro_id = 0 [03:26] wat [03:26] * wgrant blames lifeless [03:27] wtf is this [03:30] I don't recognise that [03:30] parent_distro sounds like UDD [03:30] It's in the structsub searching that you reworked [03:30] No [03:31] sorry, I mean DD [03:31] Still not :) [03:31] It's just the distro that owns the series [03:31] In this case [03:31] The constraint that uses it doesn't make much sense [03:31] Not to mention the whole defaulting to 0 thing [03:32] you've annotated the pre-move search to see it was me ? [03:32] No/ [03:34] revno: 14485 [merge] [03:34] I was on lave [03:34] leave [03:34] wgrant: sinzui [03:35] Still your fault :) [03:35] hah! [03:37] rick_h: btw telling folk that an ssh key *can't* be used anymore isn't important [03:37] rick_h: it doesn't open up security holes ;) [03:38] I think it's still interesting. [03:39] *adding* one is crucially important [03:39] Certainly [03:44] lifeless: yes, deryck thought that if someone removed your ssh key and potentially locked you out of things like bzr access you'd want to know why. [03:44] lifeless: but I agree, it's on the lower end of the notification scale [03:46] I think its nice in an auditord sense [03:46] I'm not sure its worth an email [03:47] not to worry, I'm keeping it [03:47] lifeless: yes, this discussion brought out a new bug I filed on adding some auditing support because if someone does change your email we can't verify what it was/you are who you say you are easiliy [03:47] stevenk has an auditord nearly ready [03:47] lifeless: or that we can generate a list of all the data a malicious user changed so that we make sure we can revert all the changes, etc [03:48] rick_h: log files :> [03:48] lifeless: ah, good to know then [03:48] lifeless: on the sql queries? Not sure what all we have there. [03:48] lifeless: it was brought up that we could try to do some sql backup/etc pulling, but painful [03:49] lifeless: but yea, that was deemed outside the scope of the current branch and the idea is to prevent it from happening [03:49] huh, no. [03:49] we email on these things [03:49] they are all important [03:49] so there should be email logs for them all [03:49] not in smtp, in our email engines. [03:50] that said, its 3 settings [03:50] preferred mail [03:50] (and all contact addresses arguably) [03:50] ssh keys [03:50] gpg keys [03:50] right, but we can restore data based off the email logs? [03:50] I think saying to someone 'we've nuked the settings, please set up the way you want' is a simple and solid answer [03:50] its not like restoring complex data [03:50] lifeless: ah ok [03:51] or to put it another way [03:51] even if we *did* restore it [03:51] any Ubuntu dev worth their salt would manually verify it all anyway. [03:51] right, I think the idea was more someone jumps in irc "I can't login!" "who are you? what's your email?...that doesn't match what I've got, let's see if it's changed and what it was?" [03:51] rick_h: another thing we should notify on is adding oauth tokens [03:52] rick_h: an audit trail for that would be good, and I'm +1 on having an audit trail [03:52] lifeless: ok, haven't seen taht [03:52] (again, StevenK :)) [03:52] lifeless: yea, cool [03:52] rick_h: when you allow an app access to your account, would you object to being told that that has happened ? [03:53] sorry, haven't seen the oauth stuff, cool on the WIP audit trail [03:56] lifeless: not sure, I've got to check the oauth use. Is that just for apps? scripts using api? [03:56] lifeless: I guess it should be rare I get these right? Changing keys, email is pretty rare. Annually maybe in practice worst case? [03:57] right [03:57] generally its one-time setup [03:59] rick_h: did you do GPG keys when you did this ? [03:59] rick_h: or do they already notify ? [03:59] yea, they already get an email because you have to decrypt your email with the key to verify [03:59] so gpg keys aren't in this [03:59] does that go to the preferred address [04:00] or the address in the key [04:00] preferred [04:00] cause if it goes to the address in the key it would be useless [04:00] I believe, I'll have to dbl check I guess [04:00] I believe it goes to the address in the key [04:01] so we know they aren't impersonating someone via the key [04:03] k, I'll double check. It's hidden in some browser action so I've got to find it again. [04:03] now where is that doc on Launchpads voice [04:04] https://dev.launchpad.net/UserInterfaceWording/CanonicalStyleGuide?highlight=%28voice%29 looks like what you might mean? [04:05] https://dev.launchpad.net/UserInterfaceWording [04:07] bah, I fail at going to bed during the right day [04:08] I'm really gone to bed this time. Will look over the wording docs lifeless, thanks for pointing out. Looks like I wrapped wrongly off the bat [04:09] rick_h: :P [04:09] rick_h: gnight [04:24] oh wow [04:24] ? [04:24] thats a fun hole [04:25] we validate a preferred but unvalidated email address when a different preferred email address is set [04:25] Preferred but unvalidated? [04:25] what? [04:26] lifeless: Anything setting an unvalidated email to PREFERRED without first validating it is a bug. [04:26] Preferred > validated [04:30] wgrant: as I said [04:30] 'thats a fun hole' [04:31] person.py [04:31] look for existing_preferred_email.status = EmailAddressStatus.VALIDATED [04:31] lifeless: An existing preferred email is by definition already validated. [04:31] Because we're setting it from preferred to validated [04:31] wgrant: oh, I see, right [04:31] so this could be more clear [04:31] And preferred is stricter than validated [04:31] also, we have to totally different concepts conflated. Win. [04:31] Yes. [04:31] Win [04:32] I would model this with preferred as a pointer from person, constrainted by trigger checks to be validated. [04:32] anyhow [04:32] thats then, this is now [04:32] The idea was to avoid triggers, I believe. [04:32] Or you could also have a preferred flag [04:33] and a composite unique index [04:33] or whatever [04:33] With UNIQUE emailaddress(person) WHERE preferred [04:33] Right [04:33] It's insane. [04:33] But 2005 [04:33] So we sort of have to live with it [04:34] It doesn't sound that hard to shift away. [04:34] its straight forward [04:34] little returns [04:34] probably not the most significant thing to improve right now [04:34] More straight-forward than bug.{private,security_related} -> bug.information_type :-) [04:35] No [04:35] Only a few places touch that [04:35] There are dozens upon dozens of places that touch emailaddress.status == 4 [04:45] rick_h: I've pushed an opinionated branch to https://code.launchpad.net/~rharding/launchpad/email_notice_959482/+merge/99995 [04:45] rick_h: its about half the size, a significant chunk of which was the event overhead + the need for dedicated per-event tests. [04:46] wallyworld_: O hai. [04:46] rick_h: let me know what you think, I obviously don't want to be blocking you in any way. [04:47] wallyworld_: Since I moved the ChoiceSource stuff into its own module, Y.one('body') seems to return the enclosing div. [04:47] StevenK: that seems strange at first thought [04:48] wallyworld_: It seems to be strange. If I change it to a private type, the enclosing div turns red. [04:49] StevenK: Y.one('body') should return the html element [04:49] Yes, agreed. [04:50] fiik what's up then :-( [04:51] have you looked at the html source in the browser to make sure there isn't anything stupid in there? [04:51] or tun it through an html validator? [04:51] run [04:52] Now it works. I think it's toying with me. [04:52] like my pc hardware [04:53] wallyworld_: Oh, could I get your help with the JS fallback? If I set href to the link in the TAL, the JS never fires. [04:53] StevenK: you have set the onclick handler? [04:53] I thought the ChoiceSource does that for me? [04:54] yes, should do [04:54] i have a quick look at the source code to see if it is doing anything dumb [04:56] Instance i-4ea81529 starting................................................................................................................................................................................... started on ec2-67-202-7-98.compute-1.amazonaws.com [04:56] Started in 15 minutes 46 seconds [04:56] odd [04:56] wgrant: Orsum. [04:57] wallyworld_: Sigh, that works now too. [04:57] StevenK: maybe browser caching issue? [04:57] Hm. Maybe. [05:03] AssertionError: The vocabulary must implement IEnumeratedType [05:03] :-( [06:16] blink [06:17] lp.bugs.model.bugtasksearch._build_pending_bugwatch_elsewhere_clause [06:17] First bit, 'if params.product' [06:18] AND RelatedBugTask.id = BugTask.id [06:18] Am I not thinking it through correctly, or is that inverted? [06:23] Ah [06:23] @property [06:23] def pending_bugwatches_url(self): [06:24] if not IProduct.providedBy(self.context): [06:24] return None [06:24] So that codepath is never exercised, I guess [06:27] wgrant: did you sstart a thread on tag sort order on the list ? [06:28] No, I'll do that now. [07:44] good morning === almaisan-away is now known as al-maisan [08:45] Does anyone have a spare second to run a test for me in their lp dev environment? [08:45] gmb: p'rhaps [08:46] lifeless, bin/test -t externalbugtracker-bug-imports.txt if you'd be so kind. [08:47] hah, out of date schema [08:47] making new [08:47] lifeless, Oh ,wait. [08:47] No need. [08:47] I see why it's failing [08:47] It's because I'm stupid. [08:47] do -not- give me straight lines like that [08:48] :) [08:48] I won't be able to help myself if you do [08:48] Turns out that it helps if you're running tests in a Lucid (or Oneiric) container, not on Precise. My lxc container had exited and I hadn't noticed. [08:48] ah yes, that will mess you up [08:49] I don't have a non-lxc postgresql, helps avoid that confusion [08:49] Yeah, I think this is my cue to nuke mine :) [08:49] Otherwise I'm going to do this again. [08:49] ah, insanity! [08:50] :) [08:54] it's wrong to be so happy with only having 1 ticket in my RT queue but I am :D [08:54] rt queue has been fully smacked :D === al-maisan is now known as almaisan-away [11:27] jml: Hai. Are you able to QA r15074 today? [11:28] StevenK: I guess. [11:28] StevenK: I'm surprised that's how it works though [11:29] jml: Really? Developers are mostly responsible for their own QA. In this case, you should be able to ask the webops to make you an admin on qas for short amount of time. [11:30] StevenK: I'm trusted as a judge of behaviour but not of code quality? [11:30] anyway [11:30] jml: In what way, that you need a review? [11:31] StevenK: yes. [11:32] jml: Everyone has their branches reviewed, except if the change is small and well-understood, in which case, graduated reviewers can self-review. So don't feel like it's just you. [11:33] jml: And you wrote the feature so you should have a very good understanding of how it is supposed to behave. [11:33] Yeah, that's exactly how it works. [11:34] StevenK: anyway, I can QA the feature today (UK). [11:35] jml: Thanks. If you don't feel comfortable QA'ing, then please feel free to hand it off to a LP dev if they agree to do it -- we just tend to assume the developer who wrote it will do the QA. === matsubara-afk is now known as matsubara === almaisan-away is now known as al-maisan [12:54] jml: are you available to review my branch which will (soon) bring the TSFM tests inline with the improved tests on your branch? [12:54] benji: yes, roughly speaking [12:54] benji: I mean, I can do it today. A synchronous review is probably beyond me. [12:55] jml: that's cool, thanks [12:56] jml: while I have you on the line, do you know anything about TestThreadSafeForwardingResult.test_forwarding_methods and why it does what it does (testtools/tests/test_testresult.py) [12:56] I don't understand the underlying intent of the test. And the sparse comments in the test seem to disagree (or at least are irrelevent) to what the test actually verifies. [12:56] looking === rick_h changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: rick_h* | Firefighting: - | Critical bugtasks: 4*10^2 [12:58] benji: No idea, sorry. [12:58] jml: thanks anyway [12:58] benji: perhaps ask lifeless when he gets back online. [12:58] yep === jcsackett changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: jcsackett, rick_h* | Firefighting: - | Critical bugtasks: 4*10^2 [13:58] abentley: mumble? [13:59] adeuring: coming [14:02] adeuring: https://pastebin.canonical.com/63588/ === al-maisan is now known as almaisan-away [14:58] abentley: comments on teh review, please also make the LoC note in the MP. [15:01] rick_h, hey, dude. I can chat about your branch now. 2 minutes to warm coffee and meet in a hangout, cool? === salgado is now known as salgado-lunch [15:03] deryck: sounds god [15:03] good [15:08] jml: https://code.launchpad.net/~benji/testtools/modernize-tsfr/+merge/101749 awaits when you have a moment. [15:09] benji: ta [15:12] benji: have looked at it long enough to know that I need to have a detailed look. [15:12] jml: ok; thanks [15:12] benji: will try to get to it today (UK) [15:12] jml: I appreciate it. [15:18] rick_h: Thanks. done. [15:34] jcsackett: The imports in runner are explained by the comment "Avoid importing from lp.services.job.celeryjob where not needed, to avoid configuring Celery when Rabbit is not configured." [15:35] * jcsackett looks again [15:35] abentley: so it is, ignore my blind eyes. :-) [15:35] jcsackett: cool. I'll add a comment to the other one, though. [15:35] thanks, abentley [15:42] rick_h: could you please have a look at this mp: https://code.launchpad.net/~adeuring/launchpad/abort-transaction-in-job-queue/+merge/101762 ? [15:42] will do adeuring I'll put it next on my list [15:42] rick_h: thanks! [15:48] gary_poster, hi. got a sec? [15:58] Looks like wordpress's autosave feature is broken on Lp's blog and I just lost 90 minutes of writing when it's JS went tits up. It then insults me by saying "howdy" when I log back in. If I could accomplish one thing today, please let me stab the cowboyism to death. [16:00] benji: have replied. am available to chat in real-time if you want. [16:00] jml: cool, I'll read it now [16:02] jml: interesting; if your understanding of TSFR is correct, I don't see that it ever achieved it's aim, which is entirely possible [16:02] benji: I'm pretty sure it's got very close. [16:03] my understanding (gained only though reading the code) is that it intended only to make sure individual calls did not overlap, not that calls for different tests didn't overlap [16:04] benji: startTest() sets the test start tim, and then on receiving $RESULT, then runs some semaphore-protected code that does (ignoring tags): [16:04] * time(start_time) [16:04] * startTest(test) [16:04] * time(now) [16:04] * add$RESULT(test, *args, **kwargs) [16:05] * stopTest(test) [16:06] multiple threads, each with their own TSFR, but with a shared semaphore and a shared target result object, ... [16:06] imagine the target result object dumping subunit output [16:07] it *has* to be: [16:07] test: foo [16:07] success: foo [16:07]