)]}'
{"cinder/tests/unit/volume/drivers/huawei/test_huawei_drivers.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"3c54f692eeef35dadb9b39c8cb5127fcfc9dcb37","unresolved":false,"context_lines":[{"line_number":4550,"context_line":"                          \u0027ini-2\u0027, \u00271\u0027)"},{"line_number":4551,"context_line":""},{"line_number":4552,"context_line":"    def test_is_initiator_associated_to_host_true(self):"},{"line_number":4553,"context_line":"        ret \u003d self.driver.client.is_initiator_associated_to_host(\u0027ini-1\u0027, \u00271\u0027)"},{"line_number":4554,"context_line":"        self.assertFalse(ret)"},{"line_number":4555,"context_line":"        ret \u003d self.driver.client.is_initiator_associated_to_host(\u0027ini-2\u0027, \u00272\u0027)"},{"line_number":4556,"context_line":"        self.assertTrue(ret)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba5da102_7c649521","line":4553,"updated":"2016-10-31 12:13:12.000000000","message":"Please, use ddt [1] to avoid code duplication here\n\n\n[1] http://ddt.readthedocs.io/en/latest/example.html","commit_id":"7e41cc7556c791f22d7227f06e2ca75baedd0089"}],"cinder/volume/drivers/huawei/rest_client.py":[{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"4b6cb1258b8de266ddf5259183193976d1530e6e","unresolved":false,"context_lines":[{"line_number":762,"context_line":"                    msg \u003d (_(\"Initiator %(ini)s has been added to host \""},{"line_number":763,"context_line":"                             \"%(host)s.\") % {\"ini\": ininame,"},{"line_number":764,"context_line":"                                             \"host\": item[\u0027PARENTNAME\u0027]})"},{"line_number":765,"context_line":"                    LOG.error(msg)"},{"line_number":766,"context_line":"                    raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":767,"context_line":"        return True"},{"line_number":768,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_57d8d27f","line":765,"updated":"2016-07-14 02:25:09.000000000","message":"This message being put in error level is confusing. Something has been added to a host successfully, mark it as an error? Can we change this log level to info or change the message to be reflecting an actually error, please?","commit_id":"c1a05a8b3127f2f0f63e7fa19f0125f002b689d6"},{"author":{"_account_id":18026,"name":"huanan","email":"huanan@huawei.com","username":"huanan"},"change_message_id":"e3a90a82a851fca70d0a1539bb8e4f9a8c6e6756","unresolved":false,"context_lines":[{"line_number":762,"context_line":"                    msg \u003d (_(\"Initiator %(ini)s has been added to host \""},{"line_number":763,"context_line":"                             \"%(host)s.\") % {\"ini\": ininame,"},{"line_number":764,"context_line":"                                             \"host\": item[\u0027PARENTNAME\u0027]})"},{"line_number":765,"context_line":"                    LOG.error(msg)"},{"line_number":766,"context_line":"                    raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":767,"context_line":"        return True"},{"line_number":768,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad45d7e_6d6271da","line":765,"in_reply_to":"1aa78d24_57d8d27f","updated":"2016-08-08 09:25:55.000000000","message":"Msg description is not clear, here should be initiator has been added to another host.","commit_id":"c1a05a8b3127f2f0f63e7fa19f0125f002b689d6"}]}