)]}'
{"os_brick/initiator/linuxscsi.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"f1cfa0372c43b73fb4cf9002d7bf4380f837792b","unresolved":false,"context_lines":[{"line_number":243,"context_line":"            except OSError:"},{"line_number":244,"context_line":"                # A race condition in Python\u0027s posixpath:realpath just occurred"},{"line_number":245,"context_line":"                # so we can ignore it because the file was just removed between"},{"line_number":246,"context_line":"                # a check if file exists and a call to os.readlink"},{"line_number":247,"context_line":"                continue"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"        if unlink:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_8795cf42","line":246,"updated":"2017-09-26 19:36:28.000000000","message":"Should we log and/or check the actual OSError to know if it was a race or some other (permission?) issue?","commit_id":"de89635ab70b36852d638efd6e45d245217613cc"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1fa5d971b5bc35931c6e01807c022204c9c4e25a","unresolved":false,"context_lines":[{"line_number":243,"context_line":"            except OSError:"},{"line_number":244,"context_line":"                # A race condition in Python\u0027s posixpath:realpath just occurred"},{"line_number":245,"context_line":"                # so we can ignore it because the file was just removed between"},{"line_number":246,"context_line":"                # a check if file exists and a call to os.readlink"},{"line_number":247,"context_line":"                continue"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"        if unlink:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_eb7f9012","line":246,"in_reply_to":"7f515b1d_8795cf42","updated":"2017-09-27 16:39:50.000000000","message":"I don\u0027t think we do, because if link doesn\u0027t exist it returns the same string, if it exists then /dev/disk/by-id should be available for read and execute for everybody, and the same should happen with the links inside it we are checking.","commit_id":"de89635ab70b36852d638efd6e45d245217613cc"}]}