/srv/irclogs.ubuntu.com/2012/04/13/#launchpad-yellow.txt

frankbangmb: good morning, I am currently working on my introductory blog post, but i'd be really happy to stop it to pair on something, like the buildbot bug Gary mentioned09:11
gmbfrankban, Okay. I'm putting the finishing touches on fixing bug 974608, so how about we start pairing in about 30 minutes or so?09:12
_mup_Bug #974608: bugtask-find-similar.txt fails intemittently on paralleltest runs <paralleltest> <Launchpad itself:In Progress> < https://launchpad.net/bugs/974608 >09:12
frankbangmb: perfect09:12
gmbCool.09:13
gmbfrankban, I'll be ready to pair shortly; waiting for the upgrade / reboot dance to complete.09:54
frankbanok gmb09:55
gmbfrankban, I'm in https://plus.google.com/hangouts/_/extras/canonical.com/builbot-teaching when you're ready.10:15
gary_posterbac benji frankban gmb call in 212:08
gary_postergmb, you are gone in case you didn't notice12:25
gmbArse12:25
gmbgary_poster: Complete lockup. Not sure how much you heard.12:26
gary_postergmb really not so much :-/12:26
gary_posterbac, gmb will be working late so if buildbot card is tsill going and you are looking for work by then it might be a good task12:29
* gmb goes to get a cup of tea whilst the slave churns12:39
* gary_poster takes kids to school12:56
gmbfrankban, Okay, the sllave is up. I'm in https://plus.google.com/hangouts/_/extras/canonical.com/buildbot-teaching.12:56
gmbfrankban, I've added your keys to the master and slave:12:56
gmbmaster: ec2-23-21-29-239.compute-1.amazonaws.com12:56
gmbslave: ec2-174-129-115-195.compute-1.amazonaws.com12:57
frankbangreat thanks gmb12:57
bacgary_poster, benji : it looks like the problems i've been seeing with juju not properly parsing --constraints is being caused by our juju wrapper used for testing.  it is stripping the quotes, i think.13:08
benjigary_poster: jml has decided that the per-worker tagging is a good idea, so he implemented it too.  However, his implementation is optional and he likes it that way.  He suggests we get the result we want by modifying testr to use the components he's built (he made ConcurrentTestSuite pluggable and made a plugin for it) to get the effect we're after13:10
gary_posterbac, ah!13:11
gary_posterthat makes sense13:11
gary_posterbenji, eh, ok.  Do you have any objections?13:11
benjinot really13:12
bachere is what /usr/bin/juju receives are sys.argv13:12
bac['/usr/bin/juju', 'deploy', '--config=/home/bac/juju/oneiric/buildbot-slave/examples/lpbuildbot.yaml', '--repository=/home/bac/juju', 'local:buildbot-slave', '--constraints', 'cpu=8', 'mem=6G']13:12
gary_posterkinda too bad he didn't coordinate with us better, but hard to complain too much, benji13:12
gary_posterbac, yeah, that's the same kind of problem that I was trying to explain about why xvfb-run sometimes barfs13:12
benjigary_poster: I assume I should take it for granted that the code will land and attempt to set up a testr dev environment using these branches and work from there13:13
gary_posterbenji, I'd say landing's a safe bet, given he's the creator and maintainer of testtools.  Sounds like a good next step, yes13:15
gary_posterbenji, do you think you or I should say anything to jml about improved coordination?13:21
benjigary_poster: nah13:23
benji(yes, it actually took me that long to compose that reply)13:23
gary_posterbenji, lol, cool.  obviously I thought about it long enough that I felt I wanted to ask you :-)13:24
benjiYeah.  It is a little irritating, but between the (low) amount of harm inflicted (on the company, I don't really mind that much) and the probability of it happening often (low) and the probability of bringing it up actually changing anything (low), it doesn't seem worth it13:26
gary_posterright13:33
bacbenji: any idea on how to get the juju wrapper to respect quoted params?13:33
bace.g. --constraint "cpu=8 mem=4G"13:34
bacit is transforming to --constraints cpu=8 mem=4G, which causes juju to barf13:34
benjibac: hmm, let me look at it; one second13:34
benjibac: this is a problem with bash scripts that I don't understand; there must be a general solution, but I haven't been able to find it13:35
benjia hack would be to double-quote, like so: juju --constraint "'cpu=8 mem=4G'"13:36
benjibac: maybe changing $@ in the wrapper to "$@" would do the trick13:38
bacbenji, nope13:41
benjidarn13:42
benjihmm, man bash reinforces my suspicion that "$@" should do the trick13:44
benjibac: the script is a sh script, not a bash script, I wonder if that's an issue; try "$@" and change it to be run under bash13:46
bacbenji, i tried that too early.  :(13:46
benjidarn13:46
benjiyeah, I'm reading man sh now and it says the same things about "$@" that man bash says13:47
bac$@ will keep 'cpu=8 mem=4G' together as one argument, but when passed they are still white-space separated so the receiving script doesn't see them quoted13:48
benjibac: I got it to work for me, I did have to change all three instances of $@ in the wrapper to get them all though13:50
bacbenji, so it does.  my testing was flawed.  thanks.13:53
benjicool!13:53
bacbenji, pairing!13:53
benji:)13:53
benjiand now I finally know the right way to handle argument lists in bash.  I hope I never have to use that knowlege again.13:54
* benji reboots for great justice.13:59
bacbenji, so juju_wrapper has been deleted from our charms?  does it exist in any project?  It is referenced in the READMEs for the master and slave.14:17
benjibac: I didn't remember it being deleted.14:18
benjibac: revno: 36, timestamp: Wed 2012-02-29 12:10:14 +0100, message:[r=gmb] Added revision file. Removed juju_wrapper.14:19
benjiI like it when commit messages explain why an action was taken.14:20
gmbfrankban, test_command=xvfb-run ./bin/test --subunit $IDOPTION14:25
* gmb -> out for a while; back later14:29
bacgary_poster, i've manually started an lxc locally to look at the celery issue...but cannot log into it. the container is lucid but root/root nor ubuntu/ubuntu work as logins.  what am i forgetting?14:54
gary_posterbac on call there soon14:54
frankbangary_poster: I'd like more info about the goal of the card I am working on, when you have a minute14:59
gary_posterack14:59
gary_posterbac, did you create the lucid container binding your home directory, per https://dev.launchpad.net/Running/LXC?  If so, you should be able to log in as yourself.  If not, I suggest destroying that container and making a new one15:02
gary_posterfrankban, https://talkgadget.google.com/hangouts/_/extras/canonical.com/goldenhorde ?15:03
gary_posterThere now15:03
frankbangary_poster: great15:03
bacgary_poster, thx logging in as me worked.  duh.15:04
gary_posterbac cool.  frankban has an instance running too that he is about to shut down.  Is there anything you'd like him to run on that instance before he shuts down?15:21
gary_posterbac, I mean an 8 core ec2 instance, I think15:21
bacgary_poster, no, thanks15:23
gary_postercool, welcome15:26
frankbangary_poster: our lpbuildbot is bzr+ssh://bazaar.launchpad.net/~launchpad/lpbuildbot/public/, right?16:02
gary_posteryes frankban16:02
frankbancool thanks16:02
frankbangmb: for when you are backL I am running a full test run on your ec2 instances, and later I will run another, I will take care of shutting down the instances later16:11
benjiI hope the Comcast tech across the street isn't a bad omen for my connectivity.16:27
* gary_poster goes to bank16:47
bacbenji, did Comcast let you stay on-line?17:28
benjibac: apparently17:28
bacthey're nice that way17:29
* gary_poster has triumphed over Ubuntu's keyboard layout! He can enter euros € and accent graves è and accent agues é! He can write umlauts ü! HE HAS THE POWER!18:01
bacgary_poster, +118:28
gary_poster:-)18:28
bacgary_poster, have you noticed OSX now has IOS-like alternate character selection?18:29
bacif you hold down 'a' it'll show you a bunch of options for a-umlaut, etc18:30
gary_posterbac, you press the key down?  yeah, I think I had heard of it18:30
gary_posterI actually have not logged into the osx side in some time other than by accident18:30
gary_posteripad and iphone are floating my apple boat these days18:30
bacgary_poster, so celery seems very unhappy due to semaphores.  :(18:33
gary_posterbac, you have to remove that file, remember?18:33
bacgary_poster, http://pastebin.ubuntu.com/928292/18:33
gary_posteror that doesn't fix it?18:33
bacgary_poster, i did, i did18:33
gary_posterbac, when?  those are the exact symptoms that the bug is about18:34
bacwhen?  i did it before running the test, as we discussed yesterday18:35
gary_posterbac, IOW, when I say when, I mean, did you remove the dir...18:35
gary_posterok18:35
bacperhaps, maybe the work-around is a no-work-around18:35
bacor a broke-around, if you will18:35
gary_posterbac, it worked for me.  but yeah.  So maybe it is an ephemeral issue18:35
gary_posterbac, try this:18:36
gary_posterstart up lxc instance18:36
bacgary_poster, same same on my local lxc18:36
gary_posteroh18:36
gary_posterok nm18:36
gary_posterbac, try this, if you are willing & able:18:37
bacboth.18:37
gary_posterwell, one sec, lemme see if the newest lxc is around (the one that is supposed to have the actual fix...which is supposed to be equivalent to the workaround...18:37
* bac brb18:39
gary_posterbac, I don't think the newest one is there yet.  I'm going to try to dupe locally.18:41
gary_posterbac, ok, I got the skinny.  The workaround I described at the bottom of comment #1 is correct for lucid:18:53
gary_posterAdding this line to the container's fstab will fix the problem:18:53
gary_posternone dev/shm tmpfs defaults 0 018:53
* bac trying18:55
bacmy /etc/fstab was empty prior18:55
gary_posterwhat hallyn describes in comment 2 is apparently only for precise containers18:55
gary_posterbac, wait!18:55
gary_posterno18:55
gary_posteryou add it to18:55
gary_poster/var/lib/lxc/lptests/fstab18:56
gary_posterbac ^^18:56
bacok.  kill container, add it via host, restart container18:56
gary_posterbac, yes18:57
gary_posterbac, you may need to add back rootfs/dev/shm18:57
gary_posterchecking18:57
bacyep18:58
baclxc-start fails18:58
baclxc-start: No such file or directory - failed to mount 'none' on '/usr/lib/lxc/root/dev/shm'18:58
gary_poster>>> from multiprocessing.synchronize import Lock19:00
gary_poster>>> Lock()19:00
gary_poster<Lock(owner=None)>19:00
bacworks19:00
bachurrah19:00
* bac running all celery tests now19:03
bacall celery test now pass, gary_poster19:04
gary_posterbac, yay!19:05
bacthe /etc/mailname fix is still required (i think)19:06
benjigary_poster or bac: anyone want to help me flail on testr?19:06
gary_posterbac, please verify19:06
bacgary_poster, running now w/out /etc/mailname19:07
gary_postercool bac.  bac, does that mean we only have soyuz upload intermittent error and unclean reactor?19:07
gary_posterand bac, I don't think /etc/mailname is proper fix19:07
bacgary_poster, tests fail without /etc/mailname19:08
gary_posterbac, could you please set a pdb in the username method on your local machine where it works, and determine where it is getting the name?19:08
bacsure19:08
gary_posterThanks19:09
baci'll mv .bazaar first19:09
gary_posterbenji, I can in a few19:09
benjigary_poster: k19:09
* bac predicts it'll fail without .bazaar or /etc/mailname19:09
bacfrom inspecting the code19:09
gary_posterbac, I think that won't matter: in my experiments, the .bazaar was actually set to be in a tmp dir when I ran the test19:09
gary_posterbut maybe I'm wrong19:09
gary_posterand thus, an experiment :-)19:10
bacok19:10
bacand gary_poster your local machine has no /etc/mailname?19:10
gary_posterbac, right, but I *only* Have tests on lxc here19:10
bacgary_poster, have you taken your childrens to see the calder exhibit yet?  i may go this weekend19:11
baci removed /etc/mailname from my local machine (leaving .bazaar in place) and the whoami error was seen19:13
gary_posterbac, calder: no had not heard it was here.  looks cool, thank you for heads up19:15
gary_posterbac, ok, that's interesting.  so /etc/mailname is the difference.  I wonder if /etc/mailname is supposed to be added by postfix.  I think it is but could be wrong19:16
gary_posterI'm pretty sure I've never added it explicitly19:17
bacgary_poster, dpkg -L postfix does not show it19:17
bacdpkg -S /etc/mailname also does not show any package claiming it19:17
gary_posterah I was just trying to figure that sort of thing out19:17
bacif the postinstall script of postfix write it, i wonder if dpkg would know about it19:19
bacrecall when you install postfix it asks you a bunch of questions...it may be written at that time19:19
gary_posterbac, um, ok.  I guess we could change setuplxc to write localhost there...yeah, I think I've been asked.  Well, we could do dpkg-reconfigure19:19
gary_posterum19:20
gary_posterthat's not the command19:20
gary_posterwhat am I thinking of19:20
gary_posteroh it is the command19:20
gary_posterbut it is not in my lucid19:20
gary_posteryes it is19:20
gary_postersigh19:20
gary_posterbac, yes19:21
gary_posterdpkg-reconfigure lets you set up system locally19:21
gary_postersuggests localhost19:21
bacgary_poster, but we're not installing postfix (or any MTA) right?  we=setuplxc19:23
gary_posterbac, it is installed somehow19:23
bacor is it a depend of lp-dev-depend19:23
gary_postermay well be19:23
gary_posterI suspect it is19:23
gary_posterbut if you look in the container19:23
gary_posterI think you will find that postfix is installed19:23
gary_posterbenji, um, my "few minutes" is starting to look like "sometime after 4" :-/19:24
benjigary_poster: heh, ok19:24
* benji goes to get another Dr Pepper.19:24
bacgary_poster, yes, it is installed in the container but i can't figure out why19:29
bac'apt-cache showpkg postfix' doesn't show anything interesting19:29
bachmm, i wonder if postfix is just part of main installation19:30
gary_posterbac, bsd-mailx depends on default-mta19:31
gary_posterwhich postfix provides19:31
gary_posterthat's the only dependency I see19:31
bacdefault-mta is not installed in my container19:31
gary_posterbac, I did the lame-o approach of starting aptitude and pretending I wanted to uninstall postfix and seeing what complained19:33
gary_posterfor some reason my terminal won't let me select the complaint19:33
gary_posterso I could copynpaste19:33
gary_posterbut you could dupe19:33
gary_posterif that is of interes19:33
gary_postert19:33
gary_posterwhich I question19:33
baci tried 'sudo apt-get -s remove postfix'19:33
bacwhich should be the same (-s == dryrun)19:34
bacnothing was listed19:34
bacso, it looks like a developer/test box is automatically configured with an MTA and a properly configured one sets up /etc/mailname19:35
bacgary_poster, so why do you think it unwise to have our setup write that file?19:35
gary_posterbac, I am worried that others who do not use setuplxc (or lpsetup) will encounter this.  I am worried that something else really should have added this for us.  I guess it is fine.  However, I suggest filing a bug so people can find out the solution in the future (and so that people like RObert can educate us if necessary)19:37
gary_posterSo file a bug that this test fails in a lucid lxc because /etc/mailname is not set up19:38
gary_posterand then go ahead and make a card and fix it since it in setuplxc since it is easy19:38
gary_postermaybe make a card in slack for lpsetup too19:38
gary_posterI need to go get kids from school (K's foot is sprained)19:38
gary_posterback in a bit19:39
bacok19:41
gary_posterbac, everything cool?20:12
bacgary_poster, yep, just finished the MP.  want to review?20:13
gary_postersure bac20:13
bachttps://code.launchpad.net/~bac/launchpad/bug-981114/+merge/10197020:13
bacsuch good timing20:13
baci'm nearing EOW so make it snappy!  :)20:13
gary_posterbac :-P20:14
gary_posterI added a card for lpsetup btw; I saw your bug/card, thank you20:14
bacgary_poster, thx i hadn't gotten to that yet20:16
bacgary_poster, i haven't verified this works yet.  i'm about to relocate but when i get home i'll start an ec2 run and only land the change after i've proven it works.20:17
baci did show that 'localhost' in /etc/mailname was sufficient20:17
gary_posterbac, you could also do this in pure Python from the host; might have preferred that but won't ask you to change it.  As a fly-by, could you please add a comment to the line *before* yours such as "# XXX Work around for kernel bug 959352"?  That is, before20:18
gary_posterroot_sshcall('ln -s /usr/lib/graphviz /rootfs/usr/lib/graphviz')20:18
_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 >20:18
gary_posteroh wait20:18
gary_posterI think that the comment is already there20:18
gary_posternm20:18
gary_postersorry20:18
gary_posterso anyway, will approve20:18
bacgary_poster, oh, wait, doing it host-side makes sense.  6 vs 1/2 dozen?20:19
gary_posterbac, doing it without a subprocess call seems more elegant20:19
gary_posterbut no real diff20:19
gmbgary_poster, Do you know how far frankban got with the work we were doing before I had to go out?20:19
gary_poster6.1 vs half dozen20:20
gmbI meant ot ask him when I got back but forgot.20:20
gary_postergmb, yes.  He determined that the problem was buggy.20:20
bacgary_poster, ok, i'll look at doing it that way20:20
gary_postergmb, he fixed the code issue, but we still need testr bug 949950 (tracking) in order for this to work20:20
_mup_Bug #949950: Feature request: an option to show full subunit stream of running tests <Testrepository:New> < https://launchpad.net/bugs/949950 >20:21
gmbgary_poster, Ah, because the master doesn't actually get subunit output form the slave.20:21
* bac -> goes home20:21
gary_posterright20:21
gary_posterhave a good wknd bac20:21
gmbWe'd kinda figured that out when I left, but I'm glad it wasn't just that we were doing something silly.20:21
bacu220:21
gary_poster:-)20:21
gary_postergmb, how is 974608?  Can review something if that helps20:22
gary_posterbenji, can talk testr for the last half hour of the day if that helps :-)20:24
benjigary_poster: it can't hurt20:24
gary_posterI could make really unpleasant jokes.  That would hurt.20:24
benjiheh20:25
gary_posterbenji, I'm in https://talkgadget.google.com/hangouts/_/extras/canonical.com/goldenhorde20:25
gary_posterbenji, please send us tags method just to make sure we are on right track21:06
gary_posterthis weekend sometime21:06
benjigary_poster: if you can still see my terminal, it's there21:06
benji(along with a whitespace fix)21:07
gary_postergot it thanks benji21:07
benjicool21:07
* benji goes afk.21:07
* gary_poster heads out21:10
gary_posterbye21:11
gary_postergmb, meh, do you want me t try and give 4 min review of MP?  Must run soon21:12
gary_posterbut I saw you just got it in :-)21:12
gmbgary_poster, That would be awesome if you could - it's fairly mechanical anyway.21:12
gary_posterack, on ity21:12
gary_postergmb, not my best review work :-P but I thought it looked very good21:14
gary_posterThank you!21:14
gary_posterhave a great weekend gmb21:14
gmbgary_poster, Thanks! Have a good weekend; I'll see you Tuesday.21:14
gary_posterah yes, thanks for reminder21:14
gary_posterbye21:14
gmbCya21:14

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