[00:02] * mwhudson lunches === bjf is now known as bjf-afk [00:14] thumper: there was a conflict between my branch and db-devel [00:14] http://pastebin.ubuntu.com/387304/ [00:14] * thumper looks [00:14] gary and i verified that the test is present in devel and not present in db-devel, but we didn't find any record that it had been _removed_ from db-devel [00:14] and are not sure why it would have caused a conflict [00:14] leonardr: just to be sure, maybe bring it up to thumper? It looks like it is in code, so he might know. I don't see any removals of this code in February (http://bazaar.launchpad.net/%7Elaunchpad-pqm/launchpad/db-devel/changes?filter_file_id=xxbranchmergeproposa-20081124161423-sto2nyh0ew6f80p4-1) [00:15] leonardr: if you edited something at the end of the file... ?? [00:15] leonardr: sometimes it happens [00:15] merge in db-devel [00:15] keep the test [00:15] it is new [00:15] thumper: problem is, the test doesn't pass in db-devel, maybe due to sample data differences? [00:16] I don't know who fixed it, but I'm happy it is :) [00:16] um.. [00:16] what's the error? [00:16] thumper: all_comments['entries'] is empty [00:17] leonardr: where is your branch? [00:17] leonardr: I'll update db-devel [00:17] merge in your branch [00:17] and take a look [00:18] thumper: lp~leonardr/launchpad/multiversion-wadl [00:19] * thumper postpones lunch a little [00:20] leonardr: I don't get a conflict merging your branch into db-devel [00:20] leonardr: have you merged db-devel into that branch and resolved the conflict? [00:20] thumper: yes, actually, sorry. try merging the penultimate version [00:21] leonardr: that's fine, I'll just look at the test [00:26] leonardr: your branch cleanly merged into db-devel for me [00:26] leonardr: and the test passed [00:27] leonardr: with a make schema before I started [00:27] leonardr: not sure what to say... [00:27] thumper: that fits with what happened to me [00:27] leonardr: so what is the problem? [00:27] thumper: gary and i thought it was weird and wanted to alert you to it, is all [00:27] oh [00:27] ok [00:28] just a weird bzr merge bug I think [00:28] thumper, leonardr, sorry for the ruckus, just didn't want to make a mistake. [00:28] gary_poster: that's ok [00:28] leonardr: I thought the test didn't pass for you? [00:29] leonardr, thumper: so we are merging the file without the test? [00:29] oh the make schema [00:29] I wondered if that were it [00:29] gary_poster: I don't think so. I started with an up to date db-devel, merged in leonard's branch, and the test is there [00:30] so it seems the conflict resolution is fine [00:30] but I still didn't see the test in db-devel when I pulled. but if you are happy, I am happy. I'll run along now. ;-) [00:30] and the test passes [00:30] cool [00:30] so I think we are good [00:30] ok, thank you thumper and leonardr [00:30] gary_poster: did you have an up to date db-devel? [00:30] hold on, i had the wrong view of an ambiguous statement [00:30] "the test" [00:31] let me restate what happened [00:31] lib/lp/code/tests/../stories/webservice/xx-branchmergeproposal.txt [00:31] Ran 23 tests with 0 failures and 0 errors in 5.421 seconds. [00:31] thumper: yes [00:31] i merged db-devel into my branch and saw the conflict in xx-branchmergeproposal [00:31] the conflict was a set of assertions about merge proposal comments [00:32] i removed it and then 'the test' (xx-branchmergeproposal) ran [00:32] leonardr: you removed this? http://pastebin.ubuntu.com/387304/ [00:32] yes [00:32] if i did not remove it, xx-branchmergeproposal would not run [00:32] i'm fairly sure this was after a make schema (becuase the test wouldn't run at all without a make schema) [00:32] but i'll try again [00:32] hmm... [00:33] thumper, I do not see that snippet in my copy of db-devel, which is fresh [00:33] leonardr: so to confirm, your branch as I merged it doesn't have this test? [00:33] I also did not see it in loggerhead [00:33] thumper: yes [00:33] getting loggerhead link [00:33] thumper: ok, wait [00:33] * thumper looks at devel [00:34] the branch at lp:~leonardr/launchpad/multiversion-wadl does not have the test, because i had to remove it to merge in db-devel [00:34] http://bazaar.launchpad.net/%7Elaunchpad-pqm/launchpad/db-devel/annotate/head%3A/lib/lp/code/stories/webservice/xx-branchmergeproposal.txt [00:35] thumper: I do not see that snippet in loggerhead link above, nor in my fresh copy of db-devel [00:35] ok, this IS an issue [00:36] * thumper adds the test and tries again [00:37] ah [00:37] dumb arse [00:37] it was my change [00:37] * thumper smacks forhead [00:37] forehead [00:37] yes it is correct to have it remoevd [00:37] I removed it! [00:37] d'uh [00:37] the db-devel branch as descriptions for merge proposals [00:37] the initial comment doesn't exist any more [00:38] we're golden [00:38] sorry for my confusion [00:38] * leonardr considers quitting irc before another reversal [00:38] the reason you got the conflict is that you changed some text in that block [00:38] for the new api url [00:38] yeah, i changed beta to devel [00:38] and I deleted it on db-devel [00:38] right [00:39] that is why there was a conflict [00:39] it is correct for it to not be there [00:39] on db-devel [00:39] ok, i just got email that it's been merged with pqm [00:39] sweet [00:39] leonardr, thumper thanks :-) running away also. [00:40] ok [01:03] hi. any idea what should I do to fix this error: AssertionError: No IEntry adapter found for IPOTemplate. [01:04] i'm trying to export IPOTemplate via API [01:04] heh [01:04] adiroiban: jml was just asking a very similar question a few hours ago [01:05] adiroiban: what code changes have you made? [01:06] export_as_webservice_entry() in IPOTemplate [01:06] end marked a simple TextLine as exported [01:06] hmm [01:07] if you just make the first change does it give the same error? [01:07] and this one http://paste.ubuntu.com/387327/ [01:08] without the changes from http://paste.ubuntu.com/387327/, there is no error, but the langauge is not exported [01:08] s/language/template/ [01:09] it looks like the template API needs to be referred by some other property in order to be exported [01:09] yeah, I just wondered if it was the export_as_webservice_entry() was enough to cause it [01:11] adiroiban: Can you pastebin your diff and the error? [01:11] lazr.restful is very good at confusing people. [01:13] error http://paste.ubuntu.com/387331/ [01:14] diff http://paste.ubuntu.com/387332/ [01:16] adiroiban: try dropping the export_as_webservice_collection() line [01:16] and the @collection_default_content() one too [01:17] james_w: I will try... but those are for IPOTemplateSet ... not for IPOTemplate [01:17] yes [01:17] But the first references IPOTemplate. [01:18] same error [01:18] * wgrant pokes around. [01:18] * wgrant waits for buildout to do whatever it does. [01:20] lazr.restful is resposible for creating the IEntry adapter? [01:21] create_as_webservice_entry() should do everything. [01:21] Er, export_as* [01:23] for me lazr.restful is still a voodoo afair [01:25] if I'm not using all_potemplates from distroseries, there are no errors [01:25] Yeah. [01:25] so I guess the IPoTemplate is not read [01:25] since it is not listed in apidoc [01:35] wgrant: do I need to include IPOTemplate in lib/lp/translations/interfaces/webservices.py ? [01:36] * wgrant blinks. [01:37] Aaaaaaaa. [01:37] Yes. [01:37] Everything I've ever touched has been in canonical.launchpad.interfaces, so I didn't know about that registration. [01:38] what is that? :) I have exported the API for ILanguage, which is in lib/lp/services/worlddate [01:38] But it looks like Translations has removed their stuff from there. [01:38] and I did not have to added it [01:38] Right, it's imported in lib/canonical/launchpad/interfaces/__init__.py, where all of LP's interfaces used to be available. [01:39] canonical.launchpad.interfaces is webservice-registered. [01:39] But Translations stuff appears to have been largely stripped out of c.l.i (as everything eventually should be). [01:39] ok [01:39] I added IPotemplate [01:39] and it is not listed in apidoc [01:40] and wadl file [01:40] You've removed and regenerated the WADL and apidoc? [01:40] yes [01:40] * wgrant tries. [01:41] It's there for me. [01:41] wgrant: sorry [01:41] (but as p_o_template, specify export_as_webservice_entry('po_template') for it to be more sane) [01:41] s/not/now/ [01:42] Ahh. [01:42] same here... p_o_templates [01:42] I will go to sleep now [01:42] many thanks for your help [01:42] Remember to fix that name at some point. [01:42] np [01:43] james_w: has jml fixed his no IEntry addapter problem? [01:43] I believe so [01:43] his was actually a different cause I think [01:43] aha [01:43] putting this on the wiki somewhere would be a good idea [01:44] james_w: yes. I'm adding it right now [01:44] at the Prerequisites [01:44] Yeah, I've never seen that documented anywhere. [01:45] \o/ adiroiban [01:45] Only Code and Translations have their own registrations [01:46] wgrant: yep. [01:47] maybe we need to ask on ML [01:48] Why? [01:48] why the other modules are not using webservice.py [01:49] Their exports predate c.l.i's deprecation. [01:49] they should probably use it. [01:50] They shoul dprobably create and use their own modules, that is. [01:50] So we can kill off c.l.i eventually. [01:51] ok [02:21] * thumper afk for a bit [03:32] back for a bit === mwhudson_ is now known as mwhudson [08:23] GOOD MORNING [08:23] * adeuring should remove the caps lock key [08:24] :-) [08:52] adeuring: or rebind it to control, as god intended [08:55] * wgrant wonders why there is still a schoolbell directory in the tree. [09:05] delete i t. [09:05] see what happens [09:08] It's used by c.l.components.cal, which itself doesn't seem to be used. [09:08] In fact that module shouldn't even import. [09:08] dead code. [09:08] delete. [09:08] have vcs, can undelete. [09:08] Yeah, the only reference is in a module that doesn't work any more. How odd. [09:09] it was imported for the ill fated calendar support that jamesh put together [09:09] which would have been really nice, but wasn't given enough dev time [09:09] [heard that before? :P] [09:10] Yeah, I know why it *was* there. I remember that poor old feature. [09:14] seriously. [09:14] delete. klesscode. [09:15] how much of the calendar code is left in the tree? I remember it was just hidden for a while (along with the bounty system) [09:15] jamesh: dunno, possibly lots. [09:16] hey, I'll be able to run lp on my new laptop ;) [09:16] couldn't you run it on your old one? [09:16] nup [09:16] only got 2GB of disk, and only 2GB of ram [09:16] jamesh: It's gone. [09:16] with all its overhead I'd be silly to try [09:16] Except for this one odd file with two classes. [09:17] lifeless: As long as you're 32-bit and not doing stupid Soyuz stuff, 1GiB is doable, 2GiB is fine. [09:17] (RAM, that is) [09:17] Morning [09:17] lifeless I used to do LP development in 2GB of ram on a 64 bit system [09:17] * bigjools struggles with 2GB on a 64-bit box [09:17] wgrant: 64bit [09:17] I think bounties are gone as well. [09:18] bigjools: Yeah, I had to upgrade. [09:18] I'm getting a new Stinkpad soon [09:18] with 4GB and an SSD :D [09:18] Oooh SSD. [09:18] I wonder how much happier that makes the test suite. [09:18] * bigjools does a little dance [09:18] jamesh: yeah, its got heavier AFAICT; and my evo + browser chew up tonnes [09:18] ask thumper [09:18] bigjools: me too :) [09:18] lifeless: Evo eats LOTS. [09:18] bigjools: x201s, 8GB, 128SSD [09:18] lifeless: you already have one? [09:19] I'm getting a T410 [09:19] bigjools: ordered today. [09:19] bigjools: i7 640LM [09:19] * bigjools ordered a week ago [09:19] i5 though :) [09:19] bigjools: a little faster [09:19] not run everything though [09:19] I couldn't find a way of avoiding the Windows tax though, it makes me sick [09:19] lifeless: that does sound like a nice machine. Looks pretty expensive though... [09:20] bigjools: How are Windows refunds with UK Lenovo? [09:20] jamesh: massive discount [09:20] wgrant: I dunno, I will try [09:20] it's not the refund as such, it's the bloody sales figures that sticks in the throat [09:20] bigjools: i5 ~= i7, need to check specific models and chipsets to compare in more detail. [09:21] i5-540M [09:21] bigjools: Hopefully refund figures count for something. [09:21] lifeless: more than the discounts that they always seem to claim are in effect? [09:21] I doubt it [09:21] Are UK models as ridiculously overpriced as Australian ones? [09:22] probably, we pay 17.5% VAT here [09:22] however I got an awesome deal, ended up saving ~GBP300 [09:22] jamesh: yeah [09:22] jamesh: ends 5th march, 25% off [09:23] lifeless: I guess I'll have to watch for specials when I get round to looking for a new laptop [09:23] jamesh: would have been really quite cheap if I didn't get the 8GB ram and the SSD. [09:23] jamesh: if your 6 year anniversary happens in < 3 months, you could do it now :) [09:24] I paid GBP1200, would have been ~1000 without the SSD [09:24] lifeless: it's not quite that close [09:25] bigjools: I think we'll both be happy [09:25] bigjools: I wanted the small form factor [09:25] yeah I like my 14 inches [09:25] [09:25] :> [09:25] I'm happy with 12.1" [09:26] I used to want a lot more. [09:26] lifeless: I wish they went for a thinner bezel and smaller size for the newer thinkpads [09:26] My 14" isn't too bad, but I would have preferred a 12" or 13" I think. [09:26] it's not what you have it's what you do with it [09:26] * bigjools likes WXGA+ [09:26] jamesh: I'll have to show you the 201s, its 0.8" high with screen closed [09:26] jamesh: The T400 screen bezel is asymmetric and HUGE. [09:30] lifeless: that sounds like an improvement over my X60s === daniloff is now known as danilos [11:04] Morning, all. [11:15] danilos: hi. can we schedule a pre-implementation chat for bug 525371 ? [11:15] Bug #525371: API for reading POTemplates details [11:16] adiroiban, sure, let me review the languages API first :) [11:19] ok [11:32] james_w: Did you get success and failure emails about your branches that I ran through EC2 yesterday? [11:40] gmb: I did not :-( [11:42] james_w: I did, so I'll forward them to you. I guess ec2 test doesn't look up the email addresses to which to send the messages like ec2 land does. [11:42] thanks [11:48] gmb: cool, so is the SUCCESS one landed? [11:49] james_w: No, because we're in PQM freeze for release. I'll land it for you as soon as PQM reopens (ping me on Friday if you haven't heard from me about it). [11:49] aha! [11:49] I'll fix up the other one [11:49] who knew there were actually tests for sync-source.py [11:50] james_w: there was one [11:50] oh, I managed to break that one [11:50] james_w: and there were more that tested a bit of code that was only used by the tests, not actually by sync-source.py (-: === matsubara-afk is now known as matsubara [12:21] jelmer: hey [12:21] jelmer: #tr - please answer, I'm waiting on you before crashing. [13:13] Anyone know why I would get ImportError: No module named tickcount - on lucid running make run? === mrevell is now known as mrevell-lunch [13:15] jpds: dpkg -L python-tickcount ? [13:16] Does it mention a /usr/lib/pyshared/python2.5/tickcount.so ? [13:16] 0.1-0ubuntu10launchpad1 [13:16] Yep. [13:16] Can you python2.5 -c 'import tickcount' ? [13:16] No. [13:17] hrm [13:17] oddness [13:17] ls -la /usr/lib/pymodules/python2.5/tickcount.so [13:17] ? [13:18] No such file. [13:18] Are you fully updated from the launchpad PPA? [13:18] Ah, I had this issue after upgrading to lucid... [13:19] maxb: Just ran rocketfuel-setup. [13:19] You just need to re-run `sudo apt-get update/upgrade` after rocketfuel-setup. [13:19] (so it pulls in python-support from the lp ppa) [13:19] * noodles775 runs to lunch [13:20] Do that. If it doesn't work, 'sudo update-python-modules -v python-tickcount.public' should manually poke python-support into reinstalling the symlinks for that package [13:27] * bigjools stabs the branch scanner [13:31] how do I rollback a transaction in bin/harness? [13:33] james_w, transaction.rollback() or abort()? [13:33] ah, abort works, thanks [13:34] odd that it's named that way [13:34] yeah, I never remember which one is the correct [13:42] james_w: fwiw, bin/iharness is far nicer to use [13:43] why doesn't make harness run that then? [13:44] james_w, make iharness does [13:49] gmb: tests fixed for the second branch and devel merged, so should be good to go when PQM re-opens as well [13:54] james_w: Cool, thanks. === mrevell-lunch is now known as mrevell [14:14] adiroiban, btw, I won't have time to do a pre-imp for potemplate API today; if you want we can do it tomorrow or you can try getting hold of either jtv or henninge [14:15] * beuno hugs bac [14:17] beuno: and i didn't even have to make stuff up! [14:19] bac, nobody else has to know that :) [14:19] danilos: ok. no problem. [14:20] adiroiban, I'm going to go through the languages API branch now, sorry for being late and lazy [15:17] hey guys [15:17] didrocks & I are trying to expose GPG keys over the API [15:17] that means we need to give them a URL [15:18] jml: gmb just worked on a similar thing. maybe you can find his branch for reference [15:18] but basically you need to add a url in zcml, and traversal code [15:18] any thoughts on https://launchpad.net/+gpgkey/KEYID [15:18] intellectronica, that's good to know -- I'll look for that [15:19] intellectronica, any thoughts on the proposed URL? [15:19] jml: i'm not sure i understand the question [15:19] well, i'm sure i don't [15:20] intellectronica, heh ok [15:20] intellectronica, we can give GPG keys any URL we want. we have to make a decision about what the URL will be [15:20] jml: is that the GPG keys for a person? [15:21] intellectronica, yes. [15:21] intellectronica, although all GPG keys are tied to a person, afaict [15:21] jml: See http://bazaar.launchpad.net/~launchpad-pqm/launchpad/db-devel/revision/9053 for reference to how I exposed TemporaryBlobStorage via the API. [15:21] if so, then how about lp.net/~person/+gpg-keys ? [15:21] intellectronica, we need a URL for the individual objects [15:22] jml: maybe use the key's fingerprint? [15:22] you mean like "https://launchpad.net/+gpgkey/KEYID"? [15:22] so ~person/+gpg-keys/AB12045 [15:22] yeah, key id, terminology... [15:23] why put it under ~person? [15:24] jml: what's the url of an EmailAddress? [15:24] deryck: so chrome hangs forever trying to visit https://bugs.launchpad.dev/firefox/+bug/1 (and shut up, mup!), but ffox can visit it. Go figure. [15:24] Bug #1: Microsoft has a majority market share

BjornT, good question! [15:25] deryck: oh, I think it's b/c the bug is in private mode right now [15:25] and chrome didn't have the cached creds [15:25] jml: another question: does it make sense to access the collection of all gpg keys? [15:26] BjornT, I don't think so, but I wasn't planning on exposing IGPGKeySet [15:26] BjornT, actually... [15:26] jml: because it was uploaded by a person and uniquely belongs to that person, no? [15:26] do we allow more than one person to upload the same key? [15:26] BjornT, being able to fetch a key by id sounds like a useful thing, but it's yagni for me now. [15:26] intellectronica, we don't. I checked. [15:27] jml: also, consider how a new key is added, or an old one removed. [15:27] jml: in that case i think it makes sense for it to dangle from a person [15:27] jml: i would also make it possible to iterate through all the keys for a person [15:27] intellectronica, yeah, that's what we've implemented right now [15:27] by exposing a collection [15:28] intellectronica, we just need need to give the bloody things a URL :) [15:28] so it makes sense that if you iterate over ~person/+gpg-keys every item will be of the form ~person/+gpg-keys/KEYID [15:29] there's also a pendinggpgkeys collection [15:29] for each person [15:31] BjornT, how is that relevant? [15:31] BjornT, I mean, it's definitely an interesting thing to consider :) [15:32] intellectronica, do you think it makes sense to iterate over ~person/+pending-gpg-keys in the form of ~person/+gpg-keys/KEYID? [15:33] "gpgkey_fingerprint_key" UNIQUE, btree (fingerprint) [15:33] "gpgkey_owner_key" UNIQUE, btree (owner, id) [15:33] those are the constraints [15:34] jml: yes, i think it does. keys are globally unique, but in launchpad they belong to a person and it never makes much sense to refer to them without the context of a person [15:35] jml: I'm wondering, how do you "ack" a key so that it goes from pending to accepted key? isn't the "send an email thing and decrypt it?" (it's been years I've done that, I don't remember) [15:35] didrocks, I don't know. [15:36] didrocks, my brain is big enough to fit one problem at a time :) [15:37] jml: it's big enough to understand LP's mind, mine apparently not ;) [15:38] didrocks, I use my hind-brain for that. [15:38] heh [15:41] jml: i think you need to encrypt your key and paste the result somewhere. but also been a while since i've done that [15:41] i mean encrypt _with_ your key [15:41] adeuring: how do you run a windmill test? bin/test -v -v -t TEST_BASENAME doesn't seem to do it. [15:42] kfogel: https://dev.launchpad.net/Windmill [15:42] kfogel: as intellectronica says ;) [15:42] intellectronica, adeuring: d'oh. THank you. [15:43] intellectronica, adeuring: just out of curiosity: no one would object if "bin/test -t NAME_OF_WINDMILL_TEST_FILE" did the right thing, would they? [15:44] kfogel: no, not really ;) [15:44] kfogel: no, but you'd need to record somewhere which domain the test should run on [15:45] intellectronica: well, or that could be deduced from the path to the file [15:45] kfogel: but the test setup for windmill is quite complex, so it might be better to have another script that decides if that is needed for a given test and either starts the windmill stuff of invokes the oridinary test [15:48] adeuring: not sure I understand. Basically, right now we have to specify "--layer=FooWindmillLayer", but that information could be deduced from the test every (?) time. [15:48] That is, the Foo could be deduced, and the need for the --layer at all could be deduced. [15:48] intellectronica, BjornT: thanks. [15:48] probably. [15:49] adeuring: hmm, I keep getting cert error popups, and no way to say "it's okay" [15:49]