)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"bc10a2aa7a9e429fe3f20e36c318f6dd73c1f082","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"401098c0_c577d4ce","updated":"2025-04-09 14:06:20.000000000","message":"I think the linter will catch some errors here.\nWill wait for results. A tip, run \"tox -e pep8\" locally.","commit_id":"1c21518bfb312a64c7f65ae78d6ba63885a3ddb3"}],"ironic_tempest_plugin/common/waiters.py":[{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"bc10a2aa7a9e429fe3f20e36c318f6dd73c1f082","unresolved":true,"context_lines":[{"line_number":165,"context_line":"def wait_node_value_in_field(client, node_id, field, value,"},{"line_number":166,"context_line":"                             raise_if_insufficent_access\u003dTrue,"},{"line_number":167,"context_line":"                             timeout\u003dNone, interval\u003dNone,"},{"line_number":168,"context_line":"                             abort_on_error_state):"},{"line_number":169,"context_line":"    \"\"\"Waits for a node to have a field value appear."},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    :param client: an instance of tempest plugin BaremetalClient."}],"source_content_type":"text/x-python","patch_set":2,"id":"3a106315_ac2162bd","line":168,"range":{"start_line":168,"start_character":29,"end_line":168,"end_character":49},"updated":"2025-04-09 14:06:20.000000000","message":"This parameter follows parameters with defaults, so you need a default value here.","commit_id":"1c21518bfb312a64c7f65ae78d6ba63885a3ddb3"},{"author":{"_account_id":37631,"deleted":true},"change_message_id":"97a1aa9f54c31c4af05c88f069e81a55e50d5a29","unresolved":true,"context_lines":[{"line_number":165,"context_line":"def wait_node_value_in_field(client, node_id, field, value,"},{"line_number":166,"context_line":"                             raise_if_insufficent_access\u003dTrue,"},{"line_number":167,"context_line":"                             timeout\u003dNone, interval\u003dNone,"},{"line_number":168,"context_line":"                             abort_on_error_state):"},{"line_number":169,"context_line":"    \"\"\"Waits for a node to have a field value appear."},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    :param client: an instance of tempest plugin BaremetalClient."}],"source_content_type":"text/x-python","patch_set":2,"id":"507d5357_a582c550","line":168,"range":{"start_line":168,"start_character":29,"end_line":168,"end_character":49},"in_reply_to":"3a106315_ac2162bd","updated":"2025-04-09 15:37:44.000000000","message":"silly mistake from my side. Should I have used vscode instead hard to navigate through vim.","commit_id":"1c21518bfb312a64c7f65ae78d6ba63885a3ddb3"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"bc10a2aa7a9e429fe3f20e36c318f6dd73c1f082","unresolved":true,"context_lines":[{"line_number":182,"context_line":"        node \u003d utils.get_node(client, node_id\u003dnode_id)"},{"line_number":183,"context_line":"        field_value \u003d node[field]"},{"line_number":184,"context_line":"        if abort_on_error_state"},{"line_number":185,"context_line":"        or (raise_if_insufficent_access and \u0027** Redacted\u0027 in field_value):"},{"line_number":186,"context_line":"            msg \u003d (\u0027Unable to see contents of redacted field \u0027"},{"line_number":187,"context_line":"                   \u0027indicating insufficient access to execute this test.\u0027)"},{"line_number":188,"context_line":"            raise lib_exc.InsufficientAPIAccess(msg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"be65c37b_895a38fd","line":185,"range":{"start_line":185,"start_character":8,"end_line":185,"end_character":10},"updated":"2025-04-09 14:06:20.000000000","message":"I think the linter will complain on \"continuing line under indented\".","commit_id":"1c21518bfb312a64c7f65ae78d6ba63885a3ddb3"},{"author":{"_account_id":37631,"deleted":true},"change_message_id":"97a1aa9f54c31c4af05c88f069e81a55e50d5a29","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        node \u003d utils.get_node(client, node_id\u003dnode_id)"},{"line_number":183,"context_line":"        field_value \u003d node[field]"},{"line_number":184,"context_line":"        if abort_on_error_state"},{"line_number":185,"context_line":"        or (raise_if_insufficent_access and \u0027** Redacted\u0027 in field_value):"},{"line_number":186,"context_line":"            msg \u003d (\u0027Unable to see contents of redacted field \u0027"},{"line_number":187,"context_line":"                   \u0027indicating insufficient access to execute this test.\u0027)"},{"line_number":188,"context_line":"            raise lib_exc.InsufficientAPIAccess(msg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"926d2555_57463cbd","line":185,"range":{"start_line":185,"start_character":8,"end_line":185,"end_character":10},"in_reply_to":"be65c37b_895a38fd","updated":"2025-04-09 15:37:44.000000000","message":"Done","commit_id":"1c21518bfb312a64c7f65ae78d6ba63885a3ddb3"}],"ironic_tempest_plugin/tests/scenario/baremetal_manager.py":[{"author":{"_account_id":37631,"deleted":true},"change_message_id":"97a1aa9f54c31c4af05c88f069e81a55e50d5a29","unresolved":true,"context_lines":[{"line_number":140,"context_line":""},{"line_number":141,"context_line":"    @classmethod"},{"line_number":142,"context_line":"    def wait_for_agent_heartbeat(cls, node_id, timeout\u003dNone,"},{"line_number":143,"context_line":"                                abort_on_error_state\u003dTrue):"},{"line_number":144,"context_line":"        ironic_waiters.wait_node_value_in_field("},{"line_number":145,"context_line":"            cls.baremetal_client,"},{"line_number":146,"context_line":"            node_id\u003dnode_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"a62077bb_6ef5fe15","line":143,"in_reply_to":"0fb44872_2e92308b","updated":"2025-04-09 15:37:44.000000000","message":"\u003e pep8: E128 continuation line under-indented for visual indent\n\nPlease fix.","commit_id":"1c21518bfb312a64c7f65ae78d6ba63885a3ddb3"}]}