[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 12/12] qemu-iotests: add 055 internal snapsho
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH V2 12/12] qemu-iotests: add 055 internal snapshot for block device test case |
Date: |
Wed, 19 Jun 2013 09:46:49 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Jun 19, 2013 at 02:21:31PM +0800, Wenchao Xia wrote:
> δΊ 2013-6-18 22:32, Stefan Hajnoczi ει:
> >On Fri, Jun 14, 2013 at 07:39:59PM +0800, Wenchao Xia wrote:
> >>+ def createSnapshotInTransaction(self, snapshot_num):
> >>+ actions = []
> >>+ for i in range(0, len(self.expect)):
> >>+ num = self.expect[i]['snapshots_name_counter']
> >>+ for j in range(0, snapshot_num):
> >>+ name = '%s_sn%d' % (self.expect[i]['device'], num)
> >>+ num = num + 1
> >>+ self.expect[i]['snapshots'].append(name)
> >>+ actions.append({
> >>+ 'type': 'blockdev-snapshot-internal-sync',
> >>+ 'data': { 'device': self.expect[i]['device'],
> >>+ 'name': name },
> >>+ })
> >>+ self.expect[i]['snapshots_name_counter'] = num
> >>+ result = self.vm.qmp('transaction', actions = actions)
> >>+ self.assert_qmp(result, 'return', {})
> >
> >This tests the success case. Please also use the 'Abort' action to test
> >the error case.
> Is "Abort" upstreamed? If yes I will add that part.
It's part of the drive-backup series which you have already
cherry-picked from.
- [Qemu-devel] [PATCH V2 08/12] qmp: add interface blockdev-snapshot-internal-sync, (continued)
[Qemu-devel] [PATCH V2 05/12] snapshot: add paired functions for internal snapshot id and name, Wenchao Xia, 2013/06/14
[Qemu-devel] [PATCH V2 06/12] snapshot: distinguish id and name in snapshot delete, Wenchao Xia, 2013/06/14
[Qemu-devel] [PATCH V2 10/12] hmp: add interface hmp_snapshot_blkdev_internal, Wenchao Xia, 2013/06/14
[Qemu-devel] [PATCH V2 11/12] hmp: add interface hmp_snapshot_delete_blkdev_internal, Wenchao Xia, 2013/06/14
[Qemu-devel] [PATCH V2 09/12] qmp: add interface blockdev-snapshot-delete-internal-sync, Wenchao Xia, 2013/06/14