)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"765cf145b17bcf0b7dcd01645891d3048e416f50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"89de0eb9_5de730b0","updated":"2024-05-09 19:13:31.000000000","message":"+1 for now lets see if we can get a clean upstream run\n\nthis is skipped upstream but we are getting this runnign downstream now and if we can get it to pass there on the amd host we can proxy that result.","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"c0d6948cbdad84cfbbc79e07974cc18e079f06e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e6737951_a487adff","updated":"2024-05-10 18:21:23.000000000","message":"This is super bizarre - there appears to be something with _this specific patch_ (other patches pass CI just fine) that makes it fail, test_threads_isolate looks to be a deterministic failure - even if this patch isn\u0027t actually changing anything that\u0027s running in CI.","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"b5775e88f4964eeb88509e8574d0dccc94267720","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1911282d_ceee8282","updated":"2024-05-09 20:46:13.000000000","message":"recheck","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"e77cf7de15140be33f90cefdb5b806acfdc64e48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7c76b88e_ec8282ed","updated":"2024-05-07 20:24:22.000000000","message":"recheck","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7e642126603260a824de4252644a06871dd024e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"75eec461_45f42ea5","updated":"2024-05-09 19:10:04.000000000","message":"recheck  \u0027No valid host was found. There are not enough hosts available.\u0027\n\nwe might have some concurrancy issues in this patch lets see what happnes\n\nthe test failures look unrealted so this could be a job config thing","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"15b9e532c45d19a0d9100c011ed14f4d439c19a6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"96a4c64d_a62e7d46","in_reply_to":"e6737951_a487adff","updated":"2024-05-13 13:26:43.000000000","message":"so looking at the test results the issue is a key error\n\nTraceback (most recent call last):\n  File \"/opt/stack/tempest/.tox/tempest/lib/python3.10/site-packages/whitebox_tempest_plugin/api/compute/test_cpu_pinning.py\", line 385, in test_threads_isolate\n    sib \u003d pcpu_siblings[pcpu]\nKeyError: 6\n\nwhich is fun since we are lookign up pcpu6\n\nbut the toplogy of this host is really  odd\n\n\n```\n    \u003ctopology\u003e\n      \u003ccells num\u003d\u00271\u0027\u003e\n        \u003ccell id\u003d\u00270\u0027\u003e\n          \u003cmemory unit\u003d\u0027KiB\u0027\u003e7932380\u003c/memory\u003e\n          \u003cpages unit\u003d\u0027KiB\u0027 size\u003d\u00274\u0027\u003e1983095\u003c/pages\u003e\n          \u003cpages unit\u003d\u0027KiB\u0027 size\u003d\u00272048\u0027\u003e0\u003c/pages\u003e\n          \u003cpages unit\u003d\u0027KiB\u0027 size\u003d\u00271048576\u0027\u003e0\u003c/pages\u003e\n          \u003cdistances\u003e\n            \u003csibling id\u003d\u00270\u0027 value\u003d\u002710\u0027/\u003e\n          \u003c/distances\u003e\n          \u003ccpus num\u003d\u00275\u0027\u003e\n            \u003ccpu id\u003d\u00270\u0027 socket_id\u003d\u00270\u0027 die_id\u003d\u00270\u0027 core_id\u003d\u00270\u0027 siblings\u003d\u00270\u0027/\u003e\n            \u003ccpu id\u003d\u00271\u0027 socket_id\u003d\u00271\u0027 die_id\u003d\u00270\u0027 core_id\u003d\u00270\u0027 siblings\u003d\u00271\u0027/\u003e\n            \u003ccpu id\u003d\u00272\u0027 socket_id\u003d\u00272\u0027 die_id\u003d\u00270\u0027 core_id\u003d\u00270\u0027 siblings\u003d\u00272\u0027/\u003e\n            \u003ccpu id\u003d\u00273\u0027 socket_id\u003d\u00273\u0027 die_id\u003d\u00270\u0027 core_id\u003d\u00270\u0027 siblings\u003d\u00273\u0027/\u003e\n            \u003ccpu id\u003d\u00277\u0027 socket_id\u003d\u00277\u0027 die_id\u003d\u00270\u0027 core_id\u003d\u00270\u0027 siblings\u003d\u00277\u0027/\u003e\n          \u003c/cpus\u003e\n        \u003c/cell\u003e\n      \u003c/cells\u003e\n    \u003c/topology\u003e\n    \u003ccache\u003e\n      \u003cbank id\u003d\u00270\u0027 level\u003d\u00273\u0027 type\u003d\u0027both\u0027 size\u003d\u002716\u0027 unit\u003d\u0027MiB\u0027 cpus\u003d\u00270\u0027/\u003e\n      \u003cbank id\u003d\u00271\u0027 level\u003d\u00273\u0027 type\u003d\u0027both\u0027 size\u003d\u002716\u0027 unit\u003d\u0027MiB\u0027 cpus\u003d\u00271\u0027/\u003e\n      \u003cbank id\u003d\u00272\u0027 level\u003d\u00273\u0027 type\u003d\u0027both\u0027 size\u003d\u002716\u0027 unit\u003d\u0027MiB\u0027 cpus\u003d\u00272\u0027/\u003e\n      \u003cbank id\u003d\u00273\u0027 level\u003d\u00273\u0027 type\u003d\u0027both\u0027 size\u003d\u002716\u0027 unit\u003d\u0027MiB\u0027 cpus\u003d\u00273\u0027/\u003e\n      \u003cbank id\u003d\u00275\u0027 level\u003d\u00273\u0027 type\u003d\u0027both\u0027 size\u003d\u002716\u0027 unit\u003d\u0027MiB\u0027 cpus\u003d\u00275\u0027/\u003e\n      \u003cbank id\u003d\u00276\u0027 level\u003d\u00273\u0027 type\u003d\u0027both\u0027 size\u003d\u002716\u0027 unit\u003d\u0027MiB\u0027 cpus\u003d\u00276\u0027/\u003e\n      \u003cbank id\u003d\u00277\u0027 level\u003d\u00273\u0027 type\u003d\u0027both\u0027 size\u003d\u002716\u0027 unit\u003d\u0027MiB\u0027 cpus\u003d\u00277\u0027/\u003e\n    \u003c/cache\u003e\n```\n\n\nlibvirt only shows 5 of the cpus presubably because the other 3 are powered off.\n\npcpu_siblings is the result of self.get_host_cpu_siblings(host)\n\nhttps://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/api/compute/test_cpu_pinning.py#L377\n\n\nand that uses the host capablities from libvirt\n\n\nhttps://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/api/compute/test_cpu_pinning.py#L358\n\nso this is unrealted to this patch","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"4a92c7c07982dd51f6854e31f0609f2e4f5dc8db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fabdda04_4d96f4ee","updated":"2024-11-20 16:48:50.000000000","message":"@alifshit@redhat.com yes I think that did fix it, I need to retest vGPU/SR-IOV based on the ssh_client changes I did but I think this unplugs some of the other things in review.","commit_id":"8a3d5843eb44c796e120dbcc21ba9af179f2bddf"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"4731087d87d3da59ee05fa13a4cae9d89b15d267","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9281a622_194468e4","updated":"2024-11-20 03:45:33.000000000","message":"recheck\nDid https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/933454 merging fix out gate?","commit_id":"8a3d5843eb44c796e120dbcc21ba9af179f2bddf"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"8546a4948f19fd67e3c10d0f9cd01e1d0a1fb2b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"b3ccccc0_f7716788","updated":"2025-11-24 18:11:59.000000000","message":"recheck","commit_id":"5a51c7afbba189b491a587dcff0930d8be9b9aed"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"d4e4797493c26e8602ab2390fd074a8628c3e07c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"6495abe0_3d757cb3","updated":"2026-02-03 16:29:19.000000000","message":"recheck","commit_id":"b4732f6f31e6fc55e878f654145d193d6dd05b05"}],"whitebox_tempest_plugin/api/compute/base.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"05d9223b2896b8e31016aaaae4d70d38ac34c016","unresolved":true,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"        return new_image[\u0027id\u0027]"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"    def create_validateable_instance(self, validation_resources, **kwargs):"},{"line_number":113,"context_line":"        \"\"\"Create a validateable instance based on provided flavor"},{"line_number":114,"context_line":"        :param flavor: dict, attributes describing flavor"},{"line_number":115,"context_line":"        :param validation_resources: dict, parameters necessary to setup ssh"}],"source_content_type":"text/x-python","patch_set":6,"id":"8fcd80d6_03768d2b","line":112,"range":{"start_line":112,"start_character":8,"end_line":112,"end_character":36},"updated":"2024-05-13 13:16:44.000000000","message":"this already exists in the vgpu tests \nbut is not aliasing anythin and im not seing how this could cause shareing.","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"05d9223b2896b8e31016aaaae4d70d38ac34c016","unresolved":true,"context_lines":[{"line_number":472,"context_line":"        huge_pages \u003d root.findall(\u0027.memoryBacking/hugepages/page\u0027)"},{"line_number":473,"context_line":"        return huge_pages"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"    def _create_ssh_client(self, server, validation_resources):"},{"line_number":476,"context_line":"        \"\"\"Create an ssh client to execute commands on the guest instance"},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"        :param server: the ssh client will be setup to interface with the"}],"source_content_type":"text/x-python","patch_set":6,"id":"cac881f1_3a9055d5","line":475,"updated":"2024-05-13 13:16:44.000000000","message":"this would be the most likely cause of the test interaction but you are not aliaising a method in tempest\n\nhttps://codesearch.opendev.org/?q\u003d_create_ssh_client\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d\n\nso im not sure how.\n\nstypel nit you should not have an _ here if you intend this to be called out side of the current class.","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"4a92c7c07982dd51f6854e31f0609f2e4f5dc8db","unresolved":false,"context_lines":[{"line_number":472,"context_line":"        huge_pages \u003d root.findall(\u0027.memoryBacking/hugepages/page\u0027)"},{"line_number":473,"context_line":"        return huge_pages"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"    def _create_ssh_client(self, server, validation_resources):"},{"line_number":476,"context_line":"        \"\"\"Create an ssh client to execute commands on the guest instance"},{"line_number":477,"context_line":""},{"line_number":478,"context_line":"        :param server: the ssh client will be setup to interface with the"}],"source_content_type":"text/x-python","patch_set":6,"id":"38799f68_09bb1bbc","line":475,"in_reply_to":"cac881f1_3a9055d5","updated":"2024-11-20 16:48:50.000000000","message":"Acknowledged","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"}],"whitebox_tempest_plugin/api/compute/test_amd_sev.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"05d9223b2896b8e31016aaaae4d70d38ac34c016","unresolved":true,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def setup_credentials(cls):"},{"line_number":38,"context_line":"        cls.prepare_instance_network()"},{"line_number":39,"context_line":"        super(AMDSevTest, cls).setup_credentials()"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_create_amd_sev_instance(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"514cf18b_cbe5b9e9","line":38,"updated":"2024-05-13 13:16:44.000000000","message":"perhaps this is the issue?\nwhere is this network cleaned up?\n\ni have not looked at the test failreu but is it related to multipel neworks being defiend?\n\n\ndoes prepare_instance_network add a cleanup internally?","commit_id":"c849c4b414337e83e751e3d8c72abba2821849bf"}]}