)]}'
{"cinder/image/image_utils.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f8e5b52a0102a712218690a1e7d7f79370c3534b","unresolved":true,"context_lines":[{"line_number":578,"context_line":"        except TypeError:"},{"line_number":579,"context_line":"            format_raw \u003d False"},{"line_number":580,"context_line":"        data \u003d get_qemu_data(image_id, has_meta, format_raw,"},{"line_number":581,"context_line":"                             dest, True)"},{"line_number":582,"context_line":"        # We can only really do verification of the image if we have"},{"line_number":583,"context_line":"        # qemu data to use"},{"line_number":584,"context_line":"        if data is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"e337ac69_e4330f61","line":581,"range":{"start_line":581,"start_character":35,"end_line":581,"end_character":39},"updated":"2021-06-18 14:25:30.000000000","message":"I think this would be more clear as\n\n  run_as_root\u003dTrue\n\n(although i guess you only need to scroll up a page to see what True does)","commit_id":"e96f48d3692d92e80cebc2ac607fb1782162fc37"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"ee0f5b404d9eb318b0f989ffde971248559df08a","unresolved":true,"context_lines":[{"line_number":578,"context_line":"        except TypeError:"},{"line_number":579,"context_line":"            format_raw \u003d False"},{"line_number":580,"context_line":"        data \u003d get_qemu_data(image_id, has_meta, format_raw,"},{"line_number":581,"context_line":"                             dest, True)"},{"line_number":582,"context_line":"        # We can only really do verification of the image if we have"},{"line_number":583,"context_line":"        # qemu data to use"},{"line_number":584,"context_line":"        if data is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fc4128e9_32e7f350","line":581,"range":{"start_line":581,"start_character":35,"end_line":581,"end_character":39},"in_reply_to":"e337ac69_e4330f61","updated":"2021-06-18 14:32:46.000000000","message":"Well, usually we do it that way because \"run_as_root\" is an argument with a default set, but in get_qemu_data it is a plain boolean that must be passed because it has no default.  That tells me it shouldn\u0027t be passed as a keyword...","commit_id":"e96f48d3692d92e80cebc2ac607fb1782162fc37"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f70cefa73b255b45415e1c76d6a383d0693d08db","unresolved":true,"context_lines":[{"line_number":578,"context_line":"        except TypeError:"},{"line_number":579,"context_line":"            format_raw \u003d False"},{"line_number":580,"context_line":"        data \u003d get_qemu_data(image_id, has_meta, format_raw,"},{"line_number":581,"context_line":"                             dest, True)"},{"line_number":582,"context_line":"        # We can only really do verification of the image if we have"},{"line_number":583,"context_line":"        # qemu data to use"},{"line_number":584,"context_line":"        if data is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f580f23_b7cc90c6","line":581,"range":{"start_line":581,"start_character":35,"end_line":581,"end_character":39},"in_reply_to":"fc4128e9_32e7f350","updated":"2021-06-18 14:36:22.000000000","message":"OK, thanks for looking.","commit_id":"e96f48d3692d92e80cebc2ac607fb1782162fc37"}]}