)]}'
{"nova/tests/functional/libvirt/test_pci_sriov_servers.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"dbfcdf1a1e178edb83fc10a59e74ca5eedd11a5d","unresolved":false,"context_lines":[{"line_number":367,"context_line":"    # PciPassthroughFilter is invoked in _run_build_test pass in the"},{"line_number":368,"context_line":"    # numa affinity tests otherwise the NUMATopologyFilter will eliminate"},{"line_number":369,"context_line":"    # all hosts before we execute the PciPassthroughFilter."},{"line_number":370,"context_line":"    ADDITIONAL_FILTERS \u003d [\u0027PciPassthroughFilter\u0027, \u0027NUMATopologyFilter\u0027]"},{"line_number":371,"context_line":"    ALIAS_NAME \u003d \u0027a1\u0027"},{"line_number":372,"context_line":"    PCI_PASSTHROUGH_WHITELIST \u003d [jsonutils.dumps("},{"line_number":373,"context_line":"        {"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_9f170eef","side":"PARENT","line":370,"updated":"2020-09-07 12:05:36.000000000","message":"Doesn\u0027t the intention of this test is to check if the PciPassthroughFilter eliminate all the hosts?","commit_id":"f7e2d1e08f9f6dd580b5881106db5a446e27b0ab"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5126ecf23c1b31f005dd28d656a1314245740fa4","unresolved":false,"context_lines":[{"line_number":367,"context_line":"    # PciPassthroughFilter is invoked in _run_build_test pass in the"},{"line_number":368,"context_line":"    # numa affinity tests otherwise the NUMATopologyFilter will eliminate"},{"line_number":369,"context_line":"    # all hosts before we execute the PciPassthroughFilter."},{"line_number":370,"context_line":"    ADDITIONAL_FILTERS \u003d [\u0027PciPassthroughFilter\u0027, \u0027NUMATopologyFilter\u0027]"},{"line_number":371,"context_line":"    ALIAS_NAME \u003d \u0027a1\u0027"},{"line_number":372,"context_line":"    PCI_PASSTHROUGH_WHITELIST \u003d [jsonutils.dumps("},{"line_number":373,"context_line":"        {"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_fc9a93c2","side":"PARENT","line":370,"in_reply_to":"9f560f44_9f170eef","updated":"2020-09-07 14:58:29.000000000","message":"No, the test expects the \u0027NUMATopologyFilter\u0027 to eliminate hosts. We had arranged things this way simply because \u0027_run_build_test\u0027 was always checking that the \u0027PCIFilter\u0027 had been called and that would obviously not happen if the \u0027NUMATopologyFilter\u0027 had already eliminated the host. Since we now have the conditional in \u0027_test_policy\u0027, we no longer need to worry about that","commit_id":"f7e2d1e08f9f6dd580b5881106db5a446e27b0ab"}]}
