)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"f9c95259527919bba725092632f2c4b8195843ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bdd90f0b_1548abc2","updated":"2025-04-02 11:41:48.000000000","message":"Tests looks good. We need to wait for the os-traits release to get them passing on the gate.","commit_id":"eae4a4b56b7971c9a507a5d383ad480e797c52ab"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"5d5cc1d19c6dd775b3c8a6b4925dbcd2d0866f33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1aa702e3_c96a931f","updated":"2025-04-02 14:49:32.000000000","message":"thanks for the extra test","commit_id":"7d5d44f9f5e388619bcdd1b0570d8d39a37e4f11"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1e981f97eb5542a1d42b375be88df25b673fd82c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"25d251db_f36da63e","updated":"2025-04-08 14:33:29.000000000","message":"I reviewed the series late but I\u0027m happy with this functional patch at least 😊","commit_id":"ee67362728e09ed92a9b0a5e1e75c8303fbe0373"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"bd11faa0446de4d53a85e1ab631c2a361923aa30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"bd52583b_0264af13","updated":"2025-04-02 20:37:49.000000000","message":"These are now passing with the requirements bump. The u-c patch is now in the gate. Since the patch is already separate and some review on the base patch has already passed muster, I\u0027m inclined to keep this separate. However, if the preference is to squash this into that patch let me know.","commit_id":"ee67362728e09ed92a9b0a5e1e75c8303fbe0373"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"f401cfa85c978025b25ef08a331d9eced07f0d09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"97f67358_4f271be2","updated":"2025-04-08 00:46:04.000000000","message":"recheck dep patches merged","commit_id":"ee67362728e09ed92a9b0a5e1e75c8303fbe0373"}],"nova/tests/functional/libvirt/test_pci_in_placement.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"4bcf366398545160655759ec47026c6d35f56796","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d9bdb3a5_152490c1","line":2093,"updated":"2025-04-02 12:06:49.000000000","message":"One think that might make sense to cover as well is marking an in-use device as OTU and restarting the compute, then asserting that the in-use device is reserved. This is different as it does not happen during the claim but during the update_available_resources call happening during init_host.\n\nMaybe you could add a negative test that migration does not work and refer to the placement fix you are working on that expected to fix it.","commit_id":"eae4a4b56b7971c9a507a5d383ad480e797c52ab"}]}