)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9c6a5ed688968d682aa7e86976a558c51bb27d9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"63ae9729_9f9d0c96","updated":"2024-09-14 18:33:02.000000000","message":"Thanks; this is a good refactor; question inline regarding a similar statement.","commit_id":"5afa62f8da043f86c2ad147e3d87fc2b5a5051ac"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2938250afc5f4b5be70b1cca953d2182330c0874","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d0e660e0_ffdaf2f3","updated":"2024-09-17 05:30:44.000000000","message":"Thank you @kajinamit@oss.nttdata.com","commit_id":"beb70781a2164e988727110cdc308dffbb7319aa"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"340ecd91ff221482d94ba510488ad35afce31b7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b83f3de4_198d003c","updated":"2025-02-24 15:36:29.000000000","message":"LGTM, thanks for refactoring this!","commit_id":"786ec6316cc79246cc0910f3f395d02f7b274a76"}],"manila/share/drivers/dell_emc/driver.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9c6a5ed688968d682aa7e86976a558c51bb27d9d","unresolved":true,"context_lines":[{"line_number":91,"context_line":"        self.dhss_mandatory_security_service_association \u003d ("},{"line_number":92,"context_line":"            self.plugin.dhss_mandatory_security_service_association)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        if hasattr(self.plugin, \u0027ipv6_implemented\u0027):"},{"line_number":95,"context_line":"            self.ipv6_implemented \u003d self.plugin.ipv6_implemented"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"        self.revert_to_snap_support \u003d getattr("},{"line_number":98,"context_line":"            self.plugin, \u0027revert_to_snap_support\u0027, False)"}],"source_content_type":"text/x-python","patch_set":1,"id":"32e325f3_a04b429e","line":95,"range":{"start_line":94,"start_character":8,"end_line":95,"end_character":64},"updated":"2024-09-14 18:33:02.000000000","message":"This could be transformed too?","commit_id":"5afa62f8da043f86c2ad147e3d87fc2b5a5051ac"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4db23d027885f6e0ce6a18eff8fed5ff41f1398d","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        self.dhss_mandatory_security_service_association \u003d ("},{"line_number":92,"context_line":"            self.plugin.dhss_mandatory_security_service_association)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        if hasattr(self.plugin, \u0027ipv6_implemented\u0027):"},{"line_number":95,"context_line":"            self.ipv6_implemented \u003d self.plugin.ipv6_implemented"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"        self.revert_to_snap_support \u003d getattr("},{"line_number":98,"context_line":"            self.plugin, \u0027revert_to_snap_support\u0027, False)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1cee5b8d_8cc74ddf","line":95,"range":{"start_line":94,"start_character":8,"end_line":95,"end_character":64},"in_reply_to":"32e325f3_a04b429e","updated":"2024-09-15 05:33:32.000000000","message":"I didn\u0027t touch this initially because the default is maintained in the base class, but now I think it\u0027s better to have explicit default here to avoid unexpected problems caused by the default change. I\u0027ve updated the change to cover this, too.","commit_id":"5afa62f8da043f86c2ad147e3d87fc2b5a5051ac"}]}