)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"73c5649f809c0811f38e18759f349a43fd92da32","unresolved":true,"context_lines":[{"line_number":10,"context_line":"addresses on the same bus (0000:25:00.4 and 0000:25:00.5), making"},{"line_number":11,"context_line":"it appear that VFs under the same parent PF could have different"},{"line_number":12,"context_line":"resource_class values. This contradicts the documented restriction"},{"line_number":13,"context_line":"and causes confusion (LP#2145783)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Fix the second device_spec to use bus 27 instead of bus 25, making"},{"line_number":16,"context_line":"it clear the two VFs belong to different parent PFs (different"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3848e3bd_8cc86ff8","line":13,"range":{"start_line":13,"start_character":20,"end_line":13,"end_character":34},"updated":"2026-04-29 13:47:19.000000000","message":"nit:\n\n```suggestion\nand causes confusion.\n```\n\nWe already reference the bug below. This is redundant.","commit_id":"3271c9503fbb5c5b13cc4e1b12022e3450ecd0cc"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"73c5649f809c0811f38e18759f349a43fd92da32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"12ea42c0_4482184c","updated":"2026-04-29 13:47:19.000000000","message":"Soft -1, but I think this should either be simplier and split into two or is incomplete.","commit_id":"3271c9503fbb5c5b13cc4e1b12022e3450ecd0cc"}],"doc/source/admin/pci-passthrough.rst":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"73c5649f809c0811f38e18759f349a43fd92da32","unresolved":true,"context_lines":[{"line_number":547,"context_line":""},{"line_number":548,"context_line":"  [pci]"},{"line_number":549,"context_line":"  device_spec \u003d { \"vendor_id\": \"10de\", \"product_id\": \"25b6\", \"address\": \"0000:25:00.4\", \"resource_class\": \"CUSTOM_A16_16A\", \"managed\": \"no\" }"},{"line_number":550,"context_line":"  device_spec \u003d { \"vendor_id\": \"10de\", \"product_id\": \"25b6\", \"address\": {\"domain\": \"0000\", \"bus\": \"27\", \"slot\": \"00\", \"function\": \"[4-5]\"}, \"resource_class\": \"CUSTOM_A16_8A\", \"managed\":\"no\", \"live_migratable\": \"yes\"}"},{"line_number":551,"context_line":"  alias \u003d { \"device_type\": \"type-VF\", \"resource_class\": \"CUSTOM_A16_16A\", \"name\": \"A16_16A\" }"},{"line_number":552,"context_line":"  alias \u003d { \"device_type\": \"type-VF\", \"resource_class\": \"CUSTOM_A16_8A\", \"name\": \"A16_8A\" }"},{"line_number":553,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5d18c25c_d36dc607","line":550,"range":{"start_line":550,"start_character":60,"end_line":550,"end_character":138},"updated":"2026-04-29 13:47:19.000000000","message":"Is this a real thing? I don\u0027t see any release notes about adding support for this and the docs for this option (in `nova/conf/pci.py`) don\u0027t suggest it\u0027s supported.\n\nhttps://github.com/openstack/nova/blob/a6633225dba2dd4bb88d3efa932a95c5e17d37dd/nova/conf/pci.py#L209-L217\n\nIf it is a real thing, perhaps you could drag this out to a separate follow-up PR and also update the docs for this option in `nova/conf/pci.py`?\n\n**LATER:** Okay, there\u0027s actually an example of this in the docs.\n\nhttps://github.com/openstack/nova/blob/a6633225dba2dd4bb88d3efa932a95c5e17d37dd/nova/conf/pci.py#L309-L312\n\nWe should update the blurb for that option to say, e.g.\n\n\u003e PCI address of the device. This can be a string or an object. If a string,\n\u003e both traditional glob style and regular expression syntax is supported.\n\u003e If an object, it should contain the following keys: domain, bus, slot, function.\n\nYou can do this here or in a follow-up.","commit_id":"3271c9503fbb5c5b13cc4e1b12022e3450ecd0cc"}]}