)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31280,"name":"lkk","email":"likangkang01@inspur.com","username":"likangkang"},"change_message_id":"59163cbe12778fb43c1ea932766de75f67557e74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"24155aa7_28994bb5","updated":"2021-11-24 02:28:23.000000000","message":"recheck","commit_id":"411d00e17a626f0b349ad7ee41ec320b0b5f438a"},{"author":{"_account_id":31280,"name":"lkk","email":"likangkang01@inspur.com","username":"likangkang"},"change_message_id":"930b442d3693c2703af713b456c96381079c7bb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"504730b8_8b8e1916","updated":"2021-11-23 01:20:05.000000000","message":"recheck","commit_id":"411d00e17a626f0b349ad7ee41ec320b0b5f438a"},{"author":{"_account_id":31280,"name":"lkk","email":"likangkang01@inspur.com","username":"likangkang"},"change_message_id":"f6fa72d5215a1a2896c6063e88f61e48178a897a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"504bf14f_4428ff31","updated":"2021-11-24 10:15:48.000000000","message":"recheck","commit_id":"411d00e17a626f0b349ad7ee41ec320b0b5f438a"},{"author":{"_account_id":31280,"name":"lkk","email":"likangkang01@inspur.com","username":"likangkang"},"change_message_id":"f7eb909153d1cf3395081e1da41a94bc65937009","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5b88cea0_0394b373","updated":"2021-11-19 03:18:34.000000000","message":"recheck","commit_id":"411d00e17a626f0b349ad7ee41ec320b0b5f438a"},{"author":{"_account_id":31280,"name":"lkk","email":"likangkang01@inspur.com","username":"likangkang"},"change_message_id":"c94023c4c1d4a58e6c5d875edb13081195f88487","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7df04536_e8495680","updated":"2021-11-22 00:49:07.000000000","message":"recheck","commit_id":"411d00e17a626f0b349ad7ee41ec320b0b5f438a"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"fd931f3178e1bbfe8d52b7d725bab0010614aced","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1cb59a08_7834a627","updated":"2022-01-05 23:51:21.000000000","message":"Hi, \n\nThanks for attempting these tests. These are quite useful, it looks like they failed because they were hitting the incorrect API endpoint. Can you please take a look at the inline comment and restore this effort?","commit_id":"f51e6d3a46f4e756a9dba483e1ef4f05648428c8"},{"author":{"_account_id":31280,"name":"lkk","email":"likangkang01@inspur.com","username":"likangkang"},"change_message_id":"8350f5d75479e45329f48d931ba605065f1407c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"87cd6223_54f2f8c5","updated":"2022-01-07 00:54:37.000000000","message":"Already restore, is that ok?","commit_id":"64390b1cd0ffda8fa2567106cc424a8617733957"}],"manila_tempest_tests/tests/api/test_limits.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"fd931f3178e1bbfe8d52b7d725bab0010614aced","unresolved":true,"context_lines":[{"line_number":27,"context_line":"    def test_limits_keys(self):"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"        # list limits"},{"line_number":30,"context_line":"        limits \u003d self.shares_client.get_limits()[\u0027limits\u0027]"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        # verify response"},{"line_number":33,"context_line":"        keys \u003d [\"rate\", \"absolute\"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"9c14e1b4_b2c2a7e5","line":30,"range":{"start_line":30,"start_character":22,"end_line":30,"end_character":35},"updated":"2022-01-05 23:51:21.000000000","message":"This is a \u0027shares_client\u0027 call so far - only the \"v1\" API being tested here;\n\nthe manila \"v2\" API is an extension of the v1 API, with API micro-versions. \n\nI would support adding a \"get_limits\" call to the v2 client here: https://github.com/openstack/manila-tempest-plugin/blob/98b3e8782d20d193974173c5763236bb0478a289/manila_tempest_tests/services/share/v2/json/shares_client.py#L35\n\nand modifying all tests here to just test the v2 API (testing the v1 api is no longer preferred since we\u0027ve deprecated that endpoint and may remove it entirely in the future).\n\nThat will allow your tests to work","commit_id":"f51e6d3a46f4e756a9dba483e1ef4f05648428c8"},{"author":{"_account_id":31280,"name":"lkk","email":"likangkang01@inspur.com","username":"likangkang"},"change_message_id":"8350f5d75479e45329f48d931ba605065f1407c4","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    def test_limits_keys(self):"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"        # list limits"},{"line_number":30,"context_line":"        limits \u003d self.shares_client.get_limits()[\u0027limits\u0027]"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        # verify response"},{"line_number":33,"context_line":"        keys \u003d [\"rate\", \"absolute\"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a3722b8_b5cf4ea0","line":30,"range":{"start_line":30,"start_character":22,"end_line":30,"end_character":35},"in_reply_to":"9c14e1b4_b2c2a7e5","updated":"2022-01-07 00:54:37.000000000","message":"Done","commit_id":"f51e6d3a46f4e756a9dba483e1ef4f05648428c8"}]}
