)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"11c398a8961de8b60a37ec602a4bfc36641f57ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a42b1c8e_0989a988","updated":"2026-02-11 03:50:04.000000000","message":"LGTM","commit_id":"ec029fb1d49741986687afd85cbda88f1a18838d"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"8d5d2eb691e6392ccb6a37a4c69195032c003044","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9300f824_be9446af","updated":"2026-02-10 17:40:25.000000000","message":"We need https://review.opendev.org/c/openstack/requirements/+/976259 to merge for the grenade job to pass here.","commit_id":"ec029fb1d49741986687afd85cbda88f1a18838d"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"be7605bc11a08ddccf81a42bed0a7e942ce861df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ad0acd04_9f87e2c1","updated":"2026-02-10 13:58:55.000000000","message":"recheck","commit_id":"ec029fb1d49741986687afd85cbda88f1a18838d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"dfb0c8c7addbd9a30827e2230c9fefc35dbe879a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"adc7b340_5d45ec1f","updated":"2026-02-13 17:57:36.000000000","message":"Thanks, Takashi! This is a big patch, and we appreciate you taking it on.\n\nA question about something I noticed inline; otherwise, everything looks ok to me.","commit_id":"2db0584eb8a2abc12a7c59616dfa5d86c434b9c9"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"86e7625cad40a6c590b1bff790333250fc29d9ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"658ce5b0_b0c580f7","updated":"2026-02-13 20:37:50.000000000","message":"recheck openstack-tox-py313 timed out during fc zonemanager tests, possibly something not mocked correctly, pretty sure it\u0027s unrelated to this patch","commit_id":"2db0584eb8a2abc12a7c59616dfa5d86c434b9c9"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"043fa8d27e4bbd26609f23a994e19272f9e094f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"5ad5ce14_1f837752","updated":"2026-02-16 14:31:18.000000000","message":"My concern has been addressed, LGTM.  Thanks, Takashi!\n\nRaising to top priority because the cross-cinder-py312 job is being made non-voting [0] until this patch has merged, which basically leaves us unprotected.\n\n[0] https://review.opendev.org/c/openstack/requirements/+/976839","commit_id":"143c5c12fb60ef7130ddff0fc9b9ec23bc452d49"}],"cinder/service.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"dfb0c8c7addbd9a30827e2230c9fefc35dbe879a","unresolved":true,"context_lines":[{"line_number":643,"context_line":""},{"line_number":644,"context_line":""},{"line_number":645,"context_line":"def get_launcher() -\u003e service.ProcessLauncher:"},{"line_number":646,"context_line":"    return Launcher()"}],"source_content_type":"text/x-python","patch_set":8,"id":"70b16fb1_3adb819e","line":646,"range":{"start_line":646,"start_character":4,"end_line":646,"end_character":21},"updated":"2026-02-13 17:57:36.000000000","message":"in the old code, when os.name !\u003d \u0027nt\u0027, we were returning process_launcher() ... shouldn\u0027t we continue to do that?\n\nI think we can remove the Launcher class, which was added for windows by change https://review.opendev.org/#/q/I46c6b332e614b6 (or if that seems too drastic for this pach, you could add a TODO to look into it later)","commit_id":"2db0584eb8a2abc12a7c59616dfa5d86c434b9c9"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8da5c9ea6c2be53aa5782487fd45ec54d033c47c","unresolved":false,"context_lines":[{"line_number":643,"context_line":""},{"line_number":644,"context_line":""},{"line_number":645,"context_line":"def get_launcher() -\u003e service.ProcessLauncher:"},{"line_number":646,"context_line":"    return Launcher()"}],"source_content_type":"text/x-python","patch_set":8,"id":"b8484dc6_7a77fc39","line":646,"range":{"start_line":646,"start_character":4,"end_line":646,"end_character":21},"in_reply_to":"70b16fb1_3adb819e","updated":"2026-02-15 04:31:14.000000000","message":"Oh yes. It\u0027s a good spot. I guess oslo.service eventually selected ProcessLaundher internally and this explicit setting is not needed, but I\u0027ll leave that change now for safety.","commit_id":"2db0584eb8a2abc12a7c59616dfa5d86c434b9c9"}]}