)]}'
{"glance/async_/flows/plugins/image_conversion.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"58b727f524c41651f701ef9b8cccc6cdb8e02b60","unresolved":true,"context_lines":[{"line_number":140,"context_line":"                                   container_format\u003d\u0027bare\u0027)"},{"line_number":141,"context_line":"        new_size \u003d os.stat(dest_path).st_size"},{"line_number":142,"context_line":"        action.set_image_attribute(size\u003dnew_size)"},{"line_number":143,"context_line":"        LOG.info(\u0027Updated image %s size\u003d%i disk_format\u003d%s\u0027 % ("},{"line_number":144,"context_line":"            self.image_id, new_size, target_format))"},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"        os.remove(src_path)"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"5a319bb9_7266f8cf","line":144,"range":{"start_line":143,"start_character":17,"end_line":144,"end_character":51},"updated":"2021-06-04 15:09:59.000000000","message":"Should be enclosed with _LI for translations","commit_id":"2204b938009e27869892cba5aff76489a237feba"}],"glance/tests/unit/async_/flows/plugins/test_image_conversion.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"6f6e608492ac81f6b3e09cf63051e228dd80095c","unresolved":true,"context_lines":[{"line_number":125,"context_line":"        self.assertEqual(\u0027bare\u0027, image.container_format)"},{"line_number":126,"context_line":"        self.assertEqual(\u0027qcow2\u0027, image.disk_format)"},{"line_number":127,"context_line":"        self.assertEqual(123, image.virtual_size)"},{"line_number":128,"context_line":"        self.assertEqual(123, image.size)"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def _setup_image_convert_info_fail(self):"},{"line_number":131,"context_line":"        image_convert \u003d image_conversion._ConvertImage(self.context,"}],"source_content_type":"text/x-python","patch_set":6,"id":"6ff21f1c_b52a7157","line":128,"range":{"start_line":128,"start_character":0,"end_line":128,"end_character":41},"updated":"2021-06-04 01:20:04.000000000","message":"Nitpicking, but if a new version of the patch is required, maybe we could use different values for virtual_size and size, as you did in the other test file?","commit_id":"f187f6756e923fd122b2328aa8c476b52df15719"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"5f5cab148b0f5fff9518f67e4c5db048dc02f048","unresolved":true,"context_lines":[{"line_number":125,"context_line":"        self.assertEqual(\u0027bare\u0027, image.container_format)"},{"line_number":126,"context_line":"        self.assertEqual(\u0027qcow2\u0027, image.disk_format)"},{"line_number":127,"context_line":"        self.assertEqual(123, image.virtual_size)"},{"line_number":128,"context_line":"        self.assertEqual(123, image.size)"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def _setup_image_convert_info_fail(self):"},{"line_number":131,"context_line":"        image_convert \u003d image_conversion._ConvertImage(self.context,"}],"source_content_type":"text/x-python","patch_set":6,"id":"a21fa4f3_f0f22aca","line":128,"range":{"start_line":128,"start_character":0,"end_line":128,"end_character":41},"in_reply_to":"6ff21f1c_b52a7157","updated":"2021-06-04 13:50:10.000000000","message":"Yeah, I dunno why I did that.","commit_id":"f187f6756e923fd122b2328aa8c476b52df15719"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"58b727f524c41651f701ef9b8cccc6cdb8e02b60","unresolved":true,"context_lines":[{"line_number":94,"context_line":"    @mock.patch.object(os, \u0027remove\u0027)"},{"line_number":95,"context_line":"    def test_image_convert_success(self, mock_os_remove, mock_os_stat):"},{"line_number":96,"context_line":"        mock_os_remove.return_value \u003d None"},{"line_number":97,"context_line":"        mock_os_stat.return_value \u003d mock.MagicMock()"},{"line_number":98,"context_line":"        mock_os_stat.return_value.st_size \u003d 123"},{"line_number":99,"context_line":"        image_convert \u003d image_conversion._ConvertImage(self.context,"},{"line_number":100,"context_line":"                                                       self.task.task_id,"}],"source_content_type":"text/x-python","patch_set":7,"id":"88727db7_b32b03ad","line":97,"range":{"start_line":97,"start_character":8,"end_line":97,"end_character":52},"updated":"2021-06-04 15:09:59.000000000","message":"is this really required ?? as below it is overriding?","commit_id":"2204b938009e27869892cba5aff76489a237feba"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"b6278848b4216b1c01e67fbeae5423b5a0d8d22d","unresolved":true,"context_lines":[{"line_number":94,"context_line":"    @mock.patch.object(os, \u0027remove\u0027)"},{"line_number":95,"context_line":"    def test_image_convert_success(self, mock_os_remove, mock_os_stat):"},{"line_number":96,"context_line":"        mock_os_remove.return_value \u003d None"},{"line_number":97,"context_line":"        mock_os_stat.return_value \u003d mock.MagicMock()"},{"line_number":98,"context_line":"        mock_os_stat.return_value.st_size \u003d 123"},{"line_number":99,"context_line":"        image_convert \u003d image_conversion._ConvertImage(self.context,"},{"line_number":100,"context_line":"                                                       self.task.task_id,"}],"source_content_type":"text/x-python","patch_set":7,"id":"5150baac_0cc8f0bc","line":97,"range":{"start_line":97,"start_character":8,"end_line":97,"end_character":52},"in_reply_to":"88727db7_b32b03ad","updated":"2021-06-04 16:24:13.000000000","message":"The line below is not overriding it, but you\u0027re right it\u0027s not required :) Probably left over from some debugging.","commit_id":"2204b938009e27869892cba5aff76489a237feba"}]}