/srv/irclogs.ubuntu.com/2012/05/08/#launchpad-yellow.txt

gary_posterbac benji frankban (hi gmb!) I'm going to run vote.  I think I will be back in time for 8:10.11:49
gary_posterIf not it will be soon after11:49
benjik11:49
bacok gary_poster11:49
baci hope gmb isn't reading irc atm11:50
bacwhy does "download as text" on paste.ubuntu.com require authentication?11:56
bacthey say there is no bad press:  https://plus.google.com/102921374554385564572/posts/PdiEdDE5LeK  (re: stevenk's comment)12:06
gary_posterheh, lovely12:08
gary_posterbac benji frankban call in one or two (asap_12:09
gary_poster)12:09
benjioh, frankban and gary_poster: I meant to mention, I saw a branch that used a negative exit status, that's somewhat odd, and almost certainly transmuted into an unsigned value (see http://www.gnu.org/software/libc/manual/html_node/Exit-Status.html)12:32
bacbenji: that was probably my branch12:33
benjiah, sorry12:34
benjiI would have responded to the MP but I was reading it on my phone and didn't feel like typing out a response on it.12:34
gary_posterbenji, ah, I was not aware that it was a problem.  Thanks for the heads up.12:35
baci'll change it to 1 instead of -112:35
gary_poster+, ah, 112:35
benji:)12:35
bacgary_poster: short line at the polls?12:36
gary_posterbac, very.  I was #81.  The only amendment sign there was "against" but our neighborhood in general leans that way, so...not particularly indicative of anything except that I'm glad my neighborhood leans that way.12:39
gary_poster#81 for the day, that is.12:39
frankbangary_poster: UnicodeDecodeError during my test run: http://ec2-107-20-111-209.compute-1.amazonaws.com:8010/waterfall12:48
frankbanit doesn't seem to be related to lpsetup, but to lpbuildbot.12:49
gary_posterfrankban, weird :-/ ummm...maybe the stdout stream gives us a clue.  looking12:53
gary_posterfrankban, "<area shape="poly" ...title="A title with non&#45;ascii characters áã" .../>" is in stdio...12:55
frankbangary_poster: it seems to be a failure of lib/lp/blueprints/stories/blueprints/xx-non-ascii-imagemap.txt12:55
frankbanyes12:55
gary_posterfrankban, so I'm not sure what the right solution is, but obviously it is that error_text is...utf8 I assume...and we need to be able to handle it in format_error (and log)12:57
gary_posterfrankban, could you make a card for that please?12:57
frankbangary_poster: sure, In the summary there is also an error that I remember: cmapx" not recognized. Use one of: dia hpgl mif mp pcl pic vtx12:57
frankbanmaybe some workaround is missing in lpsetup12:58
gary_posterfrankban, see bug 95935212:58
_mup_Bug #959352: Ephemeral containers have "/rootfs" prefix in /proc/self/maps entries <apport-collected> <kernel-request-3.2.0-21.34> <kernel-unable-to-test-upstream> <needs-upstream-testing> <precise> <staging> <linux (Ubuntu):Incomplete> <lxc (Ubuntu):Confirmed> < https://launchpad.net/bugs/959352 >12:58
frankbangary_poster: cool, workaround found12:59
gary_poster:-) cool12:59
frankbangary_poster: does launchpad-database-...-9.1 work?13:00
gary_posterfrankban, yes!  good thinking13:00
gary_posterand port is fine automatically13:00
frankbangary_poster: cool, I asked because I have a lpsetup branch ready with that fix, and now I am going to add the fix for bug 959352 too.13:01
_mup_Bug #959352: Ephemeral containers have "/rootfs" prefix in /proc/self/maps entries <apport-collected> <kernel-request-3.2.0-21.34> <kernel-unable-to-test-upstream> <needs-upstream-testing> <precise> <staging> <linux (Ubuntu):Incomplete> <lxc (Ubuntu):Confirmed> < https://launchpad.net/bugs/959352 >13:01
gary_postercool frankban sounds good13:01
gary_posterfrankban, what you are working on is kind of a "real" card.  I already had a meta card for that task.  I moved the meta task over into the active lane and I dragged your card to active.  We also technically have 90% of test runs green (better than that even) so we can officially think about these stretch goal cards.  I'll think about them later today, but for now...13:14
gary_posterbac, benji, frankban, frankban has started a stretch goal card, and we actually can officially do that.  This means that we are now going to aim for one bug card at a time.  people should either be working on that, or on the stretch goal feature...or on the IS interactio, I guess. :-/13:15
frankbangary_poster: ok. I will send a summary email for the meta task later.13:17
benjigary_poster: I can't quite parse the above.  What is the "that" in "people should either be working on that"?13:17
gary_posterbenji, the stretch goal card (the active meta card), via a "meta-task subtask"13:18
gary_posterso, the next time you are looking for a card, consider the stretch goal.  frankban, the email would be great.  we also need subtasks put in the "subtask" lane for people to grab13:18
benjiah, I didn't realize there was a meta card involved13:19
gary_posterif you'd like me to help you identify those with you, I'd be happy to.13:19
gary_posterbenji, yeah, I decided that those stretch goals seemed pretty much like meta cards.13:19
gary_posterfrankban, "if you'd like me to help you identify those with you, I'd be happy to" was addressed to you, and "those" are the subtasks for the kanban board :-)13:20
frankbangary_poster: I think I have already identified the main sub tasks, and I have branches ready for those. As I mentioned, I was actually running parallel tests in a buildbot environment created by lpsetup.13:20
benjiI've just barely started on my card (bug 994694); shall I put it back and work on the meta/streach card instead (which would put us back under the WIP limit which we somehow exceeded)13:20
_mup_Bug #994694: GeneratorExit exception throughout paralleltest subunit output <paralleltest> <Launchpad itself:Triaged> < https://launchpad.net/bugs/994694 >13:20
gary_posterbenji, not sure yet.  keep trucking, and I'll chat with frankban about what needs to be done.13:21
gary_posterfrankban, right.  so you've pretty much done everything already? :-)13:21
gary_posterbenji, "keep trucking" == "keep working on that bug" ;-)13:21
* gary_poster has been on the lookout for his use of vague idioms lately13:22
frankbangary_poster: probably. To be sure I need to 1. fix lpsetup with postgres 0.1 and the rootfs workaround and 2. complete a test run13:24
bacgary_poster invents "Mud flap management"13:25
frankbanpostgres 9.1! 0.1 is quite deprecated13:25
gary_posterfrankban, :-) ack.  So at most, at your EoD if you are not done and there are tasks that others could do, send out an email or talk to me about it and we'll see if anyone can take over for you.13:26
gary_posterfrankban, otherwise, I'll get out of your way.  Thanks!13:26
gary_posterbenji, <snort>13:26
gary_posterIn other news, 12 green runs in a row!13:26
benjiWe're "done" with fixing test bugs when we reach 95% passing runs, right?  Over how many runs is that?13:30
bacbenji: 19, no?13:34
bacbenji: i mean, i thought it was 95% green runs...so 1 failure in 20 full runs13:35
gary_posteryes, benji, what bac says is how I interpret it13:35
gary_poster"done" means that we could deploy with that13:35
gary_posterwe should keep running at that point and fix what we find13:35
gary_posterbut it means that stage is "good enough"13:36
bacand i assume we can ignore any non-green runs that also fail on the real buildbot causing testfix mode.  we shouldn't be penalized for those.13:36
gary_posterright, bac13:36
gary_posterthough there are so few commits in the LP tree that it hasn't been an issue.  OTOH maybe the UDS session will change that. :-)13:37
gary_posterbac were you able to dupe your bug?13:44
gary_poster(curiosity(13:44
gary_poster)13:44
bacgary_poster: no.13:44
gary_posterthat jibes with my experience then.13:44
bacgary_poster: so should we just keep an eye on this bug and see if it happens again?13:47
gary_posterbac...you got me.  I thought you were very brave grabbing it, because I wasn't sure what to do with it.  Mm, lemme look at it again.13:48
gary_posterbac, you could ask abentley if he has any bright ideas as to what to investigate, or if he's going to be replacing this code with celery tomorrow so nevermind, or something.13:49
gary_posterbac, no other bright ideas other than running the test 10 times in a loop and seeing if you can dupe it then.  I ran it three or four times in a loop and was unable to, after I didn't get any isolation results.13:51
gary_posterif neither of those lead anywhere bac, then, yeah, maybe dropping it for later would make sense.13:51
bacok, i'll hammer it and see13:54
frankbancould anybody please review https://code.launchpad.net/~frankban/lpsetup/postgres-9.1/+merge/105070 ? That's the branch I'd like to package and use for the next test run13:54
gary_posterbac, we have our first failure in 13 runs, and it's an old friend: http://ec2-23-22-54-190.compute-1.amazonaws.com:8010/builders/lucid_lp/builds/1/steps/shell_8/logs/summary .  We haven't seen these in quite some time, which suggests that our previous efforts improved things.  I wonder if the timeout needs to just be crazy or something.13:54
bacfrankban: what is the ETA for the death of setuplxc?13:54
bacfrankban: your MP is approved13:54
bac(i'd already done it)13:54
bacer13:54
frankbanbac: oh, thanks, and yes, I will change the exit code.13:57
* gary_poster started data center test run @ 13:05 Z (note for later)13:58
frankbanbac: ETA for the death of setuplxc -> we have a meta card to replace it, so I think we could stop supporting it (at least for non-blocking bugs). but maybe gary_poster can better answer your question13:59
gary_posterfrankban, well, when will you merge a branch to our charm that switches to lpsetup?14:00
gary_posterAssuming that you do that after everything works, that14:00
gary_posteris the moment I'm +1 on deleting setuplxc and forgetting about it14:01
gary_posterbut yeah, actually:14:01
gary_posterwe should only make changes to it for blocking bugs, I agree with that, bac, frankban14:01
gary_postereven before the switch14:02
gary_poster(which I expect to happen this week and maybe even tomorrow if we are lucky, from what frankban says)14:02
bacgary_poster: ok, then i won't bother with the exit_code -1 problem14:02
gary_posterok bac, cool14:02
bacgary_poster: it is fixed in lpsetup14:02
gary_postercool bac, I saw that in the MP, thanks14:02
gary_posterbac, do you think I should reopen bug 974617 or file a new bug.  It is the same old problem, IMO.  I think I see one more way to have a race condition, and a correlating approach to address it.14:36
_mup_Bug #974617: test_operationalerror_view_integration fails intermittently in parallel tests <paralleltest> <qa-untestable> <Launchpad itself:Fix Released by bac> <Python PGBouncer:Triaged> < https://launchpad.net/bugs/974617 >14:36
bacgary_poster: same bug14:37
gary_postercool, agree, wasn't sure, thanks14:37
bacgary_poster: i'm running a set of 20 tests for bug 994777 and so far i have seen one failure15:03
_mup_Bug #994777: lp.services.job.tests.test_runner.TestTwistedJobRunner.test_timeout_short fails rarely/intermittently in parallel tests <paralleltest> <Launchpad itself:In Progress by bac> < https://launchpad.net/bugs/994777 >15:03
gary_posterbac, wow15:04
baci have a set of 84 tests in my layer15:04
gary_posterbac, I'm not sure I understand.  (1) 20 tests or 84 tests?  (2) neither of us were able to dupe before; what are you doing differently?15:05
bac20 full runs of 84 individual tests in my --load-list15:05
bacgary_poster: i've done nothing differently but run the test repeatedly as you suggested earlier.15:07
gary_posteroh I see15:07
gary_posterbac, did you also try running the test by itself 20 times, out of curiosity?15:08
bacgary_poster: no but that's a good idea15:08
gary_posterbac, benji, I'm trying to copy the testing cabal's new version of the packages we need, and I'm getting this error: "testrepository 0.0.5+bzr153~ppa11~precise1 in precise (version older than the testrepository 0.0.5++z~ppa6~precise1 in precise published in precise)".  I'm strongly inclined to delete the ++z packages because the naming is just wrong for the present and the future AFAICT.  Do either of you disagree15:15
gary_poster or have other advice?15:15
benjigary_poster: it may be difficult to kill the ++z versions because of all the caching and other storage of version info that goes on, but I don't know enough about it to be sure it won't work in our narrow case15:16
bacgary_poster: i agree it is a mess that needs to be sorted out.  i don't have a plan.15:17
gary_posterbenji, it will be an annoyance for our own machines, if we installed them, but I think we can cope with it.  It shouldn't be a problem for the juju machines since they are torn down and rebuilt.15:17
gary_posterI plan to cope with the fallout locally first15:18
gary_posterso I can hopefully save others the pain15:18
gary_posterok, I'll give it a whirl, I think15:18
gary_posterthanks15:18
benjigary_poster: sounds like a plan15:18
gary_postercool15:18
bacgary_poster: the PPA machinery may interfere.  i *think* it remembers the versioning of previously published packages and enforces that the versioning move forward.15:18
bacbut it is worth trying15:18
gary_posterbac, even if it is deleted?  yeah, ok, giving it a whirl15:18
gary_posterthe PPA machinery let me delete and copy as I wished15:21
gary_posterit is building now15:21
gary_posterafter that I will try locally15:21
bacgary_poster: it doesn't throw up until publish time.  stupid.15:47
gary_posterbac :-(15:47
bacgary_poster: i am running 20 tests with just the failing twisted test.  so far i have seen one failure15:47
gary_posterbac, "it doesn't throw up until publish time": I thought you meant https://launchpad.net/~yellow/+archive/ppa/+packages but that looks ok.  you must mean your bug, but I'm not sure what publish time is for that job thing15:49
bacsorry gary_poster.  i'm not communicating very well today.15:50
bacthe PPA may build fine but then version issues are only caught after the build during the publish phase.15:50
gary_posters'ok, I'm throwing myself at some buildbot issue and feeling a bit numb myself15:51
gary_posterah ok15:51
* bac -> sandwhich15:51
gary_posterbut we're still fine in this particular case, right bac, given what you see on that link?15:51
gary_posters'ok, go have a sandwich  (which wich?)15:51
bacoh, yes, that does look nice15:52
gary_postergreat15:52
baci'm not sure what i'll order at Sandwhich15:52
bacprobably a sandwich15:53
gary_posterheh15:53
gary_posterbenji, if you want to be distracted a bit, take a glance at http://ec2-23-22-54-190.compute-1.amazonaws.com:8010/waterfall , top run15:54
gary_posterthe subunit stream has all tests15:54
gary_postereven a failure15:55
gary_posterhttp://pastebin.ubuntu.com/975994/15:55
gary_posterbut worker and stats are not complete15:55
* benji looks15:55
gary_postermust run to lunch/babysitting15:55
gary_posterfrankban, approved https://code.launchpad.net/~frankban/charms/oneiric/buildbot-slave/use-lpsetup/+merge/105086 with small comments17:13
gary_posteralso approved https://code.launchpad.net/~frankban/lpbuildbot/use-lpsetup/+merge/10508517:15
frankbanthanks gary_poster: I will change the charm to be backward compatible. For your other question: in lpsetup --testing is an alias for  --use-urandom --create-scripts --install-subunit17:16
gary_posterfrankban, heh, cool!17:16
frankbananother clean run \o/17:17
gary_posteryay! :-)17:17
gary_posterfrankban, might as well add your runs to https://dev.launchpad.net/ParallelTests/ResultsLog if you are willing and have the time.  no problem if not.17:18
frankbangary_poster: the first one already added, I will add the second and third runs (started now) later. need to go, have a nice evening!17:19
gary_postercool frankban thanks.  you too!17:20
benjigary_poster: since you reviewed it, I assume you know what's going on with frankban's lpsetup branch; can I move that card from review to landing?  Also, I'm looking for something to do, so I figured I should take the next step from his email and remove setuplxc from LP.18:21
gary_posterbenji, hey.  move card: +1.  remove setuplxc: not till everything has landed please.  what next: give me a sec please, trying to keep something in my head18:21
gary_posterok, thing in my head was used :-)18:22
gary_posterum, next...18:22
* benji envisions parisitic brain worms trying to escape.18:22
gary_poster:-)18:23
gary_posterbenji, 974617 will be easy if you agree with my assessment.  The UnicodeDecodeError in Francesco's card should be a bug against lpbuildbot just so we can track it, and then that is probably easy.18:24
gary_posterI'm in the middle of filing two other bugs18:24
gary_posterand they might not be as easy18:24
* benji looks at 974617.18:24
gary_posterso maybe by the time I've filed the pertinent bits you'll be done with those and can look at one of these18:24
benjigary_poster: I like your suggestion for 974617.  I'll start on it now.18:27
gary_postercool benji18:29
frankbangary_poster: I think this is new: http://ec2-23-20-70-98.compute-1.amazonaws.com:8010/builders/lucid_lp/builds/2/steps/shell_8/logs/summary19:33
* gary_poster looks19:33
gary_posterit is frankban.  I can file the bug if you give me a few minutes before you shut the instance down19:34
gary_posteror you can file it--if you do, please include a pastebin of the worker-1 output19:34
gary_posterI'm filing now19:35
frankbanthanks gary_poster, i'll pick the number and update the result log19:35
gary_posterthank you19:36
gary_posterfrankban, bug 99672019:37
_mup_Bug #996720: lib/lp/soyuz/javascript/tests/test_lp_dynamic_dom_updater.html fails rarely/intermittently in parallel tests <paralleltest> <Launchpad itself:Triaged> < https://launchpad.net/bugs/996720 >19:37
frankbandone19:41
gary_posterbenji, you are out until next Tuesday, right?19:58
benjigary_poster: right19:58
gary_posterstarting tomorrow I mean19:58
gary_posterok cool, enjoy and safe travels19:58
gary_posterbenji, source of weird buildbot/subunit output, if you are interested: 99672919:59
gary_posterbug 99672919:59
_mup_Bug #996729: zope.testing --subunit allows bad output on stdout, which can break subunit processing <paralleltest> <Launchpad itself:Triaged> < https://launchpad.net/bugs/996729 >19:59
benjiI am interested; looking.19:59
benjigary_poster: that's a good one20:00
gary_posteryeah, I'm afraid it will be even more interesting to fix, because I expect test fallout20:00
gary_postergiven the desired solution20:00
benjimmm20:01
benjiI do think your solution is a good one though.20:01
gary_postercool20:01
bacgary_poster: regarding https://bugs.launchpad.net/launchpad/+bug/994777 I find it is reproducible at about the same rate (5%) whether it is run in the original grouping or when that test is run individually.20:23
_mup_Bug #994777: lp.services.job.tests.test_runner.TestTwistedJobRunner.test_timeout_short fails rarely/intermittently in parallel tests <paralleltest> <Launchpad itself:In Progress by bac> < https://launchpad.net/bugs/994777 >20:23
baci think it has nothing to do with paralleltest and should be classified as a generic spurious test.20:23
gary_posterbac, sorry was on call with francis20:56
gary_posterbac, can we/should we just disable it then?  Do you want to timebox it or toss it to aaron for ideas?20:56
* gary_poster back in a sec20:56
* gary_poster back21:01
benjigary_poster: https://code.launchpad.net/~benji/launchpad/bug-974617/+merge/10513021:09
gary_posterdiff updating benji :-)21:10
gary_posterthere it is...21:10
gary_posterooh, nice benji.21:11
gary_posterapproved benji21:12
gary_posterI'll land tomorrow if you need me to21:12
benjigary_poster: yeah, it would probably be best if you land it, that way you get the emails, etc.21:14
gary_postercool benji.  have fun! :-)21:15
benjigary_poster: thanks, see you later21:16

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