)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"d9d575152dc98b99dfa034b7284d50d326169e26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"95d51e63_ce3c1a01","updated":"2024-12-10 03:32:05.000000000","message":"run-HPE Storage CI","commit_id":"554c1a807d5489f3cf3b0d3d3b294c6dd8a5fee5"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"2cfb3bd782ee78b811db47dadac3b34548960689","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0fd67fef_b9723584","updated":"2025-01-13 09:11:36.000000000","message":"HPE Storage CI and Zuul have passed","commit_id":"d4872f69e074f86f9fa7c5ee5839a840e85e70b6"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"4c2cd71647df4c4e38601c497f66750b329f3af3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c44f389e_5ed3b5da","updated":"2025-01-21 00:10:56.000000000","message":"This makes sense to me as a way to compute the free capacity, but from the unit test changes and the fact that you are incrementing the driver version, it looks like we should expect differences in the get_volume_stats response for the 4.0.25 vs. earlier versions.  It\u0027s worth mentioning that in the commit message, and maybe in a release note, unless you think no one will notice? (I imagine that the volume stats are logged at debug level.)\n\nCode and tests look good; it would be good to say something about why you\u0027re making the change in the commit message.","commit_id":"d4872f69e074f86f9fa7c5ee5839a840e85e70b6"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"682dc4b8a438ec47ecf5813b8a3f5ba2a1133c83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"74ce6ae3_019c5719","in_reply_to":"c44f389e_5ed3b5da","updated":"2025-01-21 10:44:36.000000000","message":"Updated the commit message and added a release note.","commit_id":"d4872f69e074f86f9fa7c5ee5839a840e85e70b6"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"0814b79dac7828de69282ce89f8ae25313645d6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"14fed540_4218c537","updated":"2025-01-22 16:10:49.000000000","message":"Hi Brian,\nmay i request your feedback please.\nHPE Storage CI and Zuul have passed","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9acc235f694386dd5fef652934d365ccc52bc9cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e149027a_04eb1454","updated":"2025-01-26 08:48:37.000000000","message":"Thank you Brian","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"09561965f933214242d616cb4d800165d15e43cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"06a35964_6713d07f","updated":"2025-02-07 05:38:24.000000000","message":"Thank you Hemna \u0026 Brian. The patch has merged.","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"682dc4b8a438ec47ecf5813b8a3f5ba2a1133c83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d7fe571f_e0b678dc","updated":"2025-01-21 10:44:36.000000000","message":"Thanks Brian for review.\ni tried to address review comments.","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e2b13604582c12d20a7f25cea6d4b16e1493ce97","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f8ae0973_f80e2ef7","updated":"2025-01-24 16:08:17.000000000","message":"Thanks for updating the commit message; it\u0027s clear now why you\u0027re making the change.  Release note looks good, too.  Code \u0026 tests LGTM.","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"989c5d64287f85848710e144deef16fee40861ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"15abb71f_6e809969","updated":"2025-01-22 05:30:38.000000000","message":"recheck cinder-plugin-ceph-tempest","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"2e1bfc0cb856c1cfd735c4ea6d0c460f0a918203","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1e7b9b3a_2de4ca06","updated":"2025-01-22 03:18:27.000000000","message":"recheck cinder-plugin-ceph-tempest failed","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"3592799ed0878e4d70ce0e7ae39e452a1f7ae0b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e0d5f520_9de9487b","updated":"2025-01-22 09:03:36.000000000","message":"recheck cinder-plugin-ceph-tempest failed third time","commit_id":"792462cad150e179189a3564844fae40be1ae653"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"0ec338a97fee919ba21c01c744934ce260ca6e24","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8178c5ed_e18e5e74","updated":"2025-01-22 12:07:48.000000000","message":"recheck devstack-plugin-nfs-tempest-full","commit_id":"792462cad150e179189a3564844fae40be1ae653"}],"cinder/tests/unit/volume/drivers/hpe/test_hpe3par.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"4c2cd71647df4c4e38601c497f66750b329f3af3","unresolved":true,"context_lines":[{"line_number":8201,"context_line":"            self.assertFalse(stats[\u0027pools\u0027][0][\u0027QoS_support\u0027])"},{"line_number":8202,"context_line":"            self.assertEqual(86.0,"},{"line_number":8203,"context_line":"                             stats[\u0027pools\u0027][0][\u0027provisioned_capacity_gb\u0027])"},{"line_number":8204,"context_line":"            self.assertEqual(24.0, stats[\u0027pools\u0027][0][\u0027total_capacity_gb\u0027])"},{"line_number":8205,"context_line":"            self.assertEqual(3.0, stats[\u0027pools\u0027][0][\u0027free_capacity_gb\u0027])"},{"line_number":8206,"context_line":"            self.assertEqual(87.5, stats[\u0027pools\u0027][0][\u0027capacity_utilization\u0027])"},{"line_number":8207,"context_line":"            self.assertEqual(3, stats[\u0027pools\u0027][0][\u0027total_volumes\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"6d54ad94_089bd39e","side":"PARENT","line":8204,"updated":"2025-01-21 00:10:56.000000000","message":"This was really weird to have the total_capacity_gb be less than the provisioned_capacity_gb, so i can see why you want to change the computation logic","commit_id":"962fe29e778c58a8e90c78602e7249cb4b06e450"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"682dc4b8a438ec47ecf5813b8a3f5ba2a1133c83","unresolved":false,"context_lines":[{"line_number":8201,"context_line":"            self.assertFalse(stats[\u0027pools\u0027][0][\u0027QoS_support\u0027])"},{"line_number":8202,"context_line":"            self.assertEqual(86.0,"},{"line_number":8203,"context_line":"                             stats[\u0027pools\u0027][0][\u0027provisioned_capacity_gb\u0027])"},{"line_number":8204,"context_line":"            self.assertEqual(24.0, stats[\u0027pools\u0027][0][\u0027total_capacity_gb\u0027])"},{"line_number":8205,"context_line":"            self.assertEqual(3.0, stats[\u0027pools\u0027][0][\u0027free_capacity_gb\u0027])"},{"line_number":8206,"context_line":"            self.assertEqual(87.5, stats[\u0027pools\u0027][0][\u0027capacity_utilization\u0027])"},{"line_number":8207,"context_line":"            self.assertEqual(3, stats[\u0027pools\u0027][0][\u0027total_volumes\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"567afd27_da4f6478","side":"PARENT","line":8204,"in_reply_to":"6d54ad94_089bd39e","updated":"2025-01-21 10:44:36.000000000","message":"Acknowledged","commit_id":"962fe29e778c58a8e90c78602e7249cb4b06e450"}]}
