)]}'
{"sushy_tools/emulator/drivers/libvirtdriver.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"5dab73ae6ab52720239347fdb94d1091f8f266e8","unresolved":false,"context_lines":[{"line_number":80,"context_line":"    BOOT_MODE_MAP_REV \u003d {v: k for k, v in BOOT_MODE_MAP.items()}"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    # NOTE(etingof): we have these firmware blobs hardcoded here"},{"line_number":83,"context_line":"    # which seems to work for CentOS, for instance."},{"line_number":84,"context_line":"    # What\u0027s not clear is how to adapt to possible different"},{"line_number":85,"context_line":"    # boot loaders paths dependent on libvirt packaging and custom"},{"line_number":86,"context_line":"    # domain configuration..."}],"source_content_type":"text/x-python","patch_set":6,"id":"3f79a3b5_ca5d9354","line":83,"range":{"start_line":83,"start_character":29,"end_line":83,"end_character":37},"updated":"2018-11-27 23:22:33.000000000","message":"I feel like they are different on ubuntu, but I think this works for now. Thanks!","commit_id":"c09ce2ce75ff712b46267c359c96296d6c923d77"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"02d88160edf7e376695a353b603da0285121f4aa","unresolved":false,"context_lines":[{"line_number":80,"context_line":"    BOOT_MODE_MAP_REV \u003d {v: k for k, v in BOOT_MODE_MAP.items()}"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    # NOTE(etingof): we have these firmware blobs hardcoded here"},{"line_number":83,"context_line":"    # which seems to work for CentOS, for instance."},{"line_number":84,"context_line":"    # What\u0027s not clear is how to adapt to possible different"},{"line_number":85,"context_line":"    # boot loaders paths dependent on libvirt packaging and custom"},{"line_number":86,"context_line":"    # domain configuration..."}],"source_content_type":"text/x-python","patch_set":6,"id":"3f79a3b5_6b555750","line":83,"range":{"start_line":83,"start_character":29,"end_line":83,"end_character":37},"in_reply_to":"3f79a3b5_ca5d9354","updated":"2018-11-28 13:57:52.000000000","message":"Hopefully addressed in [1].\n\n1.  https://review.openstack.org/620605","commit_id":"c09ce2ce75ff712b46267c359c96296d6c923d77"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"0110f7ac0bcf6fe52db57de76c0c4966a59d42a1","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    # which seems to work for CentOS, for instance."},{"line_number":84,"context_line":"    # What\u0027s not clear is how to adapt to possible different"},{"line_number":85,"context_line":"    # boot loaders paths dependent on libvirt packaging and custom"},{"line_number":86,"context_line":"    # domain configuration..."},{"line_number":87,"context_line":"    BOOT_LOADER_MAP \u003d {"},{"line_number":88,"context_line":"        \u0027Uefi\u0027: {"},{"line_number":89,"context_line":"            \u0027x86_64\u0027: \u0027/usr/share/OVMF/OVMF_CODE.fd\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3f79a3b5_a173951b","line":86,"updated":"2018-11-28 09:33:59.000000000","message":"Probably a configuration file (that you seem to introduce in one of your patches anyway)","commit_id":"c09ce2ce75ff712b46267c359c96296d6c923d77"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"02d88160edf7e376695a353b603da0285121f4aa","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    # which seems to work for CentOS, for instance."},{"line_number":84,"context_line":"    # What\u0027s not clear is how to adapt to possible different"},{"line_number":85,"context_line":"    # boot loaders paths dependent on libvirt packaging and custom"},{"line_number":86,"context_line":"    # domain configuration..."},{"line_number":87,"context_line":"    BOOT_LOADER_MAP \u003d {"},{"line_number":88,"context_line":"        \u0027Uefi\u0027: {"},{"line_number":89,"context_line":"            \u0027x86_64\u0027: \u0027/usr/share/OVMF/OVMF_CODE.fd\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3f79a3b5_2b4bdfb1","line":86,"in_reply_to":"3f79a3b5_a173951b","updated":"2018-11-28 13:57:52.000000000","message":"https://review.openstack.org/620605","commit_id":"c09ce2ce75ff712b46267c359c96296d6c923d77"}]}