)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"634d5d65802fe41d41206156ebfedf1d05081f3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c888c359_26b0e896","updated":"2026-07-13 08:04:15.000000000","message":"Need self review and modification","commit_id":"5bdae99852ef00ec54a500e735355c7dd773120b"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"791f067e7ec60028db8d7f9ba6851d5846b4aeab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9df8daab_cccd57a5","updated":"2026-07-14 13:26:21.000000000","message":"lgtm, reuses code from the tempest library when possible and follows the same approach used in other projects","commit_id":"f360b0375a2925ddb4be07f587de14d3af8e02b6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8a4400fa92f87dde50c11e56b5f1fb0b3e52b98c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4b90e15b_21e410d9","updated":"2026-07-15 20:25:11.000000000","message":"recheck","commit_id":"f360b0375a2925ddb4be07f587de14d3af8e02b6"}],"cyborg_tempest_plugin/services/cyborg_rest_client.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"65351284244e5bfd3d40e26b4836a4b6bba91b7f","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def get_headers(self):"},{"line_number":39,"context_line":"        headers \u003d super().get_headers()"},{"line_number":40,"context_line":"        if CYBORG_MICROVERSION:"},{"line_number":41,"context_line":"            headers[self.api_microversion_header_name] \u003d ("},{"line_number":42,"context_line":"                \" \".join([\"accelerator\", CYBORG_MICROVERSION]))"},{"line_number":43,"context_line":"        return headers"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def _response_helper(self, resp, body\u003dNone):"}],"source_content_type":"text/x-python","patch_set":2,"id":"2d12487a_7dd836f9","line":42,"range":{"start_line":40,"start_character":0,"end_line":42,"end_character":63},"updated":"2026-07-14 14:34:11.000000000","message":"just an fyi this the corerct way to constuct the header\n\nhowever without https://review.opendev.org/c/openstack/cyborg/+/994574\nthis woudl not be parsed properly by cybrog\n\nso this si only workign becasue we never pass a microversion today","commit_id":"f360b0375a2925ddb4be07f587de14d3af8e02b6"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fb6290ef5c2cc694a970544ce93db0b4fd55bba5","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def get_headers(self):"},{"line_number":39,"context_line":"        headers \u003d super().get_headers()"},{"line_number":40,"context_line":"        if CYBORG_MICROVERSION:"},{"line_number":41,"context_line":"            headers[self.api_microversion_header_name] \u003d ("},{"line_number":42,"context_line":"                \" \".join([\"accelerator\", CYBORG_MICROVERSION]))"},{"line_number":43,"context_line":"        return headers"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def _response_helper(self, resp, body\u003dNone):"}],"source_content_type":"text/x-python","patch_set":2,"id":"e6504806_7a3e0af7","line":42,"range":{"start_line":40,"start_character":0,"end_line":42,"end_character":63},"in_reply_to":"2d12487a_7dd836f9","updated":"2026-07-15 04:15:54.000000000","message":"Thank you @smooney@redhat.com for the review, let me update https://review.opendev.org/c/openstack/cyborg/+/994574 review by addressing your comment and test it without tempest microversion patch.","commit_id":"f360b0375a2925ddb4be07f587de14d3af8e02b6"}],"cyborg_tempest_plugin/tests/scenario/manager.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"65351284244e5bfd3d40e26b4836a4b6bba91b7f","unresolved":true,"context_lines":[{"line_number":51,"context_line":"    @classmethod"},{"line_number":52,"context_line":"    def resource_setup(cls):"},{"line_number":53,"context_line":"        super(ScenarioTest, cls).resource_setup()"},{"line_number":54,"context_line":"        cls.request_microversion \u003d ("},{"line_number":55,"context_line":"            api_version_utils.select_request_microversion("},{"line_number":56,"context_line":"                cls.min_microversion,"},{"line_number":57,"context_line":"                CONF.cyborg.min_microversion))"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    def setUp(self):"},{"line_number":60,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":2,"id":"117fc10f_3fa539ed","line":57,"range":{"start_line":54,"start_character":7,"end_line":57,"end_character":46},"updated":"2026-07-14 14:34:11.000000000","message":"this is sort of incorrect\n\nit will work but each call to cybrog should pass it an explcity  microverion\nits not correct to use the same microversion for every request to cyborg.\n\nit might work but that is not how the api is intedned to work.\n\nwe need to make sure we get this right before we start adding tests that use this with eh wrong pattern","commit_id":"f360b0375a2925ddb4be07f587de14d3af8e02b6"}]}