)]}'
{"climate/plugins/instances/vm_plugin.py":[{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"cf212c321160640e2deb573a9de6f0a56435c1f9","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                except climate_exceptions.Timeout:"},{"line_number":75,"context_line":"                    LOG.error(\u0027Image create failed with timeout. Take a look \u0027"},{"line_number":76,"context_line":"                              \u0027at nova.\u0027)"},{"line_number":77,"context_line":"                except nova_client.exceptions.Conflict as e:"},{"line_number":78,"context_line":"                    LOG.warning(\u0027%s. Take a look \u0027"},{"line_number":79,"context_line":"                                \u0027at nova. (Request-ID: %s)\u0027 % (e.message,"},{"line_number":80,"context_line":"                                                               e.request_id))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9%2FLU%3D","line":77,"updated":"2014-03-31 18:32:43.000000000","message":"I guess you mean nova_exceptions here, module nova_client.exceptions is not available here)","commit_id":"00bfad876bf9e797a3df583b8c88cedc65c64327"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"019ae3972a50cd5c1de897a1af69450ffc0244e5","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                    LOG.error(\u0027Image create failed with timeout. Take a look \u0027"},{"line_number":76,"context_line":"                              \u0027at nova.\u0027)"},{"line_number":77,"context_line":"                except nova_exceptions.Conflict as e:"},{"line_number":78,"context_line":"                    LOG.warning(\u0027%s. Take a look \u0027"},{"line_number":79,"context_line":"                                \u0027at nova. (Request-ID: %s)\u0027 % (e.message,"},{"line_number":80,"context_line":"                                                               e.request_id))"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F9wgQ%3D","line":78,"updated":"2014-04-02 10:23:21.000000000","message":"Please define a more human-readable exception","commit_id":"6c954625e976314770b3a27a57db68a8c58a6c9d"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"60714bfaa729facb288526716d7354f217b78de6","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                    LOG.error(\u0027Image create failed with timeout. Take a look \u0027"},{"line_number":76,"context_line":"                              \u0027at nova.\u0027)"},{"line_number":77,"context_line":"                except nova_exceptions.Conflict as e:"},{"line_number":78,"context_line":"                    LOG.warning(\u0027%s. Take a look \u0027"},{"line_number":79,"context_line":"                                \u0027at nova. (Request-ID: %s)\u0027 % (e.message,"},{"line_number":80,"context_line":"                                                               e.request_id))"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F9wVw%3D","line":78,"in_reply_to":"AAAAWH%2F9wgQ%3D","updated":"2014-04-03 07:40:32.000000000","message":"well, I guess it\u0027ll be better)","commit_id":"6c954625e976314770b3a27a57db68a8c58a6c9d"}],"climate/tests/plugins/instances/test_vm_plugin.py":[{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"835ca6e101196f3c92cc7f09ef5c9a5a1d7c70b6","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        self.client.side_effect \u003d \\"},{"line_number":77,"context_line":"            self.nova.ClimateNovaClient.exceptions.NotFound"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        self.assertRaises(self.exc.TaskFailed,"},{"line_number":80,"context_line":"                          self.plugin.on_end,"},{"line_number":81,"context_line":"                          self.fake_id)"},{"line_number":82,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9%2B3g%3D","line":79,"updated":"2014-03-31 19:13:00.000000000","message":"Please change this test too.","commit_id":"00bfad876bf9e797a3df583b8c88cedc65c64327"},{"author":{"_account_id":9331,"name":"Pablo Andres Fuente","email":"pablo.andres.fuente@gmail.com","username":"pafuent"},"change_message_id":"835ca6e101196f3c92cc7f09ef5c9a5a1d7c70b6","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    def test_on_end_create_image_ko_invalid_vm_state(self):"},{"line_number":84,"context_line":"        self.client.side_effect \u003d \\"},{"line_number":85,"context_line":"            self.nova.ClimateNovaClient.exceptions.Conflict"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        self.assertRaises(self.exc.TaskFailed,"},{"line_number":88,"context_line":"                          self.plugin.on_end,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F9%2B3o%3D","line":85,"updated":"2014-03-31 19:13:00.000000000","message":"This mock configuration is not correct. on_end is raising the TaskFailed exception because this mock is not raising the Conflict exception that you want. You should mock self.client.return_value.servers.create_image","commit_id":"00bfad876bf9e797a3df583b8c88cedc65c64327"}]}