)]}'
{"nova/tests/functional/libvirt/test_pci_sriov_servers.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"964bb1cc1372742ed4e3a13eb72f607685b0de90","unresolved":true,"context_lines":[{"line_number":1099,"context_line":"        # Boot a server with a single PCI device."},{"line_number":1100,"context_line":"        extra_spec \u003d {\u0027pci_passthrough:alias\u0027: f\u0027{self.ALIAS_NAME}:1\u0027}"},{"line_number":1101,"context_line":"        pci_flavor_id \u003d self._create_flavor(extra_spec\u003dextra_spec)"},{"line_number":1102,"context_line":"        server \u003d self._create_server(flavor_id\u003dpci_flavor_id, networks\u003d\u0027none\u0027)"},{"line_number":1103,"context_line":""},{"line_number":1104,"context_line":"        # Resize it to a flavor without PCI devices. We expect this to work, as"},{"line_number":1105,"context_line":"        # test_compute1 is available."}],"source_content_type":"text/x-python","patch_set":1,"id":"9d4b7d85_d1fabdfe","line":1102,"range":{"start_line":1102,"start_character":8,"end_line":1102,"end_character":78},"updated":"2021-08-25 09:52:09.000000000","message":"nit - did you want to assert that we land on test_compute0 with PCI etc?\n\nYou can add this in the fix tbh.","commit_id":"38603a4d09a77882298423a8b20c67b283b99ad0"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"964bb1cc1372742ed4e3a13eb72f607685b0de90","unresolved":true,"context_lines":[{"line_number":1109,"context_line":"                               self._resize_server, server, flavor_id)"},{"line_number":1110,"context_line":"        self.assertEqual(500, ex.response.status_code)"},{"line_number":1111,"context_line":"        self.assertIn(\u0027NoValidHost\u0027, str(ex))"},{"line_number":1112,"context_line":"        # self._confirm_resize(server)"},{"line_number":1113,"context_line":""},{"line_number":1114,"context_line":"    def _confirm_resize(self, server, host\u003d\u0027host1\u0027):"},{"line_number":1115,"context_line":"        # NOTE(sbauza): Unfortunately, _cleanup_resize() in libvirt checks the"}],"source_content_type":"text/x-python","patch_set":1,"id":"5706b1ca_1ac8581b","line":1112,"range":{"start_line":1112,"start_character":8,"end_line":1112,"end_character":38},"updated":"2021-08-25 09:52:09.000000000","message":"nit - Add an assert that we end up on test_compute1\n\nAgain you can add this in the fix.","commit_id":"38603a4d09a77882298423a8b20c67b283b99ad0"}]}