)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"5880a84aa85cc4c2af36e0fd3287a4bbcf39a96b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Lee Yarwood \u003clyarwood@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-04-02 13:23:49 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix multipath device discovery when UFN is enabled."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This currently returns an invalid path of `/dev/mapper/${WWID}`"},{"line_number":10,"context_line":"when UFN is enabled leading to failures later on when we attempt to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baa041b7_eec55a5b","line":7,"updated":"2015-04-08 11:02:25.000000000","message":"If you provide another patch, please, replace UFN with its full name \"user_friendly_names\".","commit_id":"e82b39baa4ef382415d54dc85b99fc2554ac56a7"}],"nova/storage/linuxscsi.py":[{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"d9e4417879592517d6a74d116444fead61cd441d","unresolved":false,"context_lines":[{"line_number":112,"context_line":""},{"line_number":113,"context_line":"            # Use the WWID, mpathN or custom alias to build the device path."},{"line_number":114,"context_line":"            # This should always be the first item on the first line."},{"line_number":115,"context_line":"            mdev \u003d \u0027/dev/mapper/%s\u0027 % line.split(\" \")[0]"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"            try:"},{"line_number":118,"context_line":"                os.stat(mdev)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa041b7_5b266dbb","line":115,"updated":"2015-04-01 21:12:52.000000000","message":"Your comment is correct, but unfortunately you still have to keep mdev_id, its used later in this method.","commit_id":"6433bcae025d9d2aba5c273817ce73c04f0fa31f"},{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"d9e4417879592517d6a74d116444fead61cd441d","unresolved":false,"context_lines":[{"line_number":144,"context_line":"                devices.append(dev)"},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"    if mdev is not None:"},{"line_number":147,"context_line":"        info \u003d {\"device\": mdev,"},{"line_number":148,"context_line":"                \"id\": mdev_id,"},{"line_number":149,"context_line":"                \"devices\": devices}"},{"line_number":150,"context_line":"        return info"}],"source_content_type":"text/x-python","patch_set":1,"id":"baa041b7_5818b384","line":147,"updated":"2015-04-01 21:12:52.000000000","message":"You removed mdev_id!","commit_id":"6433bcae025d9d2aba5c273817ce73c04f0fa31f"},{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"4e5e4c1c540af8c7862a31c3ff440827e6dcb32d","unresolved":false,"context_lines":[{"line_number":112,"context_line":"            # Use the WWID, mpathN or custom alias to build the device path."},{"line_number":113,"context_line":"            # This should always be the first item on the first line of"},{"line_number":114,"context_line":"            # output from `multipath -l /dev/${path}`."},{"line_number":115,"context_line":"            mdev_id \u003d lines[0].split(\" \")[0]"},{"line_number":116,"context_line":"            mdev \u003d \u0027/dev/mapper/%s\u0027 % mdev_id"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_c334c1df","line":115,"updated":"2015-04-02 11:08:01.000000000","message":"mpathN is not id, I would always use wwid as id, but build path with mpathN if exists.","commit_id":"5ad73914d39d450022ff6315094e329b50b11ad5"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"96eeb37b958822970f5090b76b8239849cbd1dea","unresolved":false,"context_lines":[{"line_number":112,"context_line":"            # Use the WWID, mpathN or custom alias to build the device path."},{"line_number":113,"context_line":"            # This should always be the first item on the first line of"},{"line_number":114,"context_line":"            # output from `multipath -l /dev/${path}`."},{"line_number":115,"context_line":"            mdev_id \u003d lines[0].split(\" \")[0]"},{"line_number":116,"context_line":"            mdev \u003d \u0027/dev/mapper/%s\u0027 % mdev_id"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_39827c01","line":115,"in_reply_to":"baa041b7_c334c1df","updated":"2015-04-02 12:13:42.000000000","message":"Yup valid point, this is actually the device name. Let me rework this quickly.","commit_id":"5ad73914d39d450022ff6315094e329b50b11ad5"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"c5cb9e43e2124b2d8c03445967f2b725e738f5c3","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                return None"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"            LOG.debug(\"Found multipath device \u003d %s\", mdev)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"            device_lines \u003d lines[3:]"},{"line_number":138,"context_line":"            for dev_line in device_lines:"},{"line_number":139,"context_line":"                if dev_line.find(\"policy\") !\u003d -1:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_2231ec05","line":136,"updated":"2015-04-08 13:46:17.000000000","message":"Please don\u0027t add random whitespace.","commit_id":"e82b39baa4ef382415d54dc85b99fc2554ac56a7"}]}