)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2cbb5b7f6f71899052446a4b45cb361b212b31bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7c598ed0_4de5fc7c","updated":"2024-03-20 12:33:10.000000000","message":"ok cool we can see that the instance on the contoler 72a54e47-495d-4244-bb46-af52c53ec3f1\nhad the expect cpu range \u003cvcpu cpuset\u003d\"0-5\"\u003e1\u003c/vcpu\u003e\n\nhttps://zuul.opendev.org/t/openstack/build/17db34012c784f47905c717a75c3023f/log/controller/logs/screen-n-cpu.txt#8482\n\nthen when we do the live migration we get the destioantion cpu in the migrate_data object\n\nsource check data is LibvirtLiveMigrateData(bdms\u003d\u003c?\u003e,block_migration\u003dTrue,disk_available_mb\u003d54272,disk_over_commit\u003dFalse,dst_cpu_shared_set_info\u003dset([2,3,4,5,6,7]),dst_numa_info\u003d\u003c?\u003e ...\n\nhttps://zuul.opendev.org/t/openstack/build/17db34012c784f47905c717a75c3023f/log/controller/logs/screen-n-cpu.txt#9104\n\nthen when updating the xml for live migration we first print the current xml\nwith \u003cvcpu cpuset\u003d\"0-5\"\u003e1\u003c/vcpu\u003e\nhttps://zuul.opendev.org/t/openstack/build/17db34012c784f47905c717a75c3023f/log/controller/logs/screen-n-cpu.txt#9383\n\nand the updated xml with \u003cvcpu placement\u003d\"static\" cpuset\u003d\"2-7\"\u003e1\u003c/vcpu\u003e\n\nhttps://zuul.opendev.org/t/openstack/build/17db34012c784f47905c717a75c3023f/log/controller/logs/screen-n-cpu.txt#9485\n\nso that show that this series works as expected for non numa instnaces\n\nresolving the long runnign bug.\n\nas i said elsewhere if we want to do more then ensure this code runs and actully validate this we need to add add the whitebox tempest plugin and add a test for this specific usecase. we can consier that in the future but i think just havign this run and exersize the code path is a good testing improvement by itself as we basically get that for free without the complixity fo adding whitebox or configuring a regex to only run this specific addtional test.","commit_id":"42eea5a55e8a59088c4082bb0b0f6eddca20fbf2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c5f489ef87a2d98aafd3cd429196717d473dee75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a1d90cef_dfba6321","updated":"2024-03-20 10:02:39.000000000","message":"this is the test configuration i mentioned on irc.\nwhile it wont assert that the xml has been updated (we would need to pull in whitebox for that) we can inspect the xml in the logs as a spot check and it will execute the code that is being modifed so if the code is broken in a way that breaks live migration then the live migration tests would fail.","commit_id":"42eea5a55e8a59088c4082bb0b0f6eddca20fbf2"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"7427c766eed551b1c6e054403838163c91921e95","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"289667c5_0a7b82f9","updated":"2024-06-13 08:38:35.000000000","message":"Not sure we have quorum, but we can try to test it.","commit_id":"ae06925ef063419d42edb39f822b9840893c7abf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e1cc8199b68a572598bc34bf13f80c69b58617ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"8c8cdb60_b39e2181","updated":"2024-06-05 10:04:06.000000000","message":"recheck timeouts and new logs","commit_id":"ae06925ef063419d42edb39f822b9840893c7abf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7bae144852b9ace6fd40def3f6382cb89fdfa96d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"a1a64a2e_c5726cc1","updated":"2024-06-25 17:47:16.000000000","message":"recheck timeouts should be resolved by https://review.opendev.org/c/openstack/devstack/+/922630","commit_id":"ae06925ef063419d42edb39f822b9840893c7abf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9f2dbafb8458fb9d328e4a7eca2726398e624ef1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"462202b4_09e98576","updated":"2024-05-27 16:43:51.000000000","message":"this only modifed nova-live-migration which passed. \nthe other failures are unrelated \n\nill recheck once it has been reviewd by others.","commit_id":"ae06925ef063419d42edb39f822b9840893c7abf"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"55c2e8f67b6743de3447f87fef986e1f5d749bc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"2b5c4a8e_06ed8450","updated":"2024-09-02 07:51:34.000000000","message":"Nova live-migration job was green and the logic of the config change make sense.","commit_id":"aca63b66e5133e38f2899fb7e79ca00072694150"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"dee9db6fdcc8091edb0258895c05d94b72544426","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"25677417_6a38ca5c","updated":"2024-09-02 07:51:05.000000000","message":"recheck the sdk job failed to create and delete VMs. That job wasn\u0027t modified here so this is unrelated","commit_id":"aca63b66e5133e38f2899fb7e79ca00072694150"}]}