)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"2434626758cd30853b9c4a140d01d317d887a851","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"973b6b04_8c6b3f23","updated":"2022-11-04 16:51:46.000000000","message":"The error in tox-functional is not related to the patch. The job timed out also here: https://review.opendev.org/c/openinfra/refstack/+/861632","commit_id":"08e9b0152f2e3f824067849bc3916e22ef14a73f"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"1badff959c4db46fcf65c8b3d4b42307e90b7b57","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"cea310bf_a720c772","updated":"2022-11-04 16:37:34.000000000","message":"This should fix the issue with the verifier() method. I just need to check something first before I remove the WIP flag. ","commit_id":"08e9b0152f2e3f824067849bc3916e22ef14a73f"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"fd291ab664707596514df83e3ae726591fca5462","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"53c642fa_ab6a3ba0","updated":"2023-02-01 10:53:43.000000000","message":"looks good, 2 questions inline ","commit_id":"87a12dfc18b119f25dad063191d1dd01b13889ac"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"b4cde7620988f9da746b84bc8a8787a3dfc11ca3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"7e92509b_fc4c19cb","updated":"2023-02-01 08:29:09.000000000","message":"recheck \nthe error is not caused by the patch ","commit_id":"87a12dfc18b119f25dad063191d1dd01b13889ac"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"f99bd0b8b62876c42d27b9575b762463d85a51f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3077e8eb_c38b97bd","in_reply_to":"53c642fa_ab6a3ba0","updated":"2023-02-01 12:17:54.000000000","message":"Fixed:)","commit_id":"87a12dfc18b119f25dad063191d1dd01b13889ac"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"3be4736945f73d85f5cb185caef0db629a13cbaf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a7271690_7703edc1","updated":"2023-02-01 19:21:23.000000000","message":"Thanks for your hard work on this! this sounds like a lengthy whack-a-mole. ","commit_id":"d18f8ad221c41e287c247d888122cb7f93dd8f46"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"53ca8d6a45e594a90acecdc4f78ec44bededd5ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"5d969d29_37add8b1","updated":"2023-02-01 14:52:53.000000000","message":"lgtm, thanks","commit_id":"d18f8ad221c41e287c247d888122cb7f93dd8f46"}],"refstack/db/sqlalchemy/api.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"fd291ab664707596514df83e3ae726591fca5462","unresolved":true,"context_lines":[{"line_number":200,"context_line":"        meta_item \u003d (session.query(models.TestMeta)"},{"line_number":201,"context_line":"                     .filter_by(test_id\u003dtest_id)"},{"line_number":202,"context_line":"                     .filter_by(meta_key\u003dkey).first() or models.TestMeta())"},{"line_number":203,"context_line":"        session.commit()"},{"line_number":204,"context_line":"        session.close()"},{"line_number":205,"context_line":"        meta_item.test_id \u003d test_id"},{"line_number":206,"context_line":"        meta_item.meta_key \u003d key"}],"source_content_type":"text/x-python","patch_set":13,"id":"b89656c8_976c4d2c","line":203,"range":{"start_line":203,"start_character":16,"end_line":203,"end_character":22},"updated":"2023-02-01 10:53:43.000000000","message":"should we call commit here? I was under an impression that we call commit only after save or delete (a change in the database)","commit_id":"87a12dfc18b119f25dad063191d1dd01b13889ac"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"f99bd0b8b62876c42d27b9575b762463d85a51f1","unresolved":false,"context_lines":[{"line_number":200,"context_line":"        meta_item \u003d (session.query(models.TestMeta)"},{"line_number":201,"context_line":"                     .filter_by(test_id\u003dtest_id)"},{"line_number":202,"context_line":"                     .filter_by(meta_key\u003dkey).first() or models.TestMeta())"},{"line_number":203,"context_line":"        session.commit()"},{"line_number":204,"context_line":"        session.close()"},{"line_number":205,"context_line":"        meta_item.test_id \u003d test_id"},{"line_number":206,"context_line":"        meta_item.meta_key \u003d key"}],"source_content_type":"text/x-python","patch_set":13,"id":"e508949c_fd1237c9","line":203,"range":{"start_line":203,"start_character":16,"end_line":203,"end_character":22},"in_reply_to":"b89656c8_976c4d2c","updated":"2023-02-01 12:17:54.000000000","message":"Oh, yes, we can delete this. I left it here by accident. The jobs were passing because it shouldn\u0027t affect the functionality of the current version of the code (I hope :D). I double-checked the code again and removed the commits if they weren\u0027t necessary.","commit_id":"87a12dfc18b119f25dad063191d1dd01b13889ac"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"fd291ab664707596514df83e3ae726591fca5462","unresolved":true,"context_lines":[{"line_number":201,"context_line":"                     .filter_by(test_id\u003dtest_id)"},{"line_number":202,"context_line":"                     .filter_by(meta_key\u003dkey).first() or models.TestMeta())"},{"line_number":203,"context_line":"        session.commit()"},{"line_number":204,"context_line":"        session.close()"},{"line_number":205,"context_line":"        meta_item.test_id \u003d test_id"},{"line_number":206,"context_line":"        meta_item.meta_key \u003d key"},{"line_number":207,"context_line":"        meta_item.value \u003d value"}],"source_content_type":"text/x-python","patch_set":13,"id":"9c96e232_b9453dbf","line":204,"range":{"start_line":204,"start_character":8,"end_line":204,"end_character":23},"updated":"2023-02-01 10:53:43.000000000","message":"does the session need to be closed before the next commit? Just wondering, it looks strange","commit_id":"87a12dfc18b119f25dad063191d1dd01b13889ac"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"f99bd0b8b62876c42d27b9575b762463d85a51f1","unresolved":false,"context_lines":[{"line_number":201,"context_line":"                     .filter_by(test_id\u003dtest_id)"},{"line_number":202,"context_line":"                     .filter_by(meta_key\u003dkey).first() or models.TestMeta())"},{"line_number":203,"context_line":"        session.commit()"},{"line_number":204,"context_line":"        session.close()"},{"line_number":205,"context_line":"        meta_item.test_id \u003d test_id"},{"line_number":206,"context_line":"        meta_item.meta_key \u003d key"},{"line_number":207,"context_line":"        meta_item.value \u003d value"}],"source_content_type":"text/x-python","patch_set":13,"id":"11599883_2d568262","line":204,"range":{"start_line":204,"start_character":8,"end_line":204,"end_character":23},"in_reply_to":"9c96e232_b9453dbf","updated":"2023-02-01 12:17:54.000000000","message":"I left it here by accident too. Fixed.","commit_id":"87a12dfc18b119f25dad063191d1dd01b13889ac"}]}