)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1d374ad9ff29915ab0c4b785aa70cb7ca55fd333","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aefc00f3_723a6c2c","updated":"2025-02-20 13:16:57.000000000","message":"LGTM.  Thanks for the clear commit message and bug description.","commit_id":"6e83ac6eeee8f3a4b3265a4e927dca1bc190e088"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"0bb9c1f3090acfd98db05de0a2b3899c7733ed6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"99b8104a_5eab67b9","updated":"2025-02-20 14:32:50.000000000","message":"This grep usage looks like a good candidate to move to privsep instead of shelling out to run grep as a command.","commit_id":"6e83ac6eeee8f3a4b3265a4e927dca1bc190e088"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d35db5c1f7c67f10b912ca0b06dbd45e38f66436","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"99da07f9_fa0120bb","updated":"2025-02-20 13:33:59.000000000","message":"We need to remember to backport this everywhere the original went, which was to Antelope (which is Unmaintained, but the patch can be proposed and the Unmaintainers can deal with it).","commit_id":"6e83ac6eeee8f3a4b3265a4e927dca1bc190e088"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0a65a5dd9ca2d66c8880b0a4110312706b04796f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e9109fc2_091bb486","updated":"2025-02-20 14:08:29.000000000","message":"recheck 3 tests failed in lvm-lio-barbican job but looks unrelated to this patch\n\nsetUpClass (tempest.api.compute.floating_ips.test_floating_ips_actions\t\nsetUpClass (tempest.api.compute.images.test_list_image_filters\t\ntempest.api.compute.admin.test_migrations.MigrationsAdminTest\t\n\nfailure:\nurllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host\u003d\u0027213.32.73.17\u0027, port\u003d443): Read timed out. (read timeout\u003d90)","commit_id":"6e83ac6eeee8f3a4b3265a4e927dca1bc190e088"}],"os_brick/initiator/linuxfc.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1d374ad9ff29915ab0c4b785aa70cb7ca55fd333","unresolved":true,"context_lines":[{"line_number":145,"context_line":"        ctls \u003d []"},{"line_number":146,"context_line":"        luns_not_found \u003d set()"},{"line_number":147,"context_line":"        for wwpn, lun in targets:"},{"line_number":148,"context_line":"            try:"},{"line_number":149,"context_line":"                # Search for target in the fc_transport path first and if we"},{"line_number":150,"context_line":"                # don\u0027t find ctl, search for target in the fc_remote_ports path"},{"line_number":151,"context_line":"                ctl \u003d (self._get_target_fc_transport_path(path, wwpn, lun) or"}],"source_content_type":"text/x-python","patch_set":2,"id":"a01fcc1d_5da7729f","side":"PARENT","line":148,"range":{"start_line":148,"start_character":12,"end_line":148,"end_character":16},"updated":"2025-02-20 13:16:57.000000000","message":"I completely missed the implication of having the try here when I reviewed https://review.opendev.org/c/openstack/os-brick/+/906743 ... makes much more sense to have it in the functions.","commit_id":"44a40f8daeb9a341c6b6c045dcdc4c032d2070dd"}]}