[00:00] bug 449105 [00:00] Malone bug 449105 in launchpad "logging into sourceforge with openid breaks" [Undecided,New] https://launchpad.net/bugs/449105 [00:00] Bug #449105: logging into sourceforge with openid breaks [00:00] Bug #449105: logging into sourceforge with openid breaks [00:00] Bug #449105: logging into sourceforge with openid breaks [00:01] oh joy [00:01] happy mup [00:06] lifeless: I think it's SF.net misbehaving. [00:06] lifeless: It's only half-respecting the delegation. [00:06] * thumper away doing emails [00:06] wgrant: happens with apache2-mod-openid [00:07] wgrant: which I was trying to setup last week, looks exactly the same [00:07] lifeless: The proprietary Canonical fork, or an open one? [00:07] apt-get install [00:07] Aha. [00:07] * wgrant checks the source. [00:07] wgrant: what proprietary canonical fork? [00:08] lifeless: https://launchpad.net/apache-openid [00:10] wgrant: looks to be all handed back to me [00:11] wgrant: oh, and I think its a different code base anyhow - there are *two* apache openid modules [00:11] yeah, thats mpopenid, not openid [00:12] wgrant: (mod python openid is what we use, apache2-mod-openid is written in C I think) [00:12] Ah. [00:16] The author of libopkele seems to have had a vendetta against whitespace. [00:25] http://freshmeat.net/projects/libopkele/ ? [00:26] That one. [00:27] you're tracking down a root cause? [00:27] I am. [00:36] cool, thank you [01:01] * mwhudson lunches [01:10] lifeless: c-i-p bug. It's not present in the copy that I of course didn't retrieve from lp:launchpad's history, because I'm not allowed to do that. [01:10] lifeless: All the declarations in the page are fine, but c-i-p now provides XRDS for OpenID 1.0 and 1.1, as well as 2.0 (which it always has) [01:11] The XRDS overrides the declarations on the page itself, and doesn't specify the identity to delegate to. [01:11] Just the server. [01:14] c-i-p ? [01:14] oh, canonical-identity-provider? [01:15] wgrant: I presume you'll put the details into my bug? [01:15] lifeless: Yep, doing so now. [01:15] thanks! [01:16] I was very confused for a while before I noticed the XRDS link. [01:16] Because identical delegation code worked fine elsewhere. [03:17] kfogel: http://www.pmease.com/features/screentour/ [03:18] kfogel: /nice/ 'tour' presentation [03:27] wgrant: that buildd issue - *should* be sorted now [04:03] spm: Thanks. What was the issue? [04:03] firewall upgrade and subtle funkies thereof [04:04] Lovely. [04:07] excluding the ones I did - which always worked perfectly. natch - I've never known a firewall upgrade/change to go perfectly. ;-) [04:08] my mates at $job-1 went thru 3-5 iterations of pain trying to work with the gateway provider (as it so happens $job-2 :-) ) [04:08] where each iteration is around 5-8 hours from midnight [04:08] Urgh. [04:08] and that was *just* the final firewall to the corporate network [04:09] not including the 4 or so DMZ ones just for their webserver farm [04:09] each of which has 13-17 active interfaces. \o/ [04:09] I *so* don't miss working at $job-2 :-D === abentley1 is now known as abentley === ursula is now known as Ursinha [08:34] good morning [09:24] Morning! [10:07] bigjools: I need some hints on writing tests for the ddeb stuff. [10:07] heh [10:08] you mean you didn't write the tests first? :) [10:08] No, no, I am writing them first. [10:08] I'm not that crazy. [10:08] ok [10:09] you need to figure out all the places in the code where we're likely to need to keep a ddeb change in parallel [10:09] I hope this is mostly in BPPH [10:09] It should be. [10:09] and if not we should move code into BPPH [10:09] I suspect nascentupload will be a pig [10:09] Does anyone mind if I set fire to checkwatches? No? Good. [10:10] and PackageUpload too === mthaddon` is now known as mthaddon [10:10] once you figure that out, write tests to match your expected behaviour, in fact you can probably just alter the existing ones on BPPH for copy, override etc [10:10] nascentupload gets a bit hard, as BPR is immutable but BPRs will now need to refer to other BPRs created in the same upload. I guess I'll need to create the DDEBs first. [10:11] hmmm interesting, what is making BPRs immutable? [10:11] DB and Zope permissions. [10:11] No user has more than SELECT+INSERT on BPR. [10:12] Which seems sane. [10:12] well we can change it if it makes sense [10:12] Actually, that can't be right. [10:12] Because queue overrides modify the BPR, don't they? [10:12] * wgrant looks again. [10:12] nascentupload runs zopeless === mrevell changed the topic of #launchpad-dev to: This is Launchpad Development Channel | Week 1 of 3.1.10 | PQM is OPEN | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in #launchpad-reviews | Use http://paste.ubuntu.com/ for pastes | This channel is logged: http://irclogs.ubuntu.com/ [10:14] OK, so queued is the only real thing that can UPDATE BPR. I guess uploader could be allowed to. [10:14] But anyway. [10:14] wgrant: you probably won't need to [10:15] change the sql config I mean, the uploader won't have committed anything so you're only updating the storm copy [10:15] It will have flushed it. [10:15] Because storeInDatabase does queries of some kind. [10:15] bah [10:15] I've tested this; it is problematic in practice. [10:15] ok [10:15] fix security.cfg then [10:16] OK. [10:17] Now, the tests of new nascentupload functionality involve checking state at various points over a sequence of about 5 uploads. I think it might be neatest to have it all as one big doctest, although they seem to be losing favour... [10:17] hello. [10:18] yeah, I'm unsure of which is the best direction, but possibly a combo of both [10:18] hi jml [10:18] did you know there was a TV station called JML in the UK? :) [10:18] no, I didn't. [10:19] or maybe mrevell told me once [10:19] or twice [10:19] wgrant: celso was moving towards using more unit tests as they are usually more focused on the actual test conditions [10:19] bigjools: To my shame, I've told him all about the products of JML. [10:19] repeatedly [10:19] :) [10:20] wgrant: there is one doctest that's an end-to-end test though, but it's currently disabled [10:20] mrevell: ha :) [10:21] bigjools: Now, this nascentupload-supertest will also end up testing for correct supersedure and overriding in the course of what it needs to do. So it's not strictly nascentupload-only. Ew. [10:22] wgrant: that will be already tested somewhere (I forgot where) [10:23] bigjools: So I should integrate the more specific DDEB tests alongside the existing tests for those methods? [10:23] wgrant: yes, I think so, that would be the first place I'd expect to see them I think. [11:17] jml: good morning [11:17] mwhudson, hello [11:17] mwhudson, I just sent you an email. [11:17] jml: so you did [11:18] jml: thanks for the mail [11:18] mwhudson, np. === statik` is now known as statik === jml changed the topic of #launchpad-dev to: This is Launchpad Development Channel | Week 1 of 3.1.10 | I am Zero OOPS and So Can You! http://is.gd/4fkLl | PQM is OPEN | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in #launchpad-reviews | Use http://paste.ubuntu.com/ for pastes | This channel is logged: http://irclogs.ubuntu.com/ [12:43] thumper: still up? === mrevell is now known as mrevell-lunch [13:01] bigjools: I see you are working on a branch to allow customisation of the first PPA's name. Does it still default to 'ppa'? [13:05] no === bigjools is now known as bigjools-phone [13:06] bigjools: That seems like a bad idea. The display names that people come up with are bad enough. [13:09] gary_poster, good morning [13:11] jml, good morning. [13:22] gmb, if I wanted to learn about how our external bug tracking system works and what it does & doesn't do, where'd be the best place for me to look? [13:23] jml: Well, there's actually not all that much in-depth documentation for it (I've been meaning to write some). But there are some wiki pages that are relevant. Let me find them for you. [13:23] jml: Also, if you want to hurt yourself. there's lib/lp/bugs/externalbugtracker/* and lib/lp/bugs/scripts/checkwatches.py. The latter is a bit of a mind-bender. [13:24] gmb, heh :) [13:25] gmb, maybe a good idea is for me to read what docs there are & have a chat w/ you sometime after. [13:25] jml: Hmm, turns out that my dream of relevant wiki pages was optimistic... I can only find https://help.launchpad.net/FeatureHighlights/BugWatches, which says nothing useful. [13:26] jml: Although, I did write this just the other day, which should tell you a bit about what's wrong with the current setup: https://dev.launchpad.net/Bugs/CheckwatchesNG [13:26] gmb, thanks. === bigjools-phone is now known as bigjools [13:46] wgrant: they are *Personal* PAs, and it can't be any worse than "ppa" ... ! === mrevell-lunch is now known as mrevell [13:52] bigjools: The old default ppa/'PPA for Some User' seems better than what people are coming up with now. There are no examples for what is appropriate. [13:52] why do you think it's necessary to do that though? I'm not against it, just keen to know why. [13:53] as an aside, I want to change the interface description to make it obvious that the name appears in the URL and the description in the GPG key [13:54] I bet a lot of people are going to start naming their first PPA with their username. [13:54] Is the display name used in the OpenPGP key? I thought it was 'Launchpad PPA for $owner'... but I haven't checked the code. [13:54] from memory it is [13:54] but I've been wrong before :) [13:57] Urgh. You're right. [13:57] That sounds like a bug. [13:57] As it's now shared between all PPAs. [13:57] hmmm good point === EdwinGrubbs2 is now known as EdwinGrubbs === barry` is now known as barry [14:51] gary_poster: ping [14:52] barry: hey are you taking today off? [14:52] gary_poster: nope, swapping it. how about yourself? [14:52] barry: me too, cool! salgado has today off though. on call will ping you [14:52] gary_poster: np === barry` is now known as barry [15:09] barry: yo, let's start [15:09] barry, want to go to another channel? #launchpad-sprint or something silly? [15:10] gary_poster: +1 [15:10] cool === stub1 is now known as stub === intellec` is now known as intellectronica [16:12] barry, am I safe to assume there is no call today? [16:12] rockstar: yep [16:13] * rockstar has been listening to smooth jazz for too long... [16:17] * jml has been listening to _The Score_ for too long. [16:33] rockstar, abentley: https://code.edge.launchpad.net/~launchpad/launchpad/python-migration [16:35] rockstar, abentley i'm deleting that branch, so nm ;) [16:35] barry, okay. [16:36] barry, I think abentley is off on a national holiday today. [16:36] rockstar: oh right. anyway the branch had a big warning that said just: MemoryError [16:36] barry, eep. [16:37] rockstar: might be reproducible by: branching off of stable; pushing w/no stacking (or with a --stacked-on that lp will deliberately ignore); wait for the puller to bomb [16:38] barry, yeah, I could see how that might be it. [16:38] I wonder if there's an open bug about that. === andrea-bs_ is now known as andrea-bs [16:47] adeuring, i have a question about bug 402126 [16:47] Bug #402126: top level publications must be public [16:47] Malone bug 402126 in lazr.restful "top level publications must be public" [Undecided,New] https://launchpad.net/bugs/402126 [16:47] Bug #402126: top level publications must be public [16:47] Bug #402126: top level publications must be public [16:48] leonardr: yes? [16:48] presumably you tried to publish the hwdbapplication object as a top-level object, and it didn't work [16:48] what was the failure? [16:48] leonardr: I must admit that I can't remeber the details... [16:49] leonardr: give me some time to reproduce. [16:49] adeuring, sure [16:55] rockstar, i have a question about bug 326307 [16:55] Bug #326307: Need @property equivalent for the API [16:55] Malone bug 326307 in lazr.restful "Need @property equivalent for the API" [Low,Triaged] https://launchpad.net/bugs/326307 [16:55] Bug #326307: Need @property equivalent for the API [16:55] Bug #326307: Need @property equivalent for the API [16:55] what is the accessor method you'd like to publish as a property? [16:55] i'd like an example for a doc i'm writing [16:56] leonardr, looking. [16:57] leonardr, hm, it's been a while since I wanted that. Lemme look around. [16:57] I'm sure I can find something. [16:57] rockstar, thanks [17:06] leonardr, how about IBranch.getPullURL exposed as IBranch.pull_url [17:06] ok, great [17:20] leonardr: I believe this patch re-enables HWDBApplication as a top-level publication (with access restriction): http://paste.ubuntu.com/291685/ Results in this error: http://paste.ubuntu.com/291687/ [17:26] adeuring: can you just except link_name from launchpad.View protection? === EdwinGrubbs is now known as Edwin-lunch [17:27] it might be better not to show hwdb at all, but then it wouldn't show up in the wadl [17:31] leonardr: any hint for how to exempt link_name from the rule? A simple addition of "allow attrinbutes="link_name" leads to the error "Failed to load application: Conflicting configuration actions" [17:53] adeuring: unfortunately the only way i've found to do that is to restrict every field *except* the one you want to allow [17:53] there's a slight chance gary will have a better idea [17:54] leonardr: that would be good. Defining the complete set of restscited attributes would not be very convenient... === bigjools is now known as bigjools-afk [18:16] adeuring: no, you can either arrange the interfaces to suit your needs better, or define the set, I'm afraid. [18:16] that's something we could theoretically improve. if you wanted to put a bug into foundations I could see how much support there is for it. describing what you want in a clear, compelling, and general way would probably help the sales pitch when I present it to others. Think of it as trying to sell it to your team leads. :-) [18:17] gary_poster: OK; I'll add a comment to bug 402126 [18:17] Malone bug 402126 in lazr.restful "top level publications must be public" [Undecided,New] https://launchpad.net/bugs/402126 [18:17] Bug #402126: top level publications must be public [18:17] Bug #402126: top level publications must be public [18:17] Bug #402126: top level publications must be public [18:17] heh, mup + ubot3` == excitement! [18:18] adeuring: it's more of a problem in the zcml spelling [18:18] gary_poster: yes, maybe [18:18] IOW, we would be redoing some zope bits to try to do what you want. I think. :-) eh, write up what you think you want and we'll see. ;-) [18:26] Night all [18:47] bac, are you approving CPs? Is anyone to your knowledge? [18:47] (and hi btw :-) ) [18:48] gary_poster: i am not. my RM powers expired a while back. you'll have to talk to flacoste [18:49] bac ok thanks. I figured, but was worth a check. (flacoste's out on nat. holiday afaik) [19:06] * rockstar lunches === EdwinGrubbs2 is now known as EdwinGrubbs [20:16] barry: You appear to have removed all my commentary on why particular tests failed [20:17] maxb: dang. sorry for the hamfisted wiki editing. i'll try to revert that [20:17] maxb: btw, you are very welcome to join gary_poster and myself on #launchpad-sprints! [20:18] s/sprints/sprint/ [20:18] right, sorry. and changes reverted [20:31] barry: hi, can you mark this mp as reviewed so I can use "ec2 land" on it? https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-430708-registry-windmill-layer/+merge/13148 [20:33] EdwinGrubbs: done [20:33] thanks [20:36] mwhudson, when you're around, I'd like to chat with you about upgrading stacked branches. [20:45] rockstar: ok [20:46] rockstar: i'm definitely not caffeinated enough for that yet, will let you know :) [20:46] mwhudson, okay. I'm assuming we'll have to be on the phone together for it. [21:07] rockstar: are you working today? [21:07] thumper, why wouldn't I be? [21:08] rockstar: the email said it was a public holiday [21:08] thumper, what email? [21:08] the staffing email [21:08] thumper, ah, well, I guess I should have read that then. :) Yes, I'm indeed working today. [21:08] * thumper runs to drop off Maia [21:14] i guess someone should look at the launchpad failures with bzr.dev at some point [21:17] mwhudson, yeah, I keep wondering that. I get those emails and wonder if maybe one of use should respond about them. [21:20] :) [21:21] thumper did an accurate summary of the failures a while ago [21:21] the part that's really sigh-inducing is the plugin api versioning nonsense [21:22] mwhudson: skype? [21:22] thumper: as usual i'm online [21:23] thumper: did you try calling? [21:51] Intriguing. Breaking LP OpenID is Low, I see. [22:22] wgrant: ? [22:37] lifeless: The bug I diagnosed yesterday was marked as a duplicate, and the original became Low. [22:38] rockstar: I guess we are done then [22:38] thumper, oops. I didn't mean to click the hang up button. [22:38] :) [22:38] np [22:38] Stupid touchpad sensitivity... [22:44] wgrant: meep :( [22:44] wgrant: bug number? [22:47] found it [23:52] * mwhudson -> into town, back online in a bit