)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30696,"name":"zhaoleilc","email":"15247232416@163.com","username":"zhaoleilc"},"change_message_id":"80ecc553a3e717b6107919786a4a2a7656dcfff1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6a89f922_93c25b95","updated":"2022-11-06 02:09:47.000000000","message":"\u003e Patch Set 1:\n\u003e \n\u003e Merge Failed.\n\u003e \n\u003e This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.\n\nOk，I will rebase the victoria branch.","commit_id":"152e5387efb186108e4d0be434145e03b8922801"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"51b1b50c54b5e1635ef18c6d74122faf58ccdb1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b090ea94_ee9b9588","updated":"2025-06-18 14:57:51.000000000","message":"FWIW there is an older review that addresses allocated_capacity_gb getting out of date here:\n\nhttps://review.opendev.org/c/openstack/cinder/+/836083","commit_id":"a8b13c80e2c62af4c43e118ab9389af5d710534e"}],"cinder/volume/manager.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"51b1b50c54b5e1635ef18c6d74122faf58ccdb1c","unresolved":true,"context_lines":[{"line_number":2849,"context_line":"                    host\u003dvolume_host_with_pool)[1]"},{"line_number":2850,"context_line":"            self.stats[\u0027pools\u0027][pool_name][\u0027allocated_capacity_gb\u0027] \u003d int("},{"line_number":2851,"context_line":"                vol_sz_sum)"},{"line_number":2852,"context_line":"            self.stats[\u0027allocated_capacity_gb\u0027] +\u003d int(vol_sz_sum)"},{"line_number":2853,"context_line":""},{"line_number":2854,"context_line":"    def _append_volume_stats(self, vol_stats) -\u003e None:"},{"line_number":2855,"context_line":"        pools \u003d vol_stats.get(\u0027pools\u0027, None)"}],"source_content_type":"text/x-python","patch_set":3,"id":"54e64550_69f9e1b8","line":2852,"updated":"2025-06-18 14:57:51.000000000","message":"I think pulling all volumes for a host for every volume service every 60 seconds is not a good idea.   My deployment has thousands of volumes and multiple driver instances running in the same AZ.","commit_id":"a8b13c80e2c62af4c43e118ab9389af5d710534e"}]}