)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"7a584c4e74da9b74742ce4799a0585c1404afad6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benny Kopilov \u003cbkopilov@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-26 13:31:46 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Volume force-delete when volume in maintenance state"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ia54721089d4925b6f5eeeef283cb62551a6120e1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ffe62b97_b9513765","line":7,"updated":"2017-03-27 10:19:16.000000000","message":"So, why do you want to add only \u0027maintenace\u0027 state? According to the link[1] which you suggested, there some statuses like these [\"available\", \"error\", \"creating\", \"deleting\", \"in-use\", \"attaching\", \"detaching\", \"error_deleting\" and \"maintenance\"]. So I think it\u0027s good to add the reason why you just picked up \u0027maintenance\u0027 status in this patch.\n\n[1] https://docs.openstack.org/cli-reference/cinder.html#cinder-reset-state","commit_id":"53ad73a67d045d76e75c8f1414f225d6f307cbcb"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"1445bc7e42d81f2d118599c58b54f71af7f24b6a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benny Kopilov \u003cbkopilov@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-26 13:31:46 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Volume force-delete when volume in maintenance state"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ia54721089d4925b6f5eeeef283cb62551a6120e1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ffe62b97_cf0bd721","line":7,"in_reply_to":"ffe62b97_b9513765","updated":"2017-03-28 08:06:39.000000000","message":"Hi ,\nThe reason i decided to test maintenance is because there is no coverage at tempest for this state at all .\nOthers are covered by tempest but i agree that we would add more tests .\n\nThank you \nBenny","commit_id":"53ad73a67d045d76e75c8f1414f225d6f307cbcb"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"b3a6cab8911695f1a60b8e40a16577d5c9b20cbd","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benny Kopilov \u003cbkopilov@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-28 11:07:18 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Volume force-delete when volume in maintenance state"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ia54721089d4925b6f5eeeef283cb62551a6120e1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ffe62b97_19343f92","line":7,"updated":"2017-03-29 04:19:43.000000000","message":"So, please add the reason why you picked only maintenance state to this commit message.","commit_id":"cae2d74bd817a34dd852b9b5577591edd217d41d"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"bee87b4be7b27f28505399fb722a6a85a929f53f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Benny Kopilov \u003cbkopilov@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-28 11:07:18 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Volume force-delete when volume in maintenance state"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ia54721089d4925b6f5eeeef283cb62551a6120e1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ffe62b97_7976ab40","line":7,"in_reply_to":"ffe62b97_19343f92","updated":"2017-03-29 04:28:18.000000000","message":"ok","commit_id":"cae2d74bd817a34dd852b9b5577591edd217d41d"}],"tempest/api/volume/admin/test_volumes_actions.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"6de01fc438e1fd1dff8d225f925d8e19d61a30e5","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    @decorators.idempotent_id(\u0027b957cabd-1486-4e21-90cf-a9ed3c39dfb2\u0027)"},{"line_number":58,"context_line":"    def test_volume_force_delete_when_volume_is_maintenance(self):"},{"line_number":59,"context_line":"        # test force delete when status of volume is error"},{"line_number":60,"context_line":"        self._create_reset_and_force_delete_temp_volume(\u0027maintenance\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffe62b97_e3638fdb","line":59,"range":{"start_line":59,"start_character":53,"end_line":59,"end_character":58},"updated":"2017-03-27 08:15:47.000000000","message":"maintenance","commit_id":"53ad73a67d045d76e75c8f1414f225d6f307cbcb"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"32baa6060666223fa98b1a6de6d193b7c045bac4","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    @decorators.idempotent_id(\u0027b957cabd-1486-4e21-90cf-a9ed3c39dfb2\u0027)"},{"line_number":58,"context_line":"    def test_volume_force_delete_when_volume_is_maintenance(self):"},{"line_number":59,"context_line":"        # test force delete when status of volume is error"},{"line_number":60,"context_line":"        self._create_reset_and_force_delete_temp_volume(\u0027maintenance\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffe62b97_a306a785","line":59,"range":{"start_line":59,"start_character":53,"end_line":59,"end_character":58},"in_reply_to":"ffe62b97_e3638fdb","updated":"2017-03-27 08:22:35.000000000","message":"Nice catch,  fixing :)","commit_id":"53ad73a67d045d76e75c8f1414f225d6f307cbcb"}]}