)]}'
{"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":"215cf642bd77242f6a310c8306702c09bfa6acc1","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            wait_until\u003d\u0027ACTIVE\u0027,"},{"line_number":72,"context_line":"            adminPass\u003dself.image_ssh_password)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        body \u003d self.servers_client.list_volume_attachments("},{"line_number":75,"context_line":"            self.server[\u0027id\u0027])[\u0027volumeAttachments\u0027]"},{"line_number":76,"context_line":"        num \u003d len(body)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        # Record addresses so that we can ssh later"},{"line_number":79,"context_line":"        server[\u0027addresses\u0027] \u003d self.servers_client.list_addresses("}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_0496aa92","line":76,"range":{"start_line":74,"start_character":0,"end_line":76,"end_character":23},"updated":"2016-09-07 07:39:44.000000000","message":"why we need here?","commit_id":"bcc084157dd59dc4a5344218fcb06d2fb207bfa4"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"721ade9a531fa61b7445e3df3cd17e6cdd07f25a","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            wait_until\u003d\u0027ACTIVE\u0027,"},{"line_number":72,"context_line":"            adminPass\u003dself.image_ssh_password)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        body \u003d self.servers_client.list_volume_attachments("},{"line_number":75,"context_line":"            self.server[\u0027id\u0027])[\u0027volumeAttachments\u0027]"},{"line_number":76,"context_line":"        num \u003d len(body)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        # Record addresses so that we can ssh later"},{"line_number":79,"context_line":"        server[\u0027addresses\u0027] \u003d self.servers_client.list_addresses("}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_44ffd2a9","line":76,"range":{"start_line":74,"start_character":0,"end_line":76,"end_character":23},"in_reply_to":"9a89bdaa_0496aa92","updated":"2016-09-07 07:41:12.000000000","message":"this is due to a bad rebase, my bad, will fix right away","commit_id":"bcc084157dd59dc4a5344218fcb06d2fb207bfa4"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"b89c2d158f71447adf782bccee14d14916607805","unresolved":false,"context_lines":[{"line_number":146,"context_line":"    @test.idempotent_id(\u00277fa563fe-f0f7-43eb-9e22-a1ece036b513\u0027)"},{"line_number":147,"context_line":"    def test_list_get_volume_attachments(self):"},{"line_number":148,"context_line":"        # Create Server, Volume and attach that Volume to Server"},{"line_number":149,"context_line":"        server \u003d self._create_server()"},{"line_number":150,"context_line":"        num_vol \u003d len(self.servers_client.list_volume_attachments("},{"line_number":151,"context_line":"            server[\u0027id\u0027])[\u0027volumeAttachments\u0027])"},{"line_number":152,"context_line":"        self._create_and_attach_volume(server)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_b218b81f","line":149,"range":{"start_line":149,"start_character":8,"end_line":149,"end_character":38},"updated":"2016-09-07 17:02:10.000000000","message":"The server is created here and the number should be 0 as list_volume_attachments by the default.\nSo I don\u0027t think current hard-coded check is wrong.\nI am not sure why we need to add more API operation here..","commit_id":"8d6da14879c982b61eacf844805af2469cc9943e"}]}