)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"4f2b92b2d0579e52bd06d2dcb4679ac4a3194a4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"11cecb46_d65099c0","updated":"2026-11-11 14:42:16.000000000","message":"awesome small test case.Thanks","commit_id":"ee0cb7a00846af87aa6b8f8222c855a1e9d89496"}],"nova/tests/functional/libvirt/test_init_host.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"eeae698546ca3dd60fa7d313348f205bcd8a0f09","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"18a2d38b_1989b3a8","line":38,"updated":"2026-11-11 15:37:52.000000000","message":"i would have expected this to be a unit test rather then a functioal test.\n\nis the reason you made it a fucntionl test rather then asserting that its caulld in a unit test of init_host so you could trigger it form the compute agent restart?\n\nwould you mind adding a unit test as well?","commit_id":"ee0cb7a00846af87aa6b8f8222c855a1e9d89496"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3dcae36bb86be2dfd81b0df56d9872c79582ced8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5d26c41b_e1282de7","line":38,"in_reply_to":"17ce2d22_92cde8ac","updated":"2026-11-12 15:02:43.000000000","message":"Acknowledged we will cover the unit tests in https://review.opendev.org/c/openstack/nova/+/1015362/","commit_id":"ee0cb7a00846af87aa6b8f8222c855a1e9d89496"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d0b27e65ed279482be4f602041e4449b3405f57c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"17ce2d22_92cde8ac","line":38,"in_reply_to":"18a2d38b_1989b3a8","updated":"2026-11-11 15:57:48.000000000","message":"We already have unit tests of _check_multipath. What we currently lack are the tests of init_host to ensure it calls that check method.\n\nI can add unit tests of init_host to make sure it calls _check_multipath, but I still find it a bit redundant and rather e2e functional tests would be useful (to catch similar lack-of-call problems in different layers) but have no strong opinion if you find this functional test over-technology.","commit_id":"ee0cb7a00846af87aa6b8f8222c855a1e9d89496"}]}