)]}'
{"tempest/api/volume/admin/test_snapshots_actions.py":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"764cf470803c8bf9510e4120f21a815926363ea3","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        resp, temp_snapshot \u003d \\"},{"line_number":73,"context_line":"            self.client.create_snapshot(self.volume[\u0027id\u0027],"},{"line_number":74,"context_line":"                                        display_name\u003dsnp_name)"},{"line_number":75,"context_line":"        self.client.wait_for_volume_status(temp_snapshot[\u0027id\u0027], \u0027available\u0027)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        return temp_snapshot"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2FR2I%3D","line":75,"updated":"2014-01-15 15:46:44.000000000","message":"as you probably saw in the tests: self.client.wait_for_snapshot_status instead of volume_status","commit_id":"947e26df16c7c36157b09e9b28478a007383836c"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"b81dadf8dccad5ac65822980e365e5a68ed378d0","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        resp, temp_snapshot \u003d \\"},{"line_number":73,"context_line":"            self.client.create_snapshot(self.volume[\u0027id\u0027],"},{"line_number":74,"context_line":"                                        display_name\u003dsnp_name)"},{"line_number":75,"context_line":"        self.client.wait_for_volume_status(temp_snapshot[\u0027id\u0027], \u0027available\u0027)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        return temp_snapshot"},{"line_number":78,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F%2FPaU%3D","line":75,"in_reply_to":"AAAAUH%2F%2FR2I%3D","updated":"2014-01-16 00:49:37.000000000","message":"Thanks, it is my fault.","commit_id":"947e26df16c7c36157b09e9b28478a007383836c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"e147d7a9a2a61f63ac5bcb2fbae25548a6b259df","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                                                          status)"},{"line_number":67,"context_line":"        super(SnapshotsActionsTest, self).tearDown()"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def _create_temp_snapshot(self):"},{"line_number":70,"context_line":"        # Create a temp snapshot for force delete tests"},{"line_number":71,"context_line":"        snp_name \u003d data_utils.rand_name(\u0027Snapshot\u0027)"},{"line_number":72,"context_line":"        resp, temp_snapshot \u003d \\"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUH%2F%2BsVo%3D","line":69,"updated":"2014-01-23 04:23:47.000000000","message":"The method which is the same behavior exists already.\nPlease use self.create_snapshot() instead.\n\ntempest/api/volume/base.py\n 57     @classmethod\n 58     def create_snapshot(cls, volume_id\u003d1, **kwargs):\n 59         \"\"\"Wrapper utility that returns a test snapshot.\"\"\"\n 60         resp, snapshot \u003d cls.snapshots_client.create_snapshot(volume_id,\n 61                                                               **kwargs)\n 62         assert 200 \u003d\u003d resp.status\n 63         cls.snapshots.append(snapshot)\n 64         cls.snapshots_client.wait_for_snapshot_status(snapshot[\u0027id\u0027],\n 65                                                       \u0027available\u0027)\n 66         return snapshot","commit_id":"a27a0755f55e3b2b455f6fc7658f719b393d969b"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"ea2ce327a9cd1e620f93dbd4f2d6ff9149533776","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                                                          status)"},{"line_number":67,"context_line":"        super(SnapshotsActionsTest, self).tearDown()"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    def _create_temp_snapshot(self):"},{"line_number":70,"context_line":"        # Create a temp snapshot for force delete tests"},{"line_number":71,"context_line":"        snp_name \u003d data_utils.rand_name(\u0027Snapshot\u0027)"},{"line_number":72,"context_line":"        resp, temp_snapshot \u003d \\"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAUH%2F%2BdTc%3D","line":69,"in_reply_to":"AAAAUH%2F%2BsVo%3D","updated":"2014-01-26 00:48:22.000000000","message":"Thanks, I will use this method.","commit_id":"a27a0755f55e3b2b455f6fc7658f719b393d969b"}]}