)]}'
{"cyborg/tests/unit/common/test_nova_client.py":[{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"3c880a071fad8a795dc481b91d19a7443ca7255c","unresolved":true,"context_lines":[{"line_number":83,"context_line":"        self.assertRaises(exception.InvalidAPIResponse,"},{"line_number":84,"context_line":"                          nova._send_events, self.events)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def test_send_events_non_422_exception(self):"},{"line_number":87,"context_line":"        # If Nova returns HTTP 207 with event code other than 422,"},{"line_number":88,"context_line":"        # raise an exception."},{"line_number":89,"context_line":"        resp_events \u003d copy.deepcopy(self.events)"}],"source_content_type":"text/x-python","patch_set":1,"id":"adedd3fd_8b2054b0","line":86,"range":{"start_line":86,"start_character":8,"end_line":86,"end_character":42},"updated":"2021-04-08 09:51:15.000000000","message":"test_send_events_with_status_code_400()","commit_id":"32fc296494e509d0e194ee51bae996eec87667c8"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"3c880a071fad8a795dc481b91d19a7443ca7255c","unresolved":true,"context_lines":[{"line_number":84,"context_line":"                          nova._send_events, self.events)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def test_send_events_non_422_exception(self):"},{"line_number":87,"context_line":"        # If Nova returns HTTP 207 with event code other than 422,"},{"line_number":88,"context_line":"        # raise an exception."},{"line_number":89,"context_line":"        resp_events \u003d copy.deepcopy(self.events)"},{"line_number":90,"context_line":"        resp_events[0].update({\u0027status\u0027: \u0027failed\u0027, \u0027code\u0027: 400})"}],"source_content_type":"text/x-python","patch_set":1,"id":"4b8c3e8f_5ce098d7","line":87,"range":{"start_line":87,"start_character":51,"end_line":87,"end_character":65},"updated":"2021-04-08 09:51:15.000000000","message":"please re-words this sentence.","commit_id":"32fc296494e509d0e194ee51bae996eec87667c8"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"3c880a071fad8a795dc481b91d19a7443ca7255c","unresolved":true,"context_lines":[{"line_number":97,"context_line":"        self.assertRaises(exception.InvalidAPIResponse,"},{"line_number":98,"context_line":"                          nova._send_events, self.events)"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def test_send_events_423_exception(self):"},{"line_number":101,"context_line":"        # If Nova returns HTTP 207 with event code other than 422,"},{"line_number":102,"context_line":"        # raise an exception."},{"line_number":103,"context_line":"        resp_events \u003d copy.deepcopy(self.events)"}],"source_content_type":"text/x-python","patch_set":1,"id":"6331c219_1a7381fc","line":100,"range":{"start_line":100,"start_character":8,"end_line":100,"end_character":38},"updated":"2021-04-08 09:51:15.000000000","message":"It\u0027s better to say: test_send_events_with_except_status_code_423()","commit_id":"32fc296494e509d0e194ee51bae996eec87667c8"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"3c880a071fad8a795dc481b91d19a7443ca7255c","unresolved":true,"context_lines":[{"line_number":98,"context_line":"                          nova._send_events, self.events)"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def test_send_events_423_exception(self):"},{"line_number":101,"context_line":"        # If Nova returns HTTP 207 with event code other than 422,"},{"line_number":102,"context_line":"        # raise an exception."},{"line_number":103,"context_line":"        resp_events \u003d copy.deepcopy(self.events)"},{"line_number":104,"context_line":"        for ev in resp_events:"},{"line_number":105,"context_line":"            ev.update({\u0027status\u0027: \u0027failed\u0027, \u0027code\u0027: 423})"}],"source_content_type":"text/x-python","patch_set":1,"id":"4047f01d_734a4425","line":102,"range":{"start_line":101,"start_character":8,"end_line":102,"end_character":29},"updated":"2021-04-08 09:51:15.000000000","message":"In nova code https://docs.openstack.org/api-ref/compute/?expanded\u003drun-events-detail#id382, the event_response.status code {200, 400, 404, 422}, here 423 just a except code.","commit_id":"32fc296494e509d0e194ee51bae996eec87667c8"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"6c784f5d3e1c9af884ceb1c75f3a5da84afc13b0","unresolved":true,"context_lines":[{"line_number":68,"context_line":"               \u0027yet associated with a host.\u0027)"},{"line_number":69,"context_line":"        self.mock_log_info.assert_called_once_with(msg, self.instance_uuid)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def test_send_events_with_some_event_code_422_exception(self):"},{"line_number":72,"context_line":"        # If Nova returns HTTP 207 with event code 422 for some events,"},{"line_number":73,"context_line":"        # but not all, raise an exception. This is not expected to"},{"line_number":74,"context_line":"        # happen with current code."}],"source_content_type":"text/x-python","patch_set":2,"id":"09d452e3_9df0c0d4","line":71,"range":{"start_line":71,"start_character":30,"end_line":71,"end_character":35},"updated":"2021-04-09 02:13:50.000000000","message":"why?, it seems redundant.","commit_id":"ffdf908ce8cbbae0ae7aaaf3fc4b118e6ed759bf"},{"author":{"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},"change_message_id":"d85701974f8f62205749d4e1354ee4e80c39d537","unresolved":true,"context_lines":[{"line_number":68,"context_line":"               \u0027yet associated with a host.\u0027)"},{"line_number":69,"context_line":"        self.mock_log_info.assert_called_once_with(msg, self.instance_uuid)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def test_send_events_with_some_event_code_422_exception(self):"},{"line_number":72,"context_line":"        # If Nova returns HTTP 207 with event code 422 for some events,"},{"line_number":73,"context_line":"        # but not all, raise an exception. This is not expected to"},{"line_number":74,"context_line":"        # happen with current code."}],"source_content_type":"text/x-python","patch_set":2,"id":"ef50ff39_5f79c135","line":71,"range":{"start_line":71,"start_character":30,"end_line":71,"end_character":35},"in_reply_to":"09d452e3_9df0c0d4","updated":"2021-04-15 03:15:05.000000000","message":"i think it is because not all returns 422, just some of them right?","commit_id":"ffdf908ce8cbbae0ae7aaaf3fc4b118e6ed759bf"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"0ebe6df83cfd0dd514c7f24759a37163192bfd61","unresolved":true,"context_lines":[{"line_number":68,"context_line":"               \u0027yet associated with a host.\u0027)"},{"line_number":69,"context_line":"        self.mock_log_info.assert_called_once_with(msg, self.instance_uuid)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def test_send_events_with_some_event_code_422_exception(self):"},{"line_number":72,"context_line":"        # If Nova returns HTTP 207 with event code 422 for some events,"},{"line_number":73,"context_line":"        # but not all, raise an exception. This is not expected to"},{"line_number":74,"context_line":"        # happen with current code."}],"source_content_type":"text/x-python","patch_set":2,"id":"e0b6fac1_ab2123ec","line":71,"range":{"start_line":71,"start_character":30,"end_line":71,"end_character":35},"in_reply_to":"34f0c821_23a3f369","updated":"2021-06-08 00:38:28.000000000","message":"IMO, this case just test the event code 422, not need to say *some*, it\u0027s easy to have a misunderstanding. And in the docs (Line72) you have explained it, so I think you can remove from the test case name to keep it clearly.","commit_id":"ffdf908ce8cbbae0ae7aaaf3fc4b118e6ed759bf"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"cdba64e5da71337ef17bd6fe24699fd0dfc62f10","unresolved":true,"context_lines":[{"line_number":68,"context_line":"               \u0027yet associated with a host.\u0027)"},{"line_number":69,"context_line":"        self.mock_log_info.assert_called_once_with(msg, self.instance_uuid)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def test_send_events_with_some_event_code_422_exception(self):"},{"line_number":72,"context_line":"        # If Nova returns HTTP 207 with event code 422 for some events,"},{"line_number":73,"context_line":"        # but not all, raise an exception. This is not expected to"},{"line_number":74,"context_line":"        # happen with current code."}],"source_content_type":"text/x-python","patch_set":2,"id":"496924a9_bfe8dca4","line":71,"range":{"start_line":71,"start_character":30,"end_line":71,"end_character":35},"in_reply_to":"e0b6fac1_ab2123ec","updated":"2021-06-08 00:44:29.000000000","message":"ok. seems reasonable.","commit_id":"ffdf908ce8cbbae0ae7aaaf3fc4b118e6ed759bf"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"68ec5930a2fcd50a2d27f074b41231c839c1cf1a","unresolved":true,"context_lines":[{"line_number":68,"context_line":"               \u0027yet associated with a host.\u0027)"},{"line_number":69,"context_line":"        self.mock_log_info.assert_called_once_with(msg, self.instance_uuid)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def test_send_events_with_some_event_code_422_exception(self):"},{"line_number":72,"context_line":"        # If Nova returns HTTP 207 with event code 422 for some events,"},{"line_number":73,"context_line":"        # but not all, raise an exception. This is not expected to"},{"line_number":74,"context_line":"        # happen with current code."}],"source_content_type":"text/x-python","patch_set":2,"id":"34f0c821_23a3f369","line":71,"range":{"start_line":71,"start_character":30,"end_line":71,"end_character":35},"in_reply_to":"ef50ff39_5f79c135","updated":"2021-04-15 05:57:16.000000000","message":"yeah, some event codes are 422.","commit_id":"ffdf908ce8cbbae0ae7aaaf3fc4b118e6ed759bf"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"6c784f5d3e1c9af884ceb1c75f3a5da84afc13b0","unresolved":true,"context_lines":[{"line_number":83,"context_line":"        self.assertRaises(exception.InvalidAPIResponse,"},{"line_number":84,"context_line":"                          nova._send_events, self.events)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def test_send_events_with_some_event_code_400_exception(self):"},{"line_number":87,"context_line":"        # If Nova returns HTTP 207 with event code 400 for some events,"},{"line_number":88,"context_line":"        # raise an exception."},{"line_number":89,"context_line":"        resp_events \u003d copy.deepcopy(self.events)"}],"source_content_type":"text/x-python","patch_set":2,"id":"31f75135_0df0bb9e","line":86,"range":{"start_line":86,"start_character":29,"end_line":86,"end_character":34},"updated":"2021-04-09 02:13:50.000000000","message":"ditto","commit_id":"ffdf908ce8cbbae0ae7aaaf3fc4b118e6ed759bf"},{"author":{"_account_id":31412,"name":"Wenping Song","email":"songwenping@inspur.com","username":"songwenping"},"change_message_id":"cdba64e5da71337ef17bd6fe24699fd0dfc62f10","unresolved":false,"context_lines":[{"line_number":83,"context_line":"        self.assertRaises(exception.InvalidAPIResponse,"},{"line_number":84,"context_line":"                          nova._send_events, self.events)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def test_send_events_with_some_event_code_400_exception(self):"},{"line_number":87,"context_line":"        # If Nova returns HTTP 207 with event code 400 for some events,"},{"line_number":88,"context_line":"        # raise an exception."},{"line_number":89,"context_line":"        resp_events \u003d copy.deepcopy(self.events)"}],"source_content_type":"text/x-python","patch_set":2,"id":"699472ce_e109c408","line":86,"range":{"start_line":86,"start_character":29,"end_line":86,"end_character":34},"in_reply_to":"31f75135_0df0bb9e","updated":"2021-06-08 00:44:29.000000000","message":"Done","commit_id":"ffdf908ce8cbbae0ae7aaaf3fc4b118e6ed759bf"}]}