)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ecdd757cd7ae1c864449c4a8be7dd7f29e1af487","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"0a293cc8_05bddf97","updated":"2022-01-12 13:50:01.000000000","message":"recheck","commit_id":"b407077dbb64fe16b672409991e1c630bc059623"},{"author":{"_account_id":32966,"name":"jiang guodao","email":"jguodao@vmware.com","username":"jguodao"},"change_message_id":"1da242b75676b5176719f76a19a46dd3cfd96898","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"03319918_bf968c27","updated":"2022-01-13 08:55:57.000000000","message":"vmware-recheck-patch","commit_id":"b407077dbb64fe16b672409991e1c630bc059623"},{"author":{"_account_id":32966,"name":"jiang guodao","email":"jguodao@vmware.com","username":"jguodao"},"change_message_id":"a413f5239b25ded62dc8ccfd4b35068705e33073","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"101d5755_89d3e0e8","updated":"2022-01-13 08:37:13.000000000","message":"vmware-recheck-patch","commit_id":"b407077dbb64fe16b672409991e1c630bc059623"},{"author":{"_account_id":32966,"name":"jiang guodao","email":"jguodao@vmware.com","username":"jguodao"},"change_message_id":"f895be077d59ae98f0751a8306e76883634c9e54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c4e39b4f_0bf1ca04","updated":"2022-01-13 08:35:51.000000000","message":"vmware-recheck-patch","commit_id":"b407077dbb64fe16b672409991e1c630bc059623"},{"author":{"_account_id":32966,"name":"jiang guodao","email":"jguodao@vmware.com","username":"jguodao"},"change_message_id":"28b4095252ef3d18226e071bf1ea9654d500aa27","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c9f4de81_d72d1b5a","updated":"2022-01-13 08:54:45.000000000","message":"vmware-recheck-patch","commit_id":"b407077dbb64fe16b672409991e1c630bc059623"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"5ed51245ad6a104e57df2a4ebf17169623170303","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bd102864_88ec1289","updated":"2022-02-16 11:42:34.000000000","message":"I\u0027ve read this but I don\u0027t feel I have enough knowledge for a +2","commit_id":"f2660948161606ecd5a08e9851a321d8d347144c"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0a9848d5_cea026a6","updated":"2022-02-23 15:28:40.000000000","message":"OK difficult to verify the tests but as you use the right roles and given the CI says yes, +2 for me.","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"2d01fe4ff911b6eb07da0622ae11ea5b706abb2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a71aca5d_f7a3ba68","updated":"2022-02-24 16:52:59.000000000","message":"Upgrading my vote after I re-read the whole series today","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/policies/aggregates.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"40f7c9db_15563e19","updated":"2022-02-23 15:28:40.000000000","message":"++ with this file, aggregates are only for admins.","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/policies/availability_zone.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[{"line_number":36,"context_line":"        scope_types\u003d[\u0027system\u0027, \u0027project\u0027]),"},{"line_number":37,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":38,"context_line":"        name\u003dPOLICY_ROOT % \u0027detail\u0027,"},{"line_number":39,"context_line":"        check_str\u003dbase.ADMIN,"},{"line_number":40,"context_line":"        description\u003d\"List detailed availability zone information with host \""},{"line_number":41,"context_line":"        \"information\","},{"line_number":42,"context_line":"        operations\u003d["}],"source_content_type":"text/x-python","patch_set":8,"id":"54d51a3f_3d03c632","line":39,"updated":"2022-02-23 15:28:40.000000000","message":"yeah, indeed only admins can look the AZ details.","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/policies/baremetal_nodes.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ea7a2948_5911e1a5","updated":"2022-02-23 15:28:40.000000000","message":"indeed all the APIs for baremetal need to be for admins","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/policies/hosts.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6bd29a9b_509473dd","updated":"2022-02-23 15:28:40.000000000","message":"correct, again this API is only for admins","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/policies/hypervisors.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5fc0b446_e3a7245c","updated":"2022-02-23 15:28:40.000000000","message":"here too","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/policies/quota_class_sets.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e4bb84d3_4a4d4f9d","updated":"2022-02-23 15:28:40.000000000","message":"ok, admin only too for quota classes.","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/policies/services.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fb98b08f_949e07b1","updated":"2022-02-23 15:28:40.000000000","message":"again, services API is admin-only","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}],"nova/tests/unit/policies/base.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"64626f724e750e11789afdffe3ee0ebbcb882375","unresolved":true,"context_lines":[{"line_number":140,"context_line":"            # rules which has the deprecated rules."},{"line_number":141,"context_line":"            self.rules_without_deprecation.update({"},{"line_number":142,"context_line":"                \"context_is_admin\":"},{"line_number":143,"context_line":"                    \"role:admin\","},{"line_number":144,"context_line":"                \"system_admin_or_owner\":"},{"line_number":145,"context_line":"                    \"rule:system_admin_api or rule:project_member_api\","},{"line_number":146,"context_line":"                \"system_or_project_reader\":"}],"source_content_type":"text/x-python","patch_set":8,"id":"508965b2_4c7bc703","line":143,"updated":"2022-02-23 15:28:40.000000000","message":"yes we need to use again the old admin role","commit_id":"60c9e3edadb3497043d20361ff665ed72241301e"}]}