[08:35] good morning gmb [08:42] Morning frankban [09:11] gmb: could you please review https://code.launchpad.net/~frankban/launchpad/bug-1012799/+merge/110270 ? [11:09] gmb: you there? [11:10] frankban: Sort of. In a cafe, suck Internet. What's up? [11:10] gmb: could you please take a look at https://code.launchpad.net/~frankban/launchpad/bug-1012799/+merge/110270 ? [11:11] frankban: I will do shortly, sure. [11:11] thanks [11:46] frankban: r=me, but I can't log in to lp at the moment; think there's a proxy in the way. Feel free to self review and note that I gave it my approval. [11:47] ok, thanks gmb [11:57] hi gmb, is anyone reviewing your branch yet? [11:58] Bac nope, feel free. :) [11:58] gmb: have you run it against trunk since my change landed? [11:58] No. [11:58] would be nice to know if i got them all [12:01] bac: I'll spin up a slave this afternoon for it. [12:02] gmb: does your implementation have the limitations we discussed yesterday, such as '-m' doing pre-filtering so that not all tests are checked? [12:03] bac: If I understand correctly, it sits before filtering takes place, but that might only be for -t; I'll check. [12:04] gmb: if that is the case, do you think we should warn when imcompatible options are chosen? [12:05] bac: yes, that sounds sane. [12:05] gary_poster: Might be late to standup; wifi is crummy here. [12:06] Am going to switch locations. [12:06] ack gmb [12:09] bac benji frankban gmb call in 2 https://plus.google.com/hangouts/_/dc2755d503ade939d419e49991619c2a6a3ba408?authuser=1&hl=en-US === _mup__ is now known as _mup_ [12:34] gary_poster: G+ crashed. Nothing to add beyond longish lunch. [12:34] gmb, cool :-) [12:34] gmb: good news, on a version of trunk before my changes, --require-unique correctly flags duplicates. on trunk it finds none. [12:34] Woot. [12:35] benji, may I move your call to 2:00 PM? [12:35] gmb: when the first dupe is found it raises an exception and quits. did you consider making it find and report all dupes? [12:35] gary_poster: sure [12:35] thank you [12:36] gmb: i'd like to suggest that unless you rejected it for good reason [12:36] bac: hmm, no. But that wouldn't be too hard; I'll do that. [12:36] yay [12:36] bac: no, I just like to prolong developers' pain. [12:36] gmb: and finally, do you have a companion LP branch that incorporates that flag into one of the makefile targets? [12:37] bac: yes but I can't push it yet due to proxy issues [12:37] bac, which would you prefer, 1:30-2:00 or 4:30-5:00? [12:37] 1:30 [12:37] cool [12:37] bac: Will be doing that this afternoon. [12:37] gmb: great [12:37] * gmb -> bureaucracy [12:44] gmb: review done [13:02] oh poo. I forgot to have the panel on bug 994752. I'll add it to tomorrow's list [13:02] <_mup_> Bug #994752: lxc-start-ephemeral's use of dhcp lease table is fragile

< https://launchpad.net/bugs/994752 > [13:08] benji, are you working on the EBS snapshot thing? If so, I will drag your card over to Active: Coding. Also has anyone stepped up to bug 1012171 yet to your knowledge? [13:08] <_mup_> Bug #1012171: Make captured stdout and stderr available within the subunit stream < https://launchpad.net/bugs/1012171 > [13:09] gary_poster: not quite yet; I'm trying to figure out the right way to set the ignores on my re-add-logs-directory branch; re. 1012171: not yet [13:09] benji ok. Do you want me to make a bug + card for that thing you are working on? [13:10] gary_poster: sure (or we can reuse the last bug, either way is fine with me) [13:11] benji I assume you mean bug 996729? if so, ugh, I want that one to be done! is it the same cause though? [13:11] <_mup_> Bug #996729: zope.testing --subunit allows bad output on stdout, which can break subunit processing and tests

< https://launchpad.net/bugs/996729 > [13:12] gary_poster: no, not 996729; let me find the bug number [13:12] gary_poster: bug 1011793 [13:13] <_mup_> Bug #1011793: lib/lp/services/profile/profiling.txt fails rarely/intermittently in parallel tests

< https://launchpad.net/bugs/1011793 > [13:18] ack [13:19] benji I decided just to make a card. It's in Active Coding with your face on it [13:19] gary_poster: my face appreciates it [13:19] :-) [13:32] frankban, hey. https://plus.google.com/hangouts/_/d8192b69d811f42c71018b1e307a9fa8ae1a1333?authuser=1&hl=en-US when you are ready [14:16] bac, Just looking at adding an error after all the tests have been scanned... it would mean adding the error handling outside the generator, which in turn would mean looping running through the generated results... Obviously this is going to be less performant than the current incarnation, but I'm not sure how much we need care about that. [14:16] Any opinions? [14:22] hi gmb. let me look [14:31] gmb: what if tests_from_suites had another parameter that was a list of duplicates found. tests_from_suites could populate that list without complaining and then find_tests could warn about the whole lot of them if any exists. [14:32] bac, Right, that's what I've done, but tests_from_suites is a generator and is recursive, so just doing something like: [14:32] tests = tests_from_suites(..., duplicate_ids=dupes) [14:32] won't have dupes fully populated [14:32] Until someone loops over `tests` [14:32] Or at least, I don't think it will. [14:33] My tests might be too shallow to prove one way or the other; [14:34] gmb: but it looks like find_tests does loop over the results of tests_from_suites. [14:34] for test, layer_name in tests_from_suite(suite, options): [14:34] bac, Right, but then it does other stuff that we don't want to care about if there are duplicates. [14:34] Although I suppose pragmatically it doesn't actually matter. [14:34] ah [14:35] gmb: i'd call my request a 'nice to have it not too complicated' [14:36] bac, Right, I think I can see a way to do it... and actually, this being zope.testing, I don't think "Ooh, that's a bit ugly" is actually a valid complaint. [14:36] It's not exactly going to win any pageants. [14:37] Thanks :) [14:49] gmb: sorry, i couldn't resist having a go at it. [14:49] http://paste.ubuntu.com/1040891/ [14:49] bac, That's exactly what I've done :) [14:49] excellent [15:02] gmb, sorry, went over with francesco. ready at https://plus.google.com/hangouts/_/d8192b69d811f42c71018b1e307a9fa8ae1a1333?authuser=1&hl=en-US ? [15:02] gary_poster, Okay, will be with you in a minute or two. [15:03] cool gmb [16:54] gary_poster, lp:~gmb/launchpad/devel/.bzr/branches/zope.testing-p13 has an up-to-date requirement on my zope.testing patch, and is based on the latest devel. [16:55] If you coudl run that in a buildbot of some sort, I'd be grateful [16:57] on call and lunch, will do soon [16:57] gary_poster, No huge rush; I EOD in 3 :) [16:58] :-0 [16:59] :-) do you want me to land if it passes, gmb? [17:01] * bac reboots [17:03] gary_poster, Please; I'll be landing the zope.testing branch shortly. [17:03] great [17:13] gmb, do you have an MP for that already? https://code.launchpad.net/~gmb/launchpad/devel/.bzr/branches/zope.testing-p13 doesn't take me anywhere... [17:14] gmb maybe you meant lp:~gmb/launchpad/zope.testing-p13 ? [17:14] gary_poster, Yes, I did. Damn colo branches [17:14] I'll do the mp now. [17:14] thank you [17:15] gary_poster, Ah, wait, hang on... I haven't updated the make target to use the new switch. Durr. [17:15] So, which make target needs updating? [17:16] gmb, I'd expect that you'd want to change buildout-templates/bin/test.in [17:16] Aah. [17:17] * gmb looks. [17:18] gmb, the defaults dict maybe? [17:18] gary_poster, Something like: [17:18] require_unique_ids: True, [17:18] ? [17:18] Er. With quotes [17:19] gmb, maybe. you got me :-P [17:19] Heh. [17:20] Well, let's see what happens.... [17:21] gary_poster, Yep, that seems to work. [17:21] I'll update it. [17:21] awesome [17:31] gary_poster: chat? [17:31] bac, yes was making hangout https://plus.google.com/hangouts/_/5cfec4c6014cb96f4037ea07d4f0744fa15d3e92?authuser=1&hl=en-US [19:59] hey benji, i'm looking at bug 1012171 and trying to figure it out. you have time for a quick call? [19:59] <_mup_> Bug #1012171: Make captured stdout and stderr available within the subunit stream < https://launchpad.net/bugs/1012171 > [19:59] bac: sure [20:00] cool, let me grab some tea and set up a hangout... [20:00] k [20:05] benji: i invited you but the window doesn't show me the url [20:05] bac: I got the invitation, one sec [20:15] benji: it looks like it is a dict with a testtools.content.Content object as the value [20:16] hrm, it's a shame but you'll have to complicate things a little; I hope those are easy to build [20:17] should be [20:36] We have 12 in a row green. The last time we had that--the only other time we had that--was May 4-8 [20:37] yay! [20:37] (note use of exclamation mark) [20:37] lol [20:37] Also for the last three days [20:37] we have a statistic of 75% [20:38] green [20:38] which is pretty good [20:38] what was our goal again? 95%? [20:39] yeah [20:39] so we need 7 more to be 1 failure in 20 [20:39] quick, spin up 7 instances [20:39] :-) We are not supposed to switch to stretch goals till 90% but we don't have any other known bugs to work on atm [20:48] benji: shouldn't the zope.testing subunit tests pass, i.e. bin/test -t subunit ? [20:51] bac: unfortunately I have no idea; the failing tests on our fork have conditioned me not to run the tests [20:51] gah, i want to strangle my mechanic. end of week five might have it done by middle of next week. [20:52] benji: when i finished my branch there were only 4 failures. i thought subunit worked. [20:52] benji, bac, we do have instructions on what tests are supposed to pass in versions.cfg [20:52] right, I forgot about that [20:53] we really need to at least squelch the failing tests [20:53] I forgot to ask about it [20:53] we could do that pretty easilyI woud guess [20:53] would [20:54] I will add card [21:02] the expected failures per versions.cfg (reallly an odd place to track it) are http://pastebin.ubuntu.com/913757/ [21:02] i'm seeing this in trunk http://pastebin.ubuntu.com/1041435/ [21:05] hmm, that looks mighty funny [21:06] so i'm picking up zope.testrunner from my ~/.buildout. is that right? is that what we use when running tests for zope.testing? [21:08] um, I don't think so [21:08] I think that may be from dependencies [21:08] and their tests [21:08] we use older version [21:08] I have to run [21:08] but--13 green runs! new record [21:09] sorry must run; baby bored [21:11] benji: can you try grabbing our 3.9.4 trunk and running tests? i think i'm getting weird, extra bad results