)]}'
{"ironic/tests/unit/drivers/modules/ibmc/test_power.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"1eebd5169a3a41f05c9f9d72ed545a6945baabc9","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        conn \u003d self.mock_ibmc_conn(connect_ibmc)"},{"line_number":192,"context_line":"        with task_manager.acquire(self.context, self.node.uuid,"},{"line_number":193,"context_line":"                                  shared\u003dFalse) as task:"},{"line_number":194,"context_line":"            self.config(power_state_change_timeout\u003d2, group\u003d\u0027conductor\u0027)"},{"line_number":195,"context_line":"            expected_values \u003d ["},{"line_number":196,"context_line":"                (constants.SYSTEM_POWER_STATE_OFF, constants.RESET_ON),"},{"line_number":197,"context_line":"                (constants.SYSTEM_POWER_STATE_ON,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_ea90ec44","line":194,"updated":"2019-04-05 12:23:04.000000000","message":"I wonder why we need this. It looks like a red flag to me, maybe we don\u0027t mock everything we need to?","commit_id":"cc431649d00091c75beef68ee69bd0ae77ec1af2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f12128728e6825d470c015a4548a0a1bc6bbac68","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        conn \u003d self.mock_ibmc_conn(connect_ibmc)"},{"line_number":192,"context_line":"        with task_manager.acquire(self.context, self.node.uuid,"},{"line_number":193,"context_line":"                                  shared\u003dFalse) as task:"},{"line_number":194,"context_line":"            self.config(power_state_change_timeout\u003d2, group\u003d\u0027conductor\u0027)"},{"line_number":195,"context_line":"            expected_values \u003d ["},{"line_number":196,"context_line":"                (constants.SYSTEM_POWER_STATE_OFF, constants.RESET_ON),"},{"line_number":197,"context_line":"                (constants.SYSTEM_POWER_STATE_ON,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_43de7c6a","line":194,"in_reply_to":"5fc1f717_7d912752","updated":"2019-04-10 14:29:58.000000000","message":"so it is a bit of a red herring. The fundamental issue is that these tests do take a while as they are written...\n\nI think it is because of how layered this is because for the power state calls we end up invoking retrying which also again invokes eventlet and I suspect the actual patch is doing nothing as a result. Of course, I\u0027ve also reached the end of a long day.\n\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Worker Balance\n \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n - Worker 0 (1 tests) \u003d\u003e 0:00:39.184671\n - Worker 1 (1 tests) \u003d\u003e 0:00:12.813658\n - Worker 2 (1 tests) \u003d\u003e 0:00:04.669003\n - Worker 3 (1 tests) \u003d\u003e 0:00:04.122923\n - Worker 4 (1 tests) \u003d\u003e 0:00:30.427417\n - Worker 5 (1 tests) \u003d\u003e 0:00:02.517347\n - Worker 6 (1 tests) \u003d\u003e 0:00:02.928579\n - Worker 7 (34 tests) \u003d\u003e 0:00:00.933506","commit_id":"cc431649d00091c75beef68ee69bd0ae77ec1af2"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"17450d0707fc1766d1061b0e52445ed01822f2bc","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        conn \u003d self.mock_ibmc_conn(connect_ibmc)"},{"line_number":192,"context_line":"        with task_manager.acquire(self.context, self.node.uuid,"},{"line_number":193,"context_line":"                                  shared\u003dFalse) as task:"},{"line_number":194,"context_line":"            self.config(power_state_change_timeout\u003d2, group\u003d\u0027conductor\u0027)"},{"line_number":195,"context_line":"            expected_values \u003d ["},{"line_number":196,"context_line":"                (constants.SYSTEM_POWER_STATE_OFF, constants.RESET_ON),"},{"line_number":197,"context_line":"                (constants.SYSTEM_POWER_STATE_ON,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_7d912752","line":194,"in_reply_to":"5fc1f717_ea90ec44","updated":"2019-04-09 14:51:03.000000000","message":"with eventlet\u0027s sleep mocked, it seems pointless.... OR we could be mocking the wrong thing now that I think about it...","commit_id":"cc431649d00091c75beef68ee69bd0ae77ec1af2"}]}
