)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"b7163661af2d1fae353fb089155c2b30621f6491","unresolved":false,"context_lines":[{"line_number":12,"context_line":"API. If share delete operation is requested immediately after call the"},{"line_number":13,"context_line":"former API, it fails because the \"abort_volume_move\" is still in progress."},{"line_number":14,"context_line":"Now NetApp cDOT driver checks, for a period of time, if the"},{"line_number":15,"context_line":"volume-move-abort has ended before report migration cancelation success."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I76e11fef27c9723f019cfdfdc6ea86878db78776"},{"line_number":18,"context_line":"Closes-Bug: #1688620"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1f493fa4_ddd7a34a","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":17},"updated":"2020-04-29 12:53:44.000000000","message":"nit: `volume-move-abort` operation","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"35d5f8a5ec656e477325a8834e4582c7ca09ceb6","unresolved":false,"context_lines":[{"line_number":12,"context_line":"API. If share delete operation is requested immediately after call the"},{"line_number":13,"context_line":"former API, it fails because the \"abort_volume_move\" is still in progress."},{"line_number":14,"context_line":"Now NetApp cDOT driver checks, for a period of time, if the"},{"line_number":15,"context_line":"``volume-move-abort`` operationhas ended before report migration"},{"line_number":16,"context_line":"cancelation success."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I76e11fef27c9723f019cfdfdc6ea86878db78776"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1f493fa4_3355ce1e","line":15,"updated":"2020-05-04 10:59:31.000000000","message":"nit: please add the blank space here again","commit_id":"5f64aaff713a4bf2ecddc49b4f2ad10f2c9a63ba"}],"manila/share/drivers/netapp/dataontap/cluster_mode/lib_base.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"b7163661af2d1fae353fb089155c2b30621f6491","unresolved":false,"context_lines":[{"line_number":2512,"context_line":"        \"\"\"Abort an ongoing migration.\"\"\""},{"line_number":2513,"context_line":"        vserver, vserver_client \u003d self._get_vserver(share_server\u003dshare_server)"},{"line_number":2514,"context_line":"        share_volume \u003d self._get_backend_share_name(source_share[\u0027id\u0027])"},{"line_number":2515,"context_line":"        retries \u003d (self.configuration.netapp_migration_cancel_timeout / 5)"},{"line_number":2516,"context_line":""},{"line_number":2517,"context_line":"        try:"},{"line_number":2518,"context_line":"            self._get_volume_move_status(source_share, share_server)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_5df7138c","line":2515,"updated":"2020-04-29 12:53:44.000000000","message":"retries \u003d (self.configuration.netapp_migration_cancel_timeout / 5 or 1)\nJust in case there is no \u0027netapp_migration_cancel_timeout\u0027.","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"ad844e684f8c204f99278fa1d1a13040d1d360af","unresolved":false,"context_lines":[{"line_number":2512,"context_line":"        \"\"\"Abort an ongoing migration.\"\"\""},{"line_number":2513,"context_line":"        vserver, vserver_client \u003d self._get_vserver(share_server\u003dshare_server)"},{"line_number":2514,"context_line":"        share_volume \u003d self._get_backend_share_name(source_share[\u0027id\u0027])"},{"line_number":2515,"context_line":"        retries \u003d (self.configuration.netapp_migration_cancel_timeout / 5)"},{"line_number":2516,"context_line":""},{"line_number":2517,"context_line":"        try:"},{"line_number":2518,"context_line":"            self._get_volume_move_status(source_share, share_server)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_71f673d9","line":2515,"in_reply_to":"1f493fa4_5df7138c","updated":"2020-04-29 23:08:47.000000000","message":"Done!","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"b7163661af2d1fae353fb089155c2b30621f6491","unresolved":false,"context_lines":[{"line_number":2542,"context_line":"                \u0027share_move_state\u0027: move_status[\u0027state\u0027]"},{"line_number":2543,"context_line":"            }"},{"line_number":2544,"context_line":"            msg \u003d _(\"Migration cancelation was not successful. The share \""},{"line_number":2545,"context_line":"                    \"migration state failed in the state transition from \""},{"line_number":2546,"context_line":"                    \"%(share_move_state)s state to \u0027failed\u0027. Retries \""},{"line_number":2547,"context_line":"                    \"exhausted.\") % msg_args"},{"line_number":2548,"context_line":"            raise exception.NetAppException(message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_3d0e87e4","line":2545,"range":{"start_line":2545,"start_character":44,"end_line":2545,"end_character":67},"updated":"2020-04-29 12:53:44.000000000","message":"nit - what about: while transitioning?","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"ad844e684f8c204f99278fa1d1a13040d1d360af","unresolved":false,"context_lines":[{"line_number":2542,"context_line":"                \u0027share_move_state\u0027: move_status[\u0027state\u0027]"},{"line_number":2543,"context_line":"            }"},{"line_number":2544,"context_line":"            msg \u003d _(\"Migration cancelation was not successful. The share \""},{"line_number":2545,"context_line":"                    \"migration state failed in the state transition from \""},{"line_number":2546,"context_line":"                    \"%(share_move_state)s state to \u0027failed\u0027. Retries \""},{"line_number":2547,"context_line":"                    \"exhausted.\") % msg_args"},{"line_number":2548,"context_line":"            raise exception.NetAppException(message\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_11d60f29","line":2545,"range":{"start_line":2545,"start_character":44,"end_line":2545,"end_character":67},"in_reply_to":"1f493fa4_3d0e87e4","updated":"2020-04-29 23:08:47.000000000","message":"Yes, it is better.","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"}],"manila/tests/share/drivers/netapp/dataontap/cluster_mode/test_lib_base.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"b7163661af2d1fae353fb089155c2b30621f6491","unresolved":false,"context_lines":[{"line_number":5223,"context_line":"        self.mock_object(self.library, \u0027_get_backend_share_name\u0027,"},{"line_number":5224,"context_line":"                         mock.Mock(return_value\u003dfake.SHARE_NAME))"},{"line_number":5225,"context_line":"        self.mock_object(self.client, \u0027abort_volume_move\u0027)"},{"line_number":5226,"context_line":"        self.mock_object(self.client, \u0027get_volume_move_status\u0027,"},{"line_number":5227,"context_line":"                         mock.Mock(return_value\u003dstate))"},{"line_number":5228,"context_line":""},{"line_number":5229,"context_line":"        if state[\u0027state\u0027] \u003d\u003d \u0027failed\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_fdc9bf25","line":5226,"range":{"start_line":5226,"start_character":39,"end_line":5226,"end_character":61},"updated":"2020-04-29 12:53:44.000000000","message":"shouldn\u0027t this be _get_volume_move_status?","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"35d5f8a5ec656e477325a8834e4582c7ca09ceb6","unresolved":false,"context_lines":[{"line_number":5223,"context_line":"        self.mock_object(self.library, \u0027_get_backend_share_name\u0027,"},{"line_number":5224,"context_line":"                         mock.Mock(return_value\u003dfake.SHARE_NAME))"},{"line_number":5225,"context_line":"        self.mock_object(self.client, \u0027abort_volume_move\u0027)"},{"line_number":5226,"context_line":"        self.mock_object(self.client, \u0027get_volume_move_status\u0027,"},{"line_number":5227,"context_line":"                         mock.Mock(return_value\u003dstate))"},{"line_number":5228,"context_line":""},{"line_number":5229,"context_line":"        if state[\u0027state\u0027] \u003d\u003d \u0027failed\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_932f82a6","line":5226,"range":{"start_line":5226,"start_character":39,"end_line":5226,"end_character":61},"in_reply_to":"1f493fa4_31b4ebd0","updated":"2020-05-04 10:59:31.000000000","message":"Got you. Thanks for answering.","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"ad844e684f8c204f99278fa1d1a13040d1d360af","unresolved":false,"context_lines":[{"line_number":5223,"context_line":"        self.mock_object(self.library, \u0027_get_backend_share_name\u0027,"},{"line_number":5224,"context_line":"                         mock.Mock(return_value\u003dfake.SHARE_NAME))"},{"line_number":5225,"context_line":"        self.mock_object(self.client, \u0027abort_volume_move\u0027)"},{"line_number":5226,"context_line":"        self.mock_object(self.client, \u0027get_volume_move_status\u0027,"},{"line_number":5227,"context_line":"                         mock.Mock(return_value\u003dstate))"},{"line_number":5228,"context_line":""},{"line_number":5229,"context_line":"        if state[\u0027state\u0027] \u003d\u003d \u0027failed\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_31b4ebd0","line":5226,"range":{"start_line":5226,"start_character":39,"end_line":5226,"end_character":61},"in_reply_to":"1f493fa4_fdc9bf25","updated":"2020-04-29 23:08:47.000000000","message":"It\u0027s mocking the method of client_cmode, as the others migration tests. So, without _ is fine.","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"}],"releasenotes/notes/bug-1688620-netapp-migration-cancelation-fb913131eb8eb82a.yaml":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"b7163661af2d1fae353fb089155c2b30621f6491","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    NetApp cDOT driver with a very busy backend may report that the"},{"line_number":5,"context_line":"    ``migration cancelation`` operation was successful even if wasn\u0027t. Now"},{"line_number":6,"context_line":"    during the ``migration cancelation`` operation the currrent state of the"},{"line_number":7,"context_line":"    migration task will be tracked and the operation will only report success"},{"line_number":8,"context_line":"    when the operation ends correctly."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1f493fa4_3db5e7ee","line":6,"range":{"start_line":6,"start_character":55,"end_line":6,"end_character":63},"updated":"2020-04-29 12:53:44.000000000","message":"current","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"ad844e684f8c204f99278fa1d1a13040d1d360af","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    NetApp cDOT driver with a very busy backend may report that the"},{"line_number":5,"context_line":"    ``migration cancelation`` operation was successful even if wasn\u0027t. Now"},{"line_number":6,"context_line":"    during the ``migration cancelation`` operation the currrent state of the"},{"line_number":7,"context_line":"    migration task will be tracked and the operation will only report success"},{"line_number":8,"context_line":"    when the operation ends correctly."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1f493fa4_a44f1669","line":6,"range":{"start_line":6,"start_character":55,"end_line":6,"end_character":63},"in_reply_to":"1f493fa4_3db5e7ee","updated":"2020-04-29 23:08:47.000000000","message":"Done!","commit_id":"dcca320c056fa92507371827887f1430a5a86f23"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"a2a02a0d8968a60d45ebe8b91aa0c33a600555a4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    NetApp cDOT driver with a very busy backend may report that the"},{"line_number":5,"context_line":"    ``migration cancelation`` operation was successful even if wasn\u0027t. Now"},{"line_number":6,"context_line":"    during the ``migration cancelation`` operation the current state of the"},{"line_number":7,"context_line":"    migration task will be tracked and the operation will only report success"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ff570b3c_3c62e4ae","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":15},"updated":"2020-05-21 16:30:42.000000000","message":"ONTAP","commit_id":"0ee414082318d22b3ed19acad6a479cb105c30e5"}]}