)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"c4b439c699941f4417fb26771892237f8e6b87b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0a5540f5_4dd47a6e","updated":"2026-02-03 03:53:29.000000000","message":"recheck","commit_id":"6295f67ab72ca8c7712cfd6ecba31e27b43a004c"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"ce4e76a55be3c120ac58d96b11a0e466ad277efd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"72a5f454_849f31fe","updated":"2026-01-30 13:47:00.000000000","message":"recheck","commit_id":"6295f67ab72ca8c7712cfd6ecba31e27b43a004c"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"98b8ba4fffe91eab1d130f48075025aa7b047e75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b3a77f0a_32a92742","updated":"2026-02-02 20:15:30.000000000","message":"recheck","commit_id":"6295f67ab72ca8c7712cfd6ecba31e27b43a004c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"217a7e34e68fd7c43abe381a8d8cdb975201be75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"781a8526_a4e44385","updated":"2026-02-19 11:15:15.000000000","message":"by the way v2 should have worked if it had the depends on in v3\n\nso v3 was not passing because you reverted to v1 code it was passing because of the nova fix.","commit_id":"266d5fe7c21dc8192e1b977eeb821d4885000ef0"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0925cc43b297b190099f402f59370686ab32608b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b3427652_129dd877","updated":"2026-03-10 16:22:05.000000000","message":"let proceed with this to make sure we dont regress it again","commit_id":"41b4c072cd00f749cfa8577b0beae735d977c8e9"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"076740de55894a5bc1bb21a3cc6dff2f8470bfb6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"173b8b9a_9fc41593","updated":"2026-03-11 00:03:27.000000000","message":"recheck","commit_id":"41b4c072cd00f749cfa8577b0beae735d977c8e9"}],"whitebox_tempest_plugin/api/compute/test_provider_yaml.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"983e12f6991fc808671ef388972806ff83d671e6","unresolved":true,"context_lines":[{"line_number":47,"context_line":"        super(TestProviderYamlViaTraits, self).setUp()"},{"line_number":48,"context_line":"        self.rp_admin_cl \u003d self.os_admin.resource_providers_client"},{"line_number":49,"context_line":"        rps \u003d self.rp_admin_cl.list_resource_providers()[\u0027resource_providers\u0027]"},{"line_number":50,"context_line":"        parent_rps \u003d \\"},{"line_number":51,"context_line":"            [rp for rp in rps if rp.get(\u0027parent_provider_uuid\u0027) is None]"},{"line_number":52,"context_line":"        self.host1, self.host1_id \u003d \\"},{"line_number":53,"context_line":"            parent_rps[0][\u0027name\u0027], parent_rps[0][\u0027uuid\u0027]"},{"line_number":54,"context_line":"        self.host2, self.host2_id \u003d \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f640af9_d84f9d07","line":51,"range":{"start_line":50,"start_character":0,"end_line":51,"end_character":72},"updated":"2026-01-29 21:24:28.000000000","message":"terminology wise we call these root resource providers since they are the root of a tree.\n\n```suggestion\n        root_rps \u003d [\n            rp for rp in rps if rp.get(\u0027parent_provider_uuid\u0027) is None]\n```\n\nwith  that said the logic look ok.\n\nyou could also constuct the entires in the list as a touple and destucrue them\n\n```\nroot_rps \u003d [ (rp[\u0027name\u0027], rp[\u0027uuid\u0027]) \n            for rp in rps if rp.get(\u0027parent_provider_uuid\u0027) is None]\nself.host1, self.host1_id \u003d root_rps[0]\nself.host2, self.host2_id \u003d root_rps[1]\n```\n\nbut the code shoudl work as you have it.","commit_id":"44de1c3b8940f0fef54bead1a6854ac02f69d36a"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"ab0a278cf0a80204ea82f67bbc8d3d75c3032e67","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        super(TestProviderYamlViaTraits, self).setUp()"},{"line_number":48,"context_line":"        self.rp_admin_cl \u003d self.os_admin.resource_providers_client"},{"line_number":49,"context_line":"        rps \u003d self.rp_admin_cl.list_resource_providers()[\u0027resource_providers\u0027]"},{"line_number":50,"context_line":"        parent_rps \u003d \\"},{"line_number":51,"context_line":"            [rp for rp in rps if rp.get(\u0027parent_provider_uuid\u0027) is None]"},{"line_number":52,"context_line":"        self.host1, self.host1_id \u003d \\"},{"line_number":53,"context_line":"            parent_rps[0][\u0027name\u0027], parent_rps[0][\u0027uuid\u0027]"},{"line_number":54,"context_line":"        self.host2, self.host2_id \u003d \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"de216b70_a698886d","line":51,"range":{"start_line":50,"start_character":0,"end_line":51,"end_character":72},"in_reply_to":"5f640af9_d84f9d07","updated":"2026-01-29 21:28:33.000000000","message":"Fix applied.  I\u0027d definitely prefer to keep the terminology correct thank you.","commit_id":"44de1c3b8940f0fef54bead1a6854ac02f69d36a"}]}