)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"df72e0341e35a399c7e5d84176aa30cc969556c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"64c8cd21_543b2dc0","updated":"2025-02-27 15:24:05.000000000","message":"Need more work on it.","commit_id":"e0907508e5dcb29f32ce124ed247ef0ebb2083a1"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"174771923328be5f7eecae37751fdf0ac1e82724","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f5413d6d_de681231","updated":"2025-01-09 05:59:43.000000000","message":"Please do a review","commit_id":"e0907508e5dcb29f32ce124ed247ef0ebb2083a1"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"484a852162a11777013e3a37440e7063038957cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"305b6ea2_07c71fe4","updated":"2025-02-06 02:28:54.000000000","message":"Question inline.","commit_id":"e0907508e5dcb29f32ce124ed247ef0ebb2083a1"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"039afb4a6478b2342947e291e5802d4b8756d4c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f1f11d9e_fcb788b8","updated":"2025-01-29 15:46:22.000000000","message":"Since the PowerMax CI is currently down and will take some time to fix, I am attaching the local CI results to https://bugs.launchpad.net/cinder/+bug/2092261 for your review of the patch.","commit_id":"e0907508e5dcb29f32ce124ed247ef0ebb2083a1"}],"cinder/volume/drivers/dell_emc/powermax/masking.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"484a852162a11777013e3a37440e7063038957cd","unresolved":true,"context_lines":[{"line_number":2271,"context_line":"                    serial_number, child_sg_name, parent_sg_name,"},{"line_number":2272,"context_line":"                    extra_specs)"},{"line_number":2273,"context_line":""},{"line_number":2274,"context_line":"    @retry(retry_exc_tuple, interval\u003d2, retries\u003d10)"},{"line_number":2275,"context_line":"    def _check_device_moved_to_storage_group(self, device_id,"},{"line_number":2276,"context_line":"                                             target_storage_group_name,"},{"line_number":2277,"context_line":"                                             serial_number):"}],"source_content_type":"text/x-python","patch_set":1,"id":"138d1f56_f6122463","line":2274,"updated":"2025-02-06 02:28:54.000000000","message":"Like I said on one of your other patches, this is a really long time to be stuck in this function.  Would be good to know something about how you chose these numbers.","commit_id":"e0907508e5dcb29f32ce124ed247ef0ebb2083a1"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"24c00fa296efdd98014ee4163bc48a49c017944e","unresolved":true,"context_lines":[{"line_number":2271,"context_line":"                    serial_number, child_sg_name, parent_sg_name,"},{"line_number":2272,"context_line":"                    extra_specs)"},{"line_number":2273,"context_line":""},{"line_number":2274,"context_line":"    @retry(retry_exc_tuple, interval\u003d2, retries\u003d10)"},{"line_number":2275,"context_line":"    def _check_device_moved_to_storage_group(self, device_id,"},{"line_number":2276,"context_line":"                                             target_storage_group_name,"},{"line_number":2277,"context_line":"                                             serial_number):"}],"source_content_type":"text/x-python","patch_set":1,"id":"793f30a8_29582612","line":2274,"in_reply_to":"138d1f56_f6122463","updated":"2025-02-06 06:02:48.000000000","message":"Hi Brian,\n\nThanks for the update. Regarding retries, the response time varies—sometimes it succeeds in a single call, other times in two, and occasionally it takes up to six retries. Since there’s no fixed pattern for when we receive a response, I’ve set the retry value to 10 as a precaution and the six retries is occasionally it is not every we are getting.\n\nYes, waiting for more than half an hour for the retries is not reasonable, but I\u0027m confident it won\u0027t take that long. The maximum should be the seventh call. I set it to 10 as a precaution.","commit_id":"e0907508e5dcb29f32ce124ed247ef0ebb2083a1"}]}
