)]}'
{"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":"8b75a7bcd20590841a666b90de61e43394cf5e71","unresolved":false,"context_lines":[{"line_number":140,"context_line":"    def test_list_get_volume_attachments(self):"},{"line_number":141,"context_line":"        # Create Server, Volume and attach that Volume to Server"},{"line_number":142,"context_line":"        server \u003d self._create_server()"},{"line_number":143,"context_line":"        volume \u003d self._create_and_attach_volume(server, device\u003dself.device)"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        # List Volume attachment of the server"},{"line_number":146,"context_line":"        body \u003d self.servers_client.list_volume_attachments("}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_a2100950","line":143,"range":{"start_line":143,"start_character":54,"end_line":143,"end_character":74},"updated":"2016-11-23 12:03:32.000000000","message":"how about not passing the device here so that we can actually check device as optional.","commit_id":"2b74f107e57398eceba48441b6c95b75cb098204"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f31865b79d443d156dc1352ddd41e160af1cbf3c","unresolved":false,"context_lines":[{"line_number":140,"context_line":"    def test_list_get_volume_attachments(self):"},{"line_number":141,"context_line":"        # Create Server, Volume and attach that Volume to Server"},{"line_number":142,"context_line":"        server \u003d self._create_server()"},{"line_number":143,"context_line":"        volume \u003d self._create_and_attach_volume(server, device\u003dself.device)"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        # List Volume attachment of the server"},{"line_number":146,"context_line":"        body \u003d self.servers_client.list_volume_attachments("}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_fd299f8a","line":143,"range":{"start_line":143,"start_character":54,"end_line":143,"end_character":74},"in_reply_to":"5a74a57a_163808d3","updated":"2016-11-24 01:30:58.000000000","message":"saw its done in other tests in https://review.openstack.org/#/c/401042/\n\nSo looks good.","commit_id":"2b74f107e57398eceba48441b6c95b75cb098204"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"ea92322c69ffe9df5e3a0c42cd201c3049c78369","unresolved":false,"context_lines":[{"line_number":140,"context_line":"    def test_list_get_volume_attachments(self):"},{"line_number":141,"context_line":"        # Create Server, Volume and attach that Volume to Server"},{"line_number":142,"context_line":"        server \u003d self._create_server()"},{"line_number":143,"context_line":"        volume \u003d self._create_and_attach_volume(server, device\u003dself.device)"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        # List Volume attachment of the server"},{"line_number":146,"context_line":"        body \u003d self.servers_client.list_volume_attachments("}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_163808d3","line":143,"range":{"start_line":143,"start_character":54,"end_line":143,"end_character":74},"in_reply_to":"5a74a57a_a2100950","updated":"2016-11-23 21:55:59.000000000","message":"Yeah, I did think that but I know that is impossible.\nbecause we cannot specify non-device if defining self.device as the default value in _create_and_attach_volume().","commit_id":"2b74f107e57398eceba48441b6c95b75cb098204"}]}