=== Ursinha_ is now known as Ursinha === yofel is now known as Guest73835 === almaisan-away is now known as al-maisan === al-maisan is now known as almaisan-away === Guest73835 is now known as yofel === mrs_poolie is now known as cinerama === cinerama is now known as mrs_poolie === mrs_poolie is now known as cinerama === cinerama is now known as mrs_poolie [23:13] wgrant: https://code.launchpad.net/~stevenk/launchpad/re-enable-workermonitor/+merge/136072 [23:16] StevenK: Why? [23:16] mrs_poolie: \o/ [23:17] wgrant: To close a critical and have less disabled tests? [23:17] StevenK: I mean, what's the justification for just increasing the timeout? [23:17] Why should these things take a minute? [23:17] (it's a trick question; no test can ever be allowed to run for a minute) [23:18] hi StevenK :) [23:18] wgrant: Well, I'm not sold on 60, I was just seeing if increasing the timeout would stop the tests failing -- given 20 seems bad, I wasn't sure about 30. [23:19] Ah, they run subprocesses, I see [23:19] And it's a timeout, it isn't supposed to take that long, that's when we give up and fail the test, no? [23:19] Your __all__ is missing a comma [23:19] Otherwise good [23:20] Ah, so it is [23:23] wgrant: Can haz approve? [23:23] Done [23:25] Hmm, kanban actually logged me out [23:39] mrs_poolie: nice nick! [23:40] hi bigjools [23:40] mrs_poolie: fyi, the Ghiardelli chocolate did *not* last long [23:41] bigjools: lol. we still have lots here. going to give some away