)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"e91de860b306683bf5d953f710a9ec2c7855a346","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6191df7f_cfb1ffca","updated":"2026-03-17 17:38:13.000000000","message":"Fixed the comment: https://review.opendev.org/c/openstack/manila/+/973777/comment/84039c43_4bd72618/\n\nTested sync snapmirror workflow and is working as expected. \nCLI results: https://paste.opendev.org/show/be5k9pXncHZ1cfiI8rmN/\n\nThank you!!","commit_id":"fb1c4e2ffbcee866d1f8ec06fbce4da7d7ad4a30"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"31c5202c57a1302c1f2d2886d5ce10b1f6ef3ec3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4b0c2caf_f9208ddc","updated":"2026-03-18 05:04:17.000000000","message":"Thanks for addressing this @saikumar1016@gmail.com. I\u0027ve one comment inline","commit_id":"fb1c4e2ffbcee866d1f8ec06fbce4da7d7ad4a30"},{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"aa1135f20d82850d54d30d5f4858e60aa9d8a6b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b1daa74b_fe90d2ec","updated":"2026-04-06 09:57:57.000000000","message":"Hi @ces.eduardo98@gmail.com\n\nWould be able to take a look at this when you get some free time? \n\nThank you!!","commit_id":"3911d2df91f1bf858170d5d0810f822f3423bfa7"},{"author":{"_account_id":38371,"name":"Kumar Tadepalli","display_name":"Kumar Tadepalli","email":"kumart.lead@gmail.com","username":"kumart","status":"NetApp"},"change_message_id":"89c29fe8f28512cdea254184f3d78767d4686b40","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c8a45fd5_d16d883a","updated":"2026-03-18 10:09:46.000000000","message":"LGTM","commit_id":"3911d2df91f1bf858170d5d0810f822f3423bfa7"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"af391621af2a3bb2c0a02699348d4a2ea708e163","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fd5f9033_683fe98b","updated":"2026-04-22 04:34:36.000000000","message":"LGTM ..!","commit_id":"3911d2df91f1bf858170d5d0810f822f3423bfa7"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"099aa2f9ac050ace8e50cdbc9ea9724a2dd0b864","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d230e7f9_e4813fcc","updated":"2026-03-18 16:30:49.000000000","message":"LGTM, thanks Sai!","commit_id":"3911d2df91f1bf858170d5d0810f822f3423bfa7"},{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"46539a8e48bd3691d12d40b7a6e31a674d5e07b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"54fcfd75_f2687a40","updated":"2026-03-18 07:02:18.000000000","message":"Thank you so much @gouthampravi@gmail.com, addressed the comments and re-tested the workflow as well.","commit_id":"3911d2df91f1bf858170d5d0810f822f3423bfa7"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"20fc3e3d9c00deb743cad50f8689482e3c97e57c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fde47650_3fb95777","updated":"2026-04-21 13:39:36.000000000","message":"Upgrading to +2. LGTM.","commit_id":"3911d2df91f1bf858170d5d0810f822f3423bfa7"}],"manila/share/manager.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"31c5202c57a1302c1f2d2886d5ce10b1f6ef3ec3","unresolved":true,"context_lines":[{"line_number":2809,"context_line":"        replica_list \u003d [self._get_share_instance_dict(context, r)"},{"line_number":2810,"context_line":"                        for r in replica_list]"},{"line_number":2811,"context_line":""},{"line_number":2812,"context_line":"        for r in replica_list:"},{"line_number":2813,"context_line":"            r[\u0027metadata\u0027] \u003d self.db.share_replica_metadata_get("},{"line_number":2814,"context_line":"                context, r[\"id\"]) or {}"},{"line_number":2815,"context_line":""},{"line_number":2816,"context_line":"        share_replica \u003d self._get_share_instance_dict(context, share_replica)"},{"line_number":2817,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"931c7fe3_1e2f66c9","line":2814,"range":{"start_line":2812,"start_character":8,"end_line":2814,"end_character":39},"updated":"2026-03-18 05:04:17.000000000","message":"This can now be removed too","commit_id":"fb1c4e2ffbcee866d1f8ec06fbce4da7d7ad4a30"},{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"46539a8e48bd3691d12d40b7a6e31a674d5e07b6","unresolved":false,"context_lines":[{"line_number":2809,"context_line":"        replica_list \u003d [self._get_share_instance_dict(context, r)"},{"line_number":2810,"context_line":"                        for r in replica_list]"},{"line_number":2811,"context_line":""},{"line_number":2812,"context_line":"        for r in replica_list:"},{"line_number":2813,"context_line":"            r[\u0027metadata\u0027] \u003d self.db.share_replica_metadata_get("},{"line_number":2814,"context_line":"                context, r[\"id\"]) or {}"},{"line_number":2815,"context_line":""},{"line_number":2816,"context_line":"        share_replica \u003d self._get_share_instance_dict(context, share_replica)"},{"line_number":2817,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"d5f7053c_ac16abe7","line":2814,"range":{"start_line":2812,"start_character":8,"end_line":2814,"end_character":39},"in_reply_to":"931c7fe3_1e2f66c9","updated":"2026-03-18 07:02:18.000000000","message":"Done","commit_id":"fb1c4e2ffbcee866d1f8ec06fbce4da7d7ad4a30"}],"releasenotes/notes/bug-2144676-fixing-redundant-DB-lookup-for-share-replica-metadata-172a2c64533493a6.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"cdef143ef601cbfd67b147c474e76de41d1f2f8d","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Improved share replica replication efficiency by avoiding redundant"},{"line_number":5,"context_line":"    database lookups for replica metadata. Metadata is now populated"},{"line_number":6,"context_line":"    from the ``share_instance_metadata`` already available when building"},{"line_number":7,"context_line":"    the share instance/replica dict, reducing per-replica DB queries."},{"line_number":8,"context_line":"    For more details, please refer to"},{"line_number":9,"context_line":"    `Launchpad bug #2144676 \u003chttps://bugs.launchpad.net/manila/+bug/2144676\u003e`_."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"19b76b48_8dee6cb1","line":7,"range":{"start_line":5,"start_character":43,"end_line":7,"end_character":69},"updated":"2026-03-18 05:04:58.000000000","message":"we can drop this; this information isn\u0027t useful to users or operators","commit_id":"fb1c4e2ffbcee866d1f8ec06fbce4da7d7ad4a30"},{"author":{"_account_id":36179,"name":"Saikumar Pulluri","display_name":"Saikumar Pulluri","email":"saikumar1016@gmail.com","username":"pulluri"},"change_message_id":"46539a8e48bd3691d12d40b7a6e31a674d5e07b6","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Improved share replica replication efficiency by avoiding redundant"},{"line_number":5,"context_line":"    database lookups for replica metadata. Metadata is now populated"},{"line_number":6,"context_line":"    from the ``share_instance_metadata`` already available when building"},{"line_number":7,"context_line":"    the share instance/replica dict, reducing per-replica DB queries."},{"line_number":8,"context_line":"    For more details, please refer to"},{"line_number":9,"context_line":"    `Launchpad bug #2144676 \u003chttps://bugs.launchpad.net/manila/+bug/2144676\u003e`_."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"be32a92f_139b3f98","line":7,"range":{"start_line":5,"start_character":43,"end_line":7,"end_character":69},"in_reply_to":"19b76b48_8dee6cb1","updated":"2026-03-18 07:02:18.000000000","message":"Done","commit_id":"fb1c4e2ffbcee866d1f8ec06fbce4da7d7ad4a30"}]}
