)]}'
{"nova/tests/functional/libvirt/test_uefi.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"8d1a65fb3251f9c166397996ea645e1eddf4c162","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class UEFIServersTest(base.ServersTestBase):"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    # many move operations are admin-only"},{"line_number":34,"context_line":"    ADMIN_API \u003d True"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def assertInstanceHasUEFI(self, server):"},{"line_number":37,"context_line":"        ctx \u003d nova_context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":7,"id":"e3c231ad_3588d650","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":20},"updated":"2021-03-10 16:32:22.000000000","message":"nit - I assume you want to add these later but as there are none at the moment.","commit_id":"4d76a3dbb8b11ed9a47714180e25632a99757725"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"4734de2d2dcaad14de30b7d15a24496e76a8e622","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"class UEFIServersTest(base.ServersTestBase):"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    # many move operations are admin-only"},{"line_number":34,"context_line":"    ADMIN_API \u003d True"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def assertInstanceHasUEFI(self, server):"},{"line_number":37,"context_line":"        ctx \u003d nova_context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":7,"id":"43981e02_5d7d1003","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":20},"in_reply_to":"e3c231ad_3588d650","updated":"2021-03-10 19:33:02.000000000","message":"Whoops, no, that\u0027s copy-paste error from the TPM tests that I based this on. Will remove. We should add move operation tests but that requires something like https://review.opendev.org/c/openstack/nova/+/621646/ to copy the nvram (I considered this out-of-scope here since the series is already too big - will fix in a bugfix afterwards and backport)","commit_id":"4d76a3dbb8b11ed9a47714180e25632a99757725"}]}