)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"68d5a7a7c9c714b23360ee2aeda05bae541cd8d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4cc7bae6_81eb265c","updated":"2026-06-12 06:44:06.000000000","message":"LGTM","commit_id":"3630add1992f0d9ff8649045ed7644d410d6c835"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3e59c9ce929a68998700f75377741d9e8ea4a593","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"55731c0c_9659c7a3","updated":"2026-09-18 16:09:06.000000000","message":"Changes make sense.","commit_id":"5fa053f3dfb659a262cb587a17b2dd8e49bf43b3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"007fb33fcfb38cfb2906c072a98ed17ae351cc7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"39a08bb2_c717d624","updated":"2026-09-18 16:29:14.000000000","message":"Followup patch is https://review.opendev.org/c/openstack/cinder/+/1006277","commit_id":"5fa053f3dfb659a262cb587a17b2dd8e49bf43b3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f727003568e59b5e8244a1d349fd725b85269b20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f93d6873_a475c9da","updated":"2026-09-21 12:49:25.000000000","message":"This is basically a documentation change, so I\u0027ll invoke cinder\u0027s single-approval rule.  Plus, Simon did a careful review, so I don\u0027t think there\u0027s any reason to hold this up.","commit_id":"5fa053f3dfb659a262cb587a17b2dd8e49bf43b3"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@everpuredata.com","username":"sdodsley"},"change_message_id":"c1d9fe0faa0ac4fa7498b5374581c7d3293c3823","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4c66622b_405b35c0","updated":"2026-09-18 15:05:53.000000000","message":"Verified the 202 responses, the query-string ``vol_type_id`` and the\n``association_type`` key against the controller and view code; all\ncorrect. Functional tests pass locally.\n\nOne leftover: ``cinder/tests/functional/api_sample_tests/samples/qos/\nqos_show_response.json.tpl`` is no longer referenced after this change\nand should be deleted alongside the two api-ref copies.\n\nOptional: assert 202 on the associate call in\n``test_qos_show_associations`` so the status-code fix is covered by a test.","commit_id":"5fa053f3dfb659a262cb587a17b2dd8e49bf43b3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3e59c9ce929a68998700f75377741d9e8ea4a593","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"49164eca_4cdfaf4e","in_reply_to":"4c66622b_405b35c0","updated":"2026-09-18 16:09:06.000000000","message":"I think show associations is supposed to be 200?\n\nBut you are definitely right about the tpl file with the underscores; if we don\u0027t remove it now, it will persist indefinintely.  I\u0027ll post a followup patch so we don\u0027t hold this one up.","commit_id":"5fa053f3dfb659a262cb587a17b2dd8e49bf43b3"}]}