)]}'
{"magnumclient/common/httpclient.py":[{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"15ddfadc4e446b1f0902e19d7dd4f0b323ceb718","unresolved":false,"context_lines":[{"line_number":172,"context_line":"            self.log_http_response(resp)"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"        if 400 \u003c\u003d resp.status \u003c 600:"},{"line_number":175,"context_line":"            LOG.debug(\"Request returned failure status.\")"},{"line_number":176,"context_line":"            error_json \u003d self._extract_error_json(body_str)"},{"line_number":177,"context_line":"            raise exceptions.from_response("},{"line_number":178,"context_line":"                resp, error_json.get(\u0027faultstring\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_6239b0d7","line":175,"updated":"2015-01-14 22:39:45.000000000","message":"ironic is also using same logic, how does this change help resolve the related bugs?\n\nhttps://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L179","commit_id":"dbf364a75c575e9ec3dcad315434355841715067"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"ffabca194c6c8bc08b58da3edad031ab77ef1934","unresolved":false,"context_lines":[{"line_number":172,"context_line":"            self.log_http_response(resp)"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"        if 400 \u003c\u003d resp.status \u003c 600:"},{"line_number":175,"context_line":"            LOG.debug(\"Request returned failure status.\")"},{"line_number":176,"context_line":"            error_json \u003d self._extract_error_json(body_str)"},{"line_number":177,"context_line":"            raise exceptions.from_response("},{"line_number":178,"context_line":"                resp, error_json.get(\u0027faultstring\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_9db99d18","line":175,"in_reply_to":"3a961159_6239b0d7","updated":"2015-01-14 23:19:32.000000000","message":"Hi Jay,\n\nI will submit the same patch to Ironic so that we can confirm whether this fix is fine.\n\nThe problem is that there is no log handler registered if \u0027--debug\u0027 option is not provided (which is the case in most of the openstack CLI client). Therefore, we should avoid to call LOG.warn(\"...\") because no handler will log the message.","commit_id":"dbf364a75c575e9ec3dcad315434355841715067"},{"author":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"change_message_id":"57db772e818f20044d07f1cb3e84178684a9999e","unresolved":false,"context_lines":[{"line_number":172,"context_line":"            self.log_http_response(resp)"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"        if 400 \u003c\u003d resp.status \u003c 600:"},{"line_number":175,"context_line":"            LOG.debug(\"Request returned failure status.\")"},{"line_number":176,"context_line":"            error_json \u003d self._extract_error_json(body_str)"},{"line_number":177,"context_line":"            raise exceptions.from_response("},{"line_number":178,"context_line":"                resp, error_json.get(\u0027faultstring\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_0e0572c1","line":175,"in_reply_to":"3a961159_9db99d18","updated":"2015-01-15 03:03:31.000000000","message":"This is not actually fixing those other bugs, it\u0027s just working around the bug by logging less.","commit_id":"dbf364a75c575e9ec3dcad315434355841715067"}]}