)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0b1da406997d460856ddc077964bf6d4c3002be3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f7470e1d_a9137a4e","updated":"2025-10-15 20:01:14.000000000","message":"Hi @llm.tech.developer@gmail.com\n\nThank you for working on this bug fix; The fix looks correct. I have a couple of concerns. \n\n1) Are you using any sort of AI assistance in generating this code? If yes, please ensure that you\u0027re adhering to the OpenInfra AI policy: https://openinfra.org/legal/ai-policy ; you\u0027ll need to add \"Assisted-By\" or \"Generated-By\" in your commit message depending on the level of assistance\n2) Please add a release note for this change: https://docs.openstack.org/manila/latest/contributor/adding_release_notes.html#what-are-release-notes","commit_id":"e93965877aad260be05903b65be231d742dcf315"},{"author":{"_account_id":38463,"name":"llm tech","display_name":"llmtech","email":"llm.tech.developer@gmail.com","username":"llmtech"},"change_message_id":"2ae4c586e5230600d23f5a91a5473028875cf865","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ca86e71d_118e158b","in_reply_to":"203332fb_3b8e05e3","updated":"2025-10-20 10:05:11.000000000","message":"Done","commit_id":"e93965877aad260be05903b65be231d742dcf315"},{"author":{"_account_id":38463,"name":"llm tech","display_name":"llmtech","email":"llm.tech.developer@gmail.com","username":"llmtech"},"change_message_id":"67667c45b91d319d24994343637e1783cb7a7ed0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"203332fb_3b8e05e3","in_reply_to":"f7470e1d_a9137a4e","updated":"2025-10-17 06:31:23.000000000","message":"Thank you very much for the feedback @gouthampravi@gmail.com \nYes, I have been assisted by AI and especially in the case of tests, I have used AI generated code, but of course I have tried to understand what it is doing.\nI have also manually reproduced the bug, and after the correction, manually tested that it is working ok and returning the 400 Bad Request error.\nI will now add the labels in accordance with the AI Policy, and try to work on adding the release note, as you mentioned.","commit_id":"e93965877aad260be05903b65be231d742dcf315"},{"author":{"_account_id":38463,"name":"llm tech","display_name":"llmtech","email":"llm.tech.developer@gmail.com","username":"llmtech"},"change_message_id":"2ae4c586e5230600d23f5a91a5473028875cf865","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fbcfcaa0_79d812f4","updated":"2025-10-20 10:05:11.000000000","message":"Hi @gouthampravi@gmail.com I added the label Generated-By and also the release notes, as you mentioned.","commit_id":"f67919f80694e07e785e8efad5b86208faa549d5"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0bb2f26a11f76e6636f73a26a7bfc3268c10492d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1b373697_68a97202","updated":"2026-03-03 19:04:43.000000000","message":"LGTM, only minor suggestions inline.","commit_id":"f67919f80694e07e785e8efad5b86208faa549d5"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8446fd15f49fa9fcfb08fa32bc4707d9b89a7167","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a6d3d4a4_ddc92462","updated":"2026-03-05 15:35:10.000000000","message":"LGTM. One nit pick about the releasenotes.","commit_id":"f67919f80694e07e785e8efad5b86208faa549d5"}],"manila/api/v2/share_groups.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0bb2f26a11f76e6636f73a26a7bfc3268c10492d","unresolved":true,"context_lines":[{"line_number":242,"context_line":"                kwargs[\u0027availability_zone\u0027] \u003d az.name"},{"line_number":243,"context_line":"            except exception.AvailabilityZoneNotFound as e:"},{"line_number":244,"context_line":"                raise exc.HTTPNotFound(explanation\u003de.msg)"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"        if \u0027source_share_group_snapshot_id\u0027 in share_group:"},{"line_number":247,"context_line":"            source_share_group_snapshot_id \u003d share_group.get("},{"line_number":248,"context_line":"                \u0027source_share_group_snapshot_id\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"35d54683_6a045633","side":"PARENT","line":245,"updated":"2026-03-03 19:04:43.000000000","message":"removing these lines can be misleading when checking the commit history in the future, so I suggest not doing it","commit_id":"40107f961671fd55710f70eef30ea526298b5e22"}],"releasenotes/notes/bug-1781127-share-group-from-snapshot-bug-fix-8c4e5431ae6cceeb.yaml":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0bb2f26a11f76e6636f73a26a7bfc3268c10492d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The bug with share groups that can be created from snapshot with different share group type has been fixed."},{"line_number":5,"context_line":"    See `bug \u003chttps://bugs.launchpad.net/manila/+bug/1781127\u003e`_ for more details."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"64bdc908_3b196d77","line":5,"range":{"start_line":4,"start_character":0,"end_line":5,"end_character":81},"updated":"2026-03-03 19:04:43.000000000","message":"in the future, please keep release note lines under 79 characters","commit_id":"f67919f80694e07e785e8efad5b86208faa549d5"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"c9da9fde3a5f31c11507612de5be2230792ac3c3","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The bug with share groups that can be created from snapshot with different share group type has been fixed."},{"line_number":5,"context_line":"    See `bug \u003chttps://bugs.launchpad.net/manila/+bug/1781127\u003e`_ for more details."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"8037efec_f1cd7075","line":5,"range":{"start_line":4,"start_character":0,"end_line":5,"end_character":81},"in_reply_to":"0dfbdb2d_ce299e24","updated":"2026-03-06 18:09:04.000000000","message":"Good question. It shouldn\u0027t... The rule about 79 characters is pep8 related, but pep8 is not run here, so that\u0027s why we don\u0027t see the CI failing.","commit_id":"f67919f80694e07e785e8efad5b86208faa549d5"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"8446fd15f49fa9fcfb08fa32bc4707d9b89a7167","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The bug with share groups that can be created from snapshot with different share group type has been fixed."},{"line_number":5,"context_line":"    See `bug \u003chttps://bugs.launchpad.net/manila/+bug/1781127\u003e`_ for more details."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"0dfbdb2d_ce299e24","line":5,"range":{"start_line":4,"start_character":0,"end_line":5,"end_character":81},"in_reply_to":"64bdc908_3b196d77","updated":"2026-03-05 15:35:10.000000000","message":"Wouldn\u0027t this break the releasenotes CI run?","commit_id":"f67919f80694e07e785e8efad5b86208faa549d5"}]}