| === Ursinha-afk is now known as Ursinha | ||
| wgrant | stub: Yes yes yes kill BST. | 04:13 |
|---|---|---|
| stub | Annoying DBA's since 2006 | 04:22 |
| stub | spurious-apostrophes-is-us | 04:23 |
| lifeless | BST ? | 04:31 |
| stub | Bull Shit Time | 04:36 |
| stub | Better to run in Unambiguous Computer Time | 04:37 |
| lifeless | heh | 04:37 |
| lifeless | I know what it meant, I was wondering what the impact was | 04:37 |
| lifeless | didn't you set the test tz to be argentina or something ages back ? | 04:37 |
| stub | Launchpad copes, but it confuses users. wgrant was referring to the recent bug that revision timestamps are in BST (and we don't display the timezone either). | 04:47 |
| stub | https://bugs.edge.launchpad.net/launchpad-code/+bug/594591 | 04:48 |
| _mup_ | Bug #594591: What timezone does codebrowse display? <Launchpad Bazaar Integration:New> <https://launchpad.net/bugs/594591> | 04:48 |
| lifeless | oh | 04:48 |
| lifeless | in loggerhead or lp core ? | 04:49 |
| lifeless | sounds like loggerhead | 04:49 |
| stub | Yer. Elsewhere we use 'x hours ago' or just the date, which is human friendly I guess but not cache friendly. | 04:51 |
| lifeless | its friendly enough | 04:54 |
| lifeless | pull the cached object, format, ship | 04:54 |
| stub | Not when you are trying to cache the formatting | 04:54 |
| lifeless | thats true, but formatting single revs isn't where the bottlenecks for loggerhead are | 04:55 |
| lifeless | its also cachable for the granularity of your format | 04:55 |
| lifeless | e.g. 60 seconds for 'last X minutes | 04:55 |
| lifeless | '60 minutes for 'last Y hours | 04:56 |
| lifeless | and days for 'Z days ago' | 04:56 |
| lifeless | if you're getting multiple requests a second, a 60 second cache is plenty | 04:56 |
| lifeless | particularly as its self bounding - after an hour it will scale back, and so on | 04:57 |
| stub | I'm thinking more about Launchpad. Marking up out content is slow. | 05:13 |
| wgrant | Do we have a good analysis of where time is spent? | 05:15 |
| stub | Less so now that thumper switched off bug-link tooltips so rendering text containing a bug tag no longer issues database queries | 05:15 |
| thumper | stub: it does for faq references still | 05:15 |
| stub | There is a dropin TAL replacement called Chameleon, but unfortunately isn't dropin-enough so we are still using the default TAL rendering engine. | 05:16 |
| wgrant | stub: That sounds like it could easily be done with memcached... | 05:16 |
| stub | thumper: it won't when I get done | 05:16 |
| wgrant | Although python-memcached seems to be very slow. | 05:16 |
| thumper | lifeless: I'm in general agreement with you about privacy | 05:16 |
| thumper | lifeless: I think we need careful analysis of it as it will touch everything | 05:16 |
| lifeless | stub: understood; I don't want to make things slow :). We could still get 60 second caches out using the above strategy for LP. | 05:16 |
| stub | wgrant: Still sucks the first time you render it, unless you prepopulate memcache and have some way of ensuring objects don't get evicted. | 05:16 |
| * thumper is still trying to do personal 360 reviews even though they are late | 05:17 | |
| wgrant | stub: Right. | 05:17 |
| lifeless | thumper: year, I had two 360's turn up late, forgot about em, and remembdered today | 05:17 |
| lifeless | better todo than not todo | 05:17 |
| thumper | lifeless: do you have a few minutes for a skype call? | 05:30 |
| lifeless | sure | 05:30 |
| lifeless | let me grab my mic from lynne so you don't get feedback | 05:30 |
| thumper | ok | 05:30 |
| === jamesh_ is now known as jamesh | ||
| lifeless | thumper: you have mail | 05:55 |
| mtaylor | don't use python-memcached | 06:21 |
| mtaylor | you want to use the one based on libmemcached | 06:21 |
| lifeless | what is it called? | 06:24 |
| mtaylor | python-libmemcached I think | 06:26 |
| mtaylor | also, I started a new one at one point - can't remember if I ever released it | 06:26 |
| lifeless | have you played with gizzard / | 06:27 |
| mtaylor | no... | 06:35 |
| mtaylor | ew. they're using thrift | 06:37 |
| stub | The other memcached library seemed flakier than python-memcached. Not sure what the performance issue is - it seems very thin. | 06:38 |
| stub | It would be trivial to switch though - we are only using the minimal API at the moment if they are different (get/set single keys at a time) | 06:40 |
| mtaylor | the main reason to switch would be getting distributed support that's in libmemcached | 06:40 |
| stub | Possibly because of versions of libmemcached in hardy too - not sure what is there. | 06:41 |
| mtaylor | that's where all the partitioning/load-balancing is handled - oh you need hardy | 06:41 |
| mtaylor | yeah, that's gonna be old | 06:41 |
| mtaylor | I've got libmemcached backports in my ppa I believe | 06:41 |
| mtaylor | I may have deleted hardy though... | 06:41 |
| === Ursinha-afk is now known as Ursinha | ||
| mtaylor | yeah, sorry. no hardy | 06:42 |
| stub | If it is worth upgrading we can worry about it after the lucid upgrade. | 06:42 |
| stub | What is the distributed support? | 06:42 |
| mtaylor | the key hashing algorithms to distribute across multiple memcached's is implemented in libmemcached rather than server side | 06:43 |
| mtaylor | it's conceivable that the pure-python one could have re-implemented all of the hashing stuff, but rather unlikely | 06:43 |
| stub | The pure python library calculates the hashes client side, and there is a hook to enable server side if you want. | 06:45 |
| stub | no... all client side. that hook is to be compatible with an older server I think. | 06:46 |
| === almaisan-away is now known as al-maisan | ||
| === al-maisan is now known as almaisan-away | ||
| gmb | Morning kids. | 08:00 |
| spm | yo dad | 08:01 |
| wgrant | gmb: Morning. | 08:01 |
| gmb | spm, This morning, I feel old enough to be your dad. | 08:02 |
| gmb | Morning wgrant. | 08:02 |
| spm | gmb: that's a scary scary place to be, given how old my dad actually is | 08:02 |
| wgrant | Did you get an email from that ec2 test of mine you ran? | 08:02 |
| wgrant | Because it landed. | 08:02 |
| wgrant | But I didn't get an email from it. | 08:02 |
| wgrant | Nor the other one, which probably failed. | 08:03 |
| gmb | wgrant, Yes, I did; I misspelt your email on the command line, that's why you didn't get one :). | 08:03 |
| gmb | (I wrote wrant, it turned out) | 08:03 |
| wgrant | gmb: Ah. So maybe mars just didn't fire my other one off, and EC2 isn't being crap again. | 08:03 |
| spm | wrant seems more accurate somehow... | 08:03 |
| wgrant | Heh. | 08:04 |
| gmb | wgrant, Possibly. You have to deliberately put emails in as CLI arguments, so maybe he ran the tests but forgot to add you as a recipient. | 08:04 |
| wgrant | gmb: Doesn't ec2 land do that automagically? | 08:04 |
| * spm is depressed he hasn't noticed that potential childish name misspelling tease. right under his nose all this time. shame. | 08:04 | |
| wgrant | spm: Maybe I need to get wgrant.com, and start a blog named WG-rant. | 08:05 |
| spm | BWHAHAhahahahahaha | 08:05 |
| gmb | wgrant, Yes, but I ran (and I suspect mars ran) ec2 test, just out of habit when running others' code through ec2. | 08:05 |
| gmb | wgrant, That would be full of win. | 08:05 |
| gmb | You could have a deathmatch with Matt Garrett. | 08:05 |
| wgrant | He'd surely win. | 08:06 |
| gmb | Starbucks appear to have picked "bizaare Ska tracks" as their CD this morning. Time for Rhythmbox. | 08:10 |
| adeuring | good morning | 08:14 |
| stub | I need some crappy javascript to sort tables by clicking on the column headers | 08:42 |
| stub | Or some good javascript - I'm not fussy. | 08:42 |
| lifeless | we had that | 08:42 |
| lifeless | 5 years back, just uncommit | 08:42 |
| lifeless | in fact, I think some tables *still* have it | 08:42 |
| stub | But I'll worry about that after a swim | 08:44 |
| wgrant | Can someone please ec2 land https://code.edge.launchpad.net/~wgrant/launchpad/bug-592935-hide-disabled-ppas/+merge/27411 and https://code.edge.launchpad.net/~wgrant/launchpad/bug-592914-recipe-distroseries-order/+merge/27415? | 08:58 |
| jelmer | wgrant: sure | 08:59 |
| wgrant | jelmer: Thanks. | 08:59 |
| mrevell | Good morning | 09:09 |
| noodles775 | Hi stub (or anyone with sql-foo), if you have a chance, can you take a look at the following options and let me know which of the two (left join or except) would be better? http://pastebin.ubuntu.com/450490/ | 09:13 |
| noodles775 | erm, sql-fu ;) | 09:14 |
| noodles775 | (with both queries the result will be limited to a batch) | 09:15 |
| gmb | Right. Modules without an __all__, which are used in indirect imports, first against the wall when I become President of the World, okay? | 09:17 |
| wgrant | You have my vote, as long as removing all glob imports is also one of your core promises. | 09:19 |
| lifeless | where is the ec2 land code these days ? | 10:51 |
| james_w | ./utilities/ec2 | 10:52 |
| james_w | ./lib/devscripts/ec2test for the meat. | 10:53 |
| lifeless | I meant as a branch url ) | 10:54 |
| lifeless | what branch is that in | 10:54 |
| james_w | it's in the launchpad codebase | 10:55 |
| lifeless | thanks | 10:55 |
| james_w | http://bazaar.launchpad.net/~launchpad-pqm/launchpad/devel/files/head:/lib/devscripts/ec2test/ | 10:56 |
| lifeless | hah | 10:59 |
| lifeless | bad mime type, naughty bobo | 10:59 |
| lifeless | or,. | 11:00 |
| lifeless | email module being confusion, no worries | 11:00 |
| jtv | hnyarg testfix mode? | 12:36 |
| wgrant | bigjools: Is the package metadata thing being designed with the PPA index page as a possible future use case? | 12:37 |
| bigjools | no | 12:38 |
| wgrant | :( | 12:38 |
| bigjools | but it would be easy to do | 12:39 |
| bigjools | just not a priority | 12:39 |
| wgrant | Well, as long as the format isn't unowkrable. | 12:39 |
| noodles775 | It'd be great to be able to say, if the meta data is there describing the apps, use it, otherwise use the installable binaries... | 12:40 |
| wgrant | Exactly. | 12:40 |
| jtv | Anyone got time to testfix lp.code.browser.tests.test_sourcepackagerecipe.TestSourcePackageRecipeView.test_request_builds_page? | 12:42 |
| wgrant | The order thing? | 12:43 |
| jtv | wgrant: looks like | 12:44 |
| jtv | Warty & Secret changed places in some test output. | 12:44 |
| wgrant | I have a branch in EC2 which orders by distroseries.version. I wasn't aware the order wasn't stable... I was just irritated over the weekend that it was arbitrary. | 12:44 |
| wgrant | So it will probably be rejected for the thing that it fixes :( | 12:45 |
| jtv | wgrant: the irony. | 12:45 |
| jtv | wgrant: I guess this was Aaron's branch, and he's not here yet... any other interesting background you can give me? | 12:46 |
| wgrant | I don't know why it's just started breaking now. | 12:47 |
| jtv | wgrant: hmmyeah... I don't see anything in the past day of changse that would particularly have effected this. | 12:49 |
| jtv | It may be an indeterminate ordering. | 12:50 |
| jtv | What was that sound? Ah, darkness fell. Goes so fast here. | 12:50 |
| wgrant | Oh. | 12:50 |
| wgrant | Oh. I hate you. | 12:51 |
| wgrant | Gaaaah. | 12:51 |
| wgrant | So, I was told to use attrgetter('name.version') rather than lambda ds: ds.name.version | 12:51 |
| wgrant | That worked locally, although I was surprised. | 12:51 |
| wgrant | And it just failed ec2. | 12:51 |
| jelmer | wgrant: did you see those emails? | 12:51 |
| wgrant | Turns out that python2.5 doesn't like that. | 12:51 |
| wgrant | python2.6 is fine with it. | 12:51 |
| wgrant | Gahl | 12:51 |
| wgrant | So, https://code.edge.launchpad.net/~wgrant/launchpad/bug-592914-recipe-distroseries-order/+merge/27415 | 12:52 |
| jtv | wgrant: errr when you say "I hate you," who exactly are you talking to? | 12:53 |
| wgrant | That should work on 2.5. | 12:53 |
| wgrant | jtv: Python. | 12:53 |
| jtv | ah :) | 12:53 |
| wgrant | Does anybody have a 2.5-using system that they can test that on? | 12:54 |
| wgrant | 'bin/test -t sourcepackagerecipe' is sufficient. | 12:54 |
| jtv | (silence) | 12:56 |
| * mwhudson thinks attrgetter is an abomination | 12:56 | |
| jtv | wgrant: I can run it through ec2, and then just land it | 12:57 |
| * jtv wonders: if it works for "foo.bar" will it also work for "foo.invokeSomeMethod(param, arg).bar"? | 12:57 | |
| wgrant | mwhudson: This new variant seems to be. | 12:58 |
| wgrant | The old one wasn't too bad. | 12:58 |
| mwhudson | maybe i exaggerated | 12:58 |
| mwhudson | s/abomination/solution to a non-problem/ | 12:59 |
| wgrant | What, you don't hate lambdas like Launchpad policy does? | 12:59 |
| mwhudson | correct | 12:59 |
| wgrant | Good. | 12:59 |
| jtv | noodles775, I'm sure you know this: can I give utilities/ec2 options like bin/test's -t to control which tests will run? | 13:09 |
| noodles775 | I don't actually... I'd use the source to find out... | 13:10 |
| jtv | I did that, but it's not incredibly transparent. | 13:10 |
| noodles775 | jtv: utilities/ec2 --help test | 13:12 |
| noodles775 | Shows how to pass test options. | 13:12 |
| maxb | <wgrant> So, I was told to use attrgetter('name.version') rather than lambda ds: ds.name.version | 13:12 |
| maxb | What is the rationale for that? The lambda looks so much nicer to my mind | 13:12 |
| jtv | noodles775: damn, I should've thought of that. :) Thanks. | 13:12 |
| wgrant | maxb: The style guide just about forbids lambdas. | 13:12 |
| jtv | And the exception to that is recent. | 13:13 |
| * noodles775 thinks it's another case of "use it sensibly" | 13:13 | |
| noodles775 | s/it's/it should be/ | 13:13 |
| wgrant | It explicitly says that one should use attrgetter instead if possible. | 13:13 |
| maxb | And fails to provide plausible rationale for this :-( | 13:14 |
| noodles775 | which broke your test on 2.5 :/ | 13:14 |
| jtv | ISTR the review docs on the old wiki saying something about attrgetters as well... something about them being unsafe? Never really got what it was about. | 13:15 |
| maxb | "Use of the built-in hasattr function should be avoided since it swallows exceptions." ? | 13:31 |
| wgrant | http://bugs.python.org/issue504714 | 13:33 |
| mars | wgrant, I did not get a mail for that branch. Not sure why, maybe the run itself did not start, and I no longer have the console I started it in. | 13:58 |
| mars | wgrant, do you need me to do another run? Or did gmb pick it up? | 13:59 |
| gmb | mars, I didn't pick it up since I didn't know what happened to it. | 13:59 |
| gmb | Probably best to run it again. | 13:59 |
| wgrant | mars, gmb: jelmer reran it. | 14:02 |
| wgrant | But it failed. | 14:02 |
| wgrant | Since attrgetter('foo.bar') doesn't work on Python 2.5. | 14:02 |
| wgrant | So I've reverted to the lambda. | 14:03 |
| mars | wgrant, yeah, I thought of that when I saw your reply later that night | 14:03 |
| wgrant | But now devel is in testfix. | 14:03 |
| mars | I have a hard-earned "check in Python2.5" mental switch after trying to work in 2.6 for a while | 14:03 |
| mars | 2.6 has a surprisingly large number of obviously useful features that are not in 2.5. You get addicted to them. | 14:04 |
| wgrant | Heh. | 14:04 |
| wgrant | Yes... | 14:04 |
| wgrant | I think 2.4->2.5 might have been worse, though. | 14:05 |
| wgrant | But we can do away with 2.5 in a month or two... | 14:05 |
| mars | yes, looking forward to that | 14:05 |
| wgrant | Hm. | 14:08 |
| wgrant | Someone has landed that branch now, anyway. | 14:08 |
| wgrant | jtv: Was that you? | 14:08 |
| jtv | wgrant: yes | 14:08 |
| jtv | like I said | 14:08 |
| wgrant | Thanks. | 14:08 |
| jtv | Thank _you_. Easy testfix. :) | 14:09 |
| maxb | ooi, do we know when we will know when LP datacentre machines start going lucid? | 14:24 |
| wgrant | https://code.edge.launchpad.net/~wgrant/launchpad/bug-592935-hide-disabled-ppas/+merge/27411 failed EC2 because of the devel breakage. | 14:27 |
| wgrant | Can somebody please send it back? | 14:27 |
| === henninge_ is now known as henninge | ||
| mars | wgrant, I'll try and lp-land it | 14:34 |
| wgrant | I wonder if it will bother to email anybody this time. | 14:35 |
| mars | wgrant, it is on its way | 14:49 |
| mars | but the buildbot pass is a while away - 7 hours | 14:50 |
| wgrant | But it should be untestfixed already, right? | 14:52 |
| mars | yes | 14:53 |
| mars | jtv's branch did that | 14:53 |
| jtv | mars: it was wgrant's branch... I just landed it as a testfix | 14:53 |
| sinzui | gary_poster, flacoste: I need help with a CheckBoxWidget: http://pastebin.ubuntu.com/450614/ | 15:42 |
| === matsubara is now known as matsubara-lunch | ||
| gary_poster | sinzui: not sure. I'll look at setUpWidget. | 15:44 |
| sinzui | gary_poster, flacoste: I neglected to mention that I am using GhostWidget to suppress rendering. I now think (after making my failure public) that the problem is that GhostWidget is really a text widget, it does not covert values of 'on' to True | 15:45 |
| sinzui | That must be the case. I should subclass the GhostWidget or implement a BooleanGhostWidet | 15:46 |
| gary_poster | sinzui: that jibes with the suggestion that I was trying to validate before making: make the widget handle the value. | 15:46 |
| gary_poster | so, agree | 15:47 |
| sinzui | thanks gary | 15:47 |
| flacoste | sinzui: why do we need a ghostwidget in the first place? | 15:47 |
| gary_poster | welcome | 15:47 |
| sinzui | flacoste, it is a mechanism we use to allow a form to manage manage a field in a normal fashion, but permit another widget to do the rendering. We do this when we want to compose a compound widget that interleaves subordinate fields into the widget's primary field | 15:49 |
| sinzui | flacoste, eg, we show the licence_info field adjacent to the license widget because it is only relevant when the user choose two of the 25 items | 15:50 |
| flacoste | ok | 15:56 |
| sinzui | sweat a 6 line fix by converting the Ghost into a mixin | 15:57 |
| sinzui | ow sweet. I hate English | 15:58 |
| === matsubara-lunch is now known as matsubara | ||
| === jtv is now known as jtv-afk | ||
| === salgado is now known as salgado-lunch | ||
| === EdwinGrubbs is now known as Edwin-lunch | ||
| jml | how do I disable edge redirection | 17:49 |
| jpds | jml: Footer of any page on the right: "Disable edge redirect". | 17:50 |
| jml | jpds, thanks. | 17:51 |
| === Ursinha is now known as Ursinha-lunch | ||
| === beuno is now known as beuno-lunch | ||
| === henninge is now known as henninge-afk | ||
| mrevell | night | 18:20 |
| === salgado-lunch is now known as salgado | ||
| === beuno-lunch is now known as beuno | ||
| === henninge-afk is now known as henninge | ||
| === Ursinha-lunch is now known as Ursinha | ||
| === Edwin-lunch is now known as EdwinGrubbs | ||
| henninge | matsubara: you were saying you something like bug 595163 fixed elsewhere? Do remember where? | 19:36 |
| _mup_ | Bug #595163: TypeError raised when query string contains a list <oops> <Launchpad Translations:New> <https://launchpad.net/bugs/595163> | 19:36 |
| matsubara | henninge, just a sec. let me find it | 19:36 |
| matsubara | henninge, https://bugs.edge.launchpad.net/bugs/213366 | 19:41 |
| _mup_ | Bug #213366: OOPS in +builds page passing a list as the query string <oops> <Soyuz:Fix Released by cprov> <https://launchpad.net/bugs/213366> | 19:41 |
| henninge | matsubara: so, that fix turns the TypeError into an UnexpectedFormData. Is that an acceptable fix? | 20:12 |
| matsubara | henninge, yes, I think so. that way the user will be informed that what he's trying to do is wrong and we won't record an oops for it | 20:13 |
| henninge | ok, cool ;-) | 20:14 |
| henninge | matsubara: thanks, that's really easy now. | 20:14 |
| henninge | I hope | 20:14 |
| henninge | ;) | 20:14 |
| matsubara | :-) | 20:21 |
| === almaisan-away is now known as al-maisan | ||
| === al-maisan is now known as almaisan-away | ||
| maxb | thumper: Hi. Given the status of bug 327126 is it OK if I go ahead and create a small bzr-svn import from apache.org ? | 22:14 |
| thumper | maxb: yep | 22:14 |
| maxb | lets give it a go... (I have doubts) | 22:15 |
| === salgado is now known as salgado-afk | ||
| maxb | bzrlib.errors.NotBranchError: Not a branch: "/subversion/trunk/subversion/tests/cmdline/svntest". | 22:18 |
| maxb | boo | 22:18 |
| maxb | (yes I did want just the svntest library in its own branch) | 22:18 |
| thumper | maxb: we do have some apache imports working | 22:21 |
| maxb | yeah, I think I've been caught out by bzr-svn being too clever for its own good | 22:21 |
| lifeless | skynet is on the way | 22:25 |
| === matsubara is now known as matsubara-afk | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!