)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"c6c62e08017acdcecf01862b604802a7d6ac9ba9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c19108e3_6023d930","updated":"2023-11-15 04:24:04.000000000","message":"Existing messages tend to end with a period, but I don\u0027t care too much.","commit_id":"7da3d82773fe68f7ccb2187ceafed012d145799f"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ba6d191636b9d3f062594293f82353d07d44a3fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1aef9d4f_78a6e701","updated":"2024-01-05 00:19:16.000000000","message":"LGTM.","commit_id":"7da3d82773fe68f7ccb2187ceafed012d145799f"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"6996cf08f95a6923fabc89e358c8c2577d50c725","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f38fc5c7_fc646339","updated":"2024-01-06 09:38:26.000000000","message":"one question inline but overall LGTM.","commit_id":"7da3d82773fe68f7ccb2187ceafed012d145799f"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"55ede02f9d98ebf9736b3601506f0eb2d1d30e4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7d5bb827_f728c33b","updated":"2023-12-28 14:38:22.000000000","message":"run Pure Storage CI","commit_id":"7da3d82773fe68f7ccb2187ceafed012d145799f"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"c221dc9f10916531290c1cb23e1136c7a7bb1829","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ee1b2dd7_9fdd277f","updated":"2024-01-05 19:55:56.000000000","message":"run Pure Storage CI","commit_id":"7da3d82773fe68f7ccb2187ceafed012d145799f"}],"os_brick/privileged/nvmeof.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ba6d191636b9d3f062594293f82353d07d44a3fa","unresolved":true,"context_lines":[{"line_number":50,"context_line":"        # This is different from OSError\u0027s ENOENT, which is missing nvme"},{"line_number":51,"context_line":"        # command.  This ENOENT is when nvme says there isn\u0027t an nqn."},{"line_number":52,"context_line":"        except putils.ProcessExecutionError as e:"},{"line_number":53,"context_line":"            # nvme-cli\u0027s error are all over the place, so merge the output"},{"line_number":54,"context_line":"            err_msg \u003d e.stdout + \u0027\\n\u0027 + e.stderr"},{"line_number":55,"context_line":"            msg \u003d err_msg.casefold()"},{"line_number":56,"context_line":"            if \u0027error: invalid sub-command\u0027 in msg:"}],"source_content_type":"text/x-python","patch_set":1,"id":"e1214e80_f41e63eb","line":53,"updated":"2024-01-05 00:19:16.000000000","message":"This is definitely a good move!","commit_id":"7da3d82773fe68f7ccb2187ceafed012d145799f"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"6996cf08f95a6923fabc89e358c8c2577d50c725","unresolved":true,"context_lines":[{"line_number":55,"context_line":"            msg \u003d err_msg.casefold()"},{"line_number":56,"context_line":"            if \u0027error: invalid sub-command\u0027 in msg:"},{"line_number":57,"context_line":"                LOG.debug(\u0027Version too old cannot check current hostnqn.\u0027)"},{"line_number":58,"context_line":"            elif \u0027hostnqn is not available\u0027 in msg:"},{"line_number":59,"context_line":"                LOG.debug(\u0027Version too old to return hostnqn from non file \u0027"},{"line_number":60,"context_line":"                          \u0027sources\u0027)"},{"line_number":61,"context_line":"            elif e.exit_code \u003d\u003d errno.ENOENT:"}],"source_content_type":"text/x-python","patch_set":1,"id":"af0b02fa_208466ef","line":58,"range":{"start_line":58,"start_character":18,"end_line":58,"end_character":42},"updated":"2024-01-06 09:38:26.000000000","message":"out of curiosity, why are we not checking exit code 254 and instead matching the string here? I\u0027m sure there might be a reason but it\u0027s not obvious to me","commit_id":"7da3d82773fe68f7ccb2187ceafed012d145799f"}]}