)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"762a4558f18ee3bb0791cddcac74ca12f701200d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d6d2d13e_4e98472a","updated":"2024-05-28 03:44:40.000000000","message":"work in progress, but review it for me","commit_id":"200281246fd36417cb616e3496a5530d131c9202"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"fbf783f2829ca14efd4b58f58f5c89e07c03444f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f63df799_b06503a7","in_reply_to":"d6d2d13e_4e98472a","updated":"2024-05-28 16:29:09.000000000","message":"done.","commit_id":"200281246fd36417cb616e3496a5530d131c9202"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"f181e912a0aaefae9fbb49812c1c05bfff4329c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3f41f990_ca192066","updated":"2024-05-28 11:51:58.000000000","message":"Thanks for the patch.\nThere are missing issues with sqlalchemy 2.x for example https://github.com/openstack/monasca-api/blob/master/monasca_api/common/repositories/sqla/notification_method_type_repository.py#L33\n\nYou can see errors from https://zuul.opendev.org/t/openstack/build/b55e0a89897541d7ab9ee3d8a3820ccd/log/controller/logs/screen-monasca-api.txt#114-160","commit_id":"d1a3f92713974452a3175e6928f709b1c72a0423"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"409cd243cbbcec435d1ada45d00904a5453572ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7ac6df82_9462ba09","in_reply_to":"3f41f990_ca192066","updated":"2024-05-28 16:15:23.000000000","message":"oh, I skip it because no unittest, fix in next patchset, if ci is green, I try adding unittest for it","commit_id":"d1a3f92713974452a3175e6928f709b1c72a0423"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"51301e7922771fa2cf90966c00024e000342f2fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"36212c3b_19d8559d","updated":"2024-05-28 16:28:50.000000000","message":"plz review for me","commit_id":"33d8e5db59fc06fea439ed2d525ec554d88fee45"},{"author":{"_account_id":35988,"name":"Hasan Acar","display_name":"Hasan Acar","email":"hasan.acar@tubitak.gov.tr","username":"hasanacar"},"change_message_id":"228e31a98cc1f9cbd8c8b8e2b29ba5c23915f384","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ca159c98_08beb675","in_reply_to":"36212c3b_19d8559d","updated":"2024-05-30 08:28:07.000000000","message":"Most of the errors are \"tuple indices must be integers or slices, not str\". RowProxy has been removed with SQLAlchemy 2.  you should use \"_mapping\" instead of string key directly in all required query outputs. \n\nExample : (https://docs.sqlalchemy.org/en/20/changelog/migration_14.html#change-4710-orm)","commit_id":"33d8e5db59fc06fea439ed2d525ec554d88fee45"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"747f7f90a093ea88729a7ffc245f917040c98564","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e427a857_b15e9a50","updated":"2024-05-29 06:15:51.000000000","message":"recheck tempest jobs are not triggered","commit_id":"6048146fbae64c184703590446ea33aba9941dc4"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"ee2fff8b6a840d58da5f8ca0413d57f57fbf87bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3a091622_fd0bef8b","in_reply_to":"06aaa82b_80a8eb03","updated":"2024-05-29 07:41:01.000000000","message":"it seems there are changes on zuul. we can ask openstack tc.","commit_id":"6048146fbae64c184703590446ea33aba9941dc4"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"0af1729f37bf18d21d26620ae4c0e7f3c8c1c91e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"06aaa82b_80a8eb03","in_reply_to":"e427a857_b15e9a50","updated":"2024-05-29 07:01:36.000000000","message":"do you know how to re invoke pipeline?","commit_id":"6048146fbae64c184703590446ea33aba9941dc4"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"43b4c224892efc3bf6f66044f1e24505fae5bb34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"54552d17_5c607ed0","updated":"2024-05-29 08:00:46.000000000","message":"I sent an email to tc for supporting","commit_id":"f5faa4780cb7642b2cd8268860489f5f4fb80f10"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"b3032e81d38f7fcc1272f518128cf3bc628e4eb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c1c6f93c_a4f9d2c8","updated":"2024-05-29 08:55:09.000000000","message":"recheck - testing missing tempest jobs","commit_id":"f5faa4780cb7642b2cd8268860489f5f4fb80f10"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"e64454226d497fadb7a4f84ceede5a60fa3846e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"f6d7cafe_6f5d2655","updated":"2024-06-03 17:58:25.000000000","message":"recheck","commit_id":"908815723b8bed0d7a7c2e57411c34377512dfb6"},{"author":{"_account_id":20865,"name":"Gökhan","email":"gokhan.isik@tubitak.gov.tr","username":"gokhan.isik"},"change_message_id":"6a9409c8876d888f0a231bee05877bf72d5957e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"37a1a39c_1c32d856","updated":"2024-06-03 15:36:11.000000000","message":"recheck intermittent failure","commit_id":"908815723b8bed0d7a7c2e57411c34377512dfb6"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"93ca18e3d2607ca929794a27c6de8876d6c31662","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"bc08aa32_748fb192","updated":"2024-06-03 16:09:52.000000000","message":"regate","commit_id":"908815723b8bed0d7a7c2e57411c34377512dfb6"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"3f49802d752a2c97de67855c561b980c25f5038d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"ce140b9f_d4de6a24","in_reply_to":"37a1a39c_1c32d856","updated":"2024-06-03 17:41:54.000000000","message":"don\u0027t why, but let me check","commit_id":"908815723b8bed0d7a7c2e57411c34377512dfb6"}],"monasca_api/common/repositories/sqla/alarms_repository.py":[{"author":{"_account_id":35988,"name":"Hasan Acar","display_name":"Hasan Acar","email":"hasan.acar@tubitak.gov.tr","username":"hasanacar"},"change_message_id":"1eeca1d0632f4bcc354e636e284b3295c41fc5d7","unresolved":true,"context_lines":[{"line_number":221,"context_line":"                          \u0027link\u0027: bindparam(\u0027b_link\u0027),"},{"line_number":222,"context_line":"                          \u0027updated_at\u0027: func.now()}"},{"line_number":223,"context_line":"            # TODO(thuvh) find better solution to get state from row"},{"line_number":224,"context_line":"            prev_state \u003d prev_alarm._mapping[\u0027state\u0027]"},{"line_number":225,"context_line":"            if state !\u003d prev_state:"},{"line_number":226,"context_line":"                parms[\u0027b_state\u0027] \u003d state"},{"line_number":227,"context_line":"                set_values[\u0027state\u0027] \u003d bindparam(\u0027b_state\u0027)"}],"source_content_type":"text/x-python","patch_set":10,"id":"487aab63_3d57ef36","line":224,"range":{"start_line":224,"start_character":0,"end_line":224,"end_character":53},"updated":"2024-05-30 08:42:17.000000000","message":"Just like here, you need to use _mapping on the v2/reference/* as well\n\nAn example of where you need to change:\nhttps://opendev.org/openstack/monasca-api/src/commit/a24fd834d075978bd7f43346fb3b2a440c4d2064/monasca_api/v2/reference/alarms.py#L211","commit_id":"5c7268fb52ae26464d6ad36b418311720a9716da"},{"author":{"_account_id":13114,"name":"Hoai-Thu Vuong","email":"thuvh87@gmail.com","username":"thuvh"},"change_message_id":"7f13e66a118a3e0ae8949a65f2ae98f62f74e001","unresolved":false,"context_lines":[{"line_number":221,"context_line":"                          \u0027link\u0027: bindparam(\u0027b_link\u0027),"},{"line_number":222,"context_line":"                          \u0027updated_at\u0027: func.now()}"},{"line_number":223,"context_line":"            # TODO(thuvh) find better solution to get state from row"},{"line_number":224,"context_line":"            prev_state \u003d prev_alarm._mapping[\u0027state\u0027]"},{"line_number":225,"context_line":"            if state !\u003d prev_state:"},{"line_number":226,"context_line":"                parms[\u0027b_state\u0027] \u003d state"},{"line_number":227,"context_line":"                set_values[\u0027state\u0027] \u003d bindparam(\u0027b_state\u0027)"}],"source_content_type":"text/x-python","patch_set":10,"id":"351287ee_eaa867a2","line":224,"range":{"start_line":224,"start_character":0,"end_line":224,"end_character":53},"in_reply_to":"487aab63_3d57ef36","updated":"2024-06-01 19:47:30.000000000","message":"Done","commit_id":"5c7268fb52ae26464d6ad36b418311720a9716da"}]}