)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"62b36346b46ffb7b88a5cc448518c111a56d1d31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9ea3394d_04bdcdb7","updated":"2024-10-30 07:48:29.000000000","message":"I wonder if we want to rebase on https://review.opendev.org/c/openstack/watcher/+/932963 (or just wait for that to merge first).","commit_id":"5c627a3aa366b1ac10d904cf994a52f0c1e3ec0e"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fa2f18e45cc231ae27451933dac57242d9c6f0c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cdcb3250_34803063","in_reply_to":"9ea3394d_04bdcdb7","updated":"2024-11-05 00:52:37.000000000","message":"the version of sqlachemy installed shoudl not chagne between ubuntu 22.04 and 24.04 so that patch should not be affected this one.\n\nas such it should be safe to proceed with this patche without waiting for that to merge.","commit_id":"5c627a3aa366b1ac10d904cf994a52f0c1e3ec0e"}],"watcher/db/sqlalchemy/api.py":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1326d87077d14058e3c230269266a5fbcbd57712","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from oslo_config import cfg"},{"line_number":24,"context_line":"from oslo_db import exception as db_exc"},{"line_number":25,"context_line":"from oslo_db.sqlalchemy import enginefacade"},{"line_number":26,"context_line":"from oslo_db.sqlalchemy import utils as db_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"},{"line_number":28,"context_line":"from sqlalchemy.inspection import inspect"}],"source_content_type":"text/x-python","patch_set":1,"id":"e728caee_d02e0862","line":25,"updated":"2024-10-30 07:54:42.000000000","message":"adding link in case useful for other reviewers https://docs.openstack.org/oslo.db/4.13.4/api/oslo_db.sqlalchemy.enginefacade.html","commit_id":"5c627a3aa366b1ac10d904cf994a52f0c1e3ec0e"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fa2f18e45cc231ae27451933dac57242d9c6f0c7","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from oslo_config import cfg"},{"line_number":24,"context_line":"from oslo_db import exception as db_exc"},{"line_number":25,"context_line":"from oslo_db.sqlalchemy import enginefacade"},{"line_number":26,"context_line":"from oslo_db.sqlalchemy import utils as db_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"},{"line_number":28,"context_line":"from sqlalchemy.inspection import inspect"}],"source_content_type":"text/x-python","patch_set":1,"id":"26d3cb3b_0bea6607","line":25,"in_reply_to":"e728caee_d02e0862","updated":"2024-11-05 00:52:37.000000000","message":"this is consistent with the replacement in other projects like rally\n\nhttps://review.opendev.org/c/openstack/rally/+/905343/1/rally/common/db/api.py\n\nso this looks ok to me and the ci seams to agree so i think we can move forward with this.","commit_id":"5c627a3aa366b1ac10d904cf994a52f0c1e3ec0e"}]}