)]}'
{"cinder/tests/unit/volume/drivers/test_rbd.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d3f72ed58e85fef48bdceefe7b7106385f2aa905","unresolved":true,"context_lines":[{"line_number":896,"context_line":""},{"line_number":897,"context_line":"            self.driver.create_volume_from_snapshot(self.volume_a, snapshot)"},{"line_number":898,"context_line":""},{"line_number":899,"context_line":"            mock_log.info.assert_called_once_with(\u0027Using v2 Clone API\u0027)"},{"line_number":900,"context_line":"            self.assertTrue(self.driver._clone_v2_api_checked)"},{"line_number":901,"context_line":""},{"line_number":902,"context_line":"    @common_mocks"}],"source_content_type":"text/x-python","patch_set":3,"id":"61772bc0_2b0f5a10","line":899,"range":{"start_line":899,"start_character":26,"end_line":899,"end_character":49},"updated":"2021-02-11 15:15:38.000000000","message":"would it be better to loosen this test to ``mock_log.info.assert_called_with(\u0027Using v2 Clone API\u0027)`` to confirm that our msg was logged was called and will not fail if other log.info were called simultaneously\nMy concern is this shouldn\u0027t fail like the gate failure we had recently when some other log.info failed these tests","commit_id":"8755f1811b52f8e64652d3a74ac694dba3d4f873"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"1d8807b2b3ee3bfed356b5d6c3287358a3709e8c","unresolved":true,"context_lines":[{"line_number":896,"context_line":""},{"line_number":897,"context_line":"            self.driver.create_volume_from_snapshot(self.volume_a, snapshot)"},{"line_number":898,"context_line":""},{"line_number":899,"context_line":"            mock_log.info.assert_called_once_with(\u0027Using v2 Clone API\u0027)"},{"line_number":900,"context_line":"            self.assertTrue(self.driver._clone_v2_api_checked)"},{"line_number":901,"context_line":""},{"line_number":902,"context_line":"    @common_mocks"}],"source_content_type":"text/x-python","patch_set":3,"id":"49165326_182704a9","line":899,"range":{"start_line":899,"start_character":26,"end_line":899,"end_character":49},"in_reply_to":"61772bc0_2b0f5a10","updated":"2021-02-11 15:20:49.000000000","message":"Yes I think this is correct, should do something different here","commit_id":"8755f1811b52f8e64652d3a74ac694dba3d4f873"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d3f72ed58e85fef48bdceefe7b7106385f2aa905","unresolved":true,"context_lines":[{"line_number":917,"context_line":""},{"line_number":918,"context_line":"            self.driver.create_volume_from_snapshot(self.volume_a, snapshot)"},{"line_number":919,"context_line":""},{"line_number":920,"context_line":"            mock_log.warning.assert_called_once()"},{"line_number":921,"context_line":"            self.assertIn(\u0027Not using v2 clone API\u0027,"},{"line_number":922,"context_line":"                          mock_log.warning.call_args[0][0])"},{"line_number":923,"context_line":"            self.assertTrue(self.driver._clone_v2_api_checked)"}],"source_content_type":"text/x-python","patch_set":3,"id":"b8f8c96a_9a195963","line":920,"range":{"start_line":920,"start_character":29,"end_line":920,"end_character":47},"updated":"2021-02-11 15:15:38.000000000","message":"same","commit_id":"8755f1811b52f8e64652d3a74ac694dba3d4f873"}]}