)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"c31253f3bc167cd98561a93e0f0b8f11d2bf9349","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a21208da_8b244555","updated":"2023-10-03 13:13:08.000000000","message":"This makes sense to me :)","commit_id":"318b573dc7dc4322c90640ea828860d0ef160ad2"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"db941ad680e146602262668b1e539aa52c54a5c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a05a3918_ff63e604","updated":"2023-11-16 19:43:55.000000000","message":"Terry will push an update if he decides to refactor things based on the comments.","commit_id":"61935457bdfec157626d0b16a2bd43913c9e2451"}],"ovsdbapp/tests/functional/schema/ovn_southbound/event.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"efb62c9545694cd1b90e300b763378d35205c6fd","unresolved":true,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"class MatchFnConditionsEvent(event.WaitEvent):"},{"line_number":34,"context_line":"    def __init__(self, *args, timeout\u003d1, **kwargs):"},{"line_number":35,"context_line":"        super().__init__(*args, timeout\u003dtimeout, **kwargs)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    def match_fn(self, event, row, old):"}],"source_content_type":"text/x-python","patch_set":2,"id":"d78831e0_d6de87c5","line":34,"range":{"start_line":34,"start_character":29,"end_line":34,"end_character":40},"updated":"2023-11-16 18:16:35.000000000","message":"nit: do we need this?","commit_id":"61935457bdfec157626d0b16a2bd43913c9e2451"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"7de8d2c5247b0a9f488d674c6b1e0d0dbfa2e301","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"class MatchFnConditionsEvent(event.WaitEvent):"},{"line_number":34,"context_line":"    def __init__(self, *args, timeout\u003d1, **kwargs):"},{"line_number":35,"context_line":"        super().__init__(*args, timeout\u003dtimeout, **kwargs)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    def match_fn(self, event, row, old):"}],"source_content_type":"text/x-python","patch_set":2,"id":"2b0929a8_95ea1891","line":34,"range":{"start_line":34,"start_character":29,"end_line":34,"end_character":40},"in_reply_to":"aeba35d9_1ac0e5bb","updated":"2023-11-16 18:49:02.000000000","message":"But yeah, I see what you are saying. We could specify it when instantiating instead.","commit_id":"61935457bdfec157626d0b16a2bd43913c9e2451"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"94b2e2726b90ad2f66336bdac0f41b0939209be1","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"class MatchFnConditionsEvent(event.WaitEvent):"},{"line_number":34,"context_line":"    def __init__(self, *args, timeout\u003d1, **kwargs):"},{"line_number":35,"context_line":"        super().__init__(*args, timeout\u003dtimeout, **kwargs)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    def match_fn(self, event, row, old):"}],"source_content_type":"text/x-python","patch_set":2,"id":"aeba35d9_1ac0e5bb","line":34,"range":{"start_line":34,"start_character":29,"end_line":34,"end_character":40},"in_reply_to":"d78831e0_d6de87c5","updated":"2023-11-16 18:46:36.000000000","message":"If we don\u0027t, the threading.Event wait() call will wait forever.","commit_id":"61935457bdfec157626d0b16a2bd43913c9e2451"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"2a2966cfc9be9e80f2794ce3ea59fa0daa888771","unresolved":true,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"class MatchFnConditionsEvent(event.WaitEvent):"},{"line_number":34,"context_line":"    def __init__(self, *args, timeout\u003d1, **kwargs):"},{"line_number":35,"context_line":"        super().__init__(*args, timeout\u003dtimeout, **kwargs)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    def match_fn(self, event, row, old):"},{"line_number":38,"context_line":"        # This should only be called if we pass all other conditions"}],"source_content_type":"text/x-python","patch_set":2,"id":"88e75338_0be7c448","line":35,"range":{"start_line":35,"start_character":32,"end_line":35,"end_character":47},"updated":"2023-11-16 19:11:48.000000000","message":"I mean we could just pass it down here :) We don\u0027t pass the parameter to `MatchFnConditionsEvent` instance anywhere in the test","commit_id":"61935457bdfec157626d0b16a2bd43913c9e2451"}]}