qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: John Snow <jsnow@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>,
	Ishani Chugh <chugh.ishani@research.iiit.ac.in>
Cc: QEMU Developers <qemu-devel@nongnu.org>,
	Stefan Hajnoczi <stefanha@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 2/3] Test for full Backup
Date: Fri, 25 Aug 2017 11:15:37 -0400	[thread overview]
Message-ID: <4f8e1d78-52e5-8047-5899-4e2a54093dd1@redhat.com> (raw)
In-Reply-To: <CAFEAcA_K8XK_pWsy_KUDgJt8gqfAKJsHfDA6MeW651nqGz82CQ@mail.gmail.com>



On 08/25/2017 10:36 AM, Peter Maydell wrote:
> On 23 August 2017 at 14:04, Ishani Chugh
> <chugh.ishani@research.iiit.ac.in> wrote:
>> This patch is the test for full backup implementation in Backup tool.
>> The test employs two basic substests:
>> 1) Backing up an empty guest and comparing it with base image.
>> 2) Writing a pattern to the guest, creating backup and comparing
>>    with the base image.
>>
>> Signed-off-by: Ishani Chugh <chugh.ishani@research.iiit.ac.in>
>> ---
>>  tests/qemu-iotests/191     | 78 ++++++++++++++++++++++++++++++++++++++++++++++
>>  tests/qemu-iotests/191.out | 33 ++++++++++++++++++++
>>  tests/qemu-iotests/group   |  1 +
>>  3 files changed, 112 insertions(+)
>>  create mode 100755 tests/qemu-iotests/191
>>  create mode 100644 tests/qemu-iotests/191.out
>>
>> diff --git a/tests/qemu-iotests/191 b/tests/qemu-iotests/191
>> new file mode 100755
>> index 0000000..fb4cde9
>> --- /dev/null
>> +++ b/tests/qemu-iotests/191
>> @@ -0,0 +1,78 @@
>> +#!/bin/bash
>> +#
>> +# Test full backup functionality of qemu-backup tool
>> +#
>> +# Copyright (C) 2009 Red Hat, Inc.
> 
> Should this perhaps have a different copyright owner and date?
> 
> thanks
> -- PMM
> 
Yes!

Ishani, please take credit for your own work! :)

  reply	other threads:[~2017-08-25 15:15 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-23 13:04 [Qemu-devel] [PATCH 0/3] QEMU Backup Tool Ishani Chugh
2017-08-23 13:04 ` [Qemu-devel] [PATCH 1/3] backup: " Ishani Chugh
2017-08-25 14:12   ` Stefan Hajnoczi
2017-08-23 13:04 ` [Qemu-devel] [PATCH 2/3] Test for full Backup Ishani Chugh
2017-08-25 14:24   ` Stefan Hajnoczi
2017-08-25 14:36   ` Peter Maydell
2017-08-25 15:15     ` John Snow [this message]
2017-08-23 13:04 ` [Qemu-devel] [PATCH 3/3] Add manpage for QEMU Backup Tool Ishani Chugh
  -- strict thread matches above, loose matches on Subject: below --
2017-08-29 15:50 [Qemu-devel] [PATCH 0/3] " Ishani Chugh
2017-08-29 15:50 ` [Qemu-devel] [PATCH 2/3] Test for full Backup Ishani Chugh
2017-08-29 13:47 [Qemu-devel] [PATCH 0/3] QEMU Backup Tool Ishani Chugh
2017-08-29 13:48 ` [Qemu-devel] [PATCH 2/3] Test for full Backup Ishani Chugh
2017-08-21 11:10 [Qemu-devel] [PATCH 0/3] QEMU Backup Tool Ishani Chugh
2017-08-21 11:10 ` [Qemu-devel] [PATCH 2/3] Test for full Backup Ishani Chugh
2017-08-22 15:21   ` Stefan Hajnoczi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4f8e1d78-52e5-8047-5899-4e2a54093dd1@redhat.com \
    --to=jsnow@redhat.com \
    --cc=chugh.ishani@research.iiit.ac.in \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).