)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"f2b194f1af4a60f9175bae5d758c72cb2046763c","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch fixes an issue where when a tenant attempts to manage a volume"},{"line_number":10,"context_line":"that exceeds their storage quota, the clenup will cause an error due"},{"line_number":11,"context_line":"to attemptinmg to delete a non-existant volume on the backend."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #2119059"},{"line_number":14,"context_line":"Change-Id: I57acda3d94af703b52bfc923a4928b089475b476"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ce3d07ad_5acb87ae","line":11,"updated":"2025-09-05 13:44:37.000000000","message":"s/attemptinmg/attempting","commit_id":"3aad12a6c3201cde711a01e673da0ec8995e08bf"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"318ce0962b8a6a849ba304c63735d4186f3a1b9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20de1954_1cdd6e84","updated":"2025-09-04 18:34:11.000000000","message":"Small request","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"44953cb185ace2c297578806fd0b2a7045f95650","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2d3a19bb_85d913f2","updated":"2025-07-30 19:28:33.000000000","message":"run Pure Storage CI","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"bb5696ac83ff2fdd60d9e5856b9ebfa8f8090b5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3b8d120c_1318da7f","updated":"2025-07-30 13:17:58.000000000","message":"run Pure Storage CI","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"c4f369d9789d72de0b9432336da3bc73112cb746","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4e051c99_5ede2753","updated":"2025-07-31 00:04:43.000000000","message":"run Pure Storage CI","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"54c7c07a50ccca29b081cc5f213d4f7af896b870","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9d4614b5_2eec0ffa","updated":"2025-07-30 03:39:42.000000000","message":"run Pure Storage CI","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"98eef4d2b5f159e5f8afc1ba9bfaec6a9252ed6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c3a74ea6_8af3cd4a","updated":"2025-07-30 14:12:13.000000000","message":"run Pure Storage CI","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"f2b194f1af4a60f9175bae5d758c72cb2046763c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e8190ec6_37e5ac6e","updated":"2025-09-05 13:44:37.000000000","message":"Code and reno looks good, found just a typo in commit message.","commit_id":"3aad12a6c3201cde711a01e673da0ec8995e08bf"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3f32af0a338b623e3acc349de763268d4c85cbd4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a376e5d4_467306dd","updated":"2025-09-05 15:41:02.000000000","message":"Code change looks fine.  Don\u0027t want to hold it up over the commit message typo (which Fernando is quite right to point out).","commit_id":"3aad12a6c3201cde711a01e673da0ec8995e08bf"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"4694c1f4fc4650c60cd3678084b730738d42daa6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cdd0c654_49f03c67","updated":"2025-09-05 15:14:38.000000000","message":"Looks good to me.  If another reviewer asks for a an update then the typo in the commit message could be addressed, but I don\u0027t think it\u0027s a reason alone to hold up the patch.","commit_id":"3aad12a6c3201cde711a01e673da0ec8995e08bf"}],"cinder/volume/drivers/pure.py":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"318ce0962b8a6a849ba304c63735d4186f3a1b9a","unresolved":true,"context_lines":[{"line_number":961,"context_line":"        current_array \u003d self._get_current_array()"},{"line_number":962,"context_line":"        # Do a pass over remaining connections on the current array, if"},{"line_number":963,"context_line":"        # we can try and remove any remote connections too."},{"line_number":964,"context_line":"        res \u003d current_array.get_connections(volume_names\u003d[vol_name])"},{"line_number":965,"context_line":"        if res.status_code !\u003d 200:"},{"line_number":966,"context_line":"            hosts \u003d []"},{"line_number":967,"context_line":"        hosts \u003d list(res.items)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9bf3c3db_bfdf21fd","line":964,"updated":"2025-09-04 18:34:11.000000000","message":"This reads confusing to me, I would set hosts to [] first, and then reassign it if status_code is 200.  That would be clearer.\n\nOtherwise, looks good.","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"6d791a2d70fa811634e61ad1cb40253c699f88fa","unresolved":false,"context_lines":[{"line_number":961,"context_line":"        current_array \u003d self._get_current_array()"},{"line_number":962,"context_line":"        # Do a pass over remaining connections on the current array, if"},{"line_number":963,"context_line":"        # we can try and remove any remote connections too."},{"line_number":964,"context_line":"        res \u003d current_array.get_connections(volume_names\u003d[vol_name])"},{"line_number":965,"context_line":"        if res.status_code !\u003d 200:"},{"line_number":966,"context_line":"            hosts \u003d []"},{"line_number":967,"context_line":"        hosts \u003d list(res.items)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2b98f52b_db498119","line":964,"in_reply_to":"9bf3c3db_bfdf21fd","updated":"2025-09-04 22:35:21.000000000","message":"Done","commit_id":"7d6108ec84faa154d6e4da8b7f9105455ba5a3a7"}]}