)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6814a31e6bc724ca32dca445a4665b1914f12bbb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1af03ec1_9144303a","updated":"2022-04-12 14:16:55.000000000","message":"There are still failing tests here due to changes in our mocking. Need to address","commit_id":"349c7b156b0a2e349b63f001f80dc5f52bba2ede"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"3eb2ceda6b02d5a661af778e4416b19aeddd5cc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"5d983bc9_c5cee63f","updated":"2022-06-17 14:07:24.000000000","message":"Lots of CI are failing.","commit_id":"fa82e0a00af622f6a3991e8f71a3e980f2b835d8"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"502c15b89743ac7e77ab8740f4a4ebd3ae42cde1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"7685ecdd_5993b423","updated":"2022-06-17 15:47:18.000000000","message":"recheck errant failure","commit_id":"fa82e0a00af622f6a3991e8f71a3e980f2b835d8"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8bbb5204d83c28d4e44674290a564526bfabb855","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"dfd527a6_c418f51d","updated":"2022-06-23 21:23:16.000000000","message":"recheck unrelated failure","commit_id":"fa82e0a00af622f6a3991e8f71a3e980f2b835d8"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"adb37679362ad5ffbbfedd45334e80b1c4ec16f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"6cd28347_16f66294","in_reply_to":"5d983bc9_c5cee63f","updated":"2022-07-15 14:24:00.000000000","message":"Yeah, those weren\u0027t real failures","commit_id":"fa82e0a00af622f6a3991e8f71a3e980f2b835d8"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"228eb51d194dd73d83fb0eb05ee8347a7ee2947c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"65f3db3c_44021e44","updated":"2022-08-31 12:16:43.000000000","message":"recheck random timeout","commit_id":"80499857d622af49eec3192d050506d67d994ae2"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f93e74c05bf1a4bfb521cd34d83e9260f72fc7d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"a8243bd4_5fa5516b","updated":"2022-09-09 11:30:53.000000000","message":"recheck unrelated py39 failure","commit_id":"80499857d622af49eec3192d050506d67d994ae2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"5a10cc1e52b7d8d9927f17657c4a69149e76a054","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"64655849_5f6f4e77","updated":"2023-01-26 14:21:02.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"80499857d622af49eec3192d050506d67d994ae2"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"e53863aa3940400923dbc49c2ccb42f40445ca77","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"917918f3_0bf07047","updated":"2023-01-26 14:20:44.000000000","message":"run-DellEMC PowerStore CI","commit_id":"80499857d622af49eec3192d050506d67d994ae2"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"386f9e6ce7c9cce3218f454fdfd6c0d17bed11d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"03225d36_5884e39e","updated":"2026-06-14 06:17:10.000000000","message":"One concern...\"commit message says some files needed \"slightly different calling order\". Positional arg changed to keyword arg","commit_id":"8672b03d541aa00a62351f9df78b83799c85e0d3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7689fc4187d33ce9b5538506ebde3ecd1eada603","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"8499b861_27168c83","updated":"2026-06-11 19:33:25.000000000","message":"This touches a lot of files; would be good to get it merged soon so it has some time to bake during the development cycle.  Thanks for the tip about how to review the new cinder/db/api.py file, that was very useful.","commit_id":"8672b03d541aa00a62351f9df78b83799c85e0d3"}],"cinder/api/contrib/quota_classes.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7689fc4187d33ce9b5538506ebde3ecd1eada603","unresolved":true,"context_lines":[{"line_number":20,"context_line":"from cinder.api.schemas import quota_classes as schema"},{"line_number":21,"context_line":"from cinder.api import validation"},{"line_number":22,"context_line":"from cinder.db import api as db"},{"line_number":23,"context_line":"from cinder.db import api as sqla_db"},{"line_number":24,"context_line":"from cinder import exception"},{"line_number":25,"context_line":"from cinder.policies import quota_class as policy"},{"line_number":26,"context_line":"from cinder import quota"}],"source_content_type":"text/x-python","patch_set":20,"id":"523b2810_3ded6389","line":23,"range":{"start_line":23,"start_character":29,"end_line":23,"end_character":36},"updated":"2026-06-11 19:33:25.000000000","message":"nit: this only used at line 46, so you could just s/sqla_db/db/ at line 18 and remove line 23.  On the other hand, this may be a common pattern, and might be better dealt with in a followup at some point.\n\nCircling back to this ... looks like this binding of cinder.db.api to two different local names happens in most of the driver unit tests, so cleaning that up could be a good first contact project for a student or a new contributor.","commit_id":"8672b03d541aa00a62351f9df78b83799c85e0d3"}],"cinder/cmd/manage.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"386f9e6ce7c9cce3218f454fdfd6c0d17bed11d9","unresolved":true,"context_lines":[{"line_number":221,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"        try:"},{"line_number":224,"context_line":"            db.purge_deleted_rows(ctxt, age_in_days\u003dage_in_days)"},{"line_number":225,"context_line":"        except db_exc.DBReferenceError:"},{"line_number":226,"context_line":"            print(_(\"Purge command failed, check cinder-manage \""},{"line_number":227,"context_line":"                    \"logs for more details.\"))"}],"source_content_type":"text/x-python","patch_set":20,"id":"1ea1853d_ea22937b","line":224,"updated":"2026-06-14 06:17:10.000000000","message":"commit message says some files needed \"slightly different calling order\". This change altering positional arg to keyword arg","commit_id":"8672b03d541aa00a62351f9df78b83799c85e0d3"}],"cinder/db/api.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7689fc4187d33ce9b5538506ebde3ecd1eada603","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright (c) 2011 X.commerce, a business unit of eBay Inc."},{"line_number":2,"context_line":"# Copyright 2010 United States Government as represented by the"},{"line_number":3,"context_line":"# Administrator of the National Aeronautics and Space Administration."},{"line_number":4,"context_line":"# Copyright 2014 IBM Corp."},{"line_number":5,"context_line":"# All Rights Reserved."},{"line_number":6,"context_line":"#"},{"line_number":7,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":20,"id":"ddb972ef_ff47b021","line":4,"updated":"2026-06-11 19:33:25.000000000","message":"I think this is because you moved the filter functions into this file?","commit_id":"8672b03d541aa00a62351f9df78b83799c85e0d3"}]}