)]}'
{"tempest/api/compute/volumes/test_attach_volume.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"d0fb4e11f2fb83c702c50b030bea74333574dfbc","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        body \u003d self.servers_client.list_volume_attachments("},{"line_number":167,"context_line":"            server[\u0027id\u0027])[\u0027volumeAttachments\u0027]"},{"line_number":168,"context_line":"        self.assertEqual(2, len(body))"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"        body \u003d self.servers_client.show_volume_attachment("},{"line_number":171,"context_line":"            server[\u0027id\u0027],"},{"line_number":172,"context_line":"            attachment_1st[\u0027id\u0027])[\u0027volumeAttachment\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_a39f385e","line":169,"range":{"start_line":169,"start_character":0,"end_line":169,"end_character":0},"updated":"2016-11-23 08:57:04.000000000","message":"Hi , \nI think we should ssh to an instance and verify that both volumes exists and mounted.\nBenny","commit_id":"0582b4133ea40a5f2b7e02ed1d4c2c2d0b24b538"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"95b5d4ce9d649c4f52c06d48744bca1f4d4f75f4","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        body \u003d self.servers_client.list_volume_attachments("},{"line_number":167,"context_line":"            server[\u0027id\u0027])[\u0027volumeAttachments\u0027]"},{"line_number":168,"context_line":"        self.assertEqual(2, len(body))"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"        body \u003d self.servers_client.show_volume_attachment("},{"line_number":171,"context_line":"            server[\u0027id\u0027],"},{"line_number":172,"context_line":"            attachment_1st[\u0027id\u0027])[\u0027volumeAttachment\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_96a098f6","line":169,"range":{"start_line":169,"start_character":0,"end_line":169,"end_character":0},"in_reply_to":"5a74a57a_6fd6dac6","updated":"2016-11-23 21:58:31.000000000","message":"Yeah, that is possible. However this is API tests and it is enough to verify API behaviors on the test. I know the above other test verifies it but I feel it is a little overkill at the first step. and sometimes buggy.","commit_id":"0582b4133ea40a5f2b7e02ed1d4c2c2d0b24b538"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6843a3fe1a01bdf6978256c37cba6dcf35eb5353","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        body \u003d self.servers_client.list_volume_attachments("},{"line_number":167,"context_line":"            server[\u0027id\u0027])[\u0027volumeAttachments\u0027]"},{"line_number":168,"context_line":"        self.assertEqual(2, len(body))"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"        body \u003d self.servers_client.show_volume_attachment("},{"line_number":171,"context_line":"            server[\u0027id\u0027],"},{"line_number":172,"context_line":"            attachment_1st[\u0027id\u0027])[\u0027volumeAttachment\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_6fd6dac6","line":169,"range":{"start_line":169,"start_character":0,"end_line":169,"end_character":0},"in_reply_to":"5a74a57a_a39f385e","updated":"2016-11-23 11:57:55.000000000","message":"Nice point, it will be good to check the mount volume. may be we can define a private function of L124-L134 and use that in this tests also.","commit_id":"0582b4133ea40a5f2b7e02ed1d4c2c2d0b24b538"}]}