)]}'
{"cinder/tests/unit/volume/drivers/netapp/dataontap/test_nfs_base.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"74cecc47313c7916c1fdc2e19a25b16b92ff3031","unresolved":true,"context_lines":[{"line_number":742,"context_line":"            self.assertEqual(model_update[\u0027replication_status\u0027],"},{"line_number":743,"context_line":"                             actual_model_update[\u0027replication_status\u0027])"},{"line_number":744,"context_line":"        else:"},{"line_number":745,"context_line":"            self.assertIn(\u0027replication_status\u0027, actual_model_update)"},{"line_number":746,"context_line":"        self.driver._check_volume_type.assert_called_once_with("},{"line_number":747,"context_line":"            volume, self.fake_nfs_export_1, test_file, {})"},{"line_number":748,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"097cabbd_134f2ded","line":745,"range":{"start_line":745,"start_character":17,"end_line":745,"end_character":25},"updated":"2021-01-12 15:22:19.000000000","message":"Should be assertNotIn","commit_id":"1e3acf818dea6c367144743bd659541ede0b9297"}],"cinder/volume/drivers/datera/datera_common.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f25c366daed43b258bed659557f3c355fbe69d53","unresolved":true,"context_lines":[{"line_number":285,"context_line":"        try:"},{"line_number":286,"context_line":"            driver.api.tenants.create(name\u003dname)"},{"line_number":287,"context_line":"        except dfs_sdk.exceptions.ApiConflictError:"},{"line_number":288,"context_line":"            LOG.debug(\"Tenant %s already exists\", name)"},{"line_number":289,"context_line":"    return _format_tenant(name)"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3200752b_187bc71f","line":288,"range":{"start_line":288,"start_character":23,"end_line":288,"end_character":54},"updated":"2021-02-17 14:57:11.000000000","message":"I\u0027ve a question, would it be better to use string interpolation here (introduced in py3.6)\nLOG.debug(f\"Tenant {name} already exists\")","commit_id":"2ae735bff47dfc4a67cd395fac7685fcc8e79fcd"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f6e6cec966e74600a3dcd14cfcf50254959c46ed","unresolved":true,"context_lines":[{"line_number":285,"context_line":"        try:"},{"line_number":286,"context_line":"            driver.api.tenants.create(name\u003dname)"},{"line_number":287,"context_line":"        except dfs_sdk.exceptions.ApiConflictError:"},{"line_number":288,"context_line":"            LOG.debug(\"Tenant %s already exists\", name)"},{"line_number":289,"context_line":"    return _format_tenant(name)"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ee93ea51_89266df2","line":288,"range":{"start_line":288,"start_character":23,"end_line":288,"end_character":54},"in_reply_to":"3200752b_187bc71f","updated":"2021-02-17 15:16:04.000000000","message":"Not sure, I just wanted to get it matching all the other Cinder code, which basically never uses .format().","commit_id":"2ae735bff47dfc4a67cd395fac7685fcc8e79fcd"}]}