)]}'
{".zuul.yaml":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"d99bbeaa1d479917ff8c39b4880da373d9f220a6","unresolved":true,"context_lines":[{"line_number":531,"context_line":"        - devstack-platform-fedora-latest-virt-preview:"},{"line_number":532,"context_line":"            irrelevant-files: *dsvm-irrelevant-files"},{"line_number":533,"context_line":"        - devstack-plugin-ceph-compute-local-ephemeral:"},{"line_number":534,"context_line":"            irrelevant-files: *dsvm-irrelevant-files"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"8db00efe_d13177bc","side":"PARENT","line":534,"updated":"2021-02-16 13:18:07.000000000","message":"I guess these are removed only temporarily while you are working on this patch series to ease the load on the gate.","commit_id":"4af54cd38cb2454b73b4cc00f1233c83f5da5ce7"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"a6801022f2f05dc17728125bab17d236e2ff23b1","unresolved":true,"context_lines":[{"line_number":531,"context_line":"        - devstack-platform-fedora-latest-virt-preview:"},{"line_number":532,"context_line":"            irrelevant-files: *dsvm-irrelevant-files"},{"line_number":533,"context_line":"        - devstack-plugin-ceph-compute-local-ephemeral:"},{"line_number":534,"context_line":"            irrelevant-files: *dsvm-irrelevant-files"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"00a2b2de_72c2d5d8","side":"PARENT","line":534,"in_reply_to":"8db00efe_d13177bc","updated":"2021-02-16 13:46:48.000000000","message":"Yup correct, trying to avoid dansmith shouting at me ;)","commit_id":"4af54cd38cb2454b73b4cc00f1233c83f5da5ce7"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"d99bbeaa1d479917ff8c39b4880da373d9f220a6","unresolved":true,"context_lines":[{"line_number":227,"context_line":"              ide_bus: False"},{"line_number":228,"context_line":"              # Cirros doesn\u0027t support a SATA based cdrom device at present"},{"line_number":229,"context_line":"              # as used when the q35 machine type is being used by the guest."},{"line_number":230,"context_line":"              config_drive: False"},{"line_number":231,"context_line":"      devstack_localrc:"},{"line_number":232,"context_line":"        # Added in Pike."},{"line_number":233,"context_line":"        NOVA_USE_SERVICE_TOKEN: True"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"645d6545_cc139a0e","line":230,"updated":"2021-02-16 13:18:07.000000000","message":"hm, if we disable these then we loose test coverage isn\u0027t it? Can we add a test job where these are enabled while the hw_machine_type is still pc?\n\nOr are these cases covered by other running jobs already?","commit_id":"4874b428ec5af32b35d1ec48398f3372bbac3818"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"a6801022f2f05dc17728125bab17d236e2ff23b1","unresolved":true,"context_lines":[{"line_number":227,"context_line":"              ide_bus: False"},{"line_number":228,"context_line":"              # Cirros doesn\u0027t support a SATA based cdrom device at present"},{"line_number":229,"context_line":"              # as used when the q35 machine type is being used by the guest."},{"line_number":230,"context_line":"              config_drive: False"},{"line_number":231,"context_line":"      devstack_localrc:"},{"line_number":232,"context_line":"        # Added in Pike."},{"line_number":233,"context_line":"        NOVA_USE_SERVICE_TOKEN: True"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"62168e12_9b18e719","line":230,"in_reply_to":"645d6545_cc139a0e","updated":"2021-02-16 13:46:48.000000000","message":"Yes both cases are already well covered for pc and the config_drive part of this is something I hope to address within the Cirros image shortly. I should leave a TODO here highlighting that.","commit_id":"4874b428ec5af32b35d1ec48398f3372bbac3818"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"2bfc2b4528ccd37f54712b594dbf1517a9fcc909","unresolved":true,"context_lines":[{"line_number":239,"context_line":"        # Added in Ussuri."},{"line_number":240,"context_line":"        FORCE_CONFIG_DRIVE: True"},{"line_number":241,"context_line":"        # Added in Wallaby"},{"line_number":242,"context_line":"        # TODO(lyarwood): Remove the use of this image with Cirros \u003e\u003d 5.0.2"},{"line_number":243,"context_line":"        DOWNLOAD_DEFAULT_IMAGES: False"},{"line_number":244,"context_line":"        DEFAULT_IMAGE_NAME: cirros-0.5.1-dev-ahci-x86_64-disk"},{"line_number":245,"context_line":"        DEFAULT_IMAGE_FILE_NAME: cirros-0.5.1-dev-ahci-x86_64-disk.img"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"f336d683_72a4dbdf","line":242,"range":{"start_line":242,"start_character":70,"end_line":242,"end_character":75},"updated":"2021-03-04 14:59:40.000000000","message":"0.5.2","commit_id":"c06a2a6d743e9d944b5cb149e56b13c446f3944e"},{"author":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"change_message_id":"ff97bda867d2c5cd2f6f2b9d84bcb112c01cf587","unresolved":true,"context_lines":[{"line_number":202,"context_line":"        post-config:"},{"line_number":203,"context_line":"          $NOVA_CPU_CONF:"},{"line_number":204,"context_line":"            libvirt:"},{"line_number":205,"context_line":"              num_pcie_ports: 24"},{"line_number":206,"context_line":"              hw_machine_type: \"x86_64\u003dq35\""},{"line_number":207,"context_line":"            compute:"},{"line_number":208,"context_line":"              # Switch off the provider association refresh, which should"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"7ae006cf_95d03738","line":205,"range":{"start_line":205,"start_character":14,"end_line":205,"end_character":32},"updated":"2021-03-05 10:57:25.000000000","message":"It can be useful to add a note about why we\u0027re doing this.  Something like:\n\n# When using Q35, PCIe \"root ports\" need to be explicitly configured.\n# These will be used for disks, network interfaces, and hot-plugging other\n# PCIe devices. (Reference: https://libvirt.org/pci-hotplug.html#x86_64-q35)","commit_id":"3f6d38d67b4b4632c18b092516c7650dd2711229"}],"/COMMIT_MSG":[{"author":{"_account_id":6962,"name":"Kashyap Chamarthy","email":"kchamart@redhat.com","username":"kashyapc"},"change_message_id":"c76df71bcc60d2fdbb7d3407f05a5a32c6877144","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2020-04-20 09:10:07 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP nova-next: Start testing the q35 machine type"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Depends-On: https://review.opendev.org/710864"},{"line_number":10,"context_line":"Change-Id: Iad1adbc23b31dd54a96299e7a8a4b622c15eed8d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f4c43b2_3220a69b","line":9,"range":{"start_line":8,"start_character":0,"end_line":9,"end_character":45},"updated":"2020-04-20 14:56:24.000000000","message":"Does it really depend on the above change?  (Titled: \"remote_client: Use blkid -L to find config drive by filesystem label\")","commit_id":"7465a4b3e1c1a32d068cda1748ed12728bedffbf"}]}