)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"02c5a36b60c42e13afa307ababa14dae3b15a170","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c34f2c69_b8056e74","updated":"2022-01-11 15:56:45.000000000","message":"@Walt: question inline","commit_id":"059200d841fdec635d377e2fefd0b97b171f57e6"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"e8aa9f0e441f2ab2c8441925c620d7a09c31ac60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c9ba9411_10b86f79","updated":"2022-01-17 11:37:51.000000000","message":"Just small stuff removed. Looks good to me.","commit_id":"059200d841fdec635d377e2fefd0b97b171f57e6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"a2719c2640b374eeabe83b30544b1fd2b994ccc7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b86c5cb4_fd27d0bd","updated":"2021-12-17 14:21:41.000000000","message":"Looks good to me! Thanks Eric.","commit_id":"059200d841fdec635d377e2fefd0b97b171f57e6"},{"author":{"_account_id":33612,"name":"yuval","email":"yuval@lightbitslabs.com","username":"yuval"},"change_message_id":"42b3361fefaaeb8f6338178c1c9e7a29fdb1af01","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8e5726cd_97aff36b","updated":"2022-02-07 14:57:40.000000000","message":"I dont know where I can see the block logs to debug the issue but the tests are running:\nhttps://013a2bc25c5a5d413854-7365f2676f718bf07df8d3e0d7dd1275.ssl.cf5.rackcdn.com/800703/4/gate/cinder-plugin-ceph-tempest/9105fc7/job-output.txt\n\nsearch for \"... F\" - 6 tests failed and took a lot of time till failure (probably the reason for timeout).","commit_id":"36bd8f5c9773caf3c87e4683948bd32ed99ac3cf"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7179866116bffe8f269eb21ad86f30dab5976571","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"03bbfbee_3ec8a0fa","updated":"2022-02-04 16:24:39.000000000","message":"I think Walt\u0027s concern has been addressed.  Moving this along.","commit_id":"36bd8f5c9773caf3c87e4683948bd32ed99ac3cf"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"6ee24e4f8b8378387855092b82e8b9284c89e280","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2c02fc5f_f9d2b57c","updated":"2022-01-24 14:54:26.000000000","message":"Walt: please review again now that this patch has been merged:\n\n    https://review.opendev.org/c/openstack/cinder/+/824472","commit_id":"36bd8f5c9773caf3c87e4683948bd32ed99ac3cf"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"11e5195d3e1845640b68c593f10a8f4c6700be20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"72ffaa44_3d400103","updated":"2022-02-07 14:38:29.000000000","message":"recheck","commit_id":"36bd8f5c9773caf3c87e4683948bd32ed99ac3cf"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"a12ff660d8d7e40a5055c116c375dbe769d7b75d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7516ff9d_38c2eb48","updated":"2022-02-04 21:51:52.000000000","message":"recheck","commit_id":"36bd8f5c9773caf3c87e4683948bd32ed99ac3cf"}],"cinder/volume/api.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"37f52b9727da029787add1b8d64b2a570d043b9f","unresolved":true,"context_lines":[{"line_number":1325,"context_line":"        rv \u003d self.db.volume_admin_metadata_get(context, volume[\u0027id\u0027])"},{"line_number":1326,"context_line":"        LOG.info(\"Get volume admin metadata completed successfully.\","},{"line_number":1327,"context_line":"                 resource\u003dvolume)"},{"line_number":1328,"context_line":"        return dict(rv)"},{"line_number":1329,"context_line":""},{"line_number":1330,"context_line":"    def update_volume_admin_metadata(self,"},{"line_number":1331,"context_line":"                                     context: context.RequestContext,"}],"source_content_type":"text/x-python","patch_set":3,"id":"28e84593_935eaaeb","side":"PARENT","line":1328,"updated":"2021-12-22 15:47:53.000000000","message":"There is an entry in the contrib/block-box/etc/policy.json for this call.","commit_id":"cf7c2fcc75b5abdc9cb4cac5fb956fb753a4508f"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"02c5a36b60c42e13afa307ababa14dae3b15a170","unresolved":true,"context_lines":[{"line_number":1325,"context_line":"        rv \u003d self.db.volume_admin_metadata_get(context, volume[\u0027id\u0027])"},{"line_number":1326,"context_line":"        LOG.info(\"Get volume admin metadata completed successfully.\","},{"line_number":1327,"context_line":"                 resource\u003dvolume)"},{"line_number":1328,"context_line":"        return dict(rv)"},{"line_number":1329,"context_line":""},{"line_number":1330,"context_line":"    def update_volume_admin_metadata(self,"},{"line_number":1331,"context_line":"                                     context: context.RequestContext,"}],"source_content_type":"text/x-python","patch_set":3,"id":"d84143ba_218c663c","side":"PARENT","line":1328,"in_reply_to":"28e84593_935eaaeb","updated":"2022-01-11 15:56:45.000000000","message":"You should check me on this, but I don\u0027t think the policy \"volume:get_volume_admin_metadata\" is defined anywhere in cinder/policies/* ... I wonder when the last time that black-box policy file was updated?","commit_id":"cf7c2fcc75b5abdc9cb4cac5fb956fb753a4508f"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"44ae7bb03ac36a67f00399c4c38539ecb2707a7a","unresolved":false,"context_lines":[{"line_number":1325,"context_line":"        rv \u003d self.db.volume_admin_metadata_get(context, volume[\u0027id\u0027])"},{"line_number":1326,"context_line":"        LOG.info(\"Get volume admin metadata completed successfully.\","},{"line_number":1327,"context_line":"                 resource\u003dvolume)"},{"line_number":1328,"context_line":"        return dict(rv)"},{"line_number":1329,"context_line":""},{"line_number":1330,"context_line":"    def update_volume_admin_metadata(self,"},{"line_number":1331,"context_line":"                                     context: context.RequestContext,"}],"source_content_type":"text/x-python","patch_set":3,"id":"8ab1a60a_15c31b11","side":"PARENT","line":1328,"in_reply_to":"d84143ba_218c663c","updated":"2022-02-07 14:38:50.000000000","message":"Done","commit_id":"cf7c2fcc75b5abdc9cb4cac5fb956fb753a4508f"}]}