)]}'
{"tempest/lib/services/volume/v2/backups_client.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"390f49b30b6106d273af8342ad96f52e455329eb","unresolved":false,"context_lines":[{"line_number":66,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    def list_backups(self, detail\u003dFalse, **params):"},{"line_number":69,"context_line":"        \"\"\"Information for all the tenant\u0027s backups."},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        For a full list of available parameters, please refer to the official"},{"line_number":72,"context_line":"        API reference:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_1b62a317","line":69,"range":{"start_line":69,"start_character":11,"end_line":69,"end_character":52},"updated":"2017-07-10 01:10:22.000000000","message":"better to begin with a verb, like \"List all the tenant\u0027s backups\"","commit_id":"35df90444eba74946d0c51377c0378b84b364c56"}],"tempest/tests/lib/services/volume/v2/test_backups_client.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"4b2d97f2f8391bc9dbd05251c25d31496052f3fc","unresolved":false,"context_lines":[{"line_number":86,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":87,"context_line":"            resp_body,"},{"line_number":88,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":89,"context_line":"            detail\u003ddetail)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def test_list_backups_with_str_body(self):"},{"line_number":92,"context_line":"        self._test_list_backups()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_0a3704e6","line":89,"updated":"2017-07-07 18:46:55.000000000","message":"Would be nice to pass a param like marker or tenant_id or something and then do something like this [0] to confirm that the params are being passed correctly to the REST call.\n\n[0] https://github.com/openstack/tempest/blob/21dd8a5ee2ab5a068cbb20d0468bd5f444fef59a/tempest/tests/lib/services/identity/v3/test_endpoints_client.py#L77","commit_id":"35df90444eba74946d0c51377c0378b84b364c56"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"bd4ebbed267c0e42c701c97420bded15907b89f7","unresolved":false,"context_lines":[{"line_number":86,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.get\u0027,"},{"line_number":87,"context_line":"            resp_body,"},{"line_number":88,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":89,"context_line":"            detail\u003ddetail)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def test_list_backups_with_str_body(self):"},{"line_number":92,"context_line":"        self._test_list_backups()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_3790d54e","line":89,"in_reply_to":"3f1d235d_0a3704e6","updated":"2017-07-09 13:12:33.000000000","message":"+1 but m ok to do that in followup patch.","commit_id":"35df90444eba74946d0c51377c0378b84b364c56"}]}