[10:50] * gmb -> afk for an appointment [12:08] bac benji gmb https://plus.google.com/hangouts/_/caa2fb9c4a58228cb4170bbcb64e5d210e62867f?authuser=1&hl=en-US in 2 [14:06] gary_poster: wow got my ec2 bill. i'll bet yours is much more! [14:06] bac $671 [14:07] oh my [14:07] we see who the human-qa-bot is... [14:10] gary_poster, bac, benji___: I keep seeing this when I try to put my Launchpad zope.testing p16 branch through EC2: [14:10] Traceback (most recent call last): [14:10] File "/var/launchpad/ec2test-remote.py", line 280, in test [14:10] raise NonZeroExitCode(retcode) [14:10] NonZeroExitCode: Test process died with exit code 2, but no tests failed. === benji___ is now known as benji [14:10] Anyone got any ideas? [14:10] Happens every time. [14:11] gmb, um, sounds like your branch's tests don't work. Do they run locally? [14:12] Yep./ [14:12] Well. [14:12] Hmm. [14:12] Let me check again [14:12] Since I haven't pulled devel into this branch for a coupel of days. [14:14] gary_poster, But they ran in a parallel buildbot yesterday [14:15] gmb, you could also use ec2 test and run them there to see what it is seeing, or use that "stay connected after starting" flag that the ec2 command has that lets you watch what it does [14:15] Ah, good idea. [14:15] I'd forgotten about that switch. [14:15] gary_poster, Thanks for the tip. Will poke at it and get back to you. [14:16] gary_poster, bac, benji: Can one of you review https://code.launchpad.net/~gmb/lpsetup/dont-blow-away-ssh-keys-bug-1018823/+merge/113192 for me? [14:16] gmb: I can. [14:16] Thanks [14:17] gmb: why did you add doctests? are they being run now? [14:18] benji, They're used elsewhere in the codebase and the unittests I found didn't look all that comprehensive (though I could have been looking in the wrong place). I thought they were being run on merge. Or is it just nosetests without --with-doctest that's getting run. [14:18] ? [14:19] gmb: I am pretty sure the doctests are never being run [14:21] Oh. [14:21] That's annoying, cos there's a few of them. [14:21] I'll reimplement them as unit tests then. [14:22] gmb: or get nose to run them (and kill/fix the existing decoys that have rotted) [14:23] benji, nosetests --with-doctest runs cleanly on that branch. [14:24] So we could just ask bac to update the Tarmac setup ;) [14:24] 94 tests --with-doctest vs 78 without. [14:24] gmb: that makes me suspect it's not actually running the tests; try adding a failing test and see if it is actually being run [14:24] hmm, maybe they are [14:24] gmb: that should be easy enough. patches welcome. :) [14:24] I still have my doubts though. [14:25] benji, Adding: [14:25] print False [14:25] Fail [14:25] To my doctest caused it to fail. [14:26] yay! [14:26] bac, So, what do I need to patch? [14:30] gmb: the branch is at bzr+ssh://bazaar.launchpad.net/~yellow/lp-tarmac-configs/tarmac-puppet [14:30] gmb: i'm happy to make the change but we do need to get someone besides me who can maintain it. [14:31] gmb: so it is up to you, easiest and fastest is for me to make the change. best for long term is to get someone else involved. [14:35] bac, Then I'll happily get involved. [14:40] bac, Does this look okay: https://code.launchpad.net/~gmb/lp-tarmac-configs/add-nostests-to-tarmac-puppet/+merge/113221 [14:43] gmb: looks great. unfortunately, our tarmac configs are not tarmac managed so it'll need to be merged manually. [14:43] ok [14:43] bac, I'll take care of it. [14:44] grr, dyndns no longer offers free domains and i don't want to continue with lptarmac on my account. [14:44] anyone know of a similar, free service? [14:45] bac, Merged and pushed. [14:45] cool [14:46] gmb: if you want, i can tear down the server and let you start it up. would require going through the prep steps at https://dev.launchpad.net/yellow/TarmacOnCanonistack [14:46] gmb: and then you'd own it until next time [14:48] bac, That sounds cool, let's do that. [14:49] gmb: it is killed [14:50] feel free to start a hangout if you want a walk through. do note any gibberish in the doc i wrote for de-gibberishing. [14:50] bac, Okay. I need to do the canonistack setup dance first. [14:51] yep, lots of little dances involved. [16:03] gmb: i've got to run to town. email me the new ip address of the tarmac instance if you get it running or let me know if i should launch it myself. [16:06] bac, Okay. Got distracted by a review, sorry. [16:38] benji, Were my zope.testing changes okay? [16:40] gmb: zope.testing? I looked at (and am reminded, need to finish) an lpsetup branch of yours [16:40] benji, Erk, I meant lpsetup. [16:40] Sorry. Was reading the last line of the tarmac log at the time I wrote that :) [16:40] gmb: finishing now [16:40] :) [16:41] gmb: approved with one worry/suggestion [16:41] Okay [16:41] Or: okay/ruh-roh [16:42] benji, ISWYM. I'll add an assertion that it's "a" or "w" [17:35] gary_poster: I'm ready when you are.' [17:35] (and so is my stray quotation mark friend) [17:36] heh [17:36] ok cool, I'll start one up [17:37] benji https://plus.google.com/hangouts/_/5a4f3621a45ae1f8deacdb6497bd9f37aa722bb0?authuser=1&hl=en-US [18:07] gary_poster, benji: anyone care for a review break? [18:07] https://code.launchpad.net/~bac/lpsetup/addheaders/+merge/113270 [18:09] bac, on it [18:11] bac, conflict blather lines 50-57? [18:12] gary_poster: oh [18:18] bac, I suspect the "MERGE-SOURCE' version is intended, in which case it is fine. Only other comment is this: [18:18] - line 26 of diff: please feel free to remove --2a. We are. [18:18] gary_poster: ok. just pushed conflicts fix [18:18] Looks good, thank you bac. When I see conficts fix I'll approve [18:19] well, I'll make an "approve" comment [18:24] gary_poster: thanks. [18:24] gary_poster: you can approve the whole thing if you want. it is ready to land. [18:24] done bac [18:34] * bac <= petards [18:36] so if one uses DEBIAN_FRONTEND=noninteractive with apt-get will it simply not query you for stuff? [18:38] hey gmb, you still around? [18:39] gary_poster: could you change the topic and s/dyndns/no-ip/ ? i created a new account at no-ip.org for our use. otherwise i'd have to have shared my super-secret dyndns password with everyone. [18:40] sadly it'll also require changing your .ssh/config. growing pains. === gary_poster changed the topic of #launchpad-yellow to: https://dev.launchpad.net/yellow | http://launchpad.leankitkanban.com/Boards/Show/14028610 | http://irclogs.ubuntu.com/ | https://devpad.canonical.com/ | ssh tarmac@lptarmac.no-ip.org [18:40] bac, done unless you tell me I made a mistake [18:41] gary_poster: that look swell. [18:41] cool [19:13] benji, you are gone [19:13] gary_poster: trying to come back [19:27] bac dragged your lpsetup banners card to done-done (right?) [19:27] yep [19:34] gary_poster: https://code.launchpad.net/~benji/lpsetup/bug-1016645-add-command-line-options/+merge/112588 [19:35] benji approved [19:53] benji found integration test card; added your head & moved [19:55] gary_poster: k; there were conflicts on our branch, looking at them now [19:55] oh ok benji [19:55] moved it to landing then [20:05] gary_poster, benji: ever tried running 'shutdown' on an ec2 or canonistack instance? wondering if that cleans up after itself and terminates the instance, freeing resources and turning off the meter. [20:06] bac, on call, but not exactly, only partly [20:06] bac: for EC2 it does if you set up the machine to work that way when you created it (terminate on shutdown); otherwise you have to go into the console and terminate the instance [20:07] thanks! [21:04] bac, did you decide to take tomorrow off, or not? [21:04] NO! [21:04] heh [21:04] you did not decide? [21:04] i will honor our forebearers by my sweat and toil [21:04] heh [21:04] ok cool [21:05] theoretical sweat [21:05] metaphorical sweat [21:05] gary_poster: you blowing stuff up and charring meats? [21:07] naah, bac. We will go to local country club as guests of my cousin and family, and then have sort of vegetarian meal, honoring the fact that our forebears probably had to eat a lot of veggies before we all got so comparatively rich. We *might* go to downtown Raleigh to see them blow things up, but that will be getting a bit late for the young 'uns. [21:08] sounds nice. is this the raleigh country club on glenwood? [21:08] i don't think they have vegetables there. [21:13] heh [21:14] bac, no, this is the north hills one. they are more middle/lower upper class, and more into tennis than golf. I think it is a $10K+ induction fee, which is more than we are interested in, but a whole lot less than the one I think you are thinking of