)]}'
{"cinder/tests/unit/volume/drivers/test_rbd.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"a78723008c8db2b672d565cb340b9d4ce5eae2bc","unresolved":false,"context_lines":[{"line_number":1325,"context_line":"                            args \u003d [None, self.volume_a,"},{"line_number":1326,"context_line":"                                    mock_image_service, None]"},{"line_number":1327,"context_line":"                            if volume_busy:"},{"line_number":1328,"context_line":"                                mock_dv.side_effect \u003d exception.VolumeIsBusy(\"doh\")"},{"line_number":1329,"context_line":"                                self.assertRaises(exception.VolumeIsBusy,"},{"line_number":1330,"context_line":"                                                  self.driver.copy_image_to_volume,"},{"line_number":1331,"context_line":"                                                  *args)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f621f24_8c73cde7","line":1328,"updated":"2020-10-31 02:26:02.000000000","message":"pep8: E501 line too long (83 \u003e 79 characters)","commit_id":"a6c5132ba900a4f278c52559ce172afd323f0bda"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"a78723008c8db2b672d565cb340b9d4ce5eae2bc","unresolved":false,"context_lines":[{"line_number":1327,"context_line":"                            if volume_busy:"},{"line_number":1328,"context_line":"                                mock_dv.side_effect \u003d exception.VolumeIsBusy(\"doh\")"},{"line_number":1329,"context_line":"                                self.assertRaises(exception.VolumeIsBusy,"},{"line_number":1330,"context_line":"                                                  self.driver.copy_image_to_volume,"},{"line_number":1331,"context_line":"                                                  *args)"},{"line_number":1332,"context_line":"                                # 3 calls due to cfg.rados_connection_retries \u003d 3"},{"line_number":1333,"context_line":"                                self.assertEqual(mock_dv.call_count, 3)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f621f24_ec80818e","line":1330,"updated":"2020-10-31 02:26:02.000000000","message":"pep8: E501 line too long (83 \u003e 79 characters)","commit_id":"a6c5132ba900a4f278c52559ce172afd323f0bda"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"a78723008c8db2b672d565cb340b9d4ce5eae2bc","unresolved":false,"context_lines":[{"line_number":1329,"context_line":"                                self.assertRaises(exception.VolumeIsBusy,"},{"line_number":1330,"context_line":"                                                  self.driver.copy_image_to_volume,"},{"line_number":1331,"context_line":"                                                  *args)"},{"line_number":1332,"context_line":"                                # 3 calls due to cfg.rados_connection_retries \u003d 3"},{"line_number":1333,"context_line":"                                self.assertEqual(mock_dv.call_count, 3)"},{"line_number":1334,"context_line":"                            else:"},{"line_number":1335,"context_line":"                                self.driver.copy_image_to_volume(*args)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f621f24_cc7dc5b2","line":1332,"updated":"2020-10-31 02:26:02.000000000","message":"pep8: E501 line too long (81 \u003e 79 characters)","commit_id":"a6c5132ba900a4f278c52559ce172afd323f0bda"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"06c7c36a8eedfcfeb19e0c46e765b248a98f9777","unresolved":false,"context_lines":[{"line_number":1332,"context_line":"                                    self.driver.copy_image_to_volume,"},{"line_number":1333,"context_line":"                                    *args)"},{"line_number":1334,"context_line":"                                # 3 calls as cfg.rados_connection_retries \u003d 3"},{"line_number":1335,"context_line":"                                self.assertEqual(mock_dv.call_count, 3)"},{"line_number":1336,"context_line":"                            else:"},{"line_number":1337,"context_line":"                                self.driver.copy_image_to_volume(*args)"},{"line_number":1338,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_bc5da148","line":1335,"range":{"start_line":1335,"start_character":32,"end_line":1335,"end_character":71},"updated":"2020-11-10 16:03:21.000000000","message":"Thanks for adding a test.  Two things about this:\n- the ordering is usually expected, actual\n- instead of using 3 and having the comment, you could simply use self.cfg.rados_connection_retries here","commit_id":"41f4199f3291eead4ebe1234e6d6fb07b2e3788b"}]}