)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"359d9c65167f8a19990f1d12a181c38b025c71a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3cb47d10_b331d956","updated":"2022-02-24 15:48:50.000000000","message":"-1: Missing release note and unit test.","commit_id":"abcf63733c28e4e29b734674f5e91290888b918f"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9db4b2d7f7afbac6f3c139972b7f9639f63a2fdd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"530b407a_9a449e32","updated":"2022-02-25 11:30:24.000000000","message":"nit inline","commit_id":"8a9d7276e26a63dca5a75de0ce497d010e5f640c"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"188684560ff134d77c96642176ec038c18a707e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"49872f86_207841e7","updated":"2022-05-17 10:43:04.000000000","message":"This seems correct","commit_id":"53bd318710179153b0bf751785b5382582584cb4"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"5141121fd06b01bcd904a43319d888c251b92faa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e25e14ce_53a08493","updated":"2022-04-18 19:49:55.000000000","message":"recheck","commit_id":"53bd318710179153b0bf751785b5382582584cb4"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b661084ec20758c17e13c19d30b3aa5843b1a5f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0430299a_bcbf0893","updated":"2022-07-05 19:52:55.000000000","message":"It looks like all the comments have been taken into account and the CI.","commit_id":"3de124ead3e2fe5306914dd309b605c1e130828d"},{"author":{"_account_id":34201,"name":"Mounika Sreeram","email":"sreeram.mounika@ibm.com","username":"sreerammounika"},"change_message_id":"aad4f0c04976ca1b5256cb678d215e3a08219c22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"68dd62bd_5c8833b4","updated":"2022-08-10 05:47:52.000000000","message":"LGTM","commit_id":"72da8249d910a307141bb9fec663d41cc28878bd"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"82c1ed51a598c58f7a160d7b55f1b3e396ba06e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1953c494_888653e9","updated":"2022-09-17 20:23:32.000000000","message":"recheck","commit_id":"72da8249d910a307141bb9fec663d41cc28878bd"}],"cinder/volume/manager.py":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"7362152abc096ed88429ade625a82a9a87965d7e","unresolved":true,"context_lines":[{"line_number":580,"context_line":""},{"line_number":581,"context_line":"            try:"},{"line_number":582,"context_line":"                for volume in volumes:"},{"line_number":583,"context_line":"                    # available volume should also be counted into allocated"},{"line_number":584,"context_line":"                    if volume[\u0027status\u0027] in [\u0027in-use\u0027, \u0027available\u0027, \u0027attaching\u0027,"},{"line_number":585,"context_line":"                                            \u0027detaching\u0027, \u0027reserved\u0027,"},{"line_number":586,"context_line":"                                            \u0027deleting\u0027, \u0027maintenance\u0027]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"64c52a88_af579983","line":583,"updated":"2022-02-18 15:56:43.000000000","message":"nit: this comment needs to be removed or updated","commit_id":"15522db489b1607cd9a639320cf94268eadf9b37"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"2e7818e763f1c698ca222d080240b8a80b254dc9","unresolved":false,"context_lines":[{"line_number":580,"context_line":""},{"line_number":581,"context_line":"            try:"},{"line_number":582,"context_line":"                for volume in volumes:"},{"line_number":583,"context_line":"                    # available volume should also be counted into allocated"},{"line_number":584,"context_line":"                    if volume[\u0027status\u0027] in [\u0027in-use\u0027, \u0027available\u0027, \u0027attaching\u0027,"},{"line_number":585,"context_line":"                                            \u0027detaching\u0027, \u0027reserved\u0027,"},{"line_number":586,"context_line":"                                            \u0027deleting\u0027, \u0027maintenance\u0027]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1697f133_9a7b83a3","line":583,"in_reply_to":"64c52a88_af579983","updated":"2022-02-24 22:49:06.000000000","message":"Ack","commit_id":"15522db489b1607cd9a639320cf94268eadf9b37"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"359d9c65167f8a19990f1d12a181c38b025c71a9","unresolved":true,"context_lines":[{"line_number":583,"context_line":"                    # Account for volumes that have been provisioned already."},{"line_number":584,"context_line":"                    if volume[\u0027status\u0027] in [\u0027in-use\u0027, \u0027available\u0027, \u0027attaching\u0027,"},{"line_number":585,"context_line":"                                            \u0027detaching\u0027, \u0027reserved\u0027,"},{"line_number":586,"context_line":"                                            \u0027deleting\u0027, \u0027maintenance\u0027]:"},{"line_number":587,"context_line":"                        # calculate allocated capacity for driver"},{"line_number":588,"context_line":"                        self._count_allocated_capacity(ctxt, volume)"},{"line_number":589,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"0e17a3cb_31e2b68c","line":586,"updated":"2022-02-24 15:48:50.000000000","message":"-1: As discussed on IRC I think we should count all volumes that have the host field set regardless of the status they have.","commit_id":"abcf63733c28e4e29b734674f5e91290888b918f"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"2e7818e763f1c698ca222d080240b8a80b254dc9","unresolved":false,"context_lines":[{"line_number":583,"context_line":"                    # Account for volumes that have been provisioned already."},{"line_number":584,"context_line":"                    if volume[\u0027status\u0027] in [\u0027in-use\u0027, \u0027available\u0027, \u0027attaching\u0027,"},{"line_number":585,"context_line":"                                            \u0027detaching\u0027, \u0027reserved\u0027,"},{"line_number":586,"context_line":"                                            \u0027deleting\u0027, \u0027maintenance\u0027]:"},{"line_number":587,"context_line":"                        # calculate allocated capacity for driver"},{"line_number":588,"context_line":"                        self._count_allocated_capacity(ctxt, volume)"},{"line_number":589,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"92f04b75_856cb358","line":586,"in_reply_to":"0e17a3cb_31e2b68c","updated":"2022-02-24 22:49:06.000000000","message":"Done","commit_id":"abcf63733c28e4e29b734674f5e91290888b918f"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b661084ec20758c17e13c19d30b3aa5843b1a5f7","unresolved":false,"context_lines":[{"line_number":583,"context_line":"                    # Account for volumes that have been provisioned already."},{"line_number":584,"context_line":"                    if volume[\u0027status\u0027] in [\u0027in-use\u0027, \u0027available\u0027, \u0027attaching\u0027,"},{"line_number":585,"context_line":"                                            \u0027detaching\u0027, \u0027reserved\u0027,"},{"line_number":586,"context_line":"                                            \u0027deleting\u0027, \u0027maintenance\u0027]:"},{"line_number":587,"context_line":"                        # calculate allocated capacity for driver"},{"line_number":588,"context_line":"                        self._count_allocated_capacity(ctxt, volume)"},{"line_number":589,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7ca270ed_4ddf1da1","line":586,"in_reply_to":"92f04b75_856cb358","updated":"2022-07-05 19:52:55.000000000","message":"OK. I need to search for the meeting logs to check: I am not sure if this scenario is possible, but I think a volume could be in the \u0027delete\u0027 state when the volume service is restarted.","commit_id":"abcf63733c28e4e29b734674f5e91290888b918f"}],"releasenotes/notes/bug-1910767-00f20702f5fc96db.yaml":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9db4b2d7f7afbac6f3c139972b7f9639f63a2fdd","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #1910767 \u003chttps://bugs.launchpad.net/cinder/+bug/1910767\u003e`_: Fixed"},{"line_number":5,"context_line":"    Fix the calculation of the allocated capacity for the volume manager."},{"line_number":6,"context_line":"    The fix takes into account all volumes that have a host setting, not just"},{"line_number":7,"context_line":"    volumes with a status of \u0027in-use\u0027 or \u0027available\u0027."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"37710454_9e74d850","line":5,"range":{"start_line":4,"start_character":70,"end_line":5,"end_character":8},"updated":"2022-02-25 11:30:24.000000000","message":"nit: duplicate ?","commit_id":"8a9d7276e26a63dca5a75de0ce497d010e5f640c"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"a8426f9e74b8041274e4567ce4f6e43f64edd653","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #1910767 \u003chttps://bugs.launchpad.net/cinder/+bug/1910767\u003e`_: Fixed"},{"line_number":5,"context_line":"    Fix the calculation of the allocated capacity for the volume manager."},{"line_number":6,"context_line":"    The fix takes into account all volumes that have a host setting, not just"},{"line_number":7,"context_line":"    volumes with a status of \u0027in-use\u0027 or \u0027available\u0027."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"19d038b7_27e07bcd","line":5,"range":{"start_line":4,"start_character":70,"end_line":5,"end_character":8},"in_reply_to":"37710454_9e74d850","updated":"2022-02-25 12:58:09.000000000","message":"Done","commit_id":"8a9d7276e26a63dca5a75de0ce497d010e5f640c"}]}