)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"e2d926023d320c2e1601720f1eeeb3ac93a5cad5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a04ebee7_d6a231b4","updated":"2026-09-22 12:06:50.000000000","message":"Changes looks good, I have added one query, please take a look.","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"abe18c17e844845b16d4f7e5ac483c0fec4d8d87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5b12acb9_5f6a1857","updated":"2026-09-23 05:50:26.000000000","message":"LGTM - Lets wait for clean NetApp CI run.","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"739edab63332cb98a0bfb4444c888d435e124a49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"434bd3ce_909eb693","updated":"2026-09-23 10:05:13.000000000","message":"LGTM ..!","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0865f059e519d01c200b83d94b526e55af4dee15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"67b630da_c135f5d6","updated":"2026-09-23 11:28:46.000000000","message":"Thank you for working on this change, Medha. Please take a look at the comments inline","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":39161,"name":"Medha Choudhary netapp","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"917cee03190e07e28d7e5f1ded5c36a75d9ff8a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"897db050_a2d0b3ec","updated":"2026-09-23 11:24:04.000000000","message":"run-NetApp CI","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":39161,"name":"Medha Choudhary netapp","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"bdc919e53b7a04dc3ab6dd267fdb4fa7aeb1abf4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ddd34cdc_8b96ef37","in_reply_to":"5b12acb9_5f6a1857","updated":"2026-09-23 10:24:30.000000000","message":"Hi Anoop, the NetApp no-dhss fail is a pre-existing CG snapshot cleanup flake (locked snapshot on teardown), not this ss_ou change.","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":39161,"name":"Medha Choudhary netapp","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"37d91925ca019a216ba9a3241d1bbef79c6f48cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"68d070d0_6a94a9ee","in_reply_to":"67b630da_c135f5d6","updated":"2026-09-23 13:56:39.000000000","message":"done","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"4cd92f0a3dce4c8421d10229bc4fb916e02fdda0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"13fe7e10_922de095","updated":"2026-09-23 15:04:00.000000000","message":"Thanks for the update on CI run. LGTM","commit_id":"a7805c94d9f74d81215ae16181423e232023ae04"}],"manila_tempest_tests/config.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0865f059e519d01c200b83d94b526e55af4dee15","unresolved":true,"context_lines":[{"line_number":248,"context_line":"                      \"ss_dns_ip:value, ss_user:value, ss_password\u003dvalue, \""},{"line_number":249,"context_line":"                      \"ss_domain:value, ss_server:value, ss_ou:value. \""},{"line_number":250,"context_line":"                      \"ss_ou is optional and maps to the security service \""},{"line_number":251,"context_line":"                      \"organizational unit (API microversion 2.44+). \""},{"line_number":252,"context_line":"                      \"oslo.config Dict values are comma-separated, so use \""},{"line_number":253,"context_line":"                      \"semicolons in ss_ou.\"),"},{"line_number":254,"context_line":"    cfg.ListOpt(\"capability_encryption_support\","},{"line_number":255,"context_line":"                default\u003d[],"},{"line_number":256,"context_line":"                help\u003d\"Encryption support capability. Possible values are \""}],"source_content_type":"text/x-python","patch_set":3,"id":"40135f1e_a5bc50db","line":253,"range":{"start_line":251,"start_character":69,"end_line":253,"end_character":43},"updated":"2026-09-23 11:28:46.000000000","message":"Please also add a mention to this in the commit message","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":39161,"name":"Medha Choudhary netapp","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"37d91925ca019a216ba9a3241d1bbef79c6f48cd","unresolved":false,"context_lines":[{"line_number":248,"context_line":"                      \"ss_dns_ip:value, ss_user:value, ss_password\u003dvalue, \""},{"line_number":249,"context_line":"                      \"ss_domain:value, ss_server:value, ss_ou:value. \""},{"line_number":250,"context_line":"                      \"ss_ou is optional and maps to the security service \""},{"line_number":251,"context_line":"                      \"organizational unit (API microversion 2.44+). \""},{"line_number":252,"context_line":"                      \"oslo.config Dict values are comma-separated, so use \""},{"line_number":253,"context_line":"                      \"semicolons in ss_ou.\"),"},{"line_number":254,"context_line":"    cfg.ListOpt(\"capability_encryption_support\","},{"line_number":255,"context_line":"                default\u003d[],"},{"line_number":256,"context_line":"                help\u003d\"Encryption support capability. Possible values are \""}],"source_content_type":"text/x-python","patch_set":3,"id":"3194dfdb_d22d86f6","line":253,"range":{"start_line":251,"start_character":69,"end_line":253,"end_character":43},"in_reply_to":"40135f1e_a5bc50db","updated":"2026-09-23 13:56:39.000000000","message":"Done","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"}],"manila_tempest_tests/tests/api/base.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0865f059e519d01c200b83d94b526e55af4dee15","unresolved":true,"context_lines":[{"line_number":222,"context_line":"            # OU is optional. oslo.config Dict() splits on commas, so a"},{"line_number":223,"context_line":"            # multi-component DN must use semicolons in tempest.conf."},{"line_number":224,"context_line":"            ss_ou \u003d ss_config.get(\"ss_ou\")"},{"line_number":225,"context_line":"            if ss_ou:"},{"line_number":226,"context_line":"                ss_params[\"ou\"] \u003d ss_ou.replace(\";\", \",\")"},{"line_number":227,"context_line":"            ss_type \u003d ss_config.get(\"ss_type\")"},{"line_number":228,"context_line":"            security_service \u003d cls.create_security_service("}],"source_content_type":"text/x-python","patch_set":3,"id":"07de1c71_1d693f72","line":225,"range":{"start_line":225,"start_character":12,"end_line":225,"end_character":21},"updated":"2026-09-23 11:28:46.000000000","message":"Tempest is branchless, so we need to be safe in case someone mistakenly configures this.\n\n```suggestion\n            if ss_ou and utils.is_microversion_supported(\u00272.44\u0027):\n```","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":39161,"name":"Medha Choudhary netapp","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"0eb8cee10f98eea3b7107f60e96b149ee485248e","unresolved":false,"context_lines":[{"line_number":222,"context_line":"            # OU is optional. oslo.config Dict() splits on commas, so a"},{"line_number":223,"context_line":"            # multi-component DN must use semicolons in tempest.conf."},{"line_number":224,"context_line":"            ss_ou \u003d ss_config.get(\"ss_ou\")"},{"line_number":225,"context_line":"            if ss_ou:"},{"line_number":226,"context_line":"                ss_params[\"ou\"] \u003d ss_ou.replace(\";\", \",\")"},{"line_number":227,"context_line":"            ss_type \u003d ss_config.get(\"ss_type\")"},{"line_number":228,"context_line":"            security_service \u003d cls.create_security_service("}],"source_content_type":"text/x-python","patch_set":3,"id":"def927a0_9b01c4c2","line":225,"range":{"start_line":225,"start_character":12,"end_line":225,"end_character":21},"in_reply_to":"07de1c71_1d693f72","updated":"2026-09-23 13:56:54.000000000","message":"Done","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":36180,"name":"Gireesh Awasthi","display_name":"Gireesh","email":"gawasthi2010@gmail.com","username":"agireesh","status":"NetApp"},"change_message_id":"e2d926023d320c2e1601720f1eeeb3ac93a5cad5","unresolved":true,"context_lines":[{"line_number":223,"context_line":"            # multi-component DN must use semicolons in tempest.conf."},{"line_number":224,"context_line":"            ss_ou \u003d ss_config.get(\"ss_ou\")"},{"line_number":225,"context_line":"            if ss_ou:"},{"line_number":226,"context_line":"                ss_params[\"ou\"] \u003d ss_ou.replace(\";\", \",\")"},{"line_number":227,"context_line":"            ss_type \u003d ss_config.get(\"ss_type\")"},{"line_number":228,"context_line":"            security_service \u003d cls.create_security_service("},{"line_number":229,"context_line":"                ss_type,"}],"source_content_type":"text/x-python","patch_set":3,"id":"a20ccada_45b5bd82","line":226,"range":{"start_line":226,"start_character":16,"end_line":226,"end_character":57},"updated":"2026-09-22 12:06:50.000000000","message":"Why we are replacing, \";\" with\",\" ..?","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"},{"author":{"_account_id":39161,"name":"Medha Choudhary netapp","display_name":"Medha Choudhary (netapp)","email":"medhac@netapp.com","username":"medhac","status":"Netapp"},"change_message_id":"6c00bc0c3610de365d1a3114edc4e6673cfe7e94","unresolved":false,"context_lines":[{"line_number":223,"context_line":"            # multi-component DN must use semicolons in tempest.conf."},{"line_number":224,"context_line":"            ss_ou \u003d ss_config.get(\"ss_ou\")"},{"line_number":225,"context_line":"            if ss_ou:"},{"line_number":226,"context_line":"                ss_params[\"ou\"] \u003d ss_ou.replace(\";\", \",\")"},{"line_number":227,"context_line":"            ss_type \u003d ss_config.get(\"ss_type\")"},{"line_number":228,"context_line":"            security_service \u003d cls.create_security_service("},{"line_number":229,"context_line":"                ss_type,"}],"source_content_type":"text/x-python","patch_set":3,"id":"18bf6d5a_bf04f135","line":226,"range":{"start_line":226,"start_character":16,"end_line":226,"end_character":57},"in_reply_to":"a20ccada_45b5bd82","updated":"2026-09-23 05:29:07.000000000","message":"oslo.config Dict options split on commas, so the OU cannot be written with commas in tempest.conf. We use semicolons there and convert them to commas here so the API gets a normal DN.","commit_id":"fc9ee4d428f9249958d7c6047f3241a555fb8db3"}]}