)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"a30dba3ea6d05bc4ab9f049d080b20314f0e190e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"914d784c_8c070816","updated":"2022-08-22 16:42:56.000000000","message":"Good improvement, waiting for rajat\u0027s comment before approving it!\n\nThank you Eric!","commit_id":"e340a8cf838e890191700972d7b79117689816d9"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"e5c1ef2391668a13d5b8687a9674c9c22a666518","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9b38d5bd_0c362cd3","updated":"2022-08-23 05:31:48.000000000","message":"Ninja approving, since its a test related change.","commit_id":"e340a8cf838e890191700972d7b79117689816d9"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"67240ba839e510e525415dfaf5860366345841ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bb0a5a59_859e344a","updated":"2022-08-22 17:02:34.000000000","message":"One comment but LGTM","commit_id":"e340a8cf838e890191700972d7b79117689816d9"}],"glance_store/tests/unit/common/test_cinder_utils.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"67240ba839e510e525415dfaf5860366345841ec","unresolved":true,"context_lines":[{"line_number":74,"context_line":"            self.volume_api.attachment_create,"},{"line_number":75,"context_line":"            self.fake_client, self.fake_vol_id)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @mock.patch(\u0027time.sleep\u0027, new\u003dmock.Mock())"},{"line_number":78,"context_line":"    def test_attachment_create_retries(self):"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        fake_attach_id \u003d \u0027fake-attach-id\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"18e4969c_2ef0568f","line":77,"range":{"start_line":77,"start_character":34,"end_line":77,"end_character":45},"updated":"2022-08-22 17:02:34.000000000","message":"isn\u0027t the default already a MagicMock? any reason for defining new as Mock object here?","commit_id":"e340a8cf838e890191700972d7b79117689816d9"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"801a045bfd4743bfd906b51d606f1f1d92cc7038","unresolved":true,"context_lines":[{"line_number":74,"context_line":"            self.volume_api.attachment_create,"},{"line_number":75,"context_line":"            self.fake_client, self.fake_vol_id)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @mock.patch(\u0027time.sleep\u0027, new\u003dmock.Mock())"},{"line_number":78,"context_line":"    def test_attachment_create_retries(self):"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        fake_attach_id \u003d \u0027fake-attach-id\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"dec7b56e_225b7b40","line":77,"range":{"start_line":77,"start_character":34,"end_line":77,"end_character":45},"in_reply_to":"18e4969c_2ef0568f","updated":"2022-08-23 09:46:10.000000000","message":"It\u0027s more that you need the new\u003d parameter to get the decorator to not pass an unneeded parameter to the test method.","commit_id":"e340a8cf838e890191700972d7b79117689816d9"}]}