)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"16dc8c4adcd032eae30f2495d7d0f55724c19d1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2db8fb46_c6a4e9b2","updated":"2025-12-10 23:35:05.000000000","message":"LGTM, thanks for proposing this. A comment inline regarding a clarification that\u0027d be good to have if you make another patch","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"d5e4a9dfd6a33b39f5773f67c048f769583e92da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"83cd0958_812f72ce","updated":"2025-12-11 13:56:27.000000000","message":"LGTM","commit_id":"1cf1afa71d985ac654e09e4aac27fc207d6f4816"},{"author":{"_account_id":38371,"name":"Kumar Tadepalli","display_name":"Kumar Tadepalli","email":"kumart.lead@gmail.com","username":"kumart","status":"NetApp"},"change_message_id":"c193171becb2b6c7c21ee04487c64e22edcfbf64","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b8fef2e9_9e155a2d","updated":"2025-12-11 08:36:29.000000000","message":"LGTM","commit_id":"1cf1afa71d985ac654e09e4aac27fc207d6f4816"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"ee9e73c54e41b2b94af35fb3473a362226fa5ff5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"23e19ba3_509891ec","updated":"2025-12-11 13:47:26.000000000","message":"Thank you for proposing this spec. It is in line with what we discussed at the PTG and also with the way the current mount point name feature works at the moment. LGTM","commit_id":"1cf1afa71d985ac654e09e4aac27fc207d6f4816"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"ea75d1f06be870405c7d3f9601b724a2930ffa07","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8364fffd_0fed5393","updated":"2025-12-11 21:45:01.000000000","message":"LGTM. Merging as I only rebased and we had positive feedback from Anoop earlier. Let\u0027s get this in!","commit_id":"96a9d55e46e162571f42f62a145cc79981f8a7bb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"82782e67e11d863782ec58d2bb551c9c7e97b31a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fb2d5996_6dc62b26","updated":"2025-12-11 15:33:26.000000000","message":"Ty for the changes @ek121.kim@samsung.com and for the rebase Carlos. Lgtm","commit_id":"96a9d55e46e162571f42f62a145cc79981f8a7bb"}],"specs/gazpacho/spec-lite-add-human-readable-export-location-to-share-manage.rst":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"cee5dcf2eb179aeae6fdf0192baa1dc4cffa155a","unresolved":true,"context_lines":[{"line_number":8,"context_line":"-------------------------------------------------------------"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":":problem: Currently, share supports human readable export locations"},{"line_number":11,"context_line":"          by ``mount_point_name`` field."},{"line_number":12,"context_line":"          But, there is no way to determine the export location"},{"line_number":13,"context_line":"          before the share is managed, and human readable export locations"},{"line_number":14,"context_line":"          are not available for managed share."}],"source_content_type":"text/x-rst","patch_set":2,"id":"665a0ad2_f536ec1c","line":11,"updated":"2025-12-10 12:49:56.000000000","message":"Is this going to be optional field and if not provided, driver will use defaults eg /\u003cshare/volume name\u003e?","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"16dc8c4adcd032eae30f2495d7d0f55724c19d1f","unresolved":true,"context_lines":[{"line_number":8,"context_line":"-------------------------------------------------------------"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":":problem: Currently, share supports human readable export locations"},{"line_number":11,"context_line":"          by ``mount_point_name`` field."},{"line_number":12,"context_line":"          But, there is no way to determine the export location"},{"line_number":13,"context_line":"          before the share is managed, and human readable export locations"},{"line_number":14,"context_line":"          are not available for managed share."}],"source_content_type":"text/x-rst","patch_set":2,"id":"669df9d5_3622f9cc","line":11,"in_reply_to":"665a0ad2_f536ec1c","updated":"2025-12-10 23:35:05.000000000","message":"Good point to clarify that this is an optional feature, and shouldn\u0027t impact the current manage/unmanage workflows if its not used.","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":36741,"name":"Eunkyung Kim","display_name":"eunkyung","email":"ek121.kim@samsung.com","username":"eunkyung"},"change_message_id":"79eadb51bd740ce87d1ea82f94504f217fdc4617","unresolved":true,"context_lines":[{"line_number":8,"context_line":"-------------------------------------------------------------"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":":problem: Currently, share supports human readable export locations"},{"line_number":11,"context_line":"          by ``mount_point_name`` field."},{"line_number":12,"context_line":"          But, there is no way to determine the export location"},{"line_number":13,"context_line":"          before the share is managed, and human readable export locations"},{"line_number":14,"context_line":"          are not available for managed share."}],"source_content_type":"text/x-rst","patch_set":2,"id":"73a3b394_aa164cda","line":11,"in_reply_to":"669df9d5_3622f9cc","updated":"2025-12-11 02:23:36.000000000","message":"Thank you for reviewing and explaining. I\u0027ve uploaded a new patch that explains the existing workflow and that the field is optional. Please check again.","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"ce9b059630c7b31c18389b8c59ea2f4bd7018ccd","unresolved":false,"context_lines":[{"line_number":8,"context_line":"-------------------------------------------------------------"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":":problem: Currently, share supports human readable export locations"},{"line_number":11,"context_line":"          by ``mount_point_name`` field."},{"line_number":12,"context_line":"          But, there is no way to determine the export location"},{"line_number":13,"context_line":"          before the share is managed, and human readable export locations"},{"line_number":14,"context_line":"          are not available for managed share."}],"source_content_type":"text/x-rst","patch_set":2,"id":"7ee7e51f_28772162","line":11,"in_reply_to":"6bd7d966_1139931b","updated":"2025-12-11 13:56:15.000000000","message":"Acknowledged","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"ee9e73c54e41b2b94af35fb3473a362226fa5ff5","unresolved":true,"context_lines":[{"line_number":8,"context_line":"-------------------------------------------------------------"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":":problem: Currently, share supports human readable export locations"},{"line_number":11,"context_line":"          by ``mount_point_name`` field."},{"line_number":12,"context_line":"          But, there is no way to determine the export location"},{"line_number":13,"context_line":"          before the share is managed, and human readable export locations"},{"line_number":14,"context_line":"          are not available for managed share."}],"source_content_type":"text/x-rst","patch_set":2,"id":"6bd7d966_1139931b","line":11,"in_reply_to":"73a3b394_aa164cda","updated":"2025-12-11 13:47:26.000000000","message":"Thanks for adding it!","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"cee5dcf2eb179aeae6fdf0192baa1dc4cffa155a","unresolved":true,"context_lines":[{"line_number":66,"context_line":"          This implementation is not supposed to impact performance on any"},{"line_number":67,"context_line":"          aspect."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":":alternative: As an alternative, drivers could reuse the name and the"},{"line_number":70,"context_line":"              description of shares and generate human readable export"},{"line_number":71,"context_line":"              locations, but names can be duplicated and backends may fail due"},{"line_number":72,"context_line":"              to that."}],"source_content_type":"text/x-rst","patch_set":2,"id":"4313b610_7d567e88","line":69,"updated":"2025-12-10 12:49:56.000000000","message":"If the share already has a mount path on the storage, Is it expected to override the existing mount path during share manage workflow?\n\nfor eg, if a Volume is created using ONTAP System Manager, the mount path would default to /\u003cshare/volume name\u003e.","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"ce9b059630c7b31c18389b8c59ea2f4bd7018ccd","unresolved":false,"context_lines":[{"line_number":66,"context_line":"          This implementation is not supposed to impact performance on any"},{"line_number":67,"context_line":"          aspect."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":":alternative: As an alternative, drivers could reuse the name and the"},{"line_number":70,"context_line":"              description of shares and generate human readable export"},{"line_number":71,"context_line":"              locations, but names can be duplicated and backends may fail due"},{"line_number":72,"context_line":"              to that."}],"source_content_type":"text/x-rst","patch_set":2,"id":"f4a6ba15_0705b19b","line":69,"in_reply_to":"38d0d5f2_4f2f99ce","updated":"2025-12-11 13:56:15.000000000","message":"Ack","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"16dc8c4adcd032eae30f2495d7d0f55724c19d1f","unresolved":false,"context_lines":[{"line_number":66,"context_line":"          This implementation is not supposed to impact performance on any"},{"line_number":67,"context_line":"          aspect."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":":alternative: As an alternative, drivers could reuse the name and the"},{"line_number":70,"context_line":"              description of shares and generate human readable export"},{"line_number":71,"context_line":"              locations, but names can be duplicated and backends may fail due"},{"line_number":72,"context_line":"              to that."}],"source_content_type":"text/x-rst","patch_set":2,"id":"38d0d5f2_4f2f99ce","line":69,"in_reply_to":"4313b610_7d567e88","updated":"2025-12-10 23:35:05.000000000","message":"yes, that\u0027s the current workflow implemented in some drivers (such as the NetApp driver) - the volume is unmounted on the backend, and mounted again using the new name","commit_id":"1a666eb6f3792972ee254fbcd004239454e3d325"}]}
