)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8bd303c03c7b47de349d6ee74a1e7e6ad71bd502","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ce74d689_3e5020d7","updated":"2023-10-27 14:22:08.000000000","message":"Could you rebase this onto https://review.opendev.org/c/openstack/python-openstackclient/+/896915/ (or wait for it to merge, which is happening right now) and update the tests? Otherwise this looks good to me.","commit_id":"05ba68ab666f721a9d47b4ce7c18711b894d87a9"}],"openstackclient/tests/unit/volume/v3/test_block_storage_resource_filter.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c4958bc426654a04f7b555ded5e6f79ad41bf49f","unresolved":true,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"        self.app.client_manager.sdk_connection \u003d mock.Mock()"},{"line_number":29,"context_line":"        self.app.client_manager.sdk_connection.volume \u003d mock.Mock()"},{"line_number":30,"context_line":"        self.sdk_client \u003d self.app.client_manager.sdk_connection.volume"},{"line_number":31,"context_line":"        patcher \u003d mock.patch.object("},{"line_number":32,"context_line":"            sdk_utils, \u0027supports_microversion\u0027, return_value\u003dTrue"},{"line_number":33,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"9d495602_81e0c0e8","line":30,"updated":"2023-10-27 14:23:19.000000000","message":"You should be able to drop all of these when you rebase onto [1]\n\n[1] https://review.opendev.org/c/openstack/python-openstackclient/+/896915/","commit_id":"05ba68ab666f721a9d47b4ce7c18711b894d87a9"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"e4ed5f4052e4a1ff284420ba05c177558b603703","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"        self.app.client_manager.sdk_connection \u003d mock.Mock()"},{"line_number":29,"context_line":"        self.app.client_manager.sdk_connection.volume \u003d mock.Mock()"},{"line_number":30,"context_line":"        self.sdk_client \u003d self.app.client_manager.sdk_connection.volume"},{"line_number":31,"context_line":"        patcher \u003d mock.patch.object("},{"line_number":32,"context_line":"            sdk_utils, \u0027supports_microversion\u0027, return_value\u003dTrue"},{"line_number":33,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"ef2db5fc_5b66e1b8","line":30,"in_reply_to":"9d495602_81e0c0e8","updated":"2023-10-30 06:38:03.000000000","message":"Done","commit_id":"05ba68ab666f721a9d47b4ce7c18711b894d87a9"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c4958bc426654a04f7b555ded5e6f79ad41bf49f","unresolved":true,"context_lines":[{"line_number":53,"context_line":"    def setUp(self):"},{"line_number":54,"context_line":"        super().setUp()"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        self.sdk_client.resource_filters.return_value \u003d ("},{"line_number":57,"context_line":"            self.fake_resource_filters"},{"line_number":58,"context_line":"        )"},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"c0195b8d_b470df03","line":56,"range":{"start_line":56,"start_character":13,"end_line":56,"end_character":23},"updated":"2023-10-27 14:23:19.000000000","message":"You\u0027ll want to rename this to `volume_sdk_client` instead.","commit_id":"05ba68ab666f721a9d47b4ce7c18711b894d87a9"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"e4ed5f4052e4a1ff284420ba05c177558b603703","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    def setUp(self):"},{"line_number":54,"context_line":"        super().setUp()"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        self.sdk_client.resource_filters.return_value \u003d ("},{"line_number":57,"context_line":"            self.fake_resource_filters"},{"line_number":58,"context_line":"        )"},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"b9d52e8f_35abfd64","line":56,"range":{"start_line":56,"start_character":13,"end_line":56,"end_character":23},"in_reply_to":"c0195b8d_b470df03","updated":"2023-10-30 06:38:03.000000000","message":"Done","commit_id":"05ba68ab666f721a9d47b4ce7c18711b894d87a9"}]}
