/srv/irclogs.ubuntu.com/2011/07/27/#launchpad-yellow.txt

* gmb lunches10:51
danilosbac, fwiw, I am in the middle of QA for your bug fix12:01
danilosbac, successfully got epiphany-browser (universe) packages imported for oneiric, need to check that they still don't get uploaded for natty even with XS-Ubuntu-Langpack: yes set12:01
bacdanilos: i just read through the backlog on #launchpad-dev.  thanks for helping out.12:10
bacdanilos: you still trying to ensure natty fails?12:10
danilosbac, np, yeah12:10
danilosbac, btw, that price you quote for driving 450 miles, is it for the petrol only, or tolls and such as well? because it looks huge (like less than 10 mi/gallon), and I wonder if we can see that tank you're driving :P12:17
bacdanilos: that is the allowed reimbursement as set by the Federal IRS.12:17
danilosbac, ah, ok12:17
bacit is supposed to be inclusive of all costs for driving a private vehicle: petrol, wear and tear, insurance12:17
danilosbac, right, understood12:18
bacmost people find it very generous, especially if you have an efficient car12:18
danilosbac, I was just wondering if you really have a car that can eat that much gas :)12:18
baci do, but would not be driving it down!  :)12:18
daniloss/gas/petrol/12:18
danilosbac, heh, fair enough12:18
bacit is for taking the dog to the park12:18
bacand, yes, i'd be happy to get 10mpg out of it12:19
danilosheh, nice :)12:21
danilosbac, what car is it? we don't get to see many cars like that in here12:21
bacdanilos: http://www.flickr.com/photos/squishy/2289984572/12:22
danilosbac, that's a defender?12:25
bacno, pre-defender.  a 1977 Series III.   they are very closely related but this one has a teeny 4 cylinder12:26
danilosbac, ah, interesting12:28
gary_posterbac benji danilos gmb call in 1 or 212:29
gmbRighto12:29
danilosbac, hi, bigjools is stuck in something else right now, and I was unable to QA the natty package translations non-extraction (though ideally, we'd also QA that it keeps working for pre-oneiric main packages and such)13:05
danilosbac, I am unsure how much I'd be able to help further, but basically, we need https://dogfood.launchpad.net/ubuntu/+source/epiphany-browser/2.30.6-1ubuntu6/+build/2492859 to start building (that's where we are stuck at now, and until bigjools can help, there's nothing we can do), when it completes, there should either not be .translations.tar.gz tarball at https://dogfood.launchpad.net/ubuntu/oneiric/+queue for epiphany-browser, or13:09
danilosif it's there, when it's accepted (only bigjools can do that) and processed, no files should up on https://translations.dogfood.launchpad.net/ubuntu/natty/+source/epiphany-browser/+imports13:09
danilosbac, if you feel lucky, you may also decide that what QA I did is sufficient, because the new functionality to let stuff through indeed works13:09
danilosbenji, hey, you mentioned how you'd like to talk to me :)13:10
benjidanilos: I just pushed lp:~benji/launchpad/bug-734765 for you to look at13:11
benjidanilos: and here are the important SQL queries executed when rendering https://translations.launchpad.dev/ubuntu/hoary/+templates on a clean dev instance: http://paste.ubuntu.com/653099/13:12
danilosbenji, doesn't look bad at all, so only new queries are for fetching other-side POTemplate records13:16
benjidanilos: exactly, but if there are 1300 templates, then that means (potentially) 1300 new queries to render the ubuntu page13:17
danilosbenji, yeah, so you could still pre-fetch those with another left join in the similar way thus turning it all into a single query13:20
danilosbenji, you'd need a ClassAlias to use POTemplate in the same query twice, but that's not a big deal13:21
benjiwell, that's where I fell down, I can't figure out how to do that, because I've already used...13:21
benjiClassAlias is new to me, looking that up13:21
benjiooh!  that looks perfect13:22
benjihope is restored  ;)13:22
danilosbenji, you might also decide to inner join SourcePackageName while at it thus really making the entire page content fetched with a single query :)13:22
bacdanilos: ok, thanks for your help.  i'll work with julian to get it moving13:23
danilosbac, you are welcome, I am sorry I didn't get it completely done13:23
benjiyeah, I've been thinking about that; once I make it no worse than the original I'll see if I can make it a bit better without too much more effort13:23
bacdanilos: np, you got it rolling since i didn't know which packages were appropriate13:23
gmbgary_poster: Is there any procedure for creating a lazr.restful release other than JFDI?13:30
gary_postergmb, sadly yes :-) lemme see if I can find it13:30
gmbBoo13:30
gary_posterI think there is something specific to lazr.restful, but it's what I've found so far...13:32
gary_posterhttps://dev.launchpad.net/ReleaseChecklist13:32
gary_posterstill lookig13:32
benjigary_poster: is this what you're looking for? https://dev.launchpad.net/HackingLazrLibraries13:33
benjispecifivally https://dev.launchpad.net/HackingLazrLibraries#Releases13:34
gary_posterbenji, yes!  thanks13:34
gary_postergmb ^^^13:34
gmbgary_poster, benji Thanks.13:34
gary_posteryw13:34
benjinp13:34
gmbgary_poster, benji So, this seems to count as a feature change. Should it therefore go to 0.19?13:37
gmbs/it/the version number13:37
gary_postergmb, yes13:38
gmbOk13:38
benjiversion numbers are cheap (at least if you buy them from me)13:38
gary_posterwell...I can match his price13:39
danilosI'll beat whatever benji's offering!13:39
danilosheh13:39
* danilos shuts up13:39
gary_poster:-)13:39
gmbWell, I'll go do something else whilst that builds, since I've turned off site-packages...13:48
bacgary_poster: JSON cache card unblocked and in review.  \o/14:03
gary_posterbac, yay!14:03
gary_posterdanilos, small branch if you are willing and able: https://code.launchpad.net/~gary/launchpad/bug791492/+merge/6946914:07
gmbAAAAAAAAAAAH14:21
gmbFailing tests in trunk.14:21
gmbNothing to do with me so far as I can tell...14:21
* gmb digs14:21
gary_poster:-(14:21
gmbHmm.14:22
gmb2 tests fail when run all in one bundle, pass in isolation.14:22
gary_postergmb, if you used a system Python, that may be biting you.  some lazr packages could not be easilu upgraded to the buildout that handles system pythons cleanly (and that code is being ripped out in trunk because I've not been willing to maintain it)14:22
gmbgary_poster: Ah. I just used bin/test... I thought that used bin/py...14:23
gary_postergmb, it does, effectively, but bin/py is just a cover around the underlying python14:23
gmbOIC.14:24
gary_posterwith the new buildout, that cover is careful like virtualenv is14:24
gmbgary_poster: Right. So, these tests pass when run on their own (or together, just without everything else)14:24
gmbNot sure if that's likely to be a manifestation of what you described above.14:25
gary_postergmb, oh, no, that sounds like unclean tests then :-/14:25
gmbRight.14:25
gmbThe tests are both doctests under example/base/tests/14:26
gmbgary_poster: They both fail in the same way (to start with, anyway): http://pastebin.ubuntu.com/653141/14:28
gmbOh, hang on.14:29
gmbI might be havening a stupid here...14:29
gary_poster:-)14:29
danilosgary_poster, looking (sorry, dropped out for some food)14:31
gary_posterhow dare you!14:31
danilosgary_poster, sorry, it won't happen again14:34
gary_posterThat's what I like to see: abject subservience.  Oh yeah.14:35
danilosgary_poster, anyway, your branch looks good, I just don't like the hard-coded 5 for the mailman xmlrpc timeout, but if you feel that's ok (and shouldn't be a config option or some such), go for it :)14:36
gary_posterdanilos, eh, good point.  I don't even know for sure that mailman has access to the config...14:36
gary_posterI can dig around for that I guess14:37
gary_posterIf I can see mailman being configured in some way or other I can try to follow the existing pattern14:37
danilosgary_poster, excellent, thanks, it's otherwise r=me14:37
gary_posterheh, mostly I see configuration via monkeypatches, of which this is one :-)14:37
gary_postercool thanks14:37
danilosthanks for the lint fixes as well14:37
gary_postercool14:39
gary_posterdanilos, for your info and mild amusement, there is config approach and it is *really* gross (because of the "Mailman was not designed to be a library" thing) :-) .  I'm running with it.14:52
danilosgary_poster, sounds "comforting" ;)14:52
gary_posterheh14:52
gmbgary_poster: So, after much faffing I can confirm the same failures in lazr.restful 0.18.1 (the current version). I can't seem to find the root cause, though.15:09
gmbIt doesn't seem to be a test cleanliness problem15:10
gary_poster:-(15:10
gary_postergmb, nor a clean python problem?15:10
gmbgary_poster: I don't think so, but I don't know what that would look like (I haven't got a virtualenv working properly yet; working on that now)15:10
gary_postergmb, virtualenv won't help.  Well, maybe it would.  Heck, I've forgotten. :-/15:11
gmbArgh.15:11
gary_posterA clean-built Python from python.org is definitively clean.15:11
gary_posterthat said, benji or I should probably help you15:11
gary_posterI'd like to finish up the review-response I'm working on15:12
gary_posterthen I can try to dupe and dig with you15:12
gary_posterunless benji wants to distract himself :-)15:12
gmbgary_poster, that would be perfect, thanks :)15:12
gmb(My next step was to ask for someone to try and dupe this)15:12
benjilunch is distracting me perfectly well at the moment, but after that I can help15:13
gary_poster:-)15:15
gary_posterk15:15
* danilos -> out15:38
benjigmb: anything I can help with?15:42
gmbbenji: Well, if you can grab the tip of lazr.restful trunk, build it, run bin/test and see if it fails, that would be a great start15:42
benjiit would... is something interfearing with that?  ferral marsupials maybe?15:43
gmbbenji: Sadly, no. That would at least be fuzzy in a fun way. This is just fuzzy in a I-can't-figure-out-why-it-breaks-yet way.15:44
gmbConfirmation that it's not just me would be nice :)15:44
gary_posterah poop15:52
gary_posterI had an interruption when CHR was supposed to start15:53
gary_posterthen I conveniently forgot it15:53
gary_posterI'll do a bit now and a bit after lunch15:53
benjigmb: since this is a fresh OS install, my lazr.restful buildout is still chugging along.  I'll let you know how the tests fare when it gets done.15:54
gmbbenji: Cool. It took ages for me too, so that wasn't a surprise :)15:54
benjigmb: if you have at .buildout/default.cfg file that looks something like this: http://paste.ubuntu.com/653190/ then it will be faster next time15:55
benjigmb: tests done: all passed15:55
gmbbenji: Right. It's my environment then.15:55
gmbThat's a relief.15:56
gmbgary_poster: So, with benji's help, I'm satisified this is an environmental weirdness.15:56
gary_posteryay!15:56
gmbAnd so will press on...15:56
benjiI used Python 2.7.1, hopefully the results don't vary with Python version.15:56
gary_posterbleh :-(15:56
* gary_poster decides they do not :-)15:57
benjiheh15:57
gmbSssh.15:57
gmbThe tests pass as long as you run them in the right way.15:57
gmbThe first three words are where the emphasis lies.15:57
benjirerunning with 2.6.5 just to see15:57
benjiyow, 2.6.5 doesn't look good15:58
gmbDamn.15:59
gmbbenji: Is it doctests in examples/base that are failing?15:59
benjiUnicode error in field.txt looks like the root cause, there may be others.16:00
gmb?!?!?16:00
gmbHaven't seen that one.16:00
benjino error in examples/base16:00
gmbhostedfile.txt and representation-cache.txt fail for me.16:00
benjiyep, the lazr.restful trunk is definately broken under 2.6.516:02
gmbOkay.16:03
gmbSo, not just me then.16:03
gmbbenji: Oh, I see the field.txt error now. I was missing it in all the other errors.16:06
gmbbenji, gary_poster: So, um... What's the best way to proceed from here. I don't see any error in the tests themselves, and whilst it's likely because I'm running out of brainjuice I'm unsure of the best way to fix the problem.16:20
gmbThat said, 2.6.5 is the system python on Lucid, so we could really do with not having this issue...16:21
benjithe unicode error looks like a bug in Python to me -- which would explain why 2.7.1 doesn't fail in that way.  The NameError in hostedfile.txt looks really wierd, but I haven't looked closely at it either.16:22
gmbbenji: Yeah, that one's been confusing me, but I put that down to not knowing much about lazr.restful.16:24
benjisuprisingly enough, lazr.restful still needs variables to be defined before they're used16:24
gmbbenji: Well, yes. But given that the test passes when run in isolation, I figured it wasn't that simple.16:25
benjireally?  that's an interesting clue16:25
gmbbin/test -ct hostedfile.txt -t representation-cache.txt16:26
gmbpasses.16:26
gmbAh.16:28
gmbbenji: `bin/test -ct example` fails.16:28
gmbRunning the tests directly (and individually) passes.16:28
gmbOooh!16:28
gmbAh16:29
gmbbenji: So, I missed out field.txt.16:29
gmbIf you include that, the whole thing falls apart.16:29
gmbSo this is likely an isolation problem on top of that unicode error, no?16:29
benjiwell, at least running them indepenently works, that means that they don't depend on each other (but instead interfere with each other)16:29
* benji needs an IRC client with spell check.16:29
benjiyep, isolation indeed16:30
gmbgary_poster: Fun and games at lazr.restful central, then ^^16:37
gmbHow do you want me to proceed here?16:37
* gary_poster was dealing with minor child emergency, sorry16:45
* gary_poster reads traceback, gmb16:45
gmbgary_poster: child > our fussy library problems; no worries.16:46
gary_poster:-)16:46
gary_posterok, just thinking about guiding principles here...16:47
gary_posteryou could argue that we shouldn't make an official release that is known broken16:48
gary_posteryou coudl counter argue that it works in 2.716:48
gary_posterbut I'm inclined to say that the principle wins16:48
gary_posterif we take that as a given,16:48
gary_posterthen we are left with two options16:48
gary_postermake a local release within LP16:49
gary_posterand figure out the tests to make a real release.16:49
gary_posters/and/or/16:49
gary_posterThe second would be safer16:50
gary_posterand preferrable16:50
gary_posterpreferable even16:50
gary_posterbut the first option is not inconceivable.  gmb, how about this.  I expect your EoD is within an hour or so, yes?  Send me the branch and I'll give it a whirl16:51
gary_posterthis afternoon16:51
gary_posterI finished my branch and was going to go bug hunting16:52
gmbgary_poster: It's actually in lp:lazr.restful. AFAICT it's been broken since at least r189 (my commits are 190 and 191).16:52
gary_posterIf I don't get it working this afternoon, then your tomorrow morning make a local Launchpad release16:52
gary_posterand move on16:52
gmbgary_poster: That sounds fine.16:52
gary_posterok cool gmb16:53
gmbMy EoD is actually in ten minutes :)16:53
gary_posterheh ok16:53
gmbgary_poster: r189 being broken suggests that we made a release that was broken - 0.18.116:53
gmb(I suspect that we only caught this because my dev machine is still running Lucid)16:53
gary_posterif it was not already broken before that, yeah16:53
gmbOtherwise I would never have spotted the problem.16:53
gary_posterah!16:53
gary_posterinteresting16:53
gary_posterok16:54
gmbgary_poster: The field.txt failure seems to cause the other two.16:54
* gary_poster is going to have lunch16:54
gmbbin/test -ct field.txt -t hostedfile.txt -t representation-cache.txt fails horribly. Dropping field.txt removes the problem.16:54
gmbgary_poster: Okay. I'm EoDing shortly but will be on IRC for a while yet, so feel free to ping me if you've any questions.16:55
gary_postercool17:00
gary_posterthanks17:00
gary_postermy schedule:17:46
gary_posterin 15 minutes17:46
gary_posterreturn from lunch17:46
gary_posterthen do CHR17:46
gary_posterfor an hour17:46
gary_posterthen team lead call for an hour17:47
gary_posterthen I have half an hour before my EoD17:47
gary_posterIOW, it strikes me I don't have a lot of lazr.restful time.  But I'll try to squeeze it in...17:47
bacdanilos: i know you're not here, but https://dogfood.launchpad.net/ubuntu/+source/epiphany-browser/2.30.6-1ubuntu6/+build/2492859 completed successfully17:54
bacand nothing showed up at https://translations.dogfood.launchpad.net/ubuntu/natty/+source/epiphany-browser/+imports17:54
bacso i think this means the test is working as desired!  i'm marking it qa-ok.  thanks a ton for your help17:55
bacwhat did you mean, benji?20:02
bacthere was only one open question and you addressed it20:02
benjibac: the bug import?  That was indeed a question, but I got to it from the "bug import questions" link, I expect that there are other open questions20:04
bacoh20:04
benjiyeah, I see lots of open questions20:04
baci see you're right20:04
baci have all of the links as bookmarks in a tab and then open them all at once20:04
baci just saw it was a Question20:04
baci'll get to the others20:05
bacbenji: fwiw, that question about plural forms in Interlingua could be addressed here:  https://translations.launchpad.net/+languages/ia20:23
benjithanks; is that something we can put in the FAQ thing so noobs like me can benefit?20:24
bacbenji: i reckon.  i didn't go right to it but had to search around a bit.20:50
bacbenji: i suspect it comes up so rarely it isn't that important20:50
bacother than esperanto and interlingua they aren't inventing that new languages that often20:50
benjiI was thinking more for the use of the CHRer; we need a sane way of figuring out what to do with all the disperate requests we get.20:51
bacbenji: i agree.  i'd just guess that plural forms is pretty well covered now for most languages20:51
baci've never done one before...20:51
gary_postergmb, pushed fix, sent you an email with explanation21:32
gary_posternight all21:32
bacdanilos: there are several open questions related to translations that i cannot answer.  could you try to handle them during your next CHR?21:45
bachttps://answers.launchpad.net/launchpad-project/+questions?field.search_text=&field.sort=RELEVANCY&field.sort-empty-marker=1&field.actions.search=Search&field.language=en&field.language-empty-marker=1&field.status=OPEN&field.status-empty-marker=121:45

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!