)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"df0c50f5ce0f6174dc0ba98926e56284440a5d04","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add video type virtio for AArch64"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Also tweak test cases for it."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-bug: #1710766"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f436f4f_e5ab8b72","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":29},"updated":"2017-08-21 18:01:31.000000000","message":"Please write the reason why we need to select virtio on AArch64 on the commit message.\nThe bug report just shows backtrace only, and this commit message is hard to understand it.","commit_id":"21a61231c4a44f76c937a54c5702e48aa7a1c965"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4bfe0591851e6e28c6c59c8447eb1dfc81f303c9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add video type virtio for AArch64"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently just virtio type can support on AArch64,"},{"line_number":10,"context_line":"\"virrus\", \"qxl\" and \"vga\" will not work on AArch64."},{"line_number":11,"context_line":"according to libvirt upstream:"},{"line_number":12,"context_line":"https://www.redhat.com/archives/libvir-list/2016-September/msg00546.html"},{"line_number":13,"context_line":"Also tweak test cases for it."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes-bug: #1710766"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7f287b81_debef5b2","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":29},"updated":"2017-08-25 23:08:05.000000000","message":"Here is just\n\n Currently only \"virtio\" type is supported on AArch64, and the\n other \"virrus\", \"qxl\" and \"vga\" don\u0027t work on AArch64 according to\n libvirt upstream:\n https://www.redhat.com/archives/libvir-list/2016-September/msg00546.html\n\n Then this patch adds the virtio for AArch64 and tweaks the related test cases.","commit_id":"1babb467adbbc32000989899f314d35a206951a1"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"00539b8bd614920b454e308aec9ce0ed288bff01","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add video type virtio for AArch64"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently only \"virtio\" type is supported on AArch64, and the"},{"line_number":10,"context_line":"other \"virrus\", \"qxl\" and \"vga\" don\u0027t work on AArch64 according to"},{"line_number":11,"context_line":"libvirt upstream:"},{"line_number":12,"context_line":"https://www.redhat.com/archives/libvir-list/2016-September/msg00546.html"},{"line_number":13,"context_line":"Then this patch adds the virtio for AArch64 and tweaks the related test cases."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7f287b81_78e0aecd","line":10,"range":{"start_line":10,"start_character":7,"end_line":10,"end_character":8},"updated":"2017-08-31 22:28:16.000000000","message":"Typo \"cirrus\"","commit_id":"f0f09530ee9169eb29bc28d4f118676d7dc6640e"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"d0ca1db46e09eaa437d69155e951e4463be73901","unresolved":false,"context_lines":[{"line_number":4362,"context_line":"            # so use \u0027vga\u0027 instead when running on Power hardware."},{"line_number":4363,"context_line":"            video.type \u003d \u0027vga\u0027"},{"line_number":4364,"context_line":"        elif guestarch in (fields.Architecture.AARCH64):"},{"line_number":4365,"context_line":"            # NOTE(kevinz): AArch64 doesn\u0027t support \u0027cirrus\u0027 be default"},{"line_number":4366,"context_line":"            # so use \u0027virtio\u0027 instead when running on AArch64 hardware."},{"line_number":4367,"context_line":"            video.type \u003d \u0027virtio\u0027"},{"line_number":4368,"context_line":"        elif CONF.spice.enabled:"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f287b81_9493c5a3","line":4365,"updated":"2017-08-25 08:23:21.000000000","message":"We could probably say that - \"Only virtio device type is supported by AARCH64\".","commit_id":"1babb467adbbc32000989899f314d35a206951a1"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4bfe0591851e6e28c6c59c8447eb1dfc81f303c9","unresolved":false,"context_lines":[{"line_number":4362,"context_line":"            # so use \u0027vga\u0027 instead when running on Power hardware."},{"line_number":4363,"context_line":"            video.type \u003d \u0027vga\u0027"},{"line_number":4364,"context_line":"        elif guestarch in (fields.Architecture.AARCH64):"},{"line_number":4365,"context_line":"            # NOTE(kevinz): AArch64 doesn\u0027t support \u0027cirrus\u0027 be default"},{"line_number":4366,"context_line":"            # so use \u0027virtio\u0027 instead when running on AArch64 hardware."},{"line_number":4367,"context_line":"            video.type \u003d \u0027virtio\u0027"},{"line_number":4368,"context_line":"        elif CONF.spice.enabled:"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f287b81_1eeb5ddc","line":4365,"in_reply_to":"7f287b81_9493c5a3","updated":"2017-08-25 23:08:05.000000000","message":"++","commit_id":"1babb467adbbc32000989899f314d35a206951a1"}]}
