)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ab916ac6a85a817606231ed2179221050a7e7f1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6fe27a5c_bcdf4b73","updated":"2022-01-25 16:37:17.000000000","message":"+w based on nova meeting conversation","commit_id":"e14a4049bcdf6af9583d091df9ce21eec90eb0b5"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"005319471bdc5029ea8ebfdd0cb1cf0ca761eb14","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76970e96_1c615c73","updated":"2022-01-11 13:52:59.000000000","message":"LGTM","commit_id":"e14a4049bcdf6af9583d091df9ce21eec90eb0b5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3cf77c0661f8afd153348e2d4e29d99c394d15e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"73cde6ea_d89e8b14","updated":"2022-01-25 02:31:00.000000000","message":"this matches what we have in the spec which is now approved.\nill double check with gibi in the morning that he is ok with this and then +w.\n\nill also check what other traits are pending.\nif we dont have any ill propose a release to the released repo so that this can be consumed by nova.","commit_id":"e14a4049bcdf6af9583d091df9ce21eec90eb0b5"}],"os_traits/compute/__init__.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"1a2d5bad0ad449d5b046470a807f5d201c2a76d5","unresolved":true,"context_lines":[{"line_number":37,"context_line":"    \u0027SOCKET_PCI_NUMA_AFFINITY\u0027,"},{"line_number":38,"context_line":"    # The compute manager supports handling remote_managed PCI devices"},{"line_number":39,"context_line":"    # associated with SmartNIC DPUs."},{"line_number":40,"context_line":"    \u0027REMOTE_MANAGED_PORTS\u0027"},{"line_number":41,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"85cdaa5a_275a06c8","line":40,"updated":"2022-01-11 09:23:15.000000000","message":"This is according to the spec. Probably we could start bikeshedding around the namig to include PCI. But the name is not clashing with anything existing so I\u0027m OK with it.","commit_id":"e14a4049bcdf6af9583d091df9ce21eec90eb0b5"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"461b1969f56ecc7e34fd8934721dfe9cad1eea2c","unresolved":true,"context_lines":[{"line_number":37,"context_line":"    \u0027SOCKET_PCI_NUMA_AFFINITY\u0027,"},{"line_number":38,"context_line":"    # The compute manager supports handling remote_managed PCI devices"},{"line_number":39,"context_line":"    # associated with SmartNIC DPUs."},{"line_number":40,"context_line":"    \u0027REMOTE_MANAGED_PORTS\u0027"},{"line_number":41,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ca68033_08f5fbe6","line":40,"in_reply_to":"351051d7_61d75889","updated":"2022-01-11 15:17:39.000000000","message":"OK, Then I was fooled by the comment above as that mentioning PCI devs.","commit_id":"e14a4049bcdf6af9583d091df9ce21eec90eb0b5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3e221392003258a732985691d055a40cd82492bb","unresolved":true,"context_lines":[{"line_number":37,"context_line":"    \u0027SOCKET_PCI_NUMA_AFFINITY\u0027,"},{"line_number":38,"context_line":"    # The compute manager supports handling remote_managed PCI devices"},{"line_number":39,"context_line":"    # associated with SmartNIC DPUs."},{"line_number":40,"context_line":"    \u0027REMOTE_MANAGED_PORTS\u0027"},{"line_number":41,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"351051d7_61d75889","line":40,"in_reply_to":"85cdaa5a_275a06c8","updated":"2022-01-11 14:01:45.000000000","message":"well there is nothing specificly pci related about this capablity\n\nso i think including PCI woudl not be correct here.","commit_id":"e14a4049bcdf6af9583d091df9ce21eec90eb0b5"},{"author":{"_account_id":24824,"name":"Dmitrii Shcherbakov","username":"dmitriis"},"change_message_id":"60872c3c79471834a2ac5b609b63b7c5d4fbc353","unresolved":true,"context_lines":[{"line_number":37,"context_line":"    \u0027SOCKET_PCI_NUMA_AFFINITY\u0027,"},{"line_number":38,"context_line":"    # The compute manager supports handling remote_managed PCI devices"},{"line_number":39,"context_line":"    # associated with SmartNIC DPUs."},{"line_number":40,"context_line":"    \u0027REMOTE_MANAGED_PORTS\u0027"},{"line_number":41,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"4e34e32d_bfbd0740","line":40,"in_reply_to":"9ca68033_08f5fbe6","updated":"2022-01-16 11:00:02.000000000","message":"Yes, I should have probably added \"such as\" when talking about PCI.","commit_id":"e14a4049bcdf6af9583d091df9ce21eec90eb0b5"}]}