)]}'
{"tempest/api/compute/volumes/test_attach_volume.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5a938526c9cb3a6dd5d3b11936c8301a101782c2","unresolved":false,"context_lines":[{"line_number":209,"context_line":"                                       volume[\u0027id\u0027], \u0027in-use\u0027)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"        # Detach volume from server2, wait for volume status \u0027in-use\u0027"},{"line_number":212,"context_line":"        self.servers_client.detach_volume(server2[\u0027id\u0027], volume[\u0027id\u0027])"},{"line_number":213,"context_line":"        waiters.wait_for_volume_status(self.volumes_client,"},{"line_number":214,"context_line":"                                       volume[\u0027id\u0027], \u0027in-use\u0027)"},{"line_number":215,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_0d1d310f","line":212,"range":{"start_line":212,"start_character":8,"end_line":212,"end_character":70},"updated":"2017-01-16 10:02:40.000000000","message":"so this time volume will be having 2 attachment so you need to pass attachment id for server2","commit_id":"abd47ec287e423a5b0945a00c0610cee4a13e2c7"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"cb584bd28d80e723e76e22e6302714d24fe17363","unresolved":false,"context_lines":[{"line_number":209,"context_line":"                                       volume[\u0027id\u0027], \u0027in-use\u0027)"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"        # Detach volume from server2, wait for volume status \u0027in-use\u0027"},{"line_number":212,"context_line":"        self.servers_client.detach_volume(server2[\u0027id\u0027], volume[\u0027id\u0027])"},{"line_number":213,"context_line":"        waiters.wait_for_volume_status(self.volumes_client,"},{"line_number":214,"context_line":"                                       volume[\u0027id\u0027], \u0027in-use\u0027)"},{"line_number":215,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_946bc1f0","line":212,"range":{"start_line":212,"start_character":8,"end_line":212,"end_character":70},"in_reply_to":"7a3c09a3_0d1d310f","updated":"2017-01-17 01:51:49.000000000","message":"Thanks Ghanshyam, detach_volume in volumes_client uses attachment_id, and detach_volume in servers_client uses server_id as \"attachment_id\", so here server2[\u0027id\u0027] is \"attachement_id\"","commit_id":"abd47ec287e423a5b0945a00c0610cee4a13e2c7"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5a938526c9cb3a6dd5d3b11936c8301a101782c2","unresolved":false,"context_lines":[{"line_number":214,"context_line":"                                       volume[\u0027id\u0027], \u0027in-use\u0027)"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"        # Detach volume from server1, wait for volume status \u0027available\u0027"},{"line_number":217,"context_line":"        self._detach_volume(server1[\u0027id\u0027], volume[\u0027id\u0027])"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"class AttachVolumeShelveTestJSON(AttachVolumeTestJSON):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_ad36dd4f","line":217,"range":{"start_line":217,"start_character":53,"end_line":217,"end_character":56},"updated":"2017-01-16 10:02:40.000000000","message":"we should verify that volume is \u0027available\u0027 at this stage.","commit_id":"abd47ec287e423a5b0945a00c0610cee4a13e2c7"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"cb584bd28d80e723e76e22e6302714d24fe17363","unresolved":false,"context_lines":[{"line_number":214,"context_line":"                                       volume[\u0027id\u0027], \u0027in-use\u0027)"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"        # Detach volume from server1, wait for volume status \u0027available\u0027"},{"line_number":217,"context_line":"        self._detach_volume(server1[\u0027id\u0027], volume[\u0027id\u0027])"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":""},{"line_number":220,"context_line":"class AttachVolumeShelveTestJSON(AttachVolumeTestJSON):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_3454cd1f","line":217,"range":{"start_line":217,"start_character":53,"end_line":217,"end_character":56},"in_reply_to":"7a3c09a3_ad36dd4f","updated":"2017-01-17 01:51:49.000000000","message":"Yes, and here \u0027available\u0027 is checked in _detach_volume","commit_id":"abd47ec287e423a5b0945a00c0610cee4a13e2c7"}]}