)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"b797efbcda934cb7a96173e7840a77577a11308f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c1d2fd34_56443871","updated":"2022-08-25 10:09:13.000000000","message":"Would you be so kind to provide also unit tests for this?","commit_id":"287f47ce211a4529dfc000fe6e69c84624834c62"},{"author":{"_account_id":30623,"name":"suzhengwei","email":"suzhengwei@inspur.com","username":"SuZhengwei"},"change_message_id":"fe83b269c732506ba9a43ef76f0a77cfad68d2d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"505e649d_424bf35f","in_reply_to":"c1d2fd34_56443871","updated":"2022-09-09 08:09:51.000000000","message":"Done","commit_id":"287f47ce211a4529dfc000fe6e69c84624834c62"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f008159342dbe42806f1e1616d6b92d5ada17b1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ae489d21_d28f9586","updated":"2022-09-09 09:09:41.000000000","message":"Very nice! A few little issues to fix before merging.","commit_id":"9171b2da17073556e83c5e8de9cc771eb0b5838b"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"35881d6c819e0d64f7c2674324cf2f32c9ceaa50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3f1d3125_dffd8077","updated":"2022-09-09 11:20:04.000000000","message":"Fixed a typo and shortened tests\u0027 names.","commit_id":"9721deff2dd26f9b53acb21b7098524e37a654cb"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"0047b195a5363be018126ae8104d076934637cfb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"883170d8_fe9eb306","updated":"2022-09-09 11:21:43.000000000","message":"I will add a reno too.","commit_id":"9721deff2dd26f9b53acb21b7098524e37a654cb"}],"masakari/engine/drivers/taskflow/instance_failure.py":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"db565d8d802b227fbf25f1420431a9a7f1cbde19","unresolved":true,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":80,"context_line":"            except exception.Conflict as e:"},{"line_number":81,"context_line":"                LOG.warning(\"Confilct to stop instance, %s.\", str(e))"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        def _wait_for_power_off():"},{"line_number":84,"context_line":"            new_instance \u003d self.novaclient.get_server(self.context,"}],"source_content_type":"text/x-python","patch_set":3,"id":"6c65c735_009035f5","line":81,"range":{"start_line":81,"start_character":29,"end_line":81,"end_character":38},"updated":"2022-08-04 08:22:06.000000000","message":"typo here","commit_id":"fd77759813fce10b2b6e6f6c866fe156557465e4"},{"author":{"_account_id":30623,"name":"suzhengwei","email":"suzhengwei@inspur.com","username":"SuZhengwei"},"change_message_id":"7a55d83f3e12fc9e927c08c69678290861376b89","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":80,"context_line":"            except exception.Conflict as e:"},{"line_number":81,"context_line":"                LOG.warning(\"Confilct to stop instance, %s.\", str(e))"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        def _wait_for_power_off():"},{"line_number":84,"context_line":"            new_instance \u003d self.novaclient.get_server(self.context,"}],"source_content_type":"text/x-python","patch_set":3,"id":"442a0281_1f7e53d7","line":81,"range":{"start_line":81,"start_character":29,"end_line":81,"end_character":38},"in_reply_to":"6c65c735_009035f5","updated":"2022-08-04 08:45:55.000000000","message":"Done","commit_id":"fd77759813fce10b2b6e6f6c866fe156557465e4"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"a18b86448eefecfb1dfb6ee8f885b2f97d230788","unresolved":true,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":80,"context_line":"            except exception.Conflict as e:"},{"line_number":81,"context_line":"                LOG.warning(\"Conflict to stop instance, %s.\", str(e))"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        def _wait_for_power_off():"},{"line_number":84,"context_line":"            new_instance \u003d self.novaclient.get_server(self.context,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ca4e63d_fe7eb913","line":81,"updated":"2022-08-04 09:02:16.000000000","message":"it would also make sense (if possible) to check if this conflict is due to the instance being already shut down or of a different nature","commit_id":"4c3aa747bc7a587754852031153659f99bc121d6"},{"author":{"_account_id":30623,"name":"suzhengwei","email":"suzhengwei@inspur.com","username":"SuZhengwei"},"change_message_id":"fe83b269c732506ba9a43ef76f0a77cfad68d2d2","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":80,"context_line":"            except exception.Conflict as e:"},{"line_number":81,"context_line":"                LOG.warning(\"Conflict to stop instance, %s.\", str(e))"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        def _wait_for_power_off():"},{"line_number":84,"context_line":"            new_instance \u003d self.novaclient.get_server(self.context,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a977288_1f704f70","line":81,"in_reply_to":"1ca4e63d_fe7eb913","updated":"2022-09-09 08:09:51.000000000","message":"Ack","commit_id":"4c3aa747bc7a587754852031153659f99bc121d6"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"b797efbcda934cb7a96173e7840a77577a11308f","unresolved":true,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"            try:"},{"line_number":80,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":81,"context_line":"            except exception.Conflict as e:"},{"line_number":82,"context_line":"                instance \u003d self.novaclient.get_server(self.context,"},{"line_number":83,"context_line":"                                                      instance_uuid)"},{"line_number":84,"context_line":"                vm_state \u003d getattr(instance, \u0027OS-EXT-STS:vm_state\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"00f2b103_60c544ea","line":81,"range":{"start_line":81,"start_character":41,"end_line":81,"end_character":42},"updated":"2022-08-25 10:09:13.000000000","message":"does this error not contain reason already?","commit_id":"287f47ce211a4529dfc000fe6e69c84624834c62"},{"author":{"_account_id":30623,"name":"suzhengwei","email":"suzhengwei@inspur.com","username":"SuZhengwei"},"change_message_id":"fe83b269c732506ba9a43ef76f0a77cfad68d2d2","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"            try:"},{"line_number":80,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":81,"context_line":"            except exception.Conflict as e:"},{"line_number":82,"context_line":"                instance \u003d self.novaclient.get_server(self.context,"},{"line_number":83,"context_line":"                                                      instance_uuid)"},{"line_number":84,"context_line":"                vm_state \u003d getattr(instance, \u0027OS-EXT-STS:vm_state\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"11a451ed_4d15de90","line":81,"range":{"start_line":81,"start_character":41,"end_line":81,"end_character":42},"in_reply_to":"00f2b103_60c544ea","updated":"2022-09-09 08:09:51.000000000","message":"It does.","commit_id":"287f47ce211a4529dfc000fe6e69c84624834c62"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f008159342dbe42806f1e1616d6b92d5ada17b1a","unresolved":true,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":80,"context_line":"            except exception.Conflict as e:"},{"line_number":81,"context_line":"                self.update_details(str(e))"},{"line_number":82,"context_line":"                instance \u003d self.novaclient.get_server(self.context,"},{"line_number":83,"context_line":"                                                      instance_uuid)"},{"line_number":84,"context_line":"                vm_state \u003d getattr(instance, \u0027OS-EXT-STS:vm_state\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"bc790a43_a286ee8f","line":81,"range":{"start_line":81,"start_character":36,"end_line":81,"end_character":42},"updated":"2022-09-09 09:09:41.000000000","message":"If it results only in \"Conflict\", we can use \"Conflict\" here too, for clarity.","commit_id":"9171b2da17073556e83c5e8de9cc771eb0b5838b"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"d13c92cf38404814764b666655d8ce8fee095d66","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                self.novaclient.stop_server(self.context, instance.id)"},{"line_number":80,"context_line":"            except exception.Conflict as e:"},{"line_number":81,"context_line":"                self.update_details(str(e))"},{"line_number":82,"context_line":"                instance \u003d self.novaclient.get_server(self.context,"},{"line_number":83,"context_line":"                                                      instance_uuid)"},{"line_number":84,"context_line":"                vm_state \u003d getattr(instance, \u0027OS-EXT-STS:vm_state\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"c42c077e_6eeb3a81","line":81,"range":{"start_line":81,"start_character":36,"end_line":81,"end_character":42},"in_reply_to":"bc790a43_a286ee8f","updated":"2022-09-09 11:20:18.000000000","message":"Done","commit_id":"9171b2da17073556e83c5e8de9cc771eb0b5838b"}],"masakari/tests/unit/engine/drivers/taskflow/test_instance_failure_flow.py":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f008159342dbe42806f1e1616d6b92d5ada17b1a","unresolved":true,"context_lines":[{"line_number":202,"context_line":"    @mock.patch(\u0027masakari.compute.nova.novaclient\u0027)"},{"line_number":203,"context_line":"    @mock.patch(\u0027masakari.engine.drivers.taskflow.base.MasakariTask.\u0027"},{"line_number":204,"context_line":"                \u0027update_details\u0027)"},{"line_number":205,"context_line":"    def test_instance_failure_flow_stop_conflict_for_instance_already_soppped("},{"line_number":206,"context_line":"            self, _mock_notify, _mock_novaclient):"},{"line_number":207,"context_line":"        _mock_novaclient.return_value \u003d self.fake_client"},{"line_number":208,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"e27a68b3_ec8d4543","line":205,"range":{"start_line":205,"start_character":70,"end_line":205,"end_character":77},"updated":"2022-09-09 09:09:41.000000000","message":"typo","commit_id":"9171b2da17073556e83c5e8de9cc771eb0b5838b"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"d13c92cf38404814764b666655d8ce8fee095d66","unresolved":false,"context_lines":[{"line_number":202,"context_line":"    @mock.patch(\u0027masakari.compute.nova.novaclient\u0027)"},{"line_number":203,"context_line":"    @mock.patch(\u0027masakari.engine.drivers.taskflow.base.MasakariTask.\u0027"},{"line_number":204,"context_line":"                \u0027update_details\u0027)"},{"line_number":205,"context_line":"    def test_instance_failure_flow_stop_conflict_for_instance_already_soppped("},{"line_number":206,"context_line":"            self, _mock_notify, _mock_novaclient):"},{"line_number":207,"context_line":"        _mock_novaclient.return_value \u003d self.fake_client"},{"line_number":208,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"2f789694_b7e9c146","line":205,"range":{"start_line":205,"start_character":70,"end_line":205,"end_character":77},"in_reply_to":"e27a68b3_ec8d4543","updated":"2022-09-09 11:20:18.000000000","message":"Done","commit_id":"9171b2da17073556e83c5e8de9cc771eb0b5838b"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f008159342dbe42806f1e1616d6b92d5ada17b1a","unresolved":true,"context_lines":[{"line_number":213,"context_line":"                                                 vm_state\u003d\"active\")"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"        def fake_stop_server_conflict(context, uuid):"},{"line_number":216,"context_line":"            # assume that while stopping instance goes into error state"},{"line_number":217,"context_line":"            setattr(server, \u0027OS-EXT-STS:vm_state\u0027, \"stopped\")"},{"line_number":218,"context_line":"            raise exception.Conflict(reason\u003d\"Cannot \u0027stop\u0027 instance 1 \""},{"line_number":219,"context_line":"                                            \"while it is in vm_state stopped\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ceef725_eb30dd51","line":216,"range":{"start_line":216,"start_character":60,"end_line":216,"end_character":66},"updated":"2022-09-09 09:09:41.000000000","message":"stopped","commit_id":"9171b2da17073556e83c5e8de9cc771eb0b5838b"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"d13c92cf38404814764b666655d8ce8fee095d66","unresolved":false,"context_lines":[{"line_number":213,"context_line":"                                                 vm_state\u003d\"active\")"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"        def fake_stop_server_conflict(context, uuid):"},{"line_number":216,"context_line":"            # assume that while stopping instance goes into error state"},{"line_number":217,"context_line":"            setattr(server, \u0027OS-EXT-STS:vm_state\u0027, \"stopped\")"},{"line_number":218,"context_line":"            raise exception.Conflict(reason\u003d\"Cannot \u0027stop\u0027 instance 1 \""},{"line_number":219,"context_line":"                                            \"while it is in vm_state stopped\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"92ac610c_affadfc9","line":216,"range":{"start_line":216,"start_character":60,"end_line":216,"end_character":66},"in_reply_to":"1ceef725_eb30dd51","updated":"2022-09-09 11:20:18.000000000","message":"Done","commit_id":"9171b2da17073556e83c5e8de9cc771eb0b5838b"}]}