)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"55041e993e912bc6f309f6a58653df6fe298f628","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"862054cb_7e80a3b8","updated":"2022-11-17 20:07:23.000000000","message":"Agree with the direction, but there\u0027s an issue.. ","commit_id":"eb0decf2a84a06109e8d0d4c4d50a36c79fd3b15"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"e590e494fcdf70f381d6961d899bcf5b45d685aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1db07ac1_56aa62cb","updated":"2022-11-16 14:53:26.000000000","message":"recheck","commit_id":"eb0decf2a84a06109e8d0d4c4d50a36c79fd3b15"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"8f5621c72a22544bb4b3344c72168ffef1b37f50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2f289ada_760079bd","updated":"2022-11-22 18:46:55.000000000","message":"recheck","commit_id":"cfa78785b337252569c00e7b121d459d2654c29e"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"452bda6b7214f33f7196dba1315e3377532cc9f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9108281b_46268e26","updated":"2022-11-20 08:23:34.000000000","message":"recheck","commit_id":"cfa78785b337252569c00e7b121d459d2654c29e"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7460ad218116b6e901e94e3723fc230a193fa641","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"99e22fff_58105b69","updated":"2022-11-23 14:17:50.000000000","message":"recheck","commit_id":"cfa78785b337252569c00e7b121d459d2654c29e"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"74b4e23ddd2bcbf7b644078009155087d5f5ba69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a673869b_f0fe81e9","updated":"2022-12-15 21:21:33.000000000","message":"LGTM, thanks for the fix","commit_id":"4a80316fe781fd93254596cc870a6819652500ec"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c48d3544168c803cea55622b735c7db2400ad98f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"63b9b0d1_6a46418d","updated":"2023-01-04 18:11:36.000000000","message":"recheck\n\ntox4 fixes have merged","commit_id":"4a80316fe781fd93254596cc870a6819652500ec"}],"manila_tempest_tests/utils.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"55041e993e912bc6f309f6a58653df6fe298f628","unresolved":true,"context_lines":[{"line_number":198,"context_line":"    to create the share. Do not use this method in tests where you need"},{"line_number":199,"context_line":"    to test depth and breadth in the access types and access recipients."},{"line_number":200,"context_line":"    \"\"\""},{"line_number":201,"context_line":"    protocol \u003d CONF.share.enable_protocols[0]"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    if protocol in CONF.share.enable_ip_rules_for_protocols:"},{"line_number":204,"context_line":"        access_type \u003d \"ip\""}],"source_content_type":"text/x-python","patch_set":3,"id":"85cd8f66_8398e453","line":201,"range":{"start_line":201,"start_character":15,"end_line":201,"end_character":45},"updated":"2022-11-17 20:07:23.000000000","message":"This wouldn\u0027t be true always - the class sets the protocol in many cases, and it may not be the first protocol in the ``enable_protocols`` config opt. We use \"enable_protocols\" to just turn or/off protocols that need to be tested; i.e., if ``enable_protocols \u003d nfs,cifs``;  the class can be a set of NFS tests (self.protocol \u003d \"nfs\") and they\u0027ll be run because nfs is in the ``enable_protocols`` configuration option... \n\nhttps://codesearch.opendev.org/?q\u003dprotocol%20%3D\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003dopenstack/manila-tempest-plugin\n\nSo this method can be generic if you probably allow protocol to be specified as a parameter.","commit_id":"eb0decf2a84a06109e8d0d4c4d50a36c79fd3b15"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c97989da882041f0b85a22349a7baf32a36df176","unresolved":false,"context_lines":[{"line_number":198,"context_line":"    to create the share. Do not use this method in tests where you need"},{"line_number":199,"context_line":"    to test depth and breadth in the access types and access recipients."},{"line_number":200,"context_line":"    \"\"\""},{"line_number":201,"context_line":"    protocol \u003d CONF.share.enable_protocols[0]"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    if protocol in CONF.share.enable_ip_rules_for_protocols:"},{"line_number":204,"context_line":"        access_type \u003d \"ip\""}],"source_content_type":"text/x-python","patch_set":3,"id":"33da7e60_7504cf8f","line":201,"range":{"start_line":201,"start_character":15,"end_line":201,"end_character":45},"in_reply_to":"85cd8f66_8398e453","updated":"2022-12-11 09:48:21.000000000","message":"Done","commit_id":"eb0decf2a84a06109e8d0d4c4d50a36c79fd3b15"}]}