)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9ef2c04f7676988cfe9932e7e6c130f4dfd3caad","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     danielarthurt \u003cdanielarthurt@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-06-12 18:59:12 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[NetApp] Updating the release note for change I76e11fef27c9723f019cfdfdc6ea86878db78776"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch update the release note of the bugfix for the bug 1688620"},{"line_number":10,"context_line":"explaining better about the added configuration option and its use."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf51134e_2a21d64e","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":87},"updated":"2020-06-16 02:31:34.000000000","message":"The header of a commit message should preferably be under 50 characters:\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"870f8187e5c89d70105ab0d45cec48e7ee19573b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     danielarthurt \u003cdanielarthurt@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-06-12 18:59:12 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[NetApp] Updating the release note for change I76e11fef27c9723f019cfdfdc6ea86878db78776"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch update the release note of the bugfix for the bug 1688620"},{"line_number":10,"context_line":"explaining better about the added configuration option and its use."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf51134e_0ed17b55","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":87},"in_reply_to":"bf51134e_2a21d64e","updated":"2020-06-16 14:38:58.000000000","message":"Done","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9ef2c04f7676988cfe9932e7e6c130f4dfd3caad","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch update the release note of the bugfix for the bug 1688620"},{"line_number":10,"context_line":"explaining better about the added configuration option and its use."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idf9730bfc9604f906b10e58f5b767b4030d8f0db"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf51134e_aad20633","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":0},"updated":"2020-06-16 02:31:34.000000000","message":"Add:\n\nRelated-Bug: #1688620","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"870f8187e5c89d70105ab0d45cec48e7ee19573b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch update the release note of the bugfix for the bug 1688620"},{"line_number":10,"context_line":"explaining better about the added configuration option and its use."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idf9730bfc9604f906b10e58f5b767b4030d8f0db"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf51134e_6ec5978e","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":0},"in_reply_to":"bf51134e_aad20633","updated":"2020-06-16 14:38:58.000000000","message":"Done","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"}],"releasenotes/notes/bug-1688620-netapp-migration-cancelation-fb913131eb8eb82a.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9ef2c04f7676988cfe9932e7e6c130f4dfd3caad","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    NetApp ONTAP share delete operation can fail sometimes when is triggered"},{"line_number":5,"context_line":"    immediately after migration cancelation on a overloaded NetApp backend"},{"line_number":6,"context_line":"    because one of the ``migration cancel`` operation depends of an"},{"line_number":7,"context_line":"    asynchronous API."},{"line_number":8,"context_line":"    Now the NetApp driver checks if the asynchronous API has ended  its"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_0ae33207","line":5,"range":{"start_line":5,"start_character":67,"end_line":5,"end_character":74},"updated":"2020-06-16 02:31:34.000000000","message":"nit: backend.\n\nPlease split this long sentence here:\n\n Canceling an ongoing migration is an asynchronous operation\n on an ONTAP storage system.","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"870f8187e5c89d70105ab0d45cec48e7ee19573b","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    NetApp ONTAP share delete operation can fail sometimes when is triggered"},{"line_number":5,"context_line":"    immediately after migration cancelation on a overloaded NetApp backend"},{"line_number":6,"context_line":"    because one of the ``migration cancel`` operation depends of an"},{"line_number":7,"context_line":"    asynchronous API."},{"line_number":8,"context_line":"    Now the NetApp driver checks if the asynchronous API has ended  its"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_8e60ebd0","line":5,"range":{"start_line":5,"start_character":67,"end_line":5,"end_character":74},"in_reply_to":"bf51134e_0ae33207","updated":"2020-06-16 14:38:58.000000000","message":"Done!","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9ef2c04f7676988cfe9932e7e6c130f4dfd3caad","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    because one of the ``migration cancel`` operation depends of an"},{"line_number":7,"context_line":"    asynchronous API."},{"line_number":8,"context_line":"    Now the NetApp driver checks if the asynchronous API has ended  its"},{"line_number":9,"context_line":"    operation before report migration cancelation success. If the operation"},{"line_number":10,"context_line":"    of the asynchronous API did not end within the specified timeout, the"},{"line_number":11,"context_line":"    ``migration cancel`` cancel operation will be considered unsuccessful."},{"line_number":12,"context_line":"    To do so, a new configuration option ``netapp_migration_cancel_timeout``"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_6af0eeca","line":9,"range":{"start_line":9,"start_character":21,"end_line":9,"end_character":27},"updated":"2020-06-16 02:31:34.000000000","message":"reporting","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"870f8187e5c89d70105ab0d45cec48e7ee19573b","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    because one of the ``migration cancel`` operation depends of an"},{"line_number":7,"context_line":"    asynchronous API."},{"line_number":8,"context_line":"    Now the NetApp driver checks if the asynchronous API has ended  its"},{"line_number":9,"context_line":"    operation before report migration cancelation success. If the operation"},{"line_number":10,"context_line":"    of the asynchronous API did not end within the specified timeout, the"},{"line_number":11,"context_line":"    ``migration cancel`` cancel operation will be considered unsuccessful."},{"line_number":12,"context_line":"    To do so, a new configuration option ``netapp_migration_cancel_timeout``"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_ee9a47c3","line":9,"range":{"start_line":9,"start_character":21,"end_line":9,"end_character":27},"in_reply_to":"bf51134e_6af0eeca","updated":"2020-06-16 14:38:58.000000000","message":"Done","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9ef2c04f7676988cfe9932e7e6c130f4dfd3caad","unresolved":false,"context_lines":[{"line_number":9,"context_line":"    operation before report migration cancelation success. If the operation"},{"line_number":10,"context_line":"    of the asynchronous API did not end within the specified timeout, the"},{"line_number":11,"context_line":"    ``migration cancel`` cancel operation will be considered unsuccessful."},{"line_number":12,"context_line":"    To do so, a new configuration option ``netapp_migration_cancel_timeout``"},{"line_number":13,"context_line":"    has been added to define the time that the ``migration cancel`` waits"},{"line_number":14,"context_line":"    to be answered by the asynchronous API."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_8ac0e252","line":12,"range":{"start_line":12,"start_character":43,"end_line":12,"end_character":74},"updated":"2020-06-16 02:31:34.000000000","message":"This config opt should be added under the \"upgrade\" section of the release note: https://docs.openstack.org/manila/latest/contributor/adding_release_notes.html#how-do-i-add-a-release-note\n\nHaving it here is fine. But in addition perhaps add an upgrade section like this:\n\n  upgrade:\n    - | \n       The configuration option ``netapp_migration_cancel_timeout`` can be\n       specified in the NetApp backend sections to redefine the amount of\n       time the NetApp driver must attempt to wait on the asynchronous\n       operation to cancel an ongoing migration. This option is not required,\n       and it defaults to 3600 seconds, which is sufficient time in most cases.","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"},{"author":{"_account_id":31528,"name":"Daniel Tapia","email":"danielarthurt@gmail.com","username":"danielarthurt"},"change_message_id":"870f8187e5c89d70105ab0d45cec48e7ee19573b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"    operation before report migration cancelation success. If the operation"},{"line_number":10,"context_line":"    of the asynchronous API did not end within the specified timeout, the"},{"line_number":11,"context_line":"    ``migration cancel`` cancel operation will be considered unsuccessful."},{"line_number":12,"context_line":"    To do so, a new configuration option ``netapp_migration_cancel_timeout``"},{"line_number":13,"context_line":"    has been added to define the time that the ``migration cancel`` waits"},{"line_number":14,"context_line":"    to be answered by the asynchronous API."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bf51134e_4ed9b381","line":12,"range":{"start_line":12,"start_character":43,"end_line":12,"end_character":74},"in_reply_to":"bf51134e_8ac0e252","updated":"2020-06-16 14:38:58.000000000","message":"Thank you for the reference. I also saw that the release notes should not have more than 2/3 lines per section. I think that it is succinct, even with more than 3 lines per section.","commit_id":"9b627deafd8326d7a8057a8db4400b93c3ec74bf"}]}
