)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"4005b70497a11afde2b049f04e1030ea6829c7b3","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add NotAcceptable exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds missed NotAcceptable exception and handle it"},{"line_number":10,"context_line":"correctly by _error_checker() \u0027"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib9195ba86d7950f6a82812738b152b5c425fe170"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"df3967d1_b7c295e8","line":10,"updated":"2017-07-31 05:34:25.000000000","message":"What is this single quote?","commit_id":"3a03072d25ef2264c39567e845b94c8f7d57746f"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"4005b70497a11afde2b049f04e1030ea6829c7b3","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds missed NotAcceptable exception and handle it"},{"line_number":10,"context_line":"correctly by _error_checker() \u0027"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib9195ba86d7950f6a82812738b152b5c425fe170"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"df3967d1_97b79187","line":11,"updated":"2017-07-31 05:34:25.000000000","message":"I think you can add more details here. For example, why you picked 406 only? (Why you didn\u0027t pick 405?) Did you really face a actual problem with this or not?\nIt\u0027s very helpful to understand this patch.","commit_id":"3a03072d25ef2264c39567e845b94c8f7d57746f"}],"tempest/lib/common/rest_client.py":[{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"8fd67e3fe3b7eed4d5202e463703686ca7feeac0","unresolved":false,"context_lines":[{"line_number":767,"context_line":"        if resp.status \u003d\u003d 406:"},{"line_number":768,"context_line":"           if parse_resp:"},{"line_number":769,"context_line":"                resp_body \u003d self._parse_resp(resp_body)"},{"line_number":770,"context_line":"           raise exceptions.NotAcceptable(resp_body, resp\u003dresp)"},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"        if resp.status \u003d\u003d 400:"},{"line_number":773,"context_line":"            if parse_resp:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ff346bd7_aff55585","line":770,"updated":"2017-07-25 19:19:28.000000000","message":"indentation","commit_id":"bbebbd95fcc7d1d163be9516bc7f0e572e78c60c"}],"tempest/lib/exceptions.py":[{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"a8aa2e0c12603a2bd770780d72d51ec63a808dc5","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    status_code \u003d 404"},{"line_number":91,"context_line":"    message \u003d \"Object not found\""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class NotAcceptable(ClientRestClientException):"},{"line_number":94,"context_line":"    status_code \u003d 406"},{"line_number":95,"context_line":"    message \u003d \"Request not acceptable.\""},{"line_number":96,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ff346bd7_97dcc6e4","line":93,"updated":"2017-07-26 16:28:41.000000000","message":"PEP8: should have two blank lines to separate classes.","commit_id":"152d4ef10ab50f7483caa16b044ddea14e5cd8ca"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"4005b70497a11afde2b049f04e1030ea6829c7b3","unresolved":false,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"class NotAcceptable(ClientRestClientException):"},{"line_number":95,"context_line":"    status_code \u003d 406"},{"line_number":96,"context_line":"    message \u003d \"Request not acceptable.\""},{"line_number":97,"context_line":""},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"class Conflict(ClientRestClientException):"}],"source_content_type":"text/x-python","patch_set":3,"id":"df3967d1_77bc2d69","line":96,"updated":"2017-07-31 05:34:25.000000000","message":"I think just \"Not Acceptable\" is enough here. I feel \"Request\" is verbose.","commit_id":"3a03072d25ef2264c39567e845b94c8f7d57746f"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"043d331b6317c84fe785a46a6c41523d3de3c3a2","unresolved":false,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"class NotAcceptable(ClientRestClientException):"},{"line_number":95,"context_line":"    status_code \u003d 406"},{"line_number":96,"context_line":"    message \u003d \"Request not acceptable.\""},{"line_number":97,"context_line":""},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"class Conflict(ClientRestClientException):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f436f4f_ea36f8b4","line":96,"in_reply_to":"df3967d1_77bc2d69","updated":"2017-08-08 17:45:13.000000000","message":"+1 for \"Not Acceptable\" based on https://tools.ietf.org/html/rfc7231#section-6.5.6","commit_id":"3a03072d25ef2264c39567e845b94c8f7d57746f"}]}