StevenK | wgrant: https://code.launchpad.net/~stevenk/launchpad/stop-lying-please/+merge/165000 | 02:28 |
---|---|---|
wgrant | StevenK: I hope that breaks some tests | 03:42 |
StevenK | I doubt it, based on my grepping | 03:42 |
=== tasdomas_afk is now known as tasdomas | ||
lifeless | wgrant: / StevenK: hey, I would be very interested in knowning a perf comparison between ec2 and hp's test perf for lp parallel single-node test runs | 06:35 |
wgrant | lifeless: At what sort of node sizes? | 06:45 |
lifeless | wgrant: well, the ones you do ec2 test runs on .. xlarge isn't it? | 06:46 |
wgrant | lifeless: ec2 isn't parallel today | 06:46 |
lifeless | wgrant: oh! | 06:47 |
lifeless | wgrant: why not :) | 06:47 |
wgrant | lifeless: Because if we care about timeliness then we just throw it straight at buildbot | 06:47 |
wgrant | ec2 test still uses a single test process on an m2.xlarge | 06:47 |
lifeless | I would be curious if that ran any better on the same size in HPCloud, but less interested than in the actually-uses-the-cpus-case :) | 06:48 |
=== Gwaihir_ is now known as Gwaihir | ||
wgrant | StevenK: I've shuffled gina around to be a bit more sensibly testable (eg. you now don't have to run scripts/gina.py to test anything at all...), so https://code.launchpad.net/~wgrant/launchpad/gina-skippage/+merge/165048 is now pretty trivial. | 10:01 |
StevenK | wgrant: Your indentation for the test looks a little suspect | 10:10 |
wgrant | StevenK: Howso? | 10:12 |
StevenK | wgrant: You open a listcomp and don't indent another level | 10:16 |
wgrant | StevenK: Ah, I indented one by a space, and one not | 10:17 |
wgrant | Must have miscopied | 10:17 |
wgrant | Or do you object to the second too? | 10:17 |
StevenK | I'd prefer 4, but I can deal with one | 10:19 |
wgrant | It's unambiguous and I think more readable than 4 | 10:20 |
wgrant | Though it is questionable | 10:20 |
StevenK | wgrant: So fix the first one and land it, or refactor it to a function :-P | 10:20 |
wgrant | StevenK: Oh yeah, that's done | 10:44 |
StevenK | wgrant: It's been approved for a while | 11:37 |
wgrant | StevenK: Thanks | 11:50 |
=== dpm is now known as dpm-laptop | ||
=== BradCrittenden is now known as bac | ||
=== wedgwood_away is now known as wedgwood | ||
=== tasdomas is now known as tasdomas_afk | ||
=== dpm-laptop is now known as dpm | ||
=== wedgwood is now known as wedgwood_away | ||
=== wedgwood_away is now known as wedgwood | ||
=== tasdomas_afk is now known as tasdomas | ||
=== tasdomas is now known as tasdomas_afk | ||
=== deryck is now known as deryck[lunch] | ||
=== deryck[lunch] is now known as deryck | ||
=== mbarnett` is now known as mbarnett | ||
=== wedgwood is now known as wedgwood_away |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!