)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","display_name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc","status":"Senior Software Quality Engineer @ Red Hat"},"change_message_id":"ddc4628bdc740be3b43a4726294652d4c2726d13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"15af3543_2752d2cf","updated":"2026-02-05 13:51:55.000000000","message":"(my mistake for not including the +1 in my previous comment)\n* This change makes sense and all issues have been resolved. LGTM","commit_id":"00a9a81c6f4f15b1a6289c02e3b66f3726f8980c"},{"author":{"_account_id":32704,"name":"Alfredo Garcia","display_name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc","status":"Senior Software Quality Engineer @ Red Hat"},"change_message_id":"f7f5f5a989a3e7efd8a2df55c6519de33d38db7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b6b68663_00d9baab","updated":"2026-02-04 15:36:57.000000000","message":"The change makes sense","commit_id":"00a9a81c6f4f15b1a6289c02e3b66f3726f8980c"}],"tempest/api/volume/admin/test_group_snapshots.py":[{"author":{"_account_id":35153,"name":"Ashish Gupta","email":"ashigupt@redhat.com","username":"ashigupt","status":"Redhat"},"change_message_id":"b587fc3c0ffbf42a64aa4c3055d29a6082f602ab","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    def setup_clients(cls):"},{"line_number":39,"context_line":"        super(BaseGroupSnapshotsTest, cls).setup_clients()"},{"line_number":40,"context_line":"        if CONF.enforce_scope.cinder:"},{"line_number":41,"context_line":"            cls.reader_volumes_client \u003d ("},{"line_number":42,"context_line":"                cls.os_project_reader.volumes_client_latest)"},{"line_number":43,"context_line":"            cls.reader_snapshots_client \u003d ("},{"line_number":44,"context_line":"                cls.os_project_reader.snapshots_client_latest)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9fb02014_89f94187","line":41,"updated":"2026-01-30 20:34:28.000000000","message":"as mentioned in other patch, reader_volumes_client can be moved to base class and that could be reused everywhere else","commit_id":"153bc7685b3e0fbbd24f7c79c6b3382e5f9bb1e1"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"f5ee7ad1a4c99a0e6a904406bed3aff5841c7763","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    def setup_clients(cls):"},{"line_number":39,"context_line":"        super(BaseGroupSnapshotsTest, cls).setup_clients()"},{"line_number":40,"context_line":"        if CONF.enforce_scope.cinder:"},{"line_number":41,"context_line":"            cls.reader_volumes_client \u003d ("},{"line_number":42,"context_line":"                cls.os_project_reader.volumes_client_latest)"},{"line_number":43,"context_line":"            cls.reader_snapshots_client \u003d ("},{"line_number":44,"context_line":"                cls.os_project_reader.snapshots_client_latest)"}],"source_content_type":"text/x-python","patch_set":4,"id":"a470a240_762008ac","line":41,"in_reply_to":"9fb02014_89f94187","updated":"2026-02-03 13:56:35.000000000","message":"Thanks for the suggestion. I have removed the explicit reader_volumes_client definitions and called the same from base.py.","commit_id":"153bc7685b3e0fbbd24f7c79c6b3382e5f9bb1e1"}]}