[03:27] StevenK: https://code.launchpad.net/~wgrant/launchpad/dkimpy/+merge/197637 [03:34] wgrant: I'm off today [03:34] StevenK: Ah, I see. [06:16] That was smart. I'd managed to disable nearly all of test_gc with an incorrect inheritance order :-/ [06:18] No, that should be correct. wtf are my tests not being discovered? [06:22] stub: How're you running them? [06:22] stub: The test failures showed up with -t lp.services.librarianserver here [06:22] bin/test -vv test_gc [06:23] Yes, found and fixed them. But also noticed I'm only running 6 tests from librarian_gc [06:25] stub: Hm, that finds them all on 2.6 here [06:42] wgrant: Do the rest pass btw? [06:43] oh, of course they don't [06:43] stub: Everything in lp.services.librarianserver passes, apart from the GC MD5 issues, and the 2.7ism, IIRC [07:06] Seems to be removing duplicate tests, and deciding that because the tests are defined on the Base they shouldn't be run on the subclass (despite the Base not mixing in TestCase) [07:08] Hm, that's weird. [07:16] I've used the same pattern in the past, so we should confirm the test count from the precise buildbot. [07:17] I checked before, it's only about 50 below the devel one, but that includes layer setup/teardown so it would need manual diffing [07:22] are you inheriting the base you think you are? [07:28] Yeah, I'm got print statements in the setUps at the moment [07:28] LibrarianSwiftGarbageCollectionTests inherits (LibrarianGarbageCollectionTestsBase, lp.TestCase) [07:29] LibrarianGarbageCollectionTestsBase has no base classes (apart from object) [07:29] lp.TestCase inherits (testTools.TestCase, fixtures.TestCaseWithFixtures) [07:32] Cute: http://paste.ubuntu.com/6518444/ [07:34] test.py test_gc trims some tests, test.py --test=Swift can find them though. Neither get tests defined in the base class [07:36] It's probably to do with 2.7's discover support [07:36] In which case it won't affect full test runs [07:37] http://bazaar.launchpad.net/~stub/launchpad/swift-librarian/view/head:/lib/lp/services/librarianserver/tests/test_gc.py for anyone playing along at home [07:37] wgrant: I can land it and see what happens... just need to revert if the tests don't get run [07:38] Yeah, easy to check from the subunit :) [07:38] But I think it should be fine. [10:38] stub: Are you looking at the GC test failures? === wgrant_ is now known as wgrant [11:35] wgrant: Back from blinner, I'll look now [11:38] stub: Thanks. [11:39] One failed test, run twice. Maybe a trivial fix. [11:55] wgrant: Can I land http://paste.ubuntu.com/6519343/ with it? [11:56] That is how I can get the tests running locally. Certainly something crappy in test discovery in py2.7 or some test helper I have. [11:57] stub: I'd prefer not to have a workaround in that one file... [11:57] (to run them locally I just use '-t test_gc' to do pattern matching on the whole test name) [11:57] Rather than specifying a module, which will invoke 2.7's new discover logic. [11:58] Ok. [11:59] wgrant: testfix? http://paste.ubuntu.com/6519365/ [12:00] oh, two redundant lines [12:00] stub: Are content_* used anywhere? [12:00] nuking [12:00] Yeah, that [12:00] Otherwise it looks fine :) [12:01] http://paste.ubuntu.com/6519371/ [12:01] k [12:03] wgrant: Apart from large files and pulling the Swift creds from a config file rather than the environment, can you think of other stuff we need? [12:03] I don't think so. [12:03] Maybe? [12:05] I'll do the config file branch, then we can see what we trip over on staging I guess. [12:05] Yeah [12:06] Not quite sure how to fully test it there. Might want to inject a lot of files into the staging librarian, I suppose? [12:28] wgrant: Last time I checked I had about 250GB I could play with [12:30] wgrant: We need to confirm it serves files reliably (small and large), and that the disk-> swift tool works reliably in non-destructive mode. [12:31] So I guess rsync over a subtree, push it in, suck it back out from the Librarian. [12:32] Or just generate and upload some random content [12:35] But the first real test is that nothing acts any differently, since we haven't turned on the FF. [12:35] stub: Did you forget to land that with [testfix]? [12:35] It doesnt' seem to have made it in [12:36] stub: Yeah, that QA plan sounds relatively sane [12:36] Proposed commit message: [12:36] [testfix][r=stub][bug=1257636] Fix broken test_gc.py [12:36] Edit before sending? ([y]es, [n]o): Connection Timeout: disconnecting client after 300.0 seconds [12:36] Heh [12:36] attention fail ;) [12:38] The new house I'm moving too should have 1Mb/s uplink on Friday. Been stuck on 4Mb/512kb for too long. [12:44] stub: 20M/512K here :/ [12:44] Australian HFC providers have a thing for 40:1... === xnox_ is now known as xnox === bdmurray_ is now known as bdmurray === Ursinha is now known as Ursinha-afk === jpds_ is now known as jpds === Ursinha-afk is now known as Ursinha