)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0e36fdac413649ba9b4b5dce680826e4d6cf9192","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"68352bea_20026910","updated":"2023-07-18 10:49:04.000000000","message":"LGTM, thank you Chuan! Could you please follow-up with a release note for this change?","commit_id":"8d2406927e7b185c8999ad050f8ee765c3db8208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0f1487c41d190dc72deb531b813af77b4db56529","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"918baa00_ae75f88a","updated":"2023-07-17 20:20:32.000000000","message":"recheck\n\nlogs are gone\n\nThank you for this improvement! LGTM","commit_id":"8d2406927e7b185c8999ad050f8ee765c3db8208"},{"author":{"_account_id":33038,"name":"Chuan Miao","email":"chuan137@gmail.com","username":"chuan137"},"change_message_id":"829ba3080bcde1715f87e29aac5106e6c7242ac7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"626c38e9_a5ca0898","in_reply_to":"5b5b8933_bccd1bb6","updated":"2023-08-16 07:48:40.000000000","message":"Sure, release note added","commit_id":"8d2406927e7b185c8999ad050f8ee765c3db8208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"033d06f5c2d6fa8c53a1b6e0ad357d8353446e6d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5b5b8933_bccd1bb6","in_reply_to":"68352bea_20026910","updated":"2023-08-15 22:52:15.000000000","message":"This would be good to have; \n\nChuan: could you please resolve this comment?","commit_id":"8d2406927e7b185c8999ad050f8ee765c3db8208"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"4f1086acc085eb19e46db48da3921e02e7816ec7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"92497760_ab4a0fa0","updated":"2023-08-28 11:23:37.000000000","message":"LGTM, thanks for the enhancement, Chuan!","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"d84204ccda47891cf130e45a2a2bfd2c221e0e85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5dd1d7f2_e44c9129","updated":"2023-08-31 17:30:55.000000000","message":"Revert patch is fixing it: https://review.opendev.org/c/openstack/manila/+/893384","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"106c1593e6db6510e58b576084dffdabdac59acb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5b51fdec_56464941","updated":"2023-08-16 14:34:51.000000000","message":"Thanks, LGTM!","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":33038,"name":"Chuan Miao","email":"chuan137@gmail.com","username":"chuan137"},"change_message_id":"3ae8ec720206682656dce1ce9a0809b481cf6a73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c229a4ac_9c8d33e1","updated":"2024-07-01 09:00:19.000000000","message":"continue in https://review.opendev.org/c/openstack/manila/+/898306","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"}],"manila/db/sqlalchemy/api.py":[{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"d84204ccda47891cf130e45a2a2bfd2c221e0e85","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"d5ca76d8_03966c00","line":1912,"updated":"2023-08-31 17:30:55.000000000","message":"\"ShareInstance\" hasn\u0027t any \"share\" field. NetApp CI is broken because of this issue:\n\nhttps://netappopenstacklogserver.s3-us-west-1.amazonaws.com/logs/55/883655/3/upstream-check/manila-tempest-plugin-ontap-dhss/a2853dc/controller/logs/screen-m-sch.txt","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"bb42c16aa9599c09d7b65ab539276e62faebe97a","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"5b68bb3c_a23b0684","line":1912,"in_reply_to":"318f2b4f_2efcbb15","updated":"2023-09-12 19:51:50.000000000","message":"no worries :)","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":33038,"name":"Chuan Miao","email":"chuan137@gmail.com","username":"chuan137"},"change_message_id":"3ae8ec720206682656dce1ce9a0809b481cf6a73","unresolved":false,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"449996ea_b64fc4e9","line":1912,"in_reply_to":"48331132_d9600d37","updated":"2024-07-01 09:00:19.000000000","message":"Acknowledged","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":33038,"name":"Chuan Miao","email":"chuan137@gmail.com","username":"chuan137"},"change_message_id":"59f95a50e697da2ac256095097e5e8103779ac19","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"881d6018_27ca5ea3","line":1912,"in_reply_to":"52f2877f_603a34ed","updated":"2023-10-12 19:14:39.000000000","message":"Hi Goutham, can you reopen the review please? Failed to resubmit, \" ! [remote rejected]     HEAD -\u003e refs/for/master%topic\u003dbug/2020187 (change https://review.opendev.org/c/openstack/manila/+/883655 closed)\". Thanks Chuan","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"a1b087284e04b6488fe6be0cb3905ddb09a6719b","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"e7ff7a39_9f9b5906","line":1912,"in_reply_to":"5b68bb3c_a23b0684","updated":"2023-09-12 19:57:17.000000000","message":"I like this optimization; i think it\u0027s worth bringing that in as well! :)","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f44c623afad7b159304d4737958af515d487f56e","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"a3aa377f_5b9c15bd","line":1912,"in_reply_to":"881d6018_27ca5ea3","updated":"2023-10-12 21:09:46.000000000","message":"Hi Chuan; \n\nthe change has merged.. so it can\u0027t be re-opened.. You can either \n\n1) \"revert\" this change and improve it; i.e., click \"Revert\" on gerrit - it\u0027ll create a change for you, download the change locally, and make further changes necessary; or\n2) submit a different change by deleting the \"Change_ID\" from your commit message; this causes \"git review\" to make up another change id..","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b5400904fa1aded41c87808a9d421b5e74900fe7","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"48331132_d9600d37","line":1912,"in_reply_to":"a3aa377f_5b9c15bd","updated":"2023-10-12 21:10:41.000000000","message":"To follow one (1); please \"Revert\" this: https://review.opendev.org/c/openstack/manila/+/893384","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"59ccebc5a576b02437268a7c7d14fbc2c89595f8","unresolved":false,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"52f2877f_603a34ed","line":1912,"in_reply_to":"d54e30c0_3b2afea7","updated":"2023-09-26 14:36:41.000000000","message":"Hi Chuan, no - we were looking for your attention here... if you\u0027re able to resubmit the code with the missing bits, we can review and fix this bug properly again..","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":18816,"name":"Maurice Escher","display_name":"carthaca","email":"maurice.escher@sap.com","username":"mapocace"},"change_message_id":"e26eadd5ba848f308a95c0b2d4e86571a82d8d9c","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"318f2b4f_2efcbb15","line":1912,"in_reply_to":"d5ca76d8_03966c00","updated":"2023-09-07 15:16:04.000000000","message":"Sorry for that, apparently that was build on top of an optimisation that we made long ago in our fork https://github.com/sapcc/manila/commit/286120772e2feead46c16af0410b5b37628e5b3f","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"},{"author":{"_account_id":33038,"name":"Chuan Miao","email":"chuan137@gmail.com","username":"chuan137"},"change_message_id":"ee1306e34c18e1eab073b24523f5f95d9e789461","unresolved":true,"context_lines":[{"line_number":1909,"context_line":"    result \u003d model_query("},{"line_number":1910,"context_line":"        context, models.Share, func.sum(models.Share.size),"},{"line_number":1911,"context_line":"    ).join("},{"line_number":1912,"context_line":"        models.ShareInstance.share,"},{"line_number":1913,"context_line":"    ).filter(or_("},{"line_number":1914,"context_line":"        models.ShareInstance.host \u003d\u003d host,"},{"line_number":1915,"context_line":"        models.ShareInstance.host.like(\"{0}#%\".format(host)),"}],"source_content_type":"text/x-python","patch_set":3,"id":"d54e30c0_3b2afea7","line":1912,"in_reply_to":"e7ff7a39_9f9b5906","updated":"2023-09-26 14:11:45.000000000","message":"Hi, \n\nsorry for the confusions. How should I continue, can I merge the missing code and resubmit the changes, or another Change Request is expected?\n\nBR, Chuan","commit_id":"e885852648c5593d8662b30eb7cef0e08dfe0b70"}]}