)]}'
{"nova/tests/unit/virt/libvirt/fake_libvirt_data.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8a29c61f70fa22366b0b39c5714b5483ab1cd059","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"CAPABILITIES_GUEST \u003d {"},{"line_number":52,"context_line":"        \u0027i686\u0027: \u0027\u0027\u0027"},{"line_number":53,"context_line":"  \u003cguest\u003e"},{"line_number":54,"context_line":"    \u003cos_type\u003ehvm\u003c/os_type\u003e"},{"line_number":55,"context_line":"    \u003carch name\u003d\u0027i686\u0027\u003e"},{"line_number":56,"context_line":"      \u003cwordsize\u003e32\u003c/wordsize\u003e"},{"line_number":57,"context_line":"      \u003cemulator\u003e/usr/bin/qemu\u003c/emulator\u003e"},{"line_number":58,"context_line":"      \u003cmachine\u003epc-0.14\u003c/machine\u003e"},{"line_number":59,"context_line":"      \u003cmachine canonical\u003d\u0027pc-0.14\u0027\u003epc\u003c/machine\u003e"},{"line_number":60,"context_line":"      \u003cmachine\u003epc-0.13\u003c/machine\u003e"},{"line_number":61,"context_line":"      \u003cmachine\u003epc-0.12\u003c/machine\u003e"},{"line_number":62,"context_line":"      \u003cmachine\u003epc-0.11\u003c/machine\u003e"},{"line_number":63,"context_line":"      \u003cmachine\u003epc-0.10\u003c/machine\u003e"},{"line_number":64,"context_line":"      \u003cmachine\u003eisapc\u003c/machine\u003e"},{"line_number":65,"context_line":"      \u003cdomain type\u003d\u0027qemu\u0027\u003e"},{"line_number":66,"context_line":"      \u003c/domain\u003e"},{"line_number":67,"context_line":"      \u003cdomain type\u003d\u0027kvm\u0027\u003e"},{"line_number":68,"context_line":"        \u003cemulator\u003e/usr/bin/kvm\u003c/emulator\u003e"},{"line_number":69,"context_line":"        \u003cmachine\u003epc-0.14\u003c/machine\u003e"},{"line_number":70,"context_line":"        \u003cmachine canonical\u003d\u0027pc-0.14\u0027\u003epc\u003c/machine\u003e"},{"line_number":71,"context_line":"        \u003cmachine\u003epc-0.13\u003c/machine\u003e"},{"line_number":72,"context_line":"        \u003cmachine\u003epc-0.12\u003c/machine\u003e"},{"line_number":73,"context_line":"        \u003cmachine\u003epc-0.11\u003c/machine\u003e"},{"line_number":74,"context_line":"        \u003cmachine\u003epc-0.10\u003c/machine\u003e"},{"line_number":75,"context_line":"        \u003cmachine\u003eisapc\u003c/machine\u003e"},{"line_number":76,"context_line":"      \u003c/domain\u003e"},{"line_number":77,"context_line":"    \u003c/arch\u003e"},{"line_number":78,"context_line":"    \u003cfeatures\u003e"},{"line_number":79,"context_line":"      \u003ccpuselection/\u003e"},{"line_number":80,"context_line":"      \u003cdeviceboot/\u003e"},{"line_number":81,"context_line":"      \u003cpae/\u003e"},{"line_number":82,"context_line":"      \u003cnonpae/\u003e"},{"line_number":83,"context_line":"      \u003cacpi default\u003d\u0027on\u0027 toggle\u003d\u0027yes\u0027/\u003e"},{"line_number":84,"context_line":"      \u003capic default\u003d\u0027on\u0027 toggle\u003d\u0027no\u0027/\u003e"},{"line_number":85,"context_line":"    \u003c/features\u003e"},{"line_number":86,"context_line":"  \u003c/guest\u003e\u0027\u0027\u0027,"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        \u0027x86_64\u0027: \u0027\u0027\u0027"},{"line_number":89,"context_line":"  \u003cguest\u003e"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_d32a148f","line":86,"range":{"start_line":53,"start_character":0,"end_line":86,"end_character":14},"updated":"2019-08-21 16:42:25.000000000","message":"style nit: I\u0027d really liked to have seen these indented and just wrapped them in \u0027textwrap.dedent\u0027, like so:\n\n  CAPABILITIES_GUEST \u003d {\n      \u0027i686\u0027: textwrap.dedent(\"\"\"\n          \u003cguest\u003e\n            ...\n          \u003c/guest\u003e\n       \"\"\".strip(),\n       \u0027x86_64\u0027: textwrap.dedent(\"\"\"\n           ...\n       \"\"\".strip(),\n       ...\n  }\n\nIt just makes this whole thing a whole lot easier to read. Doesn\u0027t need to be done now, but perhaps in a follow-up?","commit_id":"415ed543dde3dbec7a4592aba9faf381091a4651"},{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"44ac44df39bb3478f89b88f247f354e095d9affb","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"CAPABILITIES_GUEST \u003d {"},{"line_number":52,"context_line":"        \u0027i686\u0027: \u0027\u0027\u0027"},{"line_number":53,"context_line":"  \u003cguest\u003e"},{"line_number":54,"context_line":"    \u003cos_type\u003ehvm\u003c/os_type\u003e"},{"line_number":55,"context_line":"    \u003carch name\u003d\u0027i686\u0027\u003e"},{"line_number":56,"context_line":"      \u003cwordsize\u003e32\u003c/wordsize\u003e"},{"line_number":57,"context_line":"      \u003cemulator\u003e/usr/bin/qemu\u003c/emulator\u003e"},{"line_number":58,"context_line":"      \u003cmachine\u003epc-0.14\u003c/machine\u003e"},{"line_number":59,"context_line":"      \u003cmachine canonical\u003d\u0027pc-0.14\u0027\u003epc\u003c/machine\u003e"},{"line_number":60,"context_line":"      \u003cmachine\u003epc-0.13\u003c/machine\u003e"},{"line_number":61,"context_line":"      \u003cmachine\u003epc-0.12\u003c/machine\u003e"},{"line_number":62,"context_line":"      \u003cmachine\u003epc-0.11\u003c/machine\u003e"},{"line_number":63,"context_line":"      \u003cmachine\u003epc-0.10\u003c/machine\u003e"},{"line_number":64,"context_line":"      \u003cmachine\u003eisapc\u003c/machine\u003e"},{"line_number":65,"context_line":"      \u003cdomain type\u003d\u0027qemu\u0027\u003e"},{"line_number":66,"context_line":"      \u003c/domain\u003e"},{"line_number":67,"context_line":"      \u003cdomain type\u003d\u0027kvm\u0027\u003e"},{"line_number":68,"context_line":"        \u003cemulator\u003e/usr/bin/kvm\u003c/emulator\u003e"},{"line_number":69,"context_line":"        \u003cmachine\u003epc-0.14\u003c/machine\u003e"},{"line_number":70,"context_line":"        \u003cmachine canonical\u003d\u0027pc-0.14\u0027\u003epc\u003c/machine\u003e"},{"line_number":71,"context_line":"        \u003cmachine\u003epc-0.13\u003c/machine\u003e"},{"line_number":72,"context_line":"        \u003cmachine\u003epc-0.12\u003c/machine\u003e"},{"line_number":73,"context_line":"        \u003cmachine\u003epc-0.11\u003c/machine\u003e"},{"line_number":74,"context_line":"        \u003cmachine\u003epc-0.10\u003c/machine\u003e"},{"line_number":75,"context_line":"        \u003cmachine\u003eisapc\u003c/machine\u003e"},{"line_number":76,"context_line":"      \u003c/domain\u003e"},{"line_number":77,"context_line":"    \u003c/arch\u003e"},{"line_number":78,"context_line":"    \u003cfeatures\u003e"},{"line_number":79,"context_line":"      \u003ccpuselection/\u003e"},{"line_number":80,"context_line":"      \u003cdeviceboot/\u003e"},{"line_number":81,"context_line":"      \u003cpae/\u003e"},{"line_number":82,"context_line":"      \u003cnonpae/\u003e"},{"line_number":83,"context_line":"      \u003cacpi default\u003d\u0027on\u0027 toggle\u003d\u0027yes\u0027/\u003e"},{"line_number":84,"context_line":"      \u003capic default\u003d\u0027on\u0027 toggle\u003d\u0027no\u0027/\u003e"},{"line_number":85,"context_line":"    \u003c/features\u003e"},{"line_number":86,"context_line":"  \u003c/guest\u003e\u0027\u0027\u0027,"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        \u0027x86_64\u0027: \u0027\u0027\u0027"},{"line_number":89,"context_line":"  \u003cguest\u003e"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_99a50fb7","line":86,"range":{"start_line":53,"start_character":0,"end_line":86,"end_character":14},"in_reply_to":"7faddb67_84bc50dc","updated":"2019-08-29 20:46:22.000000000","message":"Fixed in https://review.opendev.org/#/c/679339/","commit_id":"415ed543dde3dbec7a4592aba9faf381091a4651"},{"author":{"_account_id":2394,"name":"Adam Spiers","email":"aspiers@suse.com","username":"adam.spiers"},"change_message_id":"5a2c944efa280f31130074ca28011ef16185f18e","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"CAPABILITIES_GUEST \u003d {"},{"line_number":52,"context_line":"        \u0027i686\u0027: \u0027\u0027\u0027"},{"line_number":53,"context_line":"  \u003cguest\u003e"},{"line_number":54,"context_line":"    \u003cos_type\u003ehvm\u003c/os_type\u003e"},{"line_number":55,"context_line":"    \u003carch name\u003d\u0027i686\u0027\u003e"},{"line_number":56,"context_line":"      \u003cwordsize\u003e32\u003c/wordsize\u003e"},{"line_number":57,"context_line":"      \u003cemulator\u003e/usr/bin/qemu\u003c/emulator\u003e"},{"line_number":58,"context_line":"      \u003cmachine\u003epc-0.14\u003c/machine\u003e"},{"line_number":59,"context_line":"      \u003cmachine canonical\u003d\u0027pc-0.14\u0027\u003epc\u003c/machine\u003e"},{"line_number":60,"context_line":"      \u003cmachine\u003epc-0.13\u003c/machine\u003e"},{"line_number":61,"context_line":"      \u003cmachine\u003epc-0.12\u003c/machine\u003e"},{"line_number":62,"context_line":"      \u003cmachine\u003epc-0.11\u003c/machine\u003e"},{"line_number":63,"context_line":"      \u003cmachine\u003epc-0.10\u003c/machine\u003e"},{"line_number":64,"context_line":"      \u003cmachine\u003eisapc\u003c/machine\u003e"},{"line_number":65,"context_line":"      \u003cdomain type\u003d\u0027qemu\u0027\u003e"},{"line_number":66,"context_line":"      \u003c/domain\u003e"},{"line_number":67,"context_line":"      \u003cdomain type\u003d\u0027kvm\u0027\u003e"},{"line_number":68,"context_line":"        \u003cemulator\u003e/usr/bin/kvm\u003c/emulator\u003e"},{"line_number":69,"context_line":"        \u003cmachine\u003epc-0.14\u003c/machine\u003e"},{"line_number":70,"context_line":"        \u003cmachine canonical\u003d\u0027pc-0.14\u0027\u003epc\u003c/machine\u003e"},{"line_number":71,"context_line":"        \u003cmachine\u003epc-0.13\u003c/machine\u003e"},{"line_number":72,"context_line":"        \u003cmachine\u003epc-0.12\u003c/machine\u003e"},{"line_number":73,"context_line":"        \u003cmachine\u003epc-0.11\u003c/machine\u003e"},{"line_number":74,"context_line":"        \u003cmachine\u003epc-0.10\u003c/machine\u003e"},{"line_number":75,"context_line":"        \u003cmachine\u003eisapc\u003c/machine\u003e"},{"line_number":76,"context_line":"      \u003c/domain\u003e"},{"line_number":77,"context_line":"    \u003c/arch\u003e"},{"line_number":78,"context_line":"    \u003cfeatures\u003e"},{"line_number":79,"context_line":"      \u003ccpuselection/\u003e"},{"line_number":80,"context_line":"      \u003cdeviceboot/\u003e"},{"line_number":81,"context_line":"      \u003cpae/\u003e"},{"line_number":82,"context_line":"      \u003cnonpae/\u003e"},{"line_number":83,"context_line":"      \u003cacpi default\u003d\u0027on\u0027 toggle\u003d\u0027yes\u0027/\u003e"},{"line_number":84,"context_line":"      \u003capic default\u003d\u0027on\u0027 toggle\u003d\u0027no\u0027/\u003e"},{"line_number":85,"context_line":"    \u003c/features\u003e"},{"line_number":86,"context_line":"  \u003c/guest\u003e\u0027\u0027\u0027,"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        \u0027x86_64\u0027: \u0027\u0027\u0027"},{"line_number":89,"context_line":"  \u003cguest\u003e"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_84bc50dc","line":86,"range":{"start_line":53,"start_character":0,"end_line":86,"end_character":14},"in_reply_to":"7faddb67_d32a148f","updated":"2019-08-21 18:59:17.000000000","message":"Ah yeah, nice idea thanks!  This was bothering me but somehow I had forgotten about dedent.","commit_id":"415ed543dde3dbec7a4592aba9faf381091a4651"}]}
