[00:10] I've cleaned everything but the eggs cache === beuno-afk is now known as beuno [00:17] gah, what? [00:18] devel works, my python2.5 branch does not [00:18] But there's nothing there that goes anywhere near lazr anything [00:19] Getting distribution for 'lazr.uri==1.0.1'. [00:19] Installing lazr.uri 1.0.1 [00:19] Caused installation of a distribution: [00:19] lazr.uri 1.0 [00:19] with a different version. [00:19] Got None. [00:19] While: [00:19] Installing. [00:19] Getting section filetemplates. [00:19] Initializing part filetemplates. [00:19] Error: There is a version conflict. [00:19] We already have: lazr.uri 1.0 [00:19] What on earth is buildout attempting to tell me? [00:25] oohkay, purging python-lazr-uri .deb made it work... [00:26] isn't buildout supposed to be insulating you from system python versions? [00:34] /home/maxb/launchpad/lp-branches/python2.5/lib/canonical/config/__init__.py:19: UserWarning: Module lazr was already imported from None, but /home/maxb/launchpad/lp-branches/python2.5/lib is being added to sys.path [00:34] !? [00:45] maxb: there was a fix to lazr.config today I believe [00:50] * maxb observes that the zope sourcedep still isn't completely obsolete, if the symlinks are to be believed [00:57] OK after cleaning absolutely everything, now I can't "make" devel eitehr [00:57] either [00:57] :( [00:57] my make is failing too in mailman [00:58] heh, at least you got that far [00:58] mwhudson: hey, build engineer [00:59] thumper: as of monday! :) [00:59] *next* monday [00:59] :) [00:59] does your make work? [00:59] dunno [00:59] running pull now [01:00] i just read something about make in mailman failing from last week, francis said he'd fixed it though [01:01] mwhudson, hello [01:02] thumper, I was just looking at https://bugs.edge.launchpad.net/launchpad-code/+bug/418290 [01:02] Bug #418290: Person's code facet has unclear section linking to the teams the person is a member of [01:02] jml: yeah [01:02] jml: hello [01:02] File "/home/tim/src/lp/devel/eggs/zope.proxy-3.5.0-py2.4-linux-i686.egg/zope/proxy/_zope_proxy_proxy.py", line 6, in __bootstrap__ [01:02] imp.load_dynamic(__name__,__file__) [01:02] ImportError: /home/tim/src/lp/devel/eggs/zope.proxy-3.5.0-py2.4-linux-i686.egg/zope/proxy/_zope_proxy_proxy.so: cannot open shared object file: No such file or directory [01:02] thumper, can we simply fix the links to link to the code listing? [01:03] jml: there was a "fix" recently that changed the fmt:link to always go to the root site [01:03] jml: we may be able to get it to go to the code site [01:03] jml: I've not yet investigated [01:03] jml: I think it is just team/fmt:link/code [01:03] (or branches) [01:04] So, what is buildout *supposed* to do if I have lazr.uri 1.0 installed as a .deb but versions.cfg says 1.0.1 ? [01:04] it is a bit of a mixup [01:04] thumper, yeah, I think the 'fix' is a good idea [01:04] maxb: look in your download-cache [01:04] thumper, but it's different from forbidding links to particular facet pages :) [01:05] thumper, and it seems that linking to the team branch listings actually is desirable [01:05] mwhudson: Hmm. it appears to be throwing its hands up in horror and bleating incomprehensible error messages instead [01:05] it is there [01:05] maxb: i'm not completely surprised [01:05] right. [01:05] jml: the bug was that normally a link to a person or team should take you to the overivew [01:05] jml: but in this case we should link to listings, and probably give it a better title too [01:05] s/title/heading [01:06] thumper, I was about to file a bug about the heading, actually :) [01:06] jml: file away and assign to me [01:06] thumper, will do. [01:08] grrr [01:08] how can I code if I can't run the tests [01:08] a rhetorical question [01:09] * jml thinks about the new project page [01:14] leonardr, hello? [01:16] leonardr, I have three simple questions for you, if you have a moment. [01:32] is spm back today? [01:32] aye [01:34] spm, welcome back :) [01:34] spm, feeling better? [01:35] jml: yeah muchly. The joy of children and bringing home colds from school.... [01:35] heh [01:36] spm, my todo list says I have a couple of things to ask you about [01:36] spm, one of them being bazaar.edge.launchpad.net [01:41] maxb: I blew away the contents of the /eggs dir, and make clean build worked ok [01:53] * mwhudson lunches [01:58] jml: since i forgot to quit irc, you may ask your questions :) [01:58] leonardr, heh, thanks :) [01:59] leonardr, first, what should I call the constant for the production API service root. I've picked LPNET_SERVICE_ROOT for now, but STABLE_ and PROD_ might also fit. [01:59] leonardr, second, what should the URL be? I've got https://api.launchpad.net/beta/ now, but the 'beta' troubles me a little. [02:00] jml: i would go with PRODUCTION or STABLE [02:00] lunch time [02:00] leonardr, third, when I authorize an app for the production service, I am taken to an edge web page (probably because of the beta user redirect). Is this a bug? [02:00] look at the other constants and do what they do. i believe they also have /beta/ [02:01] leonardr, they also have beta. [02:01] leonardr, so I should follow what they do? [02:01] yes, the constants are going to behave like "HEAD" or "trunk" in a version control system [02:01] they will point to the most current version of the web service [02:01] which right now is /beta/ [02:01] * jml prefers 'PRODUCTION' to 'STABLE', since the latter might have API stability connotations. [02:01] sounds good [02:02] #3 is a bug -- you're in the launchpad beta testers team, so your requests to lpnet redirect you to beta [02:02] flacoste filed this bug a couple weeks back [02:02] s/beta/edge/ [02:02] cool. [02:02] leonardr, thanks for that. I'll land the patch with the new constant then. [02:03] great [02:04] Is the beta team restriction in place on the lpnet API vhost? [02:04] wgrant: yes, that's the bug [02:05] leonardr: Not the same bug... I mean, api.edge.launchpad.net is restricted to beta testers. Is api.launchpad.net? [02:05] ah [02:05] i don't know. are you sure that restriction is still in place? i thought we got rid of it [02:05] I don't know. [02:05] but if it's still in place on edge, it should still be in place on lpnet [02:05] It is certainly in place on edge. [02:06] I saw somebody complaining about it on identi.ca earlier in the week. [02:06] maybe that's another thing we should change [02:06] Restricting it on edge might be reasonable. [02:07] It's not restricted on edge. I used the api long before I joined the beta-testers team [02:08] Hm. [02:08] Then why was it crashing for a non-beta-tester, I wonder.. [02:10] Huh, you're right. It's not restricted on edge (or at least wasn't 1.5 months ago). === kiko is now known as kiko-zzz [02:17] huh. So the testsuite no longer runs unit tests first? [02:20] It did for me yesterday, but that was without the new zope.testing. [02:30] oh right. [02:30] new zope.testing. [02:31] * jml has work to do! [02:34] jml: What are you doing? [02:35] wgrant, enabling parallelization of the Launchpad test suite [02:35] Ooh. Excellent. [02:35] but also, paging the new zope.testing version into my brain [02:35] so I can work on making sure it's stdlib unittest compatible [02:35] (mostly so that lifeless's unit testing work can be applied to it) [02:38] subunit? [02:47] wgrant, subunit and testresources [02:47] and ideally, much of the work in bzrlib.tests and bzr-ec2test, but let's not get ahead of ourselves [02:59] Argh. 844 line diff :( [03:01] wgrant, most reviewers can be bribed to review diffs over the allowed size [03:02] particularly with code that is an improvement [03:03] Almost half of it is new tests. [03:05] thumper, beuno: Is https://bugs.edge.launchpad.net/launchpad-code/+bug/236442 much closer to being fixed now that we have spark lines? [03:05] Bug #236442: Show how active I've been [03:06] Well, you don't have sparklines any more... [03:06] jml: I killed them [03:07] jml: I intend to bring them back, but they need fixing [03:07] thumper, that seems perfectly fair to me. [03:07] thumper, I think that the bug I mentioned doesn't benefit from sparklines so much as actual graphs. [03:07] yeah [03:09] * thumper primal screams [03:09] mwhudson, just re-discovered https://bugs.edge.launchpad.net/launchpad-code/+bug/85326 [03:09] Bug #85326: Codehosting server should initiate a pull attempt [03:09] thumper, wassup? [03:10] the query for person active reviews is an arse [03:10] we need two different base collections [03:10] and different queries [03:10] unioned [03:10] jml: "hmm" [03:15] mwhudson, it's apparently my oldest open bug on launchpad-code [03:18] jml: that's quite a surprise [03:21] mwhudson, yes. [03:29] bloop [03:32] thumper, when you've got time, I'd like to talk about Monty Taylor's post to launchpad-users [03:33] yeah, I'm actually trying to get work done right now :( [03:33] thumper, understood. [04:00] jml: i think, probably, 85326 should be wontfix/subsumed by our eventual message queue stuff [04:00] jml: in the fullness of time, the puller might not even run on the same machine as the ssh server [04:00] mwhudson, indeed. [04:00] mwhudson, but it could also be done before-hand. [04:01] jml: you mean as a temporary improvement? [04:01] mwhudson, yeah [04:01] well, yes it could [04:01] mwhudson, I don't think it would hinder any of the message queue work [04:01] and it wouldn't be too difficult a change, I think [04:02] jml: there's quite a lot of stuff in vaguely similar areas i'd rather do first, i think [04:02] (combine puller and scanner, maybe pre-exec-ing bzr lp-server processes) [04:02] maxb: I think you want http://lpdebs.canonical.com/{dapper,jaunty}/ [04:02] mwhudson, I'd combine the puller and scanner first for sure. [04:03] maxb: That has 0.11 to 0.35. [04:04] mwhudson, I think I'd probably trigger a pull before working on pre-exec, since I think waiting for a pull is a worse kind of waiting than waiting for a bzr lp-serve process to spawn [04:04] (also because I know how to trigger a pull :)) [04:04] jml: i'm interested to see how my/our puller rewrite works in practice [04:04] probably the memory issue trumps both of those [04:04] mwhudson, me too! [04:05] mwhudson, I guess it's waiting for a prod rollout? [04:05] it will reduce the latency before a pull starts to 10s in some circumstances [04:05] right [04:06] I have to confess I'd forgotten about that patch. [04:08] maxb: By 'jaunty' I of course mean 'hardy' === ursula_ is now known as Ursinha [05:40] mwhudson: (mwhudsondoyle?) ref bug 411250. Ha! it seemed to go away on it's own - perhaps thumper can enlighten? it "felt" like a -ve cache issue.... [05:40] Bug #411250: rewriter seems to negative cache for a long time [05:41] spm: there's enough moving parts for things to be able to get pretty confusing [05:41] spm: but unless it happens again, i intend to basically not think about this again :) [05:41] heh. and fair enough too. [05:42] fwiw, nothing quite like nagios generated sms's on a regular basis to encourage bug-reporting. :-D [05:52] I need a better title for: 'Reviews requested I can do' [05:53] because I can now review anything [05:53] what it really means is: [05:53] a team I'm a member of has been asked to review this proposal [05:53] but that's kinda wordy [06:02] hmm [06:02] I'm about to wander off and go and make dinner [06:02] coming back on line when the kids are in bed === thumper is now known as thumper-cooking [06:03] thumper-cooking: The key word in the existing title is 'requested'. === thumper-cooking is now known as thumper [06:03] stub: I have a different question for you [06:04] thumper-cooking: The existing title seems fine (although I think 'Requested reviews I can do' is better) [06:04] stub: we have constantly issues with the security.cfg [06:04] stub: I want to have some group defined for "bzr-identity" that has the select rights on all the right tables [06:04] stub: and have that group used in our other db users [06:04] stub: is there an example of how to set this up? [06:05] Check out garbo, garbo-hourly and garbo-daily - it will show you how inheritance works in that file. [06:05] stub: ok, ta [06:05] * thumper really leaves to cook now === thumper is now known as thumper-cooking [07:04] well i've now read all the mail that had arrived by the time i got back on tuesday morning [07:04] but not all the mail that's arrived since then :) === Ursinha is now known as Ursinha-afk [07:15] mwhudson, heh [07:28] Morning [07:39] hi [08:11] wgrant: oh great, that gets me all but 0.7 through 0.10 [08:12] yep. [08:17] lifeless: Care to cast your mind back to 2006 and speculate where launchpad-dependencies (0.8) dapper might have been kept? [08:17] in dapper [08:17] [yes, it was actually in the distro at one point] [08:17] 0.6 is the latest in the distro. [08:17] I located 0.11 to 0.35 this morning. [08:17] And >= 0.36 are in the PPA. [08:18] sorry, nothing springs to mind then [08:19] It also looks like 0.2 and 0.3 only ever lived in pre-Soyuz Dapper. [08:19] ah well, 4 missing versions is a lot better than 30ish [08:20] maxb, can you give me a file name that I could search for? [08:20] launchpad-dependencies_0.7.dsc, 0.8, 0.9, 0.10 [08:21] no joy. [08:21] It's not critical, there's info in debian/changelog [08:22] cool. [08:22] danilos, good morning [08:23] maxb: I will grab a recent Karmic live CD in the next couple of days and see if I can reproduce your failures in a really clean environment. === jtv1 is now known as jtv [08:40] meh, I have a couple of new failures, even on 2.4 [08:40] https://dev.launchpad.net/LaunchpadOnKarmic [08:42] maxb: How's the 2.5 run looking? [08:42] see the marker :-) [08:42] maxb: Ah, I see. [08:43] maxb, so few? [08:43] 2 consecutive test runs take longer than overnight :-( [08:43] jml: hmm? which? [08:44] maxb, after , there looks to be very few tests that are actually failing [08:44] I'm pleasantly surprised :) [08:44] ah, well after the marker, those are still the results from before zbuildout landed [08:45] But presumably zbuildout should make it *better*. [08:45] ohh, I see. [08:45] good morning [08:46] Oh, and the page doesn't take account of the fact I had to cheekily "pycentral pkgremove python-lazr-uri" to make the buildout work at all === thumper-cooking is now known as thumper [08:48] has the lazr packaging mess been sorted out yet? [08:48] (I guess your comment means "no", maxb) [08:48] buildout is the solution to everything *handwave* [08:49] Well, it certainly hadn't been sorted when I was attempting to get these test runs going before sleeping last night [08:49] * wgrant just up'ed devel and is about to try. [08:50] wgrant: dpkg -l python-lazr-uri ? [08:51] maxb: Not installed, thankfully. I must have an old launchpadlib. [08:51] test with stable [08:51] That is a strange package name. [08:51] lucky you! :-) [08:52] devel might well have test failures (who can say!) [08:56] jml: good point, well made :-) Though it's the buildout flat out failing to do what its raison d'etre is that's really annoying me. [09:00] * wgrant throws broken eggs at SHHH [09:02] Aha, it works. But zc.tracelog is being a bit loud. [09:08] Next meeting I intend to ask the peanut gallery to express opions on SHHH [09:08] and opinions [09:20] lifeless, push-pop-progress, right? [09:20] wgrant: I've just herded your structural subs branch off to ec2. I started a run last night but the instance crashed and burned some time this morning. [09:23] gmb: Thanks. [09:25] jml: yes [09:25] lifeless, done [09:30] g'night [09:30] Enjoy your evening :) [09:35] seeya jml, and congrats BTW [09:49] bigjools: on? [09:51] lifeless: staying alive? [09:52] oo oo oo oo [09:52] staying alive [09:52] staying alive [09:52] oo oo oo oo [09:52] :) === mpt__ is now known as mpt [10:02] morning [11:00] morning, all. [11:00] hi deryck ! [11:09] People seem to think that packaging is trivial and automatic... === henninge_ is now known as henninge [11:22] wgrant: it ought to be [11:22] Quickly will be a step in the right direction [11:23] bigjools: For simple apps, sure. [11:23] bigjools: But libraries, not so much. [11:24] .deb is way over-engineered for most stuff IMO [11:25] bigjools: what, in particular, do you think is over-engineered for packaging, say, a gnome game? [11:26] nothing in particular [11:26] There's nothing wrong with .deb for that. [11:26] The build system, perhaps. [11:26] bigjools: so what's "most stuff" then? [11:27] did you see jono's session at All Hands when he learned to package something? [11:28] yep [11:28] So you don't mean .deb. You mean the source package format. [11:28] probabl [11:28] y [11:28] and the number of people in the audience who argued over how to do something? [11:29] For setuptools or autotools applications, it's very easy with CDBS or dh7. [11:29] but then I don't have a great deal of experience packging stuff, I am probably talking out of my arse [11:29] Heh. [11:29] however, I think it could be easier [11:30] Before DH7 and CDBS were big, it was pretty awkward. [11:30] bigjools: yeah, so the process could be a lot more helpful - or *feel* a lot more intuitive with the right tools, ,but the biggest difficulty is that the problem-domain is complex [11:30] yep. [11:30] I think it's only as complex as you want to make it [11:30] bigjools: that was partly due to the nature of some of the people in the room :-) [11:30] james_w: no doubt :) [11:37] james_w: btw I owe you an apology, I was confusing you with someone else when I was talking about lpnet vs edge API usage, sorry :( [11:37] np [11:37] I'm just happy it's there :-) [11:37] yeah, I was surprised it wasn't from the start [11:44] bigjools: it doesn't count unless it's in an email (an apology that is :) [11:45] bigjools: btw, I was surprised it wasn't either, flacoste did it basically a day after we've decided to go with it, but it never went out [11:45] danilos: shaddap and do your job :) [11:47] * danilos crawls back into his corner [11:48] mars: btw, is conversions.html based on db-stable or something? if I run it over my local copy of devel, I get much better stats for translations :) [11:48] (also, conversions.html doesn't belong on devpad) [11:48] danilos: the report does not lie, you're slackers! [11:49] bigjools: heh, that much is true as well, but I've got the code from mars $HOME and tweaked it a bit so it displays good results for translations [11:49] lol [11:49] if project=translations: done=100% [11:50] wgrant: that was exactly what I was thinking while I was typing this, but Launchpad team is not used to having publicly accessible accounts [11:50] wgrant: I'll raise it with the team [11:50] bigjools: heh, exactly :) [11:51] geez, what is this zc.tracelog spam in my "make run" output now ... [11:51] bigjools: I wondered that... [11:51] bigjools: that's what wgrant loved yesterday, 9124 or 9224 or something :) [11:51] should be DEBUG not INFO [11:52] danilos: Note I also loathed it, though. [11:52] 'cause it's all eggs. [11:52] wgrant: heh, I know, I know [11:52] wgrant: I've got about the same feelings about eggs/buildout combination [12:12] adeuring: hey [12:12] hi danilos [12:12] adeuring: did you by any chance ask for some hwdb parsing runs on staging yesterday? [12:13] danilos: no. what's the problem? [12:13] adeuring: if you did, I got some logs in my email, but didn't get the logs I actually asked for :) [12:13] adeuring: ah, never mind then, it seems Chex ran the wrong script for me then [12:32] EdwinGrubbs, ping [12:48] beuno: ping [12:49] henninge, hi [12:50] beuno: did you notice this bug I assigned to you? bug 418610 [12:50] Bug #418610: New translate page needs new icons [12:50] henninge, I did not [12:51] beuno: do you think you can get something done with that before you're gone? [12:51] henninge, could you please attach a ascreenshot of where those icons would go? [12:51] beuno: sure === Ursinha-afk is now known as Ursinha [12:51] henninge, I will try [12:51] beuno: cool, thanks [13:03] wgrant: Your branch passed its tests and is now in the PQM queue. === matsubara-afk is now known as matsubara [13:04] gmb: Thanks. [13:04] np [13:05] * wgrant shall prepare the other two pieces tomorrow. === kiko-zzz is now known as kiko === abentley1 is now known as abentley === ursula_ is now known as Ursinha === abentley1 is now known as abentley === ahma_ is now known as ahma === salgado_ is now known as salgado [14:03] beuno: I attached the screen shot. [14:04] henninge, thanks === abentley1 is now known as abentley [14:37] barry, ping [14:41] deryck: pong [14:53] beuno: ping [14:54] barry, pong [14:54] beuno: hi, two things. first, can you come to the ameu reviewer's meeting in 6m? [14:55] beuno: second, i'd like to talk about what to do about link display on the page the link points to [14:57] reviewers, lurkers and beuno -> #launchpad-meeting in 3m [14:57] sure [15:00] barry, lets see about the second after the meeting [15:00] beuno: sounds good [15:00] reviewers, lurkers and beuno -> #launchpad-meeting [15:19] barry, sorry I wasn't very useful [15:19] was on a call :) [15:20] beuno: no, it was cool :) [15:21] kfogel: ping, not urgent [15:21] beuno: so, about menu links. currently, navmenus (and only those i think) hide links for the page that you are viewing. i don't like this :). i've heard that in breadcrumbs the link to the page your viewing is still shown, but it is dimmed and not clickable. i personally don't like that either (i'd rather the link be active but otherwise visually indicative). we're looking for some ui pronouncement on [15:21] and fix the tests [15:22] mrevell: pong placeholder -- on phone, but will ping when off [15:23] barry, could you tell me more about why you want them clickable? [15:24] beuno: i'm sure it's not a good reason, but i very often use it as a cheap reload. usually the active link is way closer to my cursor than that leeeettlleee teenie browser reload button way up there in the upper left [15:25] barry, I understand [15:25] the problem is, it may misslead you to think it will take you somewhere else [15:25] or "different" [15:25] beuno: do you think we could do something visual to indicate it's the page your one, but leave the link active? like grey it down a bunch or something? [15:26] barry, I'm thinking [15:26] I'm trying to weigh in the cost/benefit [15:26] "make barry happy" weighs quite a bit, but, OTOH, we have like 500k users... [15:27] beuno: :) okay, so here's the scenario... [15:28] beuno: i just clicked on "View all sprints" and i'm transported to /sprints/+all. now i kind of remember i'm on that page because i clicked it recently, but maybe someone pinged me in irc that they just added two new meetings. i need to reload the page. what are my options? (note this happens fairly often with merge proposals and bug comments) [15:29] that's a great scenario to address [15:29] now [15:29] barry, do you use gmail? [15:29] beuno: no [15:29] ok, what gmail does [15:29] as a web application [15:29] well, i have a gmail account, but... ;) [15:29] is that if something changes while your on the page [15:29] it tells you [15:29] or it just updates it [15:30] depending on the scenario [15:30] *that* is the proper solution [15:30] of course, that's a month's worth of work right there [15:30] beuno: agreed! that would be awesome, and yep, it's a lot of work [15:30] beuno: i have another use case, but it's very launchpad developer centric, so i don't expect accommodation for it ;) [15:31] barry, I still think the potential confusion out-weighs the benefits [15:31] * beuno nudges mpt [15:31] have any thoughts? ^ [15:33] beuno: so, do you agree that the link should still be displayed (i.e. not hidden)? [15:33] beuno: it feels like a bug when the link disappears [15:34] barry, this is the las breadrumb? [15:34] beuno: my current concern is the navmenus [15:34] beuno, barry: I don't think pages should contain a link to reload themselves unless it's probable that the page contents will have changed since you loaded it. (E.g. Gmail's link to reload the Inbox, or Twitter's "121 more results since you started searching. Refresh to see them.") [15:34] barry, what's a navmenu on the UI these days? :) [15:34] beuno, barry: And where that's true, it should be more prominent than any global navigation will ever be. :-) [15:35] beuno: it's what i'm calling the menu on the top level collections and their +all pages [15:35] barry, ah, top right? [15:35] beuno: on the right portlet [15:35] barry, I see [15:35] the link should not go away, no [15:36] mpt: yes. i (mostly) like the way e.g. facebook does it [15:36] beuno: cool. so it sounds like you and mpt want the link to deactivate (i.e. not be clickable) [15:37] barry, yes, I think that's the right thing to do [15:37] beuno: okay. so, if the link has an icon, and you're sitting on its page, the icon stays and we put unclickable link text instead of the . sound about right? [15:38] barry, it does [15:38] beuno, mpt fab! thanks very much, i'll make that happen [15:39] bac: ^^ (summary: page you're on will show an unclickable link in the navmenu) [15:46] barry: that sounds great. === carlos_ is now known as carlos [16:10] where should a right side portlet go in the 3.0 layout that is not an action, or an notification or anything else mentioned on the conversion page? My specific problem: the "Current details" portlet of pages like this: https://code.staging.launchpad.net/~bjornt/launchpad/dont-flash-overlay-on-bug-page/+bug/107247/+delete [16:10] Bug #107247: Launchpad bug pages trigger caret browsing in Firefox and other Gecko browsers [16:12] adeuring, if any of that information is relevant to your decision whether to delete it, move it to the main body. Otherwise, nuke it. That sound right, beuno? [16:13] Actually it's rather alarming that that page isn't telling you *what* branch you're about to unlink from *what* bug... [16:14] mpt: deleting the portlet is too simple and straightforward ;) I think it does not contain anything interesting [16:17] * beuno reads back [16:17] correct [16:17] if it's relevant information, it should go in the main body [16:18] as mpt says, that page should *really* tell you what's being deleted [16:19] beuno: right; I added explicit links to the bug and the brnach to make that absolutely clear ;) [16:20] adeuring, so the information about the branch (ie "Merged") is not interesting, I agree [16:20] the portlet should die [16:20] just be sure that we don't loose important information [16:27] intellectronica, got a moment to help with a structural-subscriptions.txt failure? If it turns out to be hard, we can drop it but I'd at least like to shake this tree and see if anything falls out. [16:27] structural-subscription-target.txt, rather [16:37] by the way, why do we have anything in lib/canonical/launchpad/interfaces/ftests? That directory should be not once but twice gone. [16:40] jtv: sure (sorry, was on the phone until now so didn't notice your ping) [16:40] intellectronica: cool, thanks. In wgrant's branch, the very last line of that test fails. [16:41] intellectronica: the test files a bug, and then compares the set of indirect subscribers to the set of structural subscribers. [16:42] It expects the difference between those sets to be empty, but now suddenly no-priv is in that set difference. [16:42] i'm not sure i'm looking at the same test as you. can you please paste me the full path again, maybe even the failure? [16:42] intellectronica: File "lib/canonical/launchpad/interfaces/ftests/structural-subscription-target.txt", line 131, in structural-subscription-target.txt [16:43] Differences (ndiff with -expected +actual): [16:43] - set([]) [16:43] + set([u'no-priv']) [16:45] jtv: it's important to remember that this test is run several times, with target being different each time === deryck is now known as deryck[lunch] [16:46] intellectronica: remember? I don't think I knew it in the first place. :) [16:46] jtv: but you will remember it from now on, right? [16:46] jtv: see lib/canonical/launchpad/interfaces/ftests/test_structuralsubscriptiontarget.py [16:47] intellectronica: I'll have dreams about it for the rest of my life. :) [16:47] * jtv sees lib/canonical/launchpad/interfaces/ftests/test_structuralsubscriptiontarget.py === beuno is now known as beuno-lunch [16:52] intellectronica: can approval of a bug nomination cause someone to become structurally subscribed to the bug? [16:53] definitely not [16:53] but maybe it makes the approver indirectly subscribed? i don't remember [16:56] I'm asking since my prime suspect here is that wgrant's branch moves the automatic approval of bug nominations from the model code to the view. [16:58] jtv: that doesn't make much sense. is this branch the one for exposing the functionality via the api? [16:58] intellectronica: yup [16:58] if yes, then i think we want to retain that behaviour [16:59] The behaviour is still present, but explicitly in the view. I think it was pre-imped with gmb. [17:00] oh ok, maybe the tought of something i didn't, then [17:01] intellectronica, jtv: Hang on, let me try to remember. There was a reason for doing it. [17:01] anyway, to debug that failure, start with determining which target type it's failing on [17:02] * jtv starts determining that [17:03] intellectronica, jtv: So, we decided to make so that the API required explicit approval due to bug https://bugs.edge.launchpad.net/malone/+bug/253597 (the idea being that the UI should eventually change to require this, too). [17:04] Bug #253597: Not possible to nominate a bug if you have permission to approve it [17:04] gmb: yes, it made sense to me as a Bugs outsider at least. But something in the branch is breaking this test. May be a forgotten .approve(...) somewhere. === matsubara is now known as matsubara-lunch [17:07] intellectronica: wouldn't you know it, it's the very last one: Hoary. [17:22] intellectronica: so a new bug for Hoary seems to have lost an indirect subscriber [17:23] who did you expect will have been indirectly subscribed? [17:24] also, is it a private bug? those don't have indirect subscribers? === salgado is now known as salgado-lunch [17:31] intellectronica: I don't see anything that'd make it private. The test doesn't say why it expects that particular result to be empty. [17:32] intellectronica: but the unexpected subscriber seems to be the person filing the bug [17:32] What kind of subscription do you get to bugs you file? [17:32] yeah, it's a bit of a mystery. i wonder who's the idiot who wrote this === Ursinha is now known as Ursinha-nom [17:33] intellectronica: well bzr only shows your name, but the code looks older than you. :) [17:33] jtv: direct [17:33] jtv: ill take that as a compliment [17:33] fine by me [17:34] i mean, the test makes sense, i just don't remember why i added it [17:35] i think i'll need to look a the branch to be able to help. what's the url? [17:35] Actually, "structural_subscribers" is just the name of the variable. It's really bug_target.bug_subscriptions. [17:35] https://code.edge.launchpad.net/~wgrant/launchpad/export-bug-nominations/+merge/10715 [17:36] jtv: well, bug_subscriptions is just a shortcut for structural subscriptions to bug mail [17:36] (in theory, structural subscriptions can be for mail from other modules, though none of them implement this) [17:39] intellectronica: at one point I knew a bit about structural subscriptions, but it's been a while... It's basically being subscribed all bugs in something (say, a product) that gets bugs attached, right? [17:40] yes === beuno-lunch is now known as beuno [17:48] Is canonical_url_examples.txt failing for anyone else? I don't see how my branch could have broken it. https://pastebin.canonical.com/21514/ === deryck[lunch] is now known as deryck [17:52] rockstar: danilo ran into that one as well... there was a mailing list thread about it. [17:52] jtv, oh, I was searching for buildbot emails. [17:52] rockstar: also, I see the vaunted sample-data change in there; see earlier thread, probably from karl [17:53] jtv, you would think that ec2 would use the sample data from the branch that provides the sample data, so I wouldn't have that problem. [17:54] you know, someone's irc nick was to be removed from the sample data to avoid making his irc client play The Entertainer using a single tone [17:54] (Not saying who, or it'll beep again :) [17:54] jtv, yeah, I know. [17:54] intellectronica: I've got it: in the Hoary case, the test creates a bug nomination without approving it. [17:55] jtv: right, and that causes the bug reporter to be indirectly subscribed? [17:55] intellectronica: I think it was what previously caused the bug reporter to be directly subscribed—and therefore indirectly as well. [17:56] Now that doesn't happen any more, but the test also subscribes that same person structurally, and assumes that the indirect subscriptions are a superset of the structural ones. [17:56] ah, i see === matsubara-lunch is now known as matsubara [18:15] intellectronica: that's it for me tonight. Thanks for walking me through this! [18:18] no problem. and if you ever find who wrote that silly test slap him on my behalf, please [18:32] how does LP feel about the OAuth session fixation attack that it is vulnerable to? [18:33] We need to fix python-oauth to allow fixed clients to be written, and this requires API changes [18:34] so I'll want to fix that in lazr.restfulclient, but I'm not sure how that fits with lp's build system [18:34] would you need to land a fixed python-oauth at the same time? === salgado-lunch is now known as salgado [18:39] barry, https://dev.launchpad.net/ReviewerSchedule?action=diff&rev2=46&rev1=45 [18:39] does that complete my task? [18:41] beuno: fantastically so! thanks. i'll add a line for you as the current sole ui mentor [18:42] * beuno is alone [18:44] salgado: ping [18:45] EdwinGrubbs, hi [18:45] beuno: hello [18:45] hi EdwinGrubbs [18:45] EdwinGrubbs, about your email, I think that we should only highlight joining the team, not leaving it [18:45] I can put together the bg graphic for joining [18:46] salgado: I was wondering if you thought it was sane to move canonical.launchpad.browser.branding to lp.registry.browser.branding. The only non-registry objects that use that are sprints. [18:47] EdwinGrubbs, sounds ok to me [18:48] beuno: the graphic would be helpful. How long do you think that will be and will I be holding off on landing until then? [18:49] EdwinGrubbs, maybe 30 minutes, if you tell me that it's easy to just show that format when you haven't joined, and the [18:49] "leave team" link can go in the actions portlet [18:49] beuno: yes, it will be easy. [18:50] EdwinGrubbs, ON IT [18:50] beuno: thanks [19:04] ok, the API change doesn't affect lplib's usage, so there's no need to change it [19:10] beuno: when you have a chance; re: navmenu links to the page you're viewing. i have two screenshots for your approval: https://devpad.canonical.com/~barry/sprints-index.png and https://devpad.canonical.com/~barry/sprints-all.png [19:21] barry, ui=me [19:22] beuno: awesome! the unlinked text has a class so we can tweak the style later if we want === Ursinha-nom is now known as Ursinha === cprov is now known as cprov-afk [20:08] EdwinGrubbs, take this baby out for a spin: http://people.canonical.com/~beuno/bg-action-add.png [20:09] beuno: awesome. [20:13] hola! [20:13] anyone around to talk about oauth? [20:41] wgrant, http://people.canonical.com/~beuno/conversions.html [20:43] dobey: the launchpad-dev might be the best place for this [20:44] james_w: i thought that's where this is [20:44] ^ mailing list [20:44] sorry [20:53] perhaps. was hoping to get a more immediate response though :) [21:03] dobey, leonardr is your man [21:04] dobey, what's your question? (salgado knows more about oauth than i do) [21:08] leonardr: so oauth.py has a lot of problems (which have become ever apparent over the last 1.5 months since i started trying to fix some of them)... and i was wondering what launchpad's opinion is on having a fork of it [21:09] are the original maintainers unresponsive? [21:10] yes [21:10] it takes forever to get any response out of upstream [21:12] and aside from the code being ugly, the 'fixes' that did go in for 1.0a are still incomplete [21:12] dobey: i don't think there'd be a problem with creating a launchpad project that keeps a code import of the official code. then you could keep your own branches there. i think that's a pretty common pattern [21:13] but i don't really know anything about how launchpad is used in such situations, i could be wrong [21:13] well i was more interested in whether launchpad would want to switch to using the fork for oauth, instead of continuing to use the broken/undermaintained oauth.py [21:14] as i understand it, launchpadlib and the server use it, for doing things via the API [21:21] launchpadlib uses it, i don't think the server does [21:22] leonardr: i guess salgado would be the person to answer that more concretely? [21:23] salgado could answer about the server side, but i don't think oauth.py even deals with the server side [21:24] no, i guess that's not true. anyway, i dunno about the server [21:24] but for the client, the package needs to be as easy to install as python-oauth is now, so that people can actually use it [21:25] you'll either need to convince whoever currently maintains python-oauth, or we'll need to include the fork as code inside launchpadlib [21:25] that would be james_w [21:26] I couldn't find oauth.py used on the server [21:26] Launchpad itself only uses OAuthRequest._split_header() [21:26] there you go [21:27] which is a trivial 5-line method, so we should probably roll our own version of that and get rid of contrib/oauth.py [21:27] james_w, how would you handle it if oauth forked and we wanted to use the new fork? what would dobey have to do to convince you? [21:27] * salgado files a bug [21:27] well, python-oauth is only there to satisfy these two projects currently [21:27] if they switch to something else then we package that instead [21:28] which two projects? launchpadlib and what? [21:28] I'd just rather they do if *soon* if they are going to [21:29] i don't think i can promise it soon, so maybe in the next release? [21:30] if it happens soon I'd even write the patch myself [21:32] well, i believe the workflow changed somewhat for oauth 1.0a, so it would be a fairly big production in which we coordinated client and server. i don't think we can do that on your timeframe and i doubt that's a patch you'd want to write [21:32] (salgado definitely knows more about this part) [21:32] well if launchpadlib just uses one 5-line method, that sounds like an easy fix [21:32] dobey: *launchpad* uses one 5-line method [21:32] launchpadlib uses a lot more [21:32] oh [21:32] what does launchpadlib use? [21:32] all the request signing stuff [21:33] if there's no change in the workflow then if you put up your branch now and salgado and james_w and i understand it, then maybe james_w could write the patch and i could update launchpadlib and we could get everything into karmic, but i don't know what the deadline is [21:34] and i question whether a recent version of launchpadlib is going to be in karmic anyway [21:34] why's that? [21:34] because there's an old version in there now [21:34] and I've been working to fix that for months now [21:34] and upgrading to a new version would require adding a whole new package (lazr.restfulclient) [21:35] that's why I've been complaining on bugs about lazr issues and the like [21:35] lazr.restfulclient is sat in NEW [21:35] I'm about to upload the new launchpadlib [21:35] hmm [21:36] james_w, are there any outstanding bugs that are causing you problems? [21:36] no, we should be good to go now [21:36] famous last words [21:37] ok [21:37] so i don't have a branch yet. but i can definitely spend all day tomorrow/friday on it if we can do it for karmic [21:37] though hopefully i can get it all done in one day [21:38] dobey: how different is it going to be, what will the benefit be, and if part of the benefit is "1.0a", are there workflow changes? [21:38] 1.0a requires changes to the server also [21:38] so there are workflow changes [21:39] yes. since the security issue in 1.0 was "the workflow allows you to steal auth" they changed the workflow to fix it :) [21:39] but i can have the client pieces continue working with 1.0 for launchpadlib, until we can get that changed [21:39] it's not too important for most current uses of LPs API [21:40] but I suggest you fix it anyway === ursula__ is now known as Ursinha [21:40] well, we're gonna fix it, but until 10 minutes ago it wasn't considered important enough to get a fix in for karmic [21:40] i'm trying to figure out if it is now that someone else is doing some of the work === Ursinha is now known as Guest82878 [21:41] dobey: the server's going to continue to support the 1.0 workflow just so old clients don't break === Guest82878 is now known as Marvin_ [21:42] launchpad doesn't use python-oauth for the server side, so any improvements you make to the server-side code we can't use [21:42] hrmm, so unless the server will bail on 1.0a requests, it's probably fine for the client to send the 1.0a stuff anyway [21:42] because i'm guessing it will just get ignored on the server, until the server supports 1.0a [21:42] dobey: probably. what's the difference between a 1.0 request and a 1.0a request? [21:43] that depends on whether the client allows the server to only talk 1.0 [21:44] without wanting to step on salgado's toes, i can say with a fair degree of certainty that launchpad will not support 1.0a by the karmic deadline [21:44] 1.0a sends a callback in the request token request, expects oauth_callback_confirmed in that response, and expects an oauth_verifier unique id from authorization that is sent back in the access token request [21:44] so the karmic launchpadlib will have to allow a server to talk 1.0 [21:45] if we can implement 1.0a on the server side and have our karmic installed base start talking 1.0a immediately, that'd be a win [21:45] and it sounds like implementing 1.0a wouldn't require any change to launchpadlib itself, only to the oauth library [21:45] either way, launchpadlib isn't using a lot of the API, and should be an easy patch, and it's easy to test [21:45] does that sound right? [21:46] yeah, launchpadlib probably doesn't need any changes to do 1.0a [21:46] i have to read over the code entirely and see [21:46] dobey: ok, go ahead and make the change, give it to james_w who will evaluate it on its own terms as a patch to python-oauth [21:48] once it's released, i can make a change to launchpadlib as long as the change is no more complicated than telling oauth "use 1.0a if it's available, but don't fail if it's not." [21:49] i'm looking at the lplib code... it might be that the only change necessary is "change the import lines" [21:50] it will want to have a way for the caller to specify a callback won't it? [21:51] yes but lplib doesn't use callbacks [21:57] i'll evaluate what lplib is doing with the API and take that into consideration, and if it's necessary to do any extra changes to it, i'll make them as well [21:58] sound good? [22:05] leonardr: launchpadlib 1.5.1 is now in karmic [22:06] dobey: if lplib doesn't use callbacks, and we need to specify one to implement 1.0a, i don't see the point of rushing to get this into karmic [22:07] leonardr: hrmm? [22:07] dobey: you said lplib doesn't use callbacks, and you also dais that 1.0a sends a callback in the token request [22:07] are those the same callback? [22:08] if they are the same callback, i don't see how implementing 1.0 on the server side will make launchpadlib start working just because we applied your patch to oauth [22:08] leonardr: 1.0a specifies the value to be either an http callback which the server will use later (after the authorization request i think), or to be oob, if you aren't using callbacks [22:10] so you can implement 1.0a without using a callback, and it's secure? or are you really just falling back to 1.0 behavior? [22:10] the callback itself doesn't make the communication secure [22:11] but can it be secure if there's no callback? [22:11] and yeah, for lplib, i think it would just be the 1.0 behavior until the server starts doing 1.0a [22:12] there's a new "verifier" which also plays a role [22:12] i'm really getting the impression that there are 3 steps here. 1) apply dobey's patch to python-oauth. 2) implement the server side. 3) change launchpadlib to accommodate the new workflow with its callbacks and verifier [22:12] leonardr: it depends on what your definition of secure is i guess [22:12] each of those being a fair amount of work [22:13] if #3 was trivial, i could see an argument for rushing to get #1 into karmic--so that everything would start working once we put #2 in place [22:13] but if i have to do a lot of work to implement #3, that's not going to make it into karmic anyway [22:13] #3 should be trivial === salgado is now known as salgado-afk [22:14] ok, take care of #1 and ping me once you're done. we'll figure out #3 [22:14] leonardr: heh, i just said you wouldn't have to do any work for lplib to do this. :) [22:14] i'm ok with doing a small amount of work [22:15] ok [22:21] beuno, do you know how often the conversion page updates? It seems out of date. [22:24] leonardr: ok, i'll bug you tomorrow, thanks [22:30] rockstar, I'm setting one up on my people.c.c account [22:30] and I'll get it to update every hour or so [22:31] http://people.canonical.com/~beuno/conversions.html [22:31] that's db-devel [22:32] up tod ate [22:35] rockstar, I can run against devel === gary_poster is now known as gary-out [22:35] beuno, okay, that'd be much better. [22:36] * beuno branches devel [22:36] I'm hoping to get the last answers ones reviewed tonight. [22:36] awesomeness [22:57] thumper, skype is up, but other difficulties compel a short outage. I'll join the call a little late [22:58] jml: ok [22:58] rockstar, http://people.canonical.com/~beuno/conversions.html [22:58] that's based on devel === matsubara is now known as matsubara-afk [23:02] jml: ping when you're ready [23:05] jtv: Damn. [23:14] jml: https://code.edge.launchpad.net/~lifeless/testtools/startTestRun/+merge/10773 [23:15] jml: (also see #bzr re subscribing to your own branches ;P) [23:15] lifeless, I've already reviewed it. [23:15] oh, cool [23:15] totally my bad [23:15] heh [23:15] jml: you own trunk [23:15] so I can't make it so [23:15] oh right. [23:15] I'll merge it in then [23:16] thanks [23:22] jml: you might want to ping whoever packages it too [23:22] FF is about-to-or-just-has happened [23:23] lifeless, ahh, good call. === Marvin_ is now known as Ursinha [23:30] intellectronica, jtv: Thanks for looking at that. Fixed. [23:32] rockstar, could you give me a little help on template migration, pretty please? [23:32] Ursinha, sure, what's up? [23:33] rockstar, I'm changing the template of a page that has an admin link on it, but when changing to the main_side template, as described here -> https://dev.launchpad.net/VersionThreeDotO/UI/Conversion#Mechanical%20changes, the link is gone even if my user has the right permissions [23:33] rockstar, what can I be possibly missing? [23:38] thumper, on the phone with Ursinha right now, gimme a sec. [23:38] k [23:40] Ursinha: are you turning into a developer!? [23:40] (if so, commiserations!) [23:50] mwhudson, thumper, jml, rockstar sorry, had some stuff to deal with. do you guys want to have a meeting? [23:50] barry, yessir [23:50] rockstar, thank you thank you thank you [23:50] mwhudson, it seems so :) [23:50] barry: i think it would be a good idea [23:50] thumper, chat after the reviewer's meeting? [23:51] rockstar, thumper, mwhudson, jml -> #launchpad-meeting in 2m [23:52] wgrant: np. thanks for fixing this bug. i know for sure that it will make many people very happy! [23:53] intellectronica: We can hope. [23:54] I thought I had run the whole lp.bugs suite over it, but I must have been confused by the other Bugs branches I was working on at the time... [23:54] jtv: /win 4 [23:54] Gah. [23:55] I'm getting timeouts when reporting bugs in Ubuntu, can somebody look at what's going on? [23:55] e.g. OOPS-1334H2895 [23:55] (both in edge and production) [23:56] jtv: All lp.bugs tests now pass with r9238. [23:57] pochu: this sort of thing is better talked about in #launchpad btw [23:57] pochu: i imagine it's the dup search query [23:57] Easy way to work around it is use fewer terms initially, then correct the summary on the second page. [23:57] mwhudson: oh okay, it's been a long time since I come here :)