From nobody Sun Feb 8 18:57:25 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=openvz.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1649438035662317.64727433177654; Fri, 8 Apr 2022 10:13:55 -0700 (PDT) Received: from localhost ([::1]:50570 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ncsBG-0001cQ-IF for importer@patchew.org; Fri, 08 Apr 2022 13:13:54 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:52992) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ncs0F-0001Kl-CK for qemu-devel@nongnu.org; Fri, 08 Apr 2022 13:02:31 -0400 Received: from mail-lf1-x12e.google.com ([2a00:1450:4864:20::12e]:35504) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1ncs0C-0005CV-7e for qemu-devel@nongnu.org; Fri, 08 Apr 2022 13:02:31 -0400 Received: by mail-lf1-x12e.google.com with SMTP id h14so10984618lfl.2 for ; Fri, 08 Apr 2022 10:02:27 -0700 (PDT) Received: from fedora.. ([185.215.60.161]) by smtp.gmail.com with ESMTPSA id g36-20020a0565123ba400b0044a2a1ccd99sm2505687lfv.20.2022.04.08.10.02.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 Apr 2022 10:02:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=openvz-org.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=3RBxIOwgZcyAShhG7noZhYSNlbV6FHterTNyISA8k8A=; b=O6H5l41nl1DH64UT7hO/flsiUzjiXw4itGgirKeenGtxUAuseyoOnlt93attJfbInn meKdC2qdshFTMMx+IsjBuSEd91/fupI/FLXAjjGC/ZPx636m/s70zPaYgHRmmnUd4mQX 2orvCSV+jB5PyNu1lULdVbd8q5/6ww+T8c1yluhPMw3/mbw+9tDQlehmwxCzbqLGYVCH zH3ajr+i2QWW8UTDFs3MWQOkU0T7Hu8HceVMu/OA+TaK7q9eBsXdfabg3WsqLFBduqu4 OjGznX1ZSbvhPAV2YwW6uhz6mH61WLKZN8vlXc9QXN1IK6HImRkhbczoVUd93gmA8HFA tL2w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=3RBxIOwgZcyAShhG7noZhYSNlbV6FHterTNyISA8k8A=; b=J4bAr/LRYHfaG9hi+mx6CYKLLcTwVvVzKMrcbFoPgOkyt0zInbxDCB+UpqMt66XMJH sHClPMb9x/DiZeKiXQgW90ak9TgBaMGiRo5HdYXfjAeyMK2rsVzgp+5hOh1D2GTk89W7 yHrN7e9Xdpy940/m8W4ND+IE8jXojY9Ato1/TDfGU9I+zqZJ2yQkj9QM+9DEeuD90+9X ZLVJPllQF1ZLUoEMWlyOfGSTU7iGxVdTSHQktJFOIQ86B2q1mle/cEbvjY7jXQYq/7w6 xY+PzEkVaUuAI/+6htAfmABmI/hUfvdN9kHE6l4Wqyw4Fqxfic0QLB0KKw2ipJeRYWHW vbBA== X-Gm-Message-State: AOAM533acAhT90x6e7dM9PWVxm69T++SHVQ8dJPcWjzr1HJrYXBm/OKk JGiN418t+t3cc8Pzb6rnHHCxAg== X-Google-Smtp-Source: ABdhPJy8wcKWmBjrGlLqalHkW/Du2w9p+bhB3oj9neqMPfvBnPI1mFuM5frAdPjqhCzgccxF7wjS0w== X-Received: by 2002:a05:6512:3b9e:b0:44a:47c6:eb16 with SMTP id g30-20020a0565123b9e00b0044a47c6eb16mr13048787lfv.486.1649437345610; Fri, 08 Apr 2022 10:02:25 -0700 (PDT) From: Vladimir Sementsov-Ogievskiy X-Google-Original-From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Subject: [PATCH 1/2] python/machine.py: upgrade vm.command() method Date: Fri, 8 Apr 2022 20:02:13 +0300 Message-Id: <20220408170214.45585-2-vsementsov@openvz.org> X-Mailer: git-send-email 2.35.1 In-Reply-To: <20220408170214.45585-1-vsementsov@openvz.org> References: <20220408170214.45585-1-vsementsov@openvz.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2a00:1450:4864:20::12e; envelope-from=vladimir.sementsov-ogievskiy@openvz.org; helo=mail-lf1-x12e.google.com X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, bleal@redhat.com, qemu-devel@nongnu.org, hreitz@redhat.com, vsementsov@openvz.org, crosa@redhat.com, jsnow@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZM-MESSAGEID: 1649438037831100003 Content-Type: text/plain; charset="utf-8" The method is not popular, we prefer use vm.qmp() and then check success by hand.. But that's not optimal. To simplify movement to vm.command() support same interface improvements like in vm.qmp() and rename to shorter vm.cmd(). Signed-off-by: Vladimir Sementsov-Ogievskiy --- python/qemu/machine/machine.py | 16 ++++++++++++--- tests/qemu-iotests/256 | 34 ++++++++++++++++---------------- tests/qemu-iotests/257 | 36 +++++++++++++++++----------------- 3 files changed, 48 insertions(+), 38 deletions(-) diff --git a/python/qemu/machine/machine.py b/python/qemu/machine/machine.py index 07ac5a710b..a3fb840b93 100644 --- a/python/qemu/machine/machine.py +++ b/python/qemu/machine/machine.py @@ -648,14 +648,24 @@ def qmp(self, cmd: str, self._quit_issued =3D True return ret =20 - def command(self, cmd: str, - conv_keys: bool =3D True, - **args: Any) -> QMPReturnValue: + def cmd(self, cmd: str, + args_dict: Optional[Dict[str, object]] =3D None, + conv_keys: Optional[bool] =3D None, + **args: Any) -> QMPReturnValue: """ Invoke a QMP command. On success return the response dict. On failure raise an exception. """ + if args_dict is not None: + assert not args + assert conv_keys is None + args =3D args_dict + conv_keys =3D False + + if conv_keys is None: + conv_keys =3D True + qmp_args =3D self._qmp_args(conv_keys, args) ret =3D self._qmp.command(cmd, **qmp_args) if cmd =3D=3D 'quit': diff --git a/tests/qemu-iotests/256 b/tests/qemu-iotests/256 index 13666813bd..fffc8ef055 100755 --- a/tests/qemu-iotests/256 +++ b/tests/qemu-iotests/256 @@ -40,25 +40,25 @@ with iotests.FilePath('img0') as img0_path, \ def create_target(filepath, name, size): basename =3D os.path.basename(filepath) nodename =3D "file_{}".format(basename) - log(vm.command('blockdev-create', job_id=3D'job1', - options=3D{ - 'driver': 'file', - 'filename': filepath, - 'size': 0, - })) + log(vm.cmd('blockdev-create', job_id=3D'job1', + options=3D{ + 'driver': 'file', + 'filename': filepath, + 'size': 0, + })) vm.run_job('job1') - log(vm.command('blockdev-add', driver=3D'file', - node_name=3Dnodename, filename=3Dfilepath)) - log(vm.command('blockdev-create', job_id=3D'job2', - options=3D{ - 'driver': iotests.imgfmt, - 'file': nodename, - 'size': size, - })) + log(vm.cmd('blockdev-add', driver=3D'file', + node_name=3Dnodename, filename=3Dfilepath)) + log(vm.cmd('blockdev-create', job_id=3D'job2', + options=3D{ + 'driver': iotests.imgfmt, + 'file': nodename, + 'size': size, + })) vm.run_job('job2') - log(vm.command('blockdev-add', driver=3Diotests.imgfmt, - node_name=3Dname, - file=3Dnodename)) + log(vm.cmd('blockdev-add', driver=3Diotests.imgfmt, + node_name=3Dname, + file=3Dnodename)) =20 log('--- Preparing images & VM ---\n') vm.add_object('iothread,id=3Diothread0') diff --git a/tests/qemu-iotests/257 b/tests/qemu-iotests/257 index e7e7a2317e..7d3720b8e5 100755 --- a/tests/qemu-iotests/257 +++ b/tests/qemu-iotests/257 @@ -160,26 +160,26 @@ class Drive: file_node_name =3D "file_{}".format(basename) vm =3D self.vm =20 - log(vm.command('blockdev-create', job_id=3D'bdc-file-job', - options=3D{ - 'driver': 'file', - 'filename': self.path, - 'size': 0, - })) + log(vm.cmd('blockdev-create', job_id=3D'bdc-file-job', + options=3D{ + 'driver': 'file', + 'filename': self.path, + 'size': 0, + })) vm.run_job('bdc-file-job') - log(vm.command('blockdev-add', driver=3D'file', - node_name=3Dfile_node_name, filename=3Dself.path)) - - log(vm.command('blockdev-create', job_id=3D'bdc-fmt-job', - options=3D{ - 'driver': fmt, - 'file': file_node_name, - 'size': size, - })) + log(vm.cmd('blockdev-add', driver=3D'file', + node_name=3Dfile_node_name, filename=3Dself.path)) + + log(vm.cmd('blockdev-create', job_id=3D'bdc-fmt-job', + options=3D{ + 'driver': fmt, + 'file': file_node_name, + 'size': size, + })) vm.run_job('bdc-fmt-job') - log(vm.command('blockdev-add', driver=3Dfmt, - node_name=3Dname, - file=3Dfile_node_name)) + log(vm.cmd('blockdev-add', driver=3Dfmt, + node_name=3Dname, + file=3Dfile_node_name)) self.fmt =3D fmt self.size =3D size self.node =3D name --=20 2.35.1 From nobody Sun Feb 8 18:57:25 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=openvz.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1649437496215833.2434463156599; Fri, 8 Apr 2022 10:04:56 -0700 (PDT) Received: from localhost ([::1]:59126 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ncs2Y-0004Hq-UI for importer@patchew.org; Fri, 08 Apr 2022 13:04:54 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:53024) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ncs0H-0001R1-Up for qemu-devel@nongnu.org; Fri, 08 Apr 2022 13:02:33 -0400 Received: from mail-lf1-x135.google.com ([2a00:1450:4864:20::135]:38500) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1ncs0D-0005Ck-CT for qemu-devel@nongnu.org; Fri, 08 Apr 2022 13:02:33 -0400 Received: by mail-lf1-x135.google.com with SMTP id b21so16128675lfb.5 for ; Fri, 08 Apr 2022 10:02:28 -0700 (PDT) Received: from fedora.. ([185.215.60.161]) by smtp.gmail.com with ESMTPSA id g36-20020a0565123ba400b0044a2a1ccd99sm2505687lfv.20.2022.04.08.10.02.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 Apr 2022 10:02:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=openvz-org.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=mmvDOHk3JJVJDN3eq+omTJ471I+S1iqtvxcVwubF8SI=; b=k9LDncvat1/uZnBsEJuMOryDZKTPCgPxX7zqENQfeHrpx7wQDS7iWBhBRSnBNbbe38 Ah3Yj6diacNehkCKbtnC1ozIcnEItCENdujAifBgTIXhBTGyV0oWwWnTLUWhDiFwneaU wvE+dh9BLs8iRbwqf42GkgK0jVaFjHSCq4TUUYIDKNjsQ3Qabm3v6R8BBiYObgERSRO5 x8Mqjydrr0aNzJndDT303nCiPkqG2j7fX2V04tvjP18p1Qf6Y26aviW5yFHbvzLQK/Gn Kx1WiHKStcVGSigsYgIbm60eP2GkMPbyb+5CXhAJdH7Sz7h8jxGi52OonkWfTb3qj5Gq rcGg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=mmvDOHk3JJVJDN3eq+omTJ471I+S1iqtvxcVwubF8SI=; b=DEO7hvT0bt+ubQoqP6j+lPFwhgfrBDVugl1nP4f2I0PgzDmX7d4q7963JsCx5frxbW vISXQ3BqY2B2OwXe14a2k8t8/U6wXMH427rUr27p1SdzHnRTh90GQShMhXzJXHmvy8jE navANiGs6jtHtyYjFf1InVQQ/VLpbFQOS/8lpGTFymd6kFGEQVTjpTADaC++k/vxLjjV aSogu67ijNZn1tzyHf5la/fQxpsCtX6m0tRQerLyQ+HuN7yD5QYfNVfXQuUP/IMJcRjO NZtbqKx+kjXOjFGzeLkD0GCd1bafhJ7vPkFJL9DBqv8bJZAm5tIZIbLl9xqrEnwgdvCO QVXQ== X-Gm-Message-State: AOAM533yfm9uRjx7PggnmWyhH+/Xipm7ZZGZCbssNXwDmzUnCBZXGthB AX2lYQe5OKK11YY8FErew+JS9Q== X-Google-Smtp-Source: ABdhPJwnKLTTYB+dO3z4MzivzYCp/pCFYiPIC5Sl+MniGPMLAzygImh/zpEyHpgxMiOa5xmxzPUcmw== X-Received: by 2002:a05:6512:ace:b0:44a:d351:5268 with SMTP id n14-20020a0565120ace00b0044ad3515268mr12959536lfu.572.1649437346579; Fri, 08 Apr 2022 10:02:26 -0700 (PDT) From: Vladimir Sementsov-Ogievskiy X-Google-Original-From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Subject: [PATCH 2/2] iotests: use vm.cmd() instead of vm.qmp() where appropriate Date: Fri, 8 Apr 2022 20:02:14 +0300 Message-Id: <20220408170214.45585-3-vsementsov@openvz.org> X-Mailer: git-send-email 2.35.1 In-Reply-To: <20220408170214.45585-1-vsementsov@openvz.org> References: <20220408170214.45585-1-vsementsov@openvz.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2a00:1450:4864:20::135; envelope-from=vladimir.sementsov-ogievskiy@openvz.org; helo=mail-lf1-x135.google.com X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, bleal@redhat.com, qemu-devel@nongnu.org, hreitz@redhat.com, vsementsov@openvz.org, crosa@redhat.com, jsnow@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZM-MESSAGEID: 1649437500384100001 Content-Type: text/plain; charset="utf-8" Checking success by hand every time is tedious. Let's use cmd() method instead which does this check for us. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Eric Blake --- tests/qemu-iotests/030 | 168 +++---- tests/qemu-iotests/040 | 167 +++--- tests/qemu-iotests/041 | 474 ++++++++---------- tests/qemu-iotests/045 | 15 +- tests/qemu-iotests/055 | 61 +-- tests/qemu-iotests/056 | 23 +- tests/qemu-iotests/093 | 41 +- tests/qemu-iotests/118 | 221 ++++---- tests/qemu-iotests/124 | 69 ++- tests/qemu-iotests/129 | 13 +- tests/qemu-iotests/132 | 5 +- tests/qemu-iotests/139 | 43 +- tests/qemu-iotests/147 | 30 +- tests/qemu-iotests/151 | 40 +- tests/qemu-iotests/155 | 53 +- tests/qemu-iotests/165 | 7 +- tests/qemu-iotests/196 | 3 +- tests/qemu-iotests/205 | 6 +- tests/qemu-iotests/245 | 245 ++++----- tests/qemu-iotests/264 | 31 +- tests/qemu-iotests/281 | 21 +- tests/qemu-iotests/295 | 27 +- tests/qemu-iotests/296 | 14 +- tests/qemu-iotests/298 | 13 +- tests/qemu-iotests/300 | 50 +- tests/qemu-iotests/iotests.py | 6 +- .../tests/migrate-bitmaps-postcopy-test | 31 +- tests/qemu-iotests/tests/migrate-bitmaps-test | 37 +- .../qemu-iotests/tests/migrate-during-backup | 40 +- .../qemu-iotests/tests/migration-permissions | 9 +- tests/qemu-iotests/tests/mirror-top-perms | 15 +- 31 files changed, 773 insertions(+), 1205 deletions(-) diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030 index 18eddcc734..7c79fe041d 100755 --- a/tests/qemu-iotests/030 +++ b/tests/qemu-iotests/030 @@ -56,8 +56,7 @@ class TestSingleDrive(iotests.QMPTestCase): def test_stream(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 self.wait_until_completed() =20 @@ -75,8 +74,7 @@ class TestSingleDrive(iotests.QMPTestCase): qemu_io('-f', iotests.imgfmt, '-rU', '-c', 'ma= p', mid_img), 'image file map matches backing file before st= reaming') =20 - result =3D self.vm.qmp('block-stream', device=3D'mid', job_id=3D's= tream-mid') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'mid', job_id=3D'stream-mid') =20 self.wait_until_completed(drive=3D'stream-mid') =20 @@ -91,8 +89,7 @@ class TestSingleDrive(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 self.pause_job('drive0', wait=3DFalse) self.vm.resume_drive('drive0') @@ -105,8 +102,7 @@ class TestSingleDrive(iotests.QMPTestCase): result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/offset', offset) =20 - result =3D self.vm.qmp('block-job-resume', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 self.wait_until_completed() =20 @@ -124,8 +120,7 @@ class TestSingleDrive(iotests.QMPTestCase): empty_map =3D qemu_io('-f', iotests.imgfmt, '-rU', '-c', 'map', te= st_img) =20 # This is a no-op: no data should ever be copied from the base ima= ge - result =3D self.vm.qmp('block-stream', device=3D'drive0', base=3Dm= id_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0', base=3Dmid_img) =20 self.wait_until_completed() =20 @@ -138,8 +133,7 @@ class TestSingleDrive(iotests.QMPTestCase): def test_stream_partial(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0', base=3Db= acking_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0', base=3Dbacking_img) =20 self.wait_until_completed() =20 @@ -165,24 +159,22 @@ class TestSingleDrive(iotests.QMPTestCase): qemu_img('create', '-f', iotests.imgfmt, ro_top_path, str(self.image_len)) =20 - result =3D self.vm.qmp('blockdev-add', - node_name=3D'ro-top', - driver=3Diotests.imgfmt, - read_only=3DTrue, - file=3D{ - 'driver': 'file', - 'filename': ro_top_path, - 'read-only': True - }, - backing=3D'mid') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + node_name=3D'ro-top', + driver=3Diotests.imgfmt, + read_only=3DTrue, + file=3D{ + 'driver': 'file', + 'filename': ro_top_path, + 'read-only': True + }, + backing=3D'mid') =20 result =3D self.vm.qmp('block-stream', job_id=3D'stream', device=3D'ro-top', base_node=3D'base') self.assert_qmp(result, 'error/desc', 'Block node is read-only= ') =20 - result =3D self.vm.qmp('blockdev-del', node_name=3D'ro-top') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'ro-top') =20 =20 class TestParallelOps(iotests.QMPTestCase): @@ -246,10 +238,9 @@ class TestParallelOps(iotests.QMPTestCase): node_name =3D 'node%d' % i job_id =3D 'stream-%s' % node_name pending_jobs.append(job_id) - result =3D self.vm.qmp('block-stream', device=3Dnode_name, - job_id=3Djob_id, bottom=3Df'node{i-1}', - speed=3D1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3Dnode_name, + job_id=3Djob_id, bottom=3Df'node{i-1}', + speed=3D1024) =20 # Do this in reverse: After unthrottling them, some jobs may finish # before we have unthrottled all of them. This will drain their @@ -261,8 +252,7 @@ class TestParallelOps(iotests.QMPTestCase): # Starting from the top (i.e. in reverse) does not have this probl= em: # When a job finishes, the ones below it are not advanced. for job in reversed(pending_jobs): - result =3D self.vm.qmp('block-job-set-speed', device=3Djob, sp= eed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3Djob, speed=3D0) =20 # Wait for all jobs to be finished. while len(pending_jobs) > 0: @@ -288,10 +278,9 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # Set a speed limit to make sure that this job blocks the rest - result =3D self.vm.qmp('block-stream', device=3D'node4', - job_id=3D'stream-node4', base=3Dself.imgs[1], - filter_node_name=3D'stream-filter', speed=3D1= 024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'node4', + job_id=3D'stream-node4', base=3Dself.imgs[1], + filter_node_name=3D'stream-filter', speed=3D1024*1024) =20 result =3D self.vm.qmp('block-stream', device=3D'node5', job_id=3D= 'stream-node5', base=3Dself.imgs[2]) self.assert_qmp(result, 'error/desc', @@ -319,8 +308,7 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_qmp(result, 'error/desc', "Node 'node2' is busy: block device is in use by block job: st= ream") =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'stream-nod= e4', speed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'stream-node4', speed= =3D0) =20 self.wait_until_completed(drive=3D'stream-node4') self.assert_no_active_block_jobs() @@ -332,8 +320,7 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # Set a speed limit to make sure that this job blocks the rest - result =3D self.vm.qmp('block-commit', device=3D'drive0', top=3Dse= lf.imgs[5], base=3Dself.imgs[3], job_id=3D'commit-node3', speed=3D1024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', top=3Dself.imgs[5],= base=3Dself.imgs[3], job_id=3D'commit-node3', speed=3D1024*1024) =20 result =3D self.vm.qmp('block-stream', device=3D'node3', job_id=3D= 'stream-node3') self.assert_qmp(result, 'error/desc', @@ -356,8 +343,7 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_qmp(result, 'error/desc', "Node 'drive0' is busy: block device is in use by block job: c= ommit") =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'commit-nod= e3', speed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'commit-node3', speed= =3D0) =20 self.wait_until_completed(drive=3D'commit-node3') =20 @@ -368,23 +354,20 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # Set a speed limit to make sure that this job blocks the rest - result =3D self.vm.qmp('block-commit', device=3D'drive0', base=3Ds= elf.imgs[3], job_id=3D'commit-drive0', speed=3D1024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', base=3Dself.imgs[3]= , job_id=3D'commit-drive0', speed=3D1024*1024) =20 result =3D self.vm.qmp('block-stream', device=3D'node5', base=3Dse= lf.imgs[3], job_id=3D'stream-node6') self.assert_qmp(result, 'error/desc', "Node 'node5' is busy: block device is in use by block job: co= mmit") =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'commit-dri= ve0', speed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'commit-drive0', speed= =3D0) =20 event =3D self.vm.event_wait(name=3D'BLOCK_JOB_READY') self.assert_qmp(event, 'data/device', 'commit-drive0') self.assert_qmp(event, 'data/type', 'commit') self.assert_qmp_absent(event, 'data/error') =20 - result =3D self.vm.qmp('block-job-complete', device=3D'commit-driv= e0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-complete', device=3D'commit-drive0') =20 self.wait_until_completed(drive=3D'commit-drive0') =20 @@ -395,18 +378,16 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # Commit from node2 into node0 - result =3D self.vm.qmp('block-commit', device=3D'drive0', - top=3Dself.imgs[2], base=3Dself.imgs[0], - filter_node_name=3D'commit-filter', speed=3D1= 024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', + top=3Dself.imgs[2], base=3Dself.imgs[0], + filter_node_name=3D'commit-filter', speed=3D1024*1024) =20 # Stream from node2 into node4 result =3D self.vm.qmp('block-stream', device=3D'node4', base_node= =3D'node2', job_id=3D'node4') self.assert_qmp(result, 'error/desc', "Cannot freeze 'backing' link to 'commit-filter'") =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D0) =20 self.wait_until_completed() self.assert_no_active_block_jobs() @@ -419,18 +400,15 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # Commit from node2 into node0 - result =3D self.vm.qmp('block-commit', device=3D'drive0', - top_node=3D'node2', base_node=3D'node0', - filter_node_name=3D'commit-filter', speed=3D1= 024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', + top_node=3D'node2', base_node=3D'node0', + filter_node_name=3D'commit-filter', speed=3D1024*1024) =20 # Stream from node2 into node4 - result =3D self.vm.qmp('block-stream', device=3D'node4', - base_node=3D'commit-filter', job_id=3D'node4') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'node4', + base_node=3D'commit-filter', job_id=3D'node4') =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D0) =20 self.vm.run_job(job=3D'drive0', auto_dismiss=3DTrue) self.vm.run_job(job=3D'node4', auto_dismiss=3DTrue) @@ -449,12 +427,10 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # Stream from node0 into node2 - result =3D self.vm.qmp('block-stream', device=3D'node2', base_node= =3D'node0', job_id=3D'node2') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'node2', base_node=3D'node0',= job_id=3D'node2') =20 # Commit from the active layer into node3 - result =3D self.vm.qmp('block-commit', device=3D'drive0', base=3Ds= elf.imgs[3]) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', base=3Dself.imgs[3]) =20 # Wait for all jobs to be finished. pending_jobs =3D ['node2', 'drive0'] @@ -481,16 +457,13 @@ class TestParallelOps(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # Stream from node0 into node4 - result =3D self.vm.qmp('block-stream', device=3D'node4', base_node= =3D'node0', job_id=3D'node4', speed=3D1024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'node4', base_node=3D'node0',= job_id=3D'node4', speed=3D1024*1024) =20 # Commit from the active layer into node5 - result =3D self.vm.qmp('block-commit', device=3D'drive0', base=3Ds= elf.imgs[5], speed=3D1024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', base=3Dself.imgs[5]= , speed=3D1024*1024) =20 for job in ['drive0', 'node4']: - result =3D self.vm.qmp('block-job-set-speed', device=3Djob, sp= eed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3Djob, speed=3D0) =20 # Wait for all jobs to be finished. pending_jobs =3D ['node4', 'drive0'] @@ -539,8 +512,7 @@ class TestParallelOps(iotests.QMPTestCase): "'base' and 'base-node' cannot be specified at the same time") =20 # Success: the base node is a backing file of the top node - result =3D self.vm.qmp('block-stream', device=3D'node4', base_node= =3D'node2', job_id=3D'stream') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'node4', base_node=3D'node2',= job_id=3D'stream') =20 self.wait_until_completed(drive=3D'stream') =20 @@ -594,8 +566,7 @@ class TestQuorum(iotests.QMPTestCase): =20 self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'node0', job_id=3D= 'stream-node0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'node0', job_id=3D'stream-nod= e0') =20 self.wait_until_completed(drive=3D'stream-node0') =20 @@ -623,8 +594,7 @@ class TestSmallerBackingFile(iotests.QMPTestCase): def test_stream(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 self.wait_until_completed() =20 @@ -681,8 +651,7 @@ class TestEIO(TestErrors): def test_report(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 completed =3D False error =3D False @@ -709,8 +678,7 @@ class TestEIO(TestErrors): def test_ignore(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0', on_error= =3D'ignore') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0', on_error=3D'ignore') =20 error =3D False completed =3D False @@ -743,8 +711,7 @@ class TestEIO(TestErrors): def test_stop(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0', on_error= =3D'stop') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0', on_error=3D'stop') =20 error =3D False completed =3D False @@ -766,8 +733,7 @@ class TestEIO(TestErrors): self.assert_qmp(result, 'return[0]/offset', self.STREA= M_BUFFER_SIZE) self.assert_qmp(result, 'return[0]/io-status', 'failed= ') =20 - result =3D self.vm.qmp('block-job-resume', device=3D'd= rive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 result =3D self.vm.qmp('query-block-jobs') if result =3D=3D {'return': []}: @@ -793,8 +759,7 @@ class TestEIO(TestErrors): def test_enospc(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0', on_error= =3D'enospc') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0', on_error=3D'enospc') =20 completed =3D False error =3D False @@ -839,8 +804,7 @@ class TestENOSPC(TestErrors): def test_enospc(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0', on_error= =3D'enospc') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0', on_error=3D'enospc') =20 error =3D False completed =3D False @@ -862,8 +826,7 @@ class TestENOSPC(TestErrors): self.assert_qmp(result, 'return[0]/offset', self.STREA= M_BUFFER_SIZE) self.assert_qmp(result, 'return[0]/io-status', 'nospac= e') =20 - result =3D self.vm.qmp('block-job-resume', device=3D'd= rive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 result =3D self.vm.qmp('query-block-jobs') if result =3D=3D {'return': []}: @@ -908,8 +871,7 @@ class TestStreamStop(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 time.sleep(0.1) events =3D self.vm.get_qmp_events(wait=3DFalse) @@ -942,11 +904,9 @@ class TestSetSpeed(iotests.QMPTestCase): def perf_test_throughput(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D8 * 1024 * 1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D8 * = 1024 * 1024) =20 self.wait_until_completed() =20 @@ -956,16 +916,14 @@ class TestSetSpeed(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 # Default speed is 0 result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/device', 'drive0') self.assert_qmp(result, 'return[0]/speed', 0) =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D8 * 1024 * 1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D8 * = 1024 * 1024) =20 # Ensure the speed we set was accepted result =3D self.vm.qmp('query-block-jobs') @@ -976,8 +934,7 @@ class TestSetSpeed(iotests.QMPTestCase): self.vm.pause_drive('drive0') =20 # Check setting speed in block-stream works - result =3D self.vm.qmp('block-stream', device=3D'drive0', speed=3D= 4 * 1024 * 1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0', speed=3D4 * 1024 * = 1024) =20 result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/device', 'drive0') @@ -994,8 +951,7 @@ class TestSetSpeed(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp('block-stream', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', device=3D'drive0') =20 result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D-1) self.assert_qmp(result, 'error/desc', "Parameter 'speed' expects a= non-negative value") diff --git a/tests/qemu-iotests/040 b/tests/qemu-iotests/040 index 0e1cfd7e49..d43f3c3130 100755 --- a/tests/qemu-iotests/040 +++ b/tests/qemu-iotests/040 @@ -61,16 +61,14 @@ class ImageCommitTestCase(iotests.QMPTestCase): def run_commit_test(self, top, base, need_ready=3DFalse, node_names=3D= False): self.assert_no_active_block_jobs() if node_names: - result =3D self.vm.qmp('block-commit', device=3D'drive0', top_= node=3Dtop, base_node=3Dbase) + self.vm.cmd('block-commit', device=3D'drive0', top_node=3Dtop,= base_node=3Dbase) else: - result =3D self.vm.qmp('block-commit', device=3D'drive0', top= =3Dtop, base=3Dbase) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', top=3Dtop, base= =3Dbase) self.wait_for_complete(need_ready) =20 def run_default_commit_test(self): self.assert_no_active_block_jobs() - result =3D self.vm.qmp('block-commit', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0') self.wait_for_complete() =20 class TestSingleDrive(ImageCommitTestCase): @@ -111,38 +109,30 @@ class TestSingleDrive(ImageCommitTestCase): =20 @iotests.skip_if_unsupported(['throttle']) def test_commit_with_filter_and_quit(self): - result =3D self.vm.qmp('object-add', qom_type=3D'throttle-group', = id=3D'tg') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', qom_type=3D'throttle-group', id=3D'tg') =20 # Add a filter outside of the backing chain - result =3D self.vm.qmp('blockdev-add', driver=3D'throttle', node_n= ame=3D'filter', throttle_group=3D'tg', file=3D'mid') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', driver=3D'throttle', node_name=3D'filt= er', throttle_group=3D'tg', file=3D'mid') =20 - result =3D self.vm.qmp('block-commit', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0') =20 # Quit immediately, thus forcing a simultaneous cancel of the # block job and a bdrv_drain_all() - result =3D self.vm.qmp('quit') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('quit') =20 # Same as above, but this time we add the filter after starting the job @iotests.skip_if_unsupported(['throttle']) def test_commit_plus_filter_and_quit(self): - result =3D self.vm.qmp('object-add', qom_type=3D'throttle-group', = id=3D'tg') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', qom_type=3D'throttle-group', id=3D'tg') =20 - result =3D self.vm.qmp('block-commit', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0') =20 # Add a filter outside of the backing chain - result =3D self.vm.qmp('blockdev-add', driver=3D'throttle', node_n= ame=3D'filter', throttle_group=3D'tg', file=3D'mid') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', driver=3D'throttle', node_name=3D'filt= er', throttle_group=3D'tg', file=3D'mid') =20 # Quit immediately, thus forcing a simultaneous cancel of the # block job and a bdrv_drain_all() - result =3D self.vm.qmp('quit') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('quit') =20 def test_device_not_found(self): result =3D self.vm.qmp('block-commit', device=3D'nonexistent', top= =3D'%s' % mid_img) @@ -215,8 +205,7 @@ class TestSingleDrive(ImageCommitTestCase): def test_top_node_in_wrong_chain(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('blockdev-add', driver=3D'null-co', node_na= me=3D'null') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', driver=3D'null-co', node_name=3D'null') =20 result =3D self.vm.qmp('block-commit', device=3D'drive0', top_node= =3D'null', base_node=3D'base') self.assert_qmp(result, 'error/class', 'GenericError') @@ -229,11 +218,9 @@ class TestSingleDrive(ImageCommitTestCase): return =20 self.assert_no_active_block_jobs() - result =3D self.vm.qmp('block-commit', device=3D'drive0', top=3Dmi= d_img, - base=3Dbacking_img, speed=3D(self.image_len /= / 4)) - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('device_del', id=3D'scsi0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', top=3Dmid_img, + base=3Dbacking_img, speed=3D(self.image_len // 4)) + self.vm.cmd('device_del', id=3D'scsi0') =20 cancelled =3D False deleted =3D False @@ -259,9 +246,8 @@ class TestSingleDrive(ImageCommitTestCase): return =20 self.assert_no_active_block_jobs() - result =3D self.vm.qmp('block-commit', device=3D'drive0', top=3Dmi= d_img, - base=3Dbacking_img, speed=3D(self.image_len /= / 4)) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', top=3Dmid_img, + base=3Dbacking_img, speed=3D(self.image_len // 4)) =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/file', test_img) @@ -396,8 +382,7 @@ class TestSetSpeed(ImageCommitTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp('block-commit', device=3D'drive0', top=3Dmi= d_img, speed=3D1024 * 1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0', top=3Dmid_img, spee= d=3D1024 * 1024) =20 # Ensure the speed we set was accepted result =3D self.vm.qmp('query-block-jobs') @@ -470,8 +455,7 @@ class TestErrorHandling(iotests.QMPTestCase): os.remove(backing_img) =20 def blockdev_add(self, **kwargs): - result =3D self.vm.qmp('blockdev-add', **kwargs) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', **kwargs) =20 def add_block_nodes(self, base_debug=3DNone, mid_debug=3DNone, top_deb= ug=3DNone): self.blockdev_add(node_name=3D'base-file', driver=3D'file', @@ -517,11 +501,9 @@ class TestErrorHandling(iotests.QMPTestCase): completed =3D True elif ev['event'] =3D=3D 'BLOCK_JOB_ERROR': if error_pauses_job: - result =3D self.vm.qmp('block-job-resume', device=3D'j= ob0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'job0') elif ev['event'] =3D=3D 'BLOCK_JOB_READY': - result =3D self.vm.qmp('block-job-complete', device=3D'job= 0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-complete', device=3D'job0') else: self.fail("Unexpected event: %s" % ev) log.append(iotests.filter_qmp_event(ev)) @@ -584,11 +566,10 @@ class TestErrorHandling(iotests.QMPTestCase): self.add_block_nodes(top_debug=3Dtop_debug, mid_debug=3Dmid_debug, base_debug=3Dbase_debug) =20 - result =3D self.vm.qmp('block-commit', job_id=3D'job0', device=3D'= top-fmt', - top_node=3D'top-fmt' if active else 'mid-fmt', - base_node=3D'mid-fmt' if active else 'base-fm= t', - on_error=3Don_error) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', job_id=3D'job0', device=3D'top-fmt', + top_node=3D'top-fmt' if active else 'mid-fmt', + base_node=3D'mid-fmt' if active else 'base-fmt', + on_error=3Don_error) =20 def testActiveReadErrorReport(self): self.prepare_and_start_job('report', top_event=3D'read_aio') @@ -761,10 +742,9 @@ class TestCommitWithFilters(iotests.QMPTestCase): self.vm =3D iotests.VM().add_device('virtio-scsi,id=3Dvio-scsi') self.vm.launch() =20 - result =3D self.vm.qmp('object-add', qom_type=3D'throttle-group', = id=3D'tg') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', qom_type=3D'throttle-group', id=3D'tg') =20 - result =3D self.vm.qmp('blockdev-add', **{ + self.vm.cmd('blockdev-add', { 'node-name': 'top-filter', 'driver': 'throttle', 'throttle-group': 'tg', @@ -806,7 +786,6 @@ class TestCommitWithFilters(iotests.QMPTestCase): } } }) - self.assert_qmp(result, 'return', {}) =20 def tearDown(self): self.vm.shutdown() @@ -823,12 +802,11 @@ class TestCommitWithFilters(iotests.QMPTestCase): return self.vm.node_info(node)['image']['filename'] =20 def test_filterless_commit(self): - result =3D self.vm.qmp('block-commit', - job_id=3D'commit', - device=3D'top-filter', - top_node=3D'cow-2', - base_node=3D'cow-1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', + job_id=3D'commit', + device=3D'top-filter', + top_node=3D'cow-2', + base_node=3D'cow-1') self.wait_until_completed(drive=3D'commit') =20 self.assertIsNotNone(self.vm.node_info('cow-3')) @@ -839,12 +817,11 @@ class TestCommitWithFilters(iotests.QMPTestCase): self.pattern_files[2] =3D self.img1 =20 def test_commit_through_filter(self): - result =3D self.vm.qmp('block-commit', - job_id=3D'commit', - device=3D'top-filter', - top_node=3D'cow-1', - base_node=3D'cow-0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', + job_id=3D'commit', + device=3D'top-filter', + top_node=3D'cow-1', + base_node=3D'cow-0') self.wait_until_completed(drive=3D'commit') =20 self.assertIsNotNone(self.vm.node_info('cow-2')) @@ -859,9 +836,8 @@ class TestCommitWithFilters(iotests.QMPTestCase): # Add a device, so the commit job finds a parent it can change # to point to the base node (so we can test that top-filter is # dropped from the graph) - result =3D self.vm.qmp('device_add', id=3D'drv0', driver=3D'scsi-h= d', - bus=3D'vio-scsi.0', drive=3D'top-filter') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', id=3D'drv0', driver=3D'scsi-hd', + bus=3D'vio-scsi.0', drive=3D'top-filter') =20 # Try to release our reference to top-filter; that should not # work because drv0 uses it @@ -869,16 +845,14 @@ class TestCommitWithFilters(iotests.QMPTestCase): self.assert_qmp(result, 'error/class', 'GenericError') self.assert_qmp(result, 'error/desc', 'Node top-filter is in use') =20 - result =3D self.vm.qmp('block-commit', - job_id=3D'commit', - device=3D'top-filter', - base_node=3D'cow-2') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', + job_id=3D'commit', + device=3D'top-filter', + base_node=3D'cow-2') self.complete_and_wait(drive=3D'commit') =20 # Try to release our reference to top-filter again - result =3D self.vm.qmp('blockdev-del', node_name=3D'top-filter') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'top-filter') =20 self.assertIsNone(self.vm.node_info('top-filter')) self.assertIsNone(self.vm.node_info('cow-3')) @@ -893,12 +867,11 @@ class TestCommitWithFilters(iotests.QMPTestCase): self.pattern_files[3] =3D self.img2 =20 def test_filtered_active_commit_without_filter(self): - result =3D self.vm.qmp('block-commit', - job_id=3D'commit', - device=3D'top-filter', - top_node=3D'cow-3', - base_node=3D'cow-2') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', + job_id=3D'commit', + device=3D'top-filter', + top_node=3D'cow-3', + base_node=3D'cow-2') self.complete_and_wait(drive=3D'commit') =20 self.assertIsNotNone(self.vm.node_info('top-filter')) @@ -923,23 +896,22 @@ class TestCommitWithOverriddenBacking(iotests.QMPTest= Case): self.vm.launch() =20 # Use base_b instead of base_a as the backing of top - result =3D self.vm.qmp('blockdev-add', **{ - 'node-name': 'top', - 'driver': iotests.imgfmt, - 'file': { - 'driver': 'file', - 'filename': self.img_top - }, - 'backing': { - 'node-name': 'base', - 'driver': iotests.imgfmt, - 'file': { - 'driver': 'file', - 'filename': self.img_base_b - } - } - }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', { + 'node-name': 'top', + 'driver': iotests.imgfmt, + 'file': { + 'driver': 'file', + 'filename': self.img_top + }, + 'backing': { + 'node-name': 'base', + 'driver': iotests.imgfmt, + 'file': { + 'driver': 'file', + 'filename': self.img_base_b + } + } + }) =20 def tearDown(self): self.vm.shutdown() @@ -959,11 +931,10 @@ class TestCommitWithOverriddenBacking(iotests.QMPTest= Case): def test_commit_to_b(self): # Try committing to base_b (which should work, since that is # actually top's backing image) - result =3D self.vm.qmp('block-commit', - job_id=3D'commit', - device=3D'top', - base=3Dself.img_base_b) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', + job_id=3D'commit', + device=3D'top', + base=3Dself.img_base_b) =20 self.vm.event_wait('BLOCK_JOB_READY') self.vm.qmp('block-job-complete', device=3D'commit') diff --git a/tests/qemu-iotests/041 b/tests/qemu-iotests/041 index 8429958bf0..4d2a699613 100755 --- a/tests/qemu-iotests/041 +++ b/tests/qemu-iotests/041 @@ -65,9 +65,8 @@ class TestSingleDrive(iotests.QMPTestCase): def test_complete(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + target=3Dself.qmp_target) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -79,9 +78,8 @@ class TestSingleDrive(iotests.QMPTestCase): def test_cancel(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + target=3Dself.qmp_target) =20 self.cancel_and_wait(force=3DTrue) result =3D self.vm.qmp('query-block') @@ -90,9 +88,8 @@ class TestSingleDrive(iotests.QMPTestCase): def test_cancel_after_ready(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + target=3Dself.qmp_target) =20 self.wait_ready_and_cancel() result =3D self.vm.qmp('query-block') @@ -104,9 +101,8 @@ class TestSingleDrive(iotests.QMPTestCase): def test_pause(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + target=3Dself.qmp_target) =20 self.pause_job('drive0') =20 @@ -117,8 +113,7 @@ class TestSingleDrive(iotests.QMPTestCase): result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/offset', offset) =20 - result =3D self.vm.qmp('block-job-resume', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 self.complete_and_wait() self.vm.shutdown() @@ -129,9 +124,8 @@ class TestSingleDrive(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 # A small buffer is rounded up automatically - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - buf_size=3D4096, target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + buf_size=3D4096, target=3Dself.qmp_target) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -145,9 +139,8 @@ class TestSingleDrive(iotests.QMPTestCase): =20 qemu_img('create', '-f', iotests.imgfmt, '-o', 'cluster_size=3D%d,= size=3D%d' % (self.image_len, self.image_len), target_img) - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - buf_size=3D65536, mode=3D'existing', target= =3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + buf_size=3D65536, mode=3D'existing', target=3Dself.qmp= _target) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -162,9 +155,8 @@ class TestSingleDrive(iotests.QMPTestCase): qemu_img('create', '-f', iotests.imgfmt, '-o', 'cluster_size=3D%d,= backing_file=3D%s' % (self.image_len, backing_img), '-F', 'raw', target_img) - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - mode=3D'existing', target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + mode=3D'existing', target=3Dself.qmp_target) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -178,9 +170,8 @@ class TestSingleDrive(iotests.QMPTestCase): def test_implicit_node(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp(self.qmp_cmd, device=3D'drive0', sync=3D'fu= ll', - target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, device=3D'drive0', sync=3D'full', + target=3Dself.qmp_target) =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/file', test_img) @@ -236,8 +227,7 @@ class TestSingleBlockdev(TestSingleDrive): args =3D {'driver': iotests.imgfmt, 'node-name': self.qmp_target, 'file': { 'filename': target_img, 'driver': 'file' } } - result =3D self.vm.qmp("blockdev-add", **args) - self.assert_qmp(result, 'return', {}) + self.vm.cmd("blockdev-add", **args) =20 def test_mirror_to_self(self): result =3D self.vm.qmp(self.qmp_cmd, job_id=3D'job0', @@ -254,10 +244,9 @@ class TestSingleBlockdev(TestSingleDrive): result =3D self.vm.qmp('block_resize', node_name=3Dnode, size= =3D65536) self.assert_qmp(result, 'error/class', 'GenericError') =20 - result =3D self.vm.qmp(self.qmp_cmd, job_id=3D'job0', device=3D'dr= ive0', - sync=3D'full', target=3Dself.qmp_target, - auto_finalize=3DFalse, auto_dismiss=3DFalse) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, job_id=3D'job0', device=3D'drive0', + sync=3D'full', target=3Dself.qmp_target, + auto_finalize=3DFalse, auto_dismiss=3DFalse) =20 result =3D self.vm.run_job('job0', auto_finalize=3DFalse, pre_finalize=3Dpre_finalize) @@ -270,14 +259,11 @@ class TestSingleBlockdev(TestSingleDrive): self.do_test_resize(None, self.qmp_target) =20 def do_test_target_size(self, size): - result =3D self.vm.qmp('block_resize', node_name=3Dself.qmp_target, - size=3Dsize) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block_resize', node_name=3Dself.qmp_target, size=3Dsi= ze) =20 - result =3D self.vm.qmp(self.qmp_cmd, job_id=3D'job0', - device=3D'drive0', sync=3D'full', auto_dismis= s=3DFalse, - target=3Dself.qmp_target) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.qmp_cmd, job_id=3D'job0', + device=3D'drive0', sync=3D'full', auto_dismiss=3DFalse, + target=3Dself.qmp_target) =20 result =3D self.vm.run_job('job0') self.assertEqual(result, 'Source and target image have different s= izes') @@ -337,9 +323,8 @@ class TestMirrorNoBacking(iotests.QMPTestCase): =20 qemu_img('create', '-f', iotests.imgfmt, '-o', 'backing_file=3D%s' % backing_img, '-F', 'raw', tar= get_img) - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - mode=3D'existing', target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + mode=3D'existing', target=3Dtarget_img) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -353,9 +338,8 @@ class TestMirrorNoBacking(iotests.QMPTestCase): =20 qemu_img('create', '-f', iotests.imgfmt, '-o', 'backing_file=3D%s' % backing_img, '-F', 'raw', tar= get_img) - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - mode=3D'existing', target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + mode=3D'existing', target=3Dtarget_img) =20 self.wait_ready_and_cancel() result =3D self.vm.qmp('query-block') @@ -374,9 +358,8 @@ class TestMirrorNoBacking(iotests.QMPTestCase): % (TestMirrorNoBacking.image_len, target_backing_i= mg), '-F', iotests.imgfmt, target_img) =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - mode=3D'existing', target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + mode=3D'existing', target=3Dtarget_img) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -409,9 +392,8 @@ class TestMirrorResized(iotests.QMPTestCase): def test_complete_top(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= top', - target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'top', + target=3Dtarget_img) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -423,9 +405,8 @@ class TestMirrorResized(iotests.QMPTestCase): def test_complete_full(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img) =20 self.complete_and_wait() result =3D self.vm.qmp('query-block') @@ -488,9 +469,8 @@ new_state =3D "1" def test_report_read(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img) =20 completed =3D False error =3D False @@ -516,9 +496,8 @@ new_state =3D "1" def test_ignore_read(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img, on_source_error=3D'ignor= e') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img, on_source_error=3D'ignore') =20 event =3D self.vm.get_qmp_event(wait=3DTrue) while event['event'] =3D=3D 'JOB_STATUS_CHANGE': @@ -541,10 +520,9 @@ new_state =3D "1" qemu_img('create', '-f', iotests.imgfmt, '-ocluster_size=3D131072,backing_file=3D%s' %(backing_img= ), '-F', 'raw', target_img) - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= top', - on_source_error=3D'ignore', - mode=3D'existing', target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'top', + on_source_error=3D'ignore', + mode=3D'existing', target=3Dtarget_img) =20 event =3D self.vm.get_qmp_event(wait=3DTrue) while event['event'] =3D=3D 'JOB_STATUS_CHANGE': @@ -568,9 +546,8 @@ new_state =3D "1" def test_stop_read(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img, on_source_error=3D'stop') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img, on_source_error=3D'stop') =20 error =3D False ready =3D False @@ -590,8 +567,7 @@ new_state =3D "1" self.assert_qmp(result, 'return[0]/status', 'paused') self.assert_qmp(result, 'return[0]/io-status', 'failed= ') =20 - result =3D self.vm.qmp('block-job-resume', device=3D'd= rive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') error =3D True elif event['event'] =3D=3D 'BLOCK_JOB_READY': self.assertTrue(error, 'job completed unexpectedly') @@ -656,9 +632,8 @@ new_state =3D "1" def test_report_write(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - mode=3D'existing', target=3Dself.target_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + mode=3D'existing', target=3Dself.target_img) =20 completed =3D False error =3D False @@ -682,10 +657,9 @@ new_state =3D "1" def test_ignore_write(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - mode=3D'existing', target=3Dself.target_img, - on_target_error=3D'ignore') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + mode=3D'existing', target=3Dself.target_img, + on_target_error=3D'ignore') =20 event =3D self.vm.event_wait(name=3D'BLOCK_JOB_ERROR') self.assertEqual(event['event'], 'BLOCK_JOB_ERROR') @@ -698,10 +672,9 @@ new_state =3D "1" def test_stop_write(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - mode=3D'existing', target=3Dself.target_img, - on_target_error=3D'stop') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + mode=3D'existing', target=3Dself.target_img, + on_target_error=3D'stop') =20 error =3D False ready =3D False @@ -721,8 +694,7 @@ new_state =3D "1" self.assert_qmp(result, 'return[0]/status', 'paused') self.assert_qmp(result, 'return[0]/io-status', 'failed= ') =20 - result =3D self.vm.qmp('block-job-resume', device=3D'd= rive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 result =3D self.vm.qmp('query-block-jobs') self.assertIn(result['return'][0]['status'], ['running= ', 'ready']) @@ -755,17 +727,15 @@ class TestSetSpeed(iotests.QMPTestCase): def test_set_speed(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img) =20 # Default speed is 0 result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/device', 'drive0') self.assert_qmp(result, 'return[0]/speed', 0) =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D8 * 1024 * 1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D8 * = 1024 * 1024) =20 # Ensure the speed we set was accepted result =3D self.vm.qmp('query-block-jobs') @@ -775,9 +745,8 @@ class TestSetSpeed(iotests.QMPTestCase): self.wait_ready_and_cancel() =20 # Check setting speed in drive-mirror works - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img, speed=3D4*1024*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img, speed=3D4*1024*1024) =20 result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/device', 'drive0') @@ -794,9 +763,8 @@ class TestSetSpeed(iotests.QMPTestCase): =20 self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img) =20 result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D-1) self.assert_qmp(result, 'error/class', 'GenericError') @@ -811,13 +779,12 @@ class TestUnbackedSource(iotests.QMPTestCase): str(TestUnbackedSource.image_len)) self.vm =3D iotests.VM() self.vm.launch() - result =3D self.vm.qmp('blockdev-add', node_name=3D'drive0', - driver=3Diotests.imgfmt, - file=3D{ - 'driver': 'file', - 'filename': test_img, - }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', node_name=3D'drive0', + driver=3Diotests.imgfmt, + file=3D{ + 'driver': 'file', + 'filename': test_img, + }) =20 def tearDown(self): self.vm.shutdown() @@ -826,28 +793,25 @@ class TestUnbackedSource(iotests.QMPTestCase): =20 def test_absolute_paths_full(self): self.assert_no_active_block_jobs() - result =3D self.vm.qmp('drive-mirror', job_id=3D'drive0', device= =3D'drive0', - sync=3D'full', target=3Dtarget_img, - mode=3D'absolute-paths') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'drive0', device=3D'drive0', + sync=3D'full', target=3Dtarget_img, + mode=3D'absolute-paths') self.complete_and_wait() self.assert_no_active_block_jobs() =20 def test_absolute_paths_top(self): self.assert_no_active_block_jobs() - result =3D self.vm.qmp('drive-mirror', job_id=3D'drive0', device= =3D'drive0', - sync=3D'top', target=3Dtarget_img, - mode=3D'absolute-paths') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'drive0', device=3D'drive0', + sync=3D'top', target=3Dtarget_img, + mode=3D'absolute-paths') self.complete_and_wait() self.assert_no_active_block_jobs() =20 def test_absolute_paths_none(self): self.assert_no_active_block_jobs() - result =3D self.vm.qmp('drive-mirror', job_id=3D'drive0', device= =3D'drive0', - sync=3D'none', target=3Dtarget_img, - mode=3D'absolute-paths') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'drive0', device=3D'drive0', + sync=3D'none', target=3Dtarget_img, + mode=3D'absolute-paths') self.complete_and_wait() self.assert_no_active_block_jobs() =20 @@ -857,14 +821,12 @@ class TestUnbackedSource(iotests.QMPTestCase): qemu_io('-c', 'write -P 42 0 64k', target_img) =20 self.assert_no_active_block_jobs() - result =3D self.vm.qmp('drive-mirror', job_id=3D'drive0', device= =3D'drive0', - sync=3D'full', target=3Dtarget_img, mode=3D'e= xisting') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'drive0', device=3D'drive0', + sync=3D'full', target=3Dtarget_img, mode=3D'existing') self.complete_and_wait() self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('blockdev-del', node_name=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'drive0') =20 self.assertTrue(iotests.compare_images(test_img, target_img), 'target image does not match source after mirrorin= g') @@ -874,26 +836,22 @@ class TestUnbackedSource(iotests.QMPTestCase): str(self.image_len)) qemu_io('-c', 'write -P 42 0 64k', target_img) =20 - result =3D self.vm.qmp('blockdev-add', node_name=3D'target', - driver=3Diotests.imgfmt, - file=3D{ - 'driver': 'file', - 'filename': target_img, - }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', node_name=3D'target', + driver=3Diotests.imgfmt, + file=3D{ + 'driver': 'file', + 'filename': target_img, + }) =20 self.assert_no_active_block_jobs() - result =3D self.vm.qmp('blockdev-mirror', job_id=3D'drive0', devic= e=3D'drive0', - sync=3D'full', target=3D'target') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-mirror', job_id=3D'drive0', device=3D'drive0= ', + sync=3D'full', target=3D'target') self.complete_and_wait() self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('blockdev-del', node_name=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'drive0') =20 - result =3D self.vm.qmp('blockdev-del', node_name=3D'target') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'target') =20 self.assertTrue(iotests.compare_images(test_img, target_img), 'target image does not match source after mirrorin= g') @@ -918,10 +876,9 @@ class TestGranularity(iotests.QMPTestCase): =20 def test_granularity(self): self.assert_no_active_block_jobs() - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', - sync=3D'full', target=3Dtarget_img, - mode=3D'absolute-paths', granularity=3D8192) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', + sync=3D'full', target=3Dtarget_img, + mode=3D'absolute-paths', granularity=3D8192) =20 event =3D self.vm.get_qmp_event(wait=3D60.0) while event['event'] =3D=3D 'JOB_STATUS_CHANGE': @@ -963,8 +920,7 @@ class TestRepairQuorum(iotests.QMPTestCase): #assemble the quorum block device from the individual files args =3D { "driver": "quorum", "node-name": "quorum0", "vote-threshold": 2, "children": [ "img0", "img1", "img2"= ] } - result =3D self.vm.qmp("blockdev-add", **args) - self.assert_qmp(result, 'return', {}) + self.vm.cmd("blockdev-add", **args) =20 =20 def tearDown(self): @@ -978,10 +934,9 @@ class TestRepairQuorum(iotests.QMPTestCase): pass =20 def test_complete(self): - result =3D self.vm.qmp('drive-mirror', job_id=3D'job0', device=3D'= quorum0', - sync=3D'full', node_name=3D"repair0", replace= s=3D"img1", - target=3Dquorum_repair_img, format=3Diotests.= imgfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'job0', device=3D'quorum0', + sync=3D'full', node_name=3D"repair0", replaces=3D"img1= ", + target=3Dquorum_repair_img, format=3Diotests.imgfmt) =20 self.complete_and_wait(drive=3D"job0") self.assert_has_block_node("repair0", quorum_repair_img) @@ -991,10 +946,9 @@ class TestRepairQuorum(iotests.QMPTestCase): 'target image does not match source after mirrorin= g') =20 def test_cancel(self): - result =3D self.vm.qmp('drive-mirror', job_id=3D'job0', device=3D'= quorum0', - sync=3D'full', node_name=3D"repair0", replace= s=3D"img1", - target=3Dquorum_repair_img, format=3Diotests.= imgfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'job0', device=3D'quorum0', + sync=3D'full', node_name=3D"repair0", replaces=3D"img1= ", + target=3Dquorum_repair_img, format=3Diotests.imgfmt) =20 self.cancel_and_wait(drive=3D"job0", force=3DTrue) # here we check that the last registered quorum file has not been @@ -1002,10 +956,9 @@ class TestRepairQuorum(iotests.QMPTestCase): self.assert_has_block_node(None, quorum_img3) =20 def test_cancel_after_ready(self): - result =3D self.vm.qmp('drive-mirror', job_id=3D'job0', device=3D'= quorum0', - sync=3D'full', node_name=3D"repair0", replace= s=3D"img1", - target=3Dquorum_repair_img, format=3Diotests.= imgfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'job0', device=3D'quorum0', + sync=3D'full', node_name=3D"repair0", replaces=3D"img1= ", + target=3Dquorum_repair_img, format=3Diotests.imgfmt) =20 self.wait_ready_and_cancel(drive=3D"job0") # here we check that the last registered quorum file has not been @@ -1016,10 +969,9 @@ class TestRepairQuorum(iotests.QMPTestCase): 'target image does not match source after mirrorin= g') =20 def test_pause(self): - result =3D self.vm.qmp('drive-mirror', job_id=3D'job0', device=3D'= quorum0', - sync=3D'full', node_name=3D"repair0", replace= s=3D"img1", - target=3Dquorum_repair_img, format=3Diotests.= imgfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'job0', device=3D'quorum0', + sync=3D'full', node_name=3D"repair0", replaces=3D"img1= ", + target=3Dquorum_repair_img, format=3Diotests.imgfmt) =20 self.pause_job('job0') =20 @@ -1030,8 +982,7 @@ class TestRepairQuorum(iotests.QMPTestCase): result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/offset', offset) =20 - result =3D self.vm.qmp('block-job-resume', device=3D'job0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'job0') =20 self.complete_and_wait(drive=3D"job0") self.vm.shutdown() @@ -1093,10 +1044,9 @@ class TestRepairQuorum(iotests.QMPTestCase): target=3Dquorum_repair_img, format=3Diotests.= imgfmt) self.assert_qmp(result, 'error/class', 'GenericError') =20 - result =3D self.vm.qmp('drive-mirror', job_id=3D'job0', device=3D'= quorum0', - sync=3D'full', node_name=3D'repair0', replace= s=3D"snap1", - target=3Dquorum_repair_img, format=3Diotests.= imgfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'job0', device=3D'quorum0', + sync=3D'full', node_name=3D'repair0', replaces=3D"snap= 1", + target=3Dquorum_repair_img, format=3Diotests.imgfmt) =20 self.complete_and_wait('job0') self.assert_has_block_node("repair0", quorum_repair_img) @@ -1107,15 +1057,13 @@ class TestRepairQuorum(iotests.QMPTestCase): Check that we cannot replace a Quorum child when it has other parents. """ - result =3D self.vm.qmp('nbd-server-start', - addr=3D{ - 'type': 'unix', - 'data': {'path': nbd_sock_path} - }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('nbd-server-start', + addr=3D{ + 'type': 'unix', + 'data': {'path': nbd_sock_path} + }) =20 - result =3D self.vm.qmp('nbd-server-add', device=3D'img1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('nbd-server-add', device=3D'img1') =20 result =3D self.vm.qmp('drive-mirror', job_id=3D'mirror', device= =3D'quorum0', sync=3D'full', node_name=3D'repair0', replace= s=3D'img1', @@ -1130,20 +1078,17 @@ class TestRepairQuorum(iotests.QMPTestCase): The same as test_with_other_parent(), but add the NBD server only when the mirror job is already running. """ - result =3D self.vm.qmp('nbd-server-start', - addr=3D{ - 'type': 'unix', - 'data': {'path': nbd_sock_path} - }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('nbd-server-start', + addr=3D{ + 'type': 'unix', + 'data': {'path': nbd_sock_path} + }) =20 - result =3D self.vm.qmp('drive-mirror', job_id=3D'mirror', device= =3D'quorum0', - sync=3D'full', node_name=3D'repair0', replace= s=3D'img1', - target=3Dquorum_repair_img, format=3Diotests.= imgfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', job_id=3D'mirror', device=3D'quorum0', + sync=3D'full', node_name=3D'repair0', replaces=3D'img1= ', + target=3Dquorum_repair_img, format=3Diotests.imgfmt) =20 - result =3D self.vm.qmp('nbd-server-add', device=3D'img1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('nbd-server-add', device=3D'img1') =20 # The full error message goes to stderr, we will check it later self.complete_and_wait('mirror', @@ -1199,9 +1144,8 @@ class TestOrphanedSource(iotests.QMPTestCase): def test_success(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('blockdev-mirror', job_id=3D'job', device= =3D'src', - sync=3D'full', target=3D'dest') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-mirror', job_id=3D'job', device=3D'src', + sync=3D'full', target=3D'dest') =20 self.complete_and_wait('job') =20 @@ -1217,27 +1161,24 @@ class TestOrphanedSource(iotests.QMPTestCase): =20 # Unshare consistent-read on the target # (The mirror job does not care) - result =3D self.vm.qmp('blockdev-add', - driver=3D'blkdebug', - node_name=3D'dest-perm', - image=3D'dest', - unshare_child_perms=3D['consistent-read']) - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('blockdev-mirror', job_id=3D'job', device= =3D'src', - sync=3D'full', target=3D'dest', - filter_node_name=3D'mirror-filter') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + driver=3D'blkdebug', + node_name=3D'dest-perm', + image=3D'dest', + unshare_child_perms=3D['consistent-read']) + + self.vm.cmd('blockdev-mirror', job_id=3D'job', device=3D'src', + sync=3D'full', target=3D'dest', + filter_node_name=3D'mirror-filter') =20 # Require consistent-read on the source # (We can only add this node once the job has started, or it # will complain that it does not want to run on non-root nodes) - result =3D self.vm.qmp('blockdev-add', - driver=3D'blkdebug', - node_name=3D'src-perm', - image=3D'src', - take_child_perms=3D['consistent-read']) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + driver=3D'blkdebug', + node_name=3D'src-perm', + image=3D'src', + take_child_perms=3D['consistent-read']) =20 # While completing, mirror will attempt to replace src by # dest, which must fail because src-perm requires @@ -1277,26 +1218,22 @@ class TestReplaces(iotests.QMPTestCase): """ Check that we can replace filter nodes. """ - result =3D self.vm.qmp('blockdev-add', **{ - 'driver': 'copy-on-read', - 'node-name': 'filter0', - 'file': { - 'driver': 'copy-on-read', - 'node-name': 'filter1', - 'file': { - 'driver': 'null-co' - } - } - }) - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('blockdev-add', - node_name=3D'target', driver=3D'null-co') - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('blockdev-mirror', job_id=3D'mirror', devic= e=3D'filter0', - target=3D'target', sync=3D'full', replaces=3D= 'filter1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', { + 'driver': 'copy-on-read', + 'node-name': 'filter0', + 'file': { + 'driver': 'copy-on-read', + 'node-name': 'filter1', + 'file': { + 'driver': 'null-co' + } + } + }) + + self.vm.cmd('blockdev-add', node_name=3D'target', driver=3D'null-c= o') + + self.vm.cmd('blockdev-mirror', job_id=3D'mirror', device=3D'filter= 0', + target=3D'target', sync=3D'full', replaces=3D'filter1') =20 self.complete_and_wait('mirror') =20 @@ -1318,16 +1255,15 @@ class TestFilters(iotests.QMPTestCase): self.vm =3D iotests.VM().add_device('virtio-scsi,id=3Dvio-scsi') self.vm.launch() =20 - result =3D self.vm.qmp('blockdev-add', **{ - 'node-name': 'target', - 'driver': iotests.imgfmt, - 'file': { - 'driver': 'file', - 'filename': target_img - }, - 'backing': None - }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', { + 'node-name': 'target', + 'driver': iotests.imgfmt, + 'file': { + 'driver': 'file', + 'filename': target_img + }, + 'backing': None + }) =20 self.filterless_chain =3D { 'node-name': 'source', @@ -1354,19 +1290,17 @@ class TestFilters(iotests.QMPTestCase): os.remove(backing_img) =20 def test_cor(self): - result =3D self.vm.qmp('blockdev-add', **{ - 'node-name': 'filter', - 'driver': 'copy-on-read', - 'file': self.filterless_chain - }) - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('blockdev-mirror', - job_id=3D'mirror', - device=3D'filter', - target=3D'target', - sync=3D'top') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', { + 'node-name': 'filter', + 'driver': 'copy-on-read', + 'file': self.filterless_chain + }) + + self.vm.cmd('blockdev-mirror', + job_id=3D'mirror', + device=3D'filter', + target=3D'target', + sync=3D'top') =20 self.complete_and_wait('mirror') =20 @@ -1383,23 +1317,20 @@ class TestFilters(iotests.QMPTestCase): assert target_map[1]['depth'] =3D=3D 0 =20 def test_implicit_mirror_filter(self): - result =3D self.vm.qmp('blockdev-add', **self.filterless_chain) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', **self.filterless_chain) =20 # We need this so we can query from above the mirror node - result =3D self.vm.qmp('device_add', - driver=3D'scsi-hd', - id=3D'virtio', - bus=3D'vio-scsi.0', - drive=3D'source') - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('blockdev-mirror', - job_id=3D'mirror', - device=3D'source', - target=3D'target', - sync=3D'top') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', + driver=3D'scsi-hd', + id=3D'virtio', + bus=3D'vio-scsi.0', + drive=3D'source') + + self.vm.cmd('blockdev-mirror', + job_id=3D'mirror', + device=3D'source', + target=3D'target', + sync=3D'top') =20 # The mirror filter is now an implicit node, so it should be # invisible when querying the backing chain @@ -1417,24 +1348,21 @@ class TestFilters(iotests.QMPTestCase): def test_explicit_mirror_filter(self): # Same test as above, but this time we give the mirror filter # a node-name so it will not be invisible - result =3D self.vm.qmp('blockdev-add', **self.filterless_chain) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', **self.filterless_chain) =20 # We need this so we can query from above the mirror node - result =3D self.vm.qmp('device_add', - driver=3D'scsi-hd', - id=3D'virtio', - bus=3D'vio-scsi.0', - drive=3D'source') - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('blockdev-mirror', - job_id=3D'mirror', - device=3D'source', - target=3D'target', - sync=3D'top', - filter_node_name=3D'mirror-filter') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', + driver=3D'scsi-hd', + id=3D'virtio', + bus=3D'vio-scsi.0', + drive=3D'source') + + self.vm.cmd('blockdev-mirror', + job_id=3D'mirror', + device=3D'source', + target=3D'target', + sync=3D'top', + filter_node_name=3D'mirror-filter') =20 # With a node-name given to it, the mirror filter should now # be visible diff --git a/tests/qemu-iotests/045 b/tests/qemu-iotests/045 index 45eb239baa..a341f21cd7 100755 --- a/tests/qemu-iotests/045 +++ b/tests/qemu-iotests/045 @@ -77,8 +77,7 @@ class TestFdSets(iotests.QMPTestCase): self.vm.shutdown() =20 def test_remove_fdset(self): - result =3D self.vm.qmp('remove-fd', fdset_id=3D2) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('remove-fd', fdset_id=3D2) result =3D self.vm.qmp('query-fdsets') self.assert_qmp(result, 'return[0]/fdset-id', 1) self.assert_qmp(result, 'return[1]/fdset-id', 0) @@ -90,8 +89,7 @@ class TestFdSets(iotests.QMPTestCase): def test_remove_fd(self): result =3D self.vm.qmp('query-fdsets') fd_image3 =3D result['return'][0]['fds'][0]['fd'] - result =3D self.vm.qmp('remove-fd', fdset_id=3D2, fd=3Dfd_image3) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('remove-fd', fdset_id=3D2, fd=3Dfd_image3) result =3D self.vm.qmp('query-fdsets') self.assert_qmp(result, 'return[0]/fdset-id', 2) self.assert_qmp(result, 'return[1]/fdset-id', 1) @@ -151,8 +149,7 @@ class TestSCMFd(iotests.QMPTestCase): =20 def test_getfd(self): self._send_fd_by_SCM() - result =3D self.vm.qmp('getfd', fdname=3D'image0:r') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('getfd', fdname=3D'image0:r') =20 def test_getfd_invalid_fdname(self): self._send_fd_by_SCM() @@ -163,10 +160,8 @@ class TestSCMFd(iotests.QMPTestCase): =20 def test_closefd(self): self._send_fd_by_SCM() - result =3D self.vm.qmp('getfd', fdname=3D'image0:r') - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('closefd', fdname=3D'image0:r') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('getfd', fdname=3D'image0:r') + self.vm.cmd('closefd', fdname=3D'image0:r') =20 def test_closefd_fd_not_found(self): fdname =3D 'image0:r' diff --git a/tests/qemu-iotests/055 b/tests/qemu-iotests/055 index 5d6b607051..378f2737fd 100755 --- a/tests/qemu-iotests/055 +++ b/tests/qemu-iotests/055 @@ -69,8 +69,7 @@ class TestSingleDrive(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp(cmd, device=3D'drive0', target=3Dtarget, sy= nc=3D'full') - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', target=3Dtarget, sync=3D'full') =20 event =3D self.cancel_and_wait(resume=3DTrue) self.assert_qmp(event, 'data/type', 'backup') @@ -85,9 +84,7 @@ class TestSingleDrive(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp(cmd, device=3D'drive0', - target=3Dtarget, sync=3D'full') - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', target=3Dtarget, sync=3D'full') =20 self.pause_job('drive0', wait=3DFalse) self.vm.resume_drive('drive0') @@ -100,8 +97,7 @@ class TestSingleDrive(iotests.QMPTestCase): result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/offset', offset) =20 - result =3D self.vm.qmp('block-job-resume', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 self.wait_until_completed() =20 @@ -123,10 +119,9 @@ class TestSingleDrive(iotests.QMPTestCase): result =3D self.vm.qmp('block_resize', node_name=3Dnode, size= =3D65536) self.assert_qmp(result, 'error/class', 'GenericError') =20 - result =3D self.vm.qmp('blockdev-backup', job_id=3D'job0', device= =3D'drive0', - target=3D'drive1', sync=3D'full', auto_finali= ze=3DFalse, - auto_dismiss=3DFalse) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-backup', job_id=3D'job0', device=3D'drive0', + target=3D'drive1', sync=3D'full', auto_finalize=3DFals= e, + auto_dismiss=3DFalse) =20 self.vm.run_job('job0', auto_finalize=3DFalse, pre_finalize=3Dpre_= finalize) =20 @@ -137,8 +132,7 @@ class TestSingleDrive(iotests.QMPTestCase): self.do_test_resize_blockdev_backup('drive1', 'target') =20 def do_test_target_size(self, size): - result =3D self.vm.qmp('block_resize', device=3D'drive1', size=3Ds= ize) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block_resize', device=3D'drive1', size=3Dsize) =20 result =3D self.vm.qmp('blockdev-backup', job_id=3D'job0', device= =3D'drive0', target=3D'drive1', sync=3D'full') @@ -219,16 +213,14 @@ class TestSetSpeed(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp(cmd, device=3D'drive0', target=3Dtarget, sy= nc=3D'full') - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', target=3Dtarget, sync=3D'full') =20 # Default speed is 0 result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/device', 'drive0') self.assert_qmp(result, 'return[0]/speed', 0) =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D8 * 1024 * 1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D8 * = 1024 * 1024) =20 # Ensure the speed we set was accepted result =3D self.vm.qmp('query-block-jobs') @@ -240,9 +232,8 @@ class TestSetSpeed(iotests.QMPTestCase): =20 # Check setting speed option works self.vm.pause_drive('drive0') - result =3D self.vm.qmp(cmd, device=3D'drive0', - target=3Dtarget, sync=3D'full', speed=3D4*102= 4*1024) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', + target=3Dtarget, sync=3D'full', speed=3D4*1024*1024) =20 result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/device', 'drive0') @@ -267,9 +258,8 @@ class TestSetSpeed(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp(cmd, device=3D'drive0', - target=3Dtarget, sync=3D'full') - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', + target=3Dtarget, sync=3D'full') =20 result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D-1) self.assert_qmp(result, 'error/class', 'GenericError') @@ -306,7 +296,7 @@ class TestSingleTransaction(iotests.QMPTestCase): def do_test_cancel(self, cmd, target): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('transaction', actions=3D[{ + self.vm.cmd('transaction', actions=3D[{ 'type': cmd, 'data': { 'device': 'drive0', 'target': target, @@ -315,8 +305,6 @@ class TestSingleTransaction(iotests.QMPTestCase): } ]) =20 - self.assert_qmp(result, 'return', {}) - event =3D self.cancel_and_wait() self.assert_qmp(event, 'data/type', 'backup') =20 @@ -329,7 +317,7 @@ class TestSingleTransaction(iotests.QMPTestCase): def do_test_pause(self, cmd, target, image): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('transaction', actions=3D[{ + self.vm.cmd('transaction', actions=3D[{ 'type': cmd, 'data': { 'device': 'drive0', 'target': target, @@ -337,12 +325,10 @@ class TestSingleTransaction(iotests.QMPTestCase): 'speed': 64 * 1024 }, } ]) - self.assert_qmp(result, 'return', {}) =20 self.pause_job('drive0', wait=3DFalse) =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D0) =20 self.pause_wait('drive0') =20 @@ -353,8 +339,7 @@ class TestSingleTransaction(iotests.QMPTestCase): result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/offset', offset) =20 - result =3D self.vm.qmp('block-job-resume', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 self.wait_until_completed() =20 @@ -519,8 +504,7 @@ class TestCompressedToQcow2(iotests.QMPTestCase): =20 self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp(cmd, device=3D'drive0', sync=3D'full', comp= ress=3DTrue, **args) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', sync=3D'full', compress=3DTrue= , **args) =20 self.wait_until_completed() =20 @@ -545,8 +529,7 @@ class TestCompressedToQcow2(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp(cmd, device=3D'drive0', sync=3D'full', comp= ress=3DTrue, **args) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', sync=3D'full', compress=3DTrue= , **args) =20 event =3D self.cancel_and_wait(resume=3DTrue) self.assert_qmp(event, 'data/type', 'backup') @@ -568,8 +551,7 @@ class TestCompressedToQcow2(iotests.QMPTestCase): self.assert_no_active_block_jobs() =20 self.vm.pause_drive('drive0') - result =3D self.vm.qmp(cmd, device=3D'drive0', sync=3D'full', comp= ress=3DTrue, **args) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, device=3D'drive0', sync=3D'full', compress=3DTrue= , **args) =20 self.pause_job('drive0', wait=3DFalse) self.vm.resume_drive('drive0') @@ -582,8 +564,7 @@ class TestCompressedToQcow2(iotests.QMPTestCase): result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/offset', offset) =20 - result =3D self.vm.qmp('block-job-resume', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D'drive0') =20 self.wait_until_completed() =20 diff --git a/tests/qemu-iotests/056 b/tests/qemu-iotests/056 index b459a3f1e8..34f08b872d 100755 --- a/tests/qemu-iotests/056 +++ b/tests/qemu-iotests/056 @@ -75,9 +75,8 @@ class TestSyncModesNoneAndTop(iotests.QMPTestCase): =20 def test_complete_top(self): self.assert_no_active_block_jobs() - result =3D self.vm.qmp('drive-backup', device=3D'drive0', sync=3D'= top', - format=3Diotests.imgfmt, target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-backup', device=3D'drive0', sync=3D'top', + format=3Diotests.imgfmt, target=3Dtarget_img) =20 self.wait_until_completed(check_offset=3DFalse) =20 @@ -89,9 +88,8 @@ class TestSyncModesNoneAndTop(iotests.QMPTestCase): def test_cancel_sync_none(self): self.assert_no_active_block_jobs() =20 - result =3D self.vm.qmp('drive-backup', device=3D'drive0', - sync=3D'none', target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-backup', device=3D'drive0', + sync=3D'none', target=3Dtarget_img) time.sleep(1) self.vm.hmp_qemu_io('drive0', 'write -P0x5e 0 512') self.vm.hmp_qemu_io('drive0', 'aio_flush') @@ -115,18 +113,15 @@ class TestBeforeWriteNotifier(iotests.QMPTestCase): =20 def test_before_write_notifier(self): self.vm.pause_drive("drive0") - result =3D self.vm.qmp('drive-backup', device=3D'drive0', - sync=3D'full', target=3Dtarget_img, - format=3D"file", speed=3D1) - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('block-job-pause', device=3D"drive0") - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-backup', device=3D'drive0', + sync=3D'full', target=3Dtarget_img, + format=3D"file", speed=3D1) + self.vm.cmd('block-job-pause', device=3D"drive0") # Speed is low enough that this must be an uncopied range, which w= ill # trigger the before write notifier self.vm.hmp_qemu_io('drive0', 'aio_write -P 1 512512 512') self.vm.resume_drive("drive0") - result =3D self.vm.qmp('block-job-resume', device=3D"drive0") - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-resume', device=3D"drive0") event =3D self.cancel_and_wait() self.assert_qmp(event, 'data/type', 'backup') =20 diff --git a/tests/qemu-iotests/093 b/tests/qemu-iotests/093 index 93274dc8cb..bf7763c8f6 100755 --- a/tests/qemu-iotests/093 +++ b/tests/qemu-iotests/093 @@ -55,8 +55,7 @@ class ThrottleTestCase(iotests.QMPTestCase): # Set the I/O throttling parameters to all drives for i in range(0, ndrives): params['device'] =3D 'drive%d' % i - result =3D self.vm.qmp("block_set_io_throttle", conv_keys=3DFa= lse, **params) - self.assert_qmp(result, 'return', {}) + self.vm.cmd("block_set_io_throttle", conv_keys=3DFalse, **para= ms) =20 def do_test_throttle(self, ndrives, seconds, params, first_drive =3D 0= ): def check_limit(limit, num): @@ -253,8 +252,7 @@ class ThrottleTestCase(iotests.QMPTestCase): # drive1 remains in the group with a throttled request. params['bps_rd'] =3D 0 params['device'] =3D 'drive0' - result =3D self.vm.qmp("block_set_io_throttle", conv_keys=3DFalse,= **params) - self.assert_qmp(result, 'return', {}) + self.vm.cmd("block_set_io_throttle", conv_keys=3DFalse, **params) =20 # Removing the I/O limits from drive0 drains its two pending reque= sts. # The read request in drive1 is still throttled. @@ -286,8 +284,7 @@ class ThrottleTestGroupNames(iotests.QMPTestCase): =20 def set_io_throttle(self, device, params): params["device"] =3D device - result =3D self.vm.qmp("block_set_io_throttle", conv_keys=3DFalse,= **params) - self.assert_qmp(result, 'return', {}) + self.vm.cmd("block_set_io_throttle", conv_keys=3DFalse, **params) =20 def verify_name(self, device, name): result =3D self.vm.qmp("query-block") @@ -379,23 +376,18 @@ class ThrottleTestRemovableMedia(iotests.QMPTestCase): =20 def test_removable_media(self): # Add a couple of dummy nodes named cd0 and cd1 - result =3D self.vm.qmp("blockdev-add", driver=3D"null-co", - read_zeroes=3DTrue, node_name=3D"cd0") - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp("blockdev-add", driver=3D"null-co", - read_zeroes=3DTrue, node_name=3D"cd1") - self.assert_qmp(result, 'return', {}) + self.vm.cmd("blockdev-add", driver=3D"null-co", + read_zeroes=3DTrue, node_name=3D"cd0") + self.vm.cmd("blockdev-add", driver=3D"null-co", + read_zeroes=3DTrue, node_name=3D"cd1") =20 # Attach a CD drive with cd0 inserted - result =3D self.vm.qmp("device_add", driver=3D"scsi-cd", - id=3D"dev0", drive=3D"cd0") - self.assert_qmp(result, 'return', {}) + self.vm.cmd("device_add", driver=3D"scsi-cd", id=3D"dev0", drive= =3D"cd0") =20 # Set I/O limits args =3D { "id": "dev0", "iops": 100, "iops_rd": 0, "iops_wr": 0, "bps": 50, "bps_rd": 0, "bps_wr": 0 } - result =3D self.vm.qmp("block_set_io_throttle", conv_keys=3DFalse,= **args) - self.assert_qmp(result, 'return', {}) + self.vm.cmd("block_set_io_throttle", conv_keys=3DFalse, **args) =20 # Check that the I/O limits have been set result =3D self.vm.qmp("query-block") @@ -403,12 +395,9 @@ class ThrottleTestRemovableMedia(iotests.QMPTestCase): self.assert_qmp(result, 'return[0]/inserted/bps', 50) =20 # Now eject cd0 and insert cd1 - result =3D self.vm.qmp("blockdev-open-tray", id=3D'dev0') - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp("blockdev-remove-medium", id=3D'dev0') - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp("blockdev-insert-medium", id=3D'dev0', node= _name=3D'cd1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd("blockdev-open-tray", id=3D'dev0') + self.vm.cmd("blockdev-remove-medium", id=3D'dev0') + self.vm.cmd("blockdev-insert-medium", id=3D'dev0', node_name=3D'cd= 1') =20 # Check that the I/O limits are still the same result =3D self.vm.qmp("query-block") @@ -416,16 +405,14 @@ class ThrottleTestRemovableMedia(iotests.QMPTestCase): self.assert_qmp(result, 'return[0]/inserted/bps', 50) =20 # Eject cd1 - result =3D self.vm.qmp("blockdev-remove-medium", id=3D'dev0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd("blockdev-remove-medium", id=3D'dev0') =20 # Check that we can't set limits if the device has no medium result =3D self.vm.qmp("block_set_io_throttle", conv_keys=3DFalse,= **args) self.assert_qmp(result, 'error/class', 'GenericError') =20 # Remove the CD drive - result =3D self.vm.qmp("device_del", id=3D'dev0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd("device_del", id=3D'dev0') =20 =20 if __name__ =3D=3D '__main__': diff --git a/tests/qemu-iotests/118 b/tests/qemu-iotests/118 index cae52ffa5e..3a232fcb27 100755 --- a/tests/qemu-iotests/118 +++ b/tests/qemu-iotests/118 @@ -74,11 +74,9 @@ class ChangeBaseClass(iotests.QMPTestCase): class GeneralChangeTestsBaseClass(ChangeBaseClass): =20 def test_blockdev_change_medium(self): - result =3D self.vm.qmp('blockdev-change-medium', - id=3Dself.device_name, filename=3Dnew_img, - format=3Diotests.imgfmt) - - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', + id=3Dself.device_name, filename=3Dnew_img, + format=3Diotests.imgfmt) =20 self.wait_for_open() self.wait_for_close() @@ -89,8 +87,7 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/image/filename', new_i= mg) =20 def test_eject(self): - result =3D self.vm.qmp('eject', id=3Dself.device_name, force=3DTru= e) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('eject', id=3Dself.device_name, force=3DTrue) =20 self.wait_for_open() =20 @@ -100,8 +97,7 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp_absent(result, 'return[0]/inserted') =20 def test_tray_eject_change(self): - result =3D self.vm.qmp('eject', id=3Dself.device_name, force=3DTru= e) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('eject', id=3Dself.device_name, force=3DTrue) =20 self.wait_for_open() =20 @@ -110,9 +106,8 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp(result, 'return[0]/tray_open', True) self.assert_qmp_absent(result, 'return[0]/inserted') =20 - result =3D self.vm.qmp('blockdev-change-medium', id=3Dself.device_= name, - filename=3Dnew_img, format=3Diotests.imgfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', id=3Dself.device_name, + filename=3Dnew_img, format=3Diotests.imgfmt) =20 self.wait_for_close() =20 @@ -122,9 +117,8 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/image/filename', new_i= mg) =20 def test_tray_open_close(self): - result =3D self.vm.qmp('blockdev-open-tray', - id=3Dself.device_name, force=3DTrue) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-open-tray', + id=3Dself.device_name, force=3DTrue) =20 self.wait_for_open() =20 @@ -136,8 +130,7 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): else: self.assert_qmp(result, 'return[0]/inserted/image/filename', o= ld_img) =20 - result =3D self.vm.qmp('blockdev-close-tray', id=3Dself.device_nam= e) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-close-tray', id=3Dself.device_name) =20 if self.has_real_tray or not self.was_empty: self.wait_for_close() @@ -151,8 +144,7 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/image/filename', o= ld_img) =20 def test_tray_eject_close(self): - result =3D self.vm.qmp('eject', id=3Dself.device_name, force=3DTru= e) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('eject', id=3Dself.device_name, force=3DTrue) =20 self.wait_for_open() =20 @@ -161,8 +153,7 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp(result, 'return[0]/tray_open', True) self.assert_qmp_absent(result, 'return[0]/inserted') =20 - result =3D self.vm.qmp('blockdev-close-tray', id=3Dself.device_nam= e) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-close-tray', id=3Dself.device_name) =20 self.wait_for_close() =20 @@ -172,9 +163,7 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp_absent(result, 'return[0]/inserted') =20 def test_tray_open_change(self): - result =3D self.vm.qmp('blockdev-open-tray', id=3Dself.device_name, - force=3DTrue) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-open-tray', id=3Dself.device_name, force=3DT= rue) =20 self.wait_for_open() =20 @@ -186,10 +175,8 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): else: self.assert_qmp(result, 'return[0]/inserted/image/filename', o= ld_img) =20 - result =3D self.vm.qmp('blockdev-change-medium', id=3Dself.device_= name, - filename=3Dnew_img, - format=3Diotests.im= gfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', id=3Dself.device_name, + filename=3Dnew_img, format=3Diotests.imgfmt) =20 self.wait_for_close() =20 @@ -199,17 +186,14 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/image/filename', new_i= mg) =20 def test_cycle(self, read_only_node=3DFalse): - result =3D self.vm.qmp('blockdev-add', - node_name=3D'new', - driver=3Diotests.imgfmt, - read_only=3Dread_only_node, - file=3D{'filename': new_img, - 'driver': 'file'}) - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('blockdev-open-tray', - id=3Dself.device_name, force=3DTrue) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + node_name=3D'new', + driver=3Diotests.imgfmt, + read_only=3Dread_only_node, + file=3D{'filename': new_img, + 'driver': 'file'}) + + self.vm.cmd('blockdev-open-tray', id=3Dself.device_name, force=3DT= rue) =20 self.wait_for_open() =20 @@ -221,26 +205,22 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): else: self.assert_qmp(result, 'return[0]/inserted/image/filename', o= ld_img) =20 - result =3D self.vm.qmp('blockdev-remove-medium', - id=3Dself.device_name) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-remove-medium', id=3Dself.device_name) =20 result =3D self.vm.qmp('query-block') if self.has_real_tray: self.assert_qmp(result, 'return[0]/tray_open', True) self.assert_qmp_absent(result, 'return[0]/inserted') =20 - result =3D self.vm.qmp('blockdev-insert-medium', - id=3Dself.device_name, node_name=3D'new') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-insert-medium', + id=3Dself.device_name, node_name=3D'new') =20 result =3D self.vm.qmp('query-block') if self.has_real_tray: self.assert_qmp(result, 'return[0]/tray_open', True) self.assert_qmp(result, 'return[0]/inserted/image/filename', new_i= mg) =20 - result =3D self.vm.qmp('blockdev-close-tray', id=3Dself.device_nam= e) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-close-tray', id=3Dself.device_name) =20 self.wait_for_close() =20 @@ -253,9 +233,8 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): self.test_cycle(True) =20 def test_close_on_closed(self): - result =3D self.vm.qmp('blockdev-close-tray', id=3Dself.device_nam= e) # Should be a no-op - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-close-tray', id=3Dself.device_name) self.assertEqual(self.vm.get_qmp_events(wait=3DFalse), []) =20 def test_remove_on_closed(self): @@ -269,12 +248,11 @@ class GeneralChangeTestsBaseClass(ChangeBaseClass): if not self.has_real_tray: return =20 - result =3D self.vm.qmp('blockdev-add', - node_name=3D'new', - driver=3Diotests.imgfmt, - file=3D{'filename': new_img, - 'driver': 'file'}) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + node_name=3D'new', + driver=3Diotests.imgfmt, + file=3D{'filename': new_img, + 'driver': 'file'}) =20 result =3D self.vm.qmp('blockdev-insert-medium', id=3Dself.device_= name, node_name=3D'new') @@ -307,15 +285,13 @@ class TestInitiallyFilled(GeneralChangeTestsBaseClass= ): os.remove(new_img) =20 def test_insert_on_filled(self): - result =3D self.vm.qmp('blockdev-add', - node_name=3D'new', - driver=3Diotests.imgfmt, - file=3D{'filename': new_img, - 'driver': 'file'}) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + node_name=3D'new', + driver=3Diotests.imgfmt, + file=3D{'filename': new_img, + 'driver': 'file'}) =20 - result =3D self.vm.qmp('blockdev-open-tray', id=3Dself.device_name) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-open-tray', id=3Dself.device_name) =20 self.wait_for_open() =20 @@ -344,14 +320,12 @@ class TestInitiallyEmpty(GeneralChangeTestsBaseClass): os.remove(new_img) =20 def test_remove_on_empty(self): - result =3D self.vm.qmp('blockdev-open-tray', id=3Dself.device_name) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-open-tray', id=3Dself.device_name) =20 self.wait_for_open() =20 - result =3D self.vm.qmp('blockdev-remove-medium', id=3Dself.device_= name) # Should be a no-op - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-remove-medium', id=3Dself.device_name) =20 # Do this in a function to avoid leaking variables like case into the glob= al # name space (otherwise tests would be run for the abstract base classes) @@ -399,11 +373,10 @@ class TestChangeReadOnly(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/ro', True) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-change-medium', id=3Dself.device_= name, - filename=3Dnew_img, - format=3Diotests.im= gfmt, - read_only_mode=3D'r= etain') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', id=3Dself.device_name, + filename=3Dnew_img, + format=3Diotests.imgfmt, + read_only_mode=3D'retain') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', True) @@ -419,11 +392,10 @@ class TestChangeReadOnly(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/ro', True) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-change-medium', id=3Dself.device_= name, - filename=3Dnew_img, - format=3Diotests.im= gfmt, - read_only_mode=3D'r= etain') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', id=3Dself.device_name, + filename=3Dnew_img, + format=3Diotests.imgfmt, + read_only_mode=3D'retain') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', True) @@ -462,12 +434,11 @@ class TestChangeReadOnly(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/ro', True) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-change-medium', - id=3Dself.device_name, - filename=3Dnew_img, - format=3Diotests.imgfmt, - read_only_mode=3D'read-write') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', + id=3Dself.device_name, + filename=3Dnew_img, + format=3Diotests.imgfmt, + read_only_mode=3D'read-write') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', False) @@ -483,12 +454,11 @@ class TestChangeReadOnly(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/ro', False) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-change-medium', - id=3Dself.device_name, - filename=3Dnew_img, - format=3Diotests.imgfmt, - read_only_mode=3D'read-only') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', + id=3Dself.device_name, + filename=3Dnew_img, + format=3Diotests.imgfmt, + read_only_mode=3D'read-only') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', True) @@ -503,12 +473,11 @@ class TestChangeReadOnly(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/ro', False) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-change-medium', - id=3Dself.device_name, - filename=3Dnew_img, - format=3Diotests.imgfmt, - read_only_mode=3D'read-only') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', + id=3Dself.device_name, + filename=3Dnew_img, + format=3Diotests.imgfmt, + read_only_mode=3D'read-only') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', True) @@ -546,11 +515,10 @@ class TestChangeReadOnly(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/ro', True) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-change-medium', id=3Dself.device_= name, - filename=3Dnew_img, - format=3Diotests.im= gfmt, - read_only_mode=3D'r= etain') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-change-medium', id=3Dself.device_name, + filename=3Dnew_img, + format=3Diotests.imgfmt, + read_only_mode=3D'retain') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', True) @@ -587,27 +555,24 @@ class TestChangeReadOnly(ChangeBaseClass): self.assert_qmp(result, 'return[0]/inserted/ro', False) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-add', - node_name=3D'new', - driver=3Diotests.imgfmt, - read_only=3DTrue, - file=3D{'filename': new_img, - 'driver': 'file'}) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + node_name=3D'new', + driver=3Diotests.imgfmt, + read_only=3DTrue, + file=3D{'filename': new_img, + 'driver': 'file'}) =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', False) self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) =20 - result =3D self.vm.qmp('blockdev-remove-medium', id=3Dself.device_= name) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-remove-medium', id=3Dself.device_name) =20 result =3D self.vm.qmp('query-block') self.assert_qmp_absent(result, 'return[0]/inserted') =20 - result =3D self.vm.qmp('blockdev-insert-medium', id=3Dself.device_= name, - node_name=3D'new') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-insert-medium', id=3Dself.device_name, + node_name=3D'new') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/ro', True) @@ -638,22 +603,19 @@ class TestBlockJobsAfterCycle(ChangeBaseClass): =20 # For device-less BBs, calling blockdev-open-tray or blockdev-clos= e-tray # is not necessary - result =3D self.vm.qmp('blockdev-remove-medium', id=3Dself.device_= name) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-remove-medium', id=3Dself.device_name) =20 result =3D self.vm.qmp('query-block') self.assert_qmp_absent(result, 'return[0]/inserted') =20 - result =3D self.vm.qmp('blockdev-add', - node_name=3D'node0', - driver=3Diotests.imgfmt, - file=3D{'filename': old_img, - 'driver': 'file'}) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + node_name=3D'node0', + driver=3Diotests.imgfmt, + file=3D{'filename': old_img, + 'driver': 'file'}) =20 - result =3D self.vm.qmp('blockdev-insert-medium', id=3Dself.device_= name, - node_name=3D'node0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-insert-medium', id=3Dself.device_name, + node_name=3D'node0') =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/image/filename', old_i= mg) @@ -670,10 +632,9 @@ class TestBlockJobsAfterCycle(ChangeBaseClass): @iotests.skip_for_formats(('vpc', 'parallels', 'qcow', 'vdi', 'vmdk', = 'raw', 'vhdx')) def test_snapshot_and_commit(self): - result =3D self.vm.qmp('blockdev-snapshot-sync', device=3D'drive0', - snapshot_file=3Dnew= _img, - format=3Diotests.im= gfmt) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-snapshot-sync', device=3D'drive0', + snapshot_file=3Dnew_img, + format=3Diotests.imgfmt) =20 result =3D self.vm.qmp('query-block') self.assert_qmp(result, 'return[0]/inserted/image/filename', new_i= mg) @@ -681,16 +642,14 @@ class TestBlockJobsAfterCycle(ChangeBaseClass): 'return[0]/inserted/image/backing-image/filename', old_img) =20 - result =3D self.vm.qmp('block-commit', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'drive0') =20 self.vm.event_wait(name=3D'BLOCK_JOB_READY') =20 result =3D self.vm.qmp('query-block-jobs') self.assert_qmp(result, 'return[0]/device', 'drive0') =20 - result =3D self.vm.qmp('block-job-complete', device=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-complete', device=3D'drive0') =20 self.vm.event_wait(name=3D'BLOCK_JOB_COMPLETED') =20 diff --git a/tests/qemu-iotests/124 b/tests/qemu-iotests/124 index 845ab5303c..3f3febd1fc 100755 --- a/tests/qemu-iotests/124 +++ b/tests/qemu-iotests/124 @@ -201,9 +201,8 @@ class TestIncrementalBackupBase(iotests.QMPTestCase): def add_bitmap(self, name, drive, **kwargs): bitmap =3D Bitmap(name, drive) self.bitmaps.append(bitmap) - result =3D self.vm.qmp('block-dirty-bitmap-add', node=3Ddrive['id'= ], - name=3Dbitmap.name, **kwargs) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-dirty-bitmap-add', node=3Ddrive['id'], + name=3Dbitmap.name, **kwargs) return bitmap =20 =20 @@ -388,13 +387,12 @@ class TestIncrementalBackup(TestIncrementalBackupBase= ): ('0x64', '32736k', '64k'))) bitmap1 =3D self.add_bitmap('bitmap1', drive0) =20 - result =3D self.vm.qmp('transaction', actions=3D[ + self.vm.cmd('transaction', actions=3D[ transaction_bitmap_clear(bitmap0.drive['id'], bitmap0.name), transaction_bitmap_clear(bitmap1.drive['id'], bitmap1.name), transaction_drive_backup(drive0['id'], drive0['backup'], sync=3D'full', format=3Ddrive0['fmt']) ]) - self.assert_qmp(result, 'return', {}) self.wait_until_completed(drive0['id']) self.files.append(drive0['backup']) =20 @@ -417,7 +415,7 @@ class TestIncrementalBackup(TestIncrementalBackupBase): ('0xcd', '32M', '124k'))) =20 # Create a blkdebug interface to this img as 'drive1' - result =3D self.vm.qmp('blockdev-add', + self.vm.cmd('blockdev-add', node_name=3Ddrive1['id'], driver=3Ddrive1['fmt'], file=3D{ @@ -440,7 +438,6 @@ class TestIncrementalBackup(TestIncrementalBackupBase): }], } ) - self.assert_qmp(result, 'return', {}) =20 # Create bitmaps and full backups for both drives drive0 =3D self.drives[0] @@ -475,9 +472,8 @@ class TestIncrementalBackup(TestIncrementalBackupBase): format=3Ddrive1['fmt'], mode=3D'exist= ing', bitmap=3Ddr1bm0.name) ] - result =3D self.vm.qmp('transaction', actions=3Dtransaction, - properties=3D{'completion-mode': 'grouped'} ) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('transaction', actions=3Dtransaction, + properties=3D{'completion-mode': 'grouped'} ) =20 # Observe that drive0's backup is cancelled and drive1 completes w= ith # an error. @@ -504,9 +500,8 @@ class TestIncrementalBackup(TestIncrementalBackupBase): target1 =3D self.prepare_backup(dr1bm0) =20 # Re-run the exact same transaction. - result =3D self.vm.qmp('transaction', actions=3Dtransaction, - properties=3D{'completion-mode':'grouped'}) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('transaction', actions=3Dtransaction, + properties=3D{'completion-mode':'grouped'}) =20 # Both should complete successfully this time. self.assertTrue(self.wait_qmp_backup(drive0['id'])) @@ -617,7 +612,7 @@ class TestIncrementalBackupBlkdebug(TestIncrementalBack= upBase): ''' =20 drive0 =3D self.drives[0] - result =3D self.vm.qmp('blockdev-add', + self.vm.cmd('blockdev-add', node_name=3Ddrive0['id'], driver=3Ddrive0['fmt'], file=3D{ @@ -640,7 +635,6 @@ class TestIncrementalBackupBlkdebug(TestIncrementalBack= upBase): }], } ) - self.assert_qmp(result, 'return', {}) =20 self.create_anchor_backup(drive0) self.add_bitmap('bitmap0', drive0) @@ -668,29 +662,28 @@ class TestIncrementalBackupBlkdebug(TestIncrementalBa= ckupBase): drive0 =3D self.drives[0] # NB: The blkdebug script here looks for a "flush, read" pattern. # The flush occurs in hmp_io_writes, and the read during the block= job. - result =3D self.vm.qmp('blockdev-add', - node_name=3Ddrive0['id'], - driver=3Ddrive0['fmt'], - file=3D{ - 'driver': 'blkdebug', - 'image': { - 'driver': 'file', - 'filename': drive0['file'] - }, - 'set-state': [{ - 'event': 'flush_to_disk', - 'state': 1, - 'new_state': 2 - }], - 'inject-error': [{ - 'event': 'read_aio', - 'errno': 5, - 'state': 2, - 'immediately': False, - 'once': True - }], - }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + node_name=3Ddrive0['id'], + driver=3Ddrive0['fmt'], + file=3D{ + 'driver': 'blkdebug', + 'image': { + 'driver': 'file', + 'filename': drive0['file'] + }, + 'set-state': [{ + 'event': 'flush_to_disk', + 'state': 1, + 'new_state': 2 + }], + 'inject-error': [{ + 'event': 'read_aio', + 'errno': 5, + 'state': 2, + 'immediately': False, + 'once': True + }], + }) self.create_anchor_backup(drive0) bitmap =3D self.add_bitmap('bitmap0', drive0) =20 diff --git a/tests/qemu-iotests/129 b/tests/qemu-iotests/129 index c75ec62ecf..cc39f2b6ed 100755 --- a/tests/qemu-iotests/129 +++ b/tests/qemu-iotests/129 @@ -55,11 +55,9 @@ class TestStopWithBlockJob(iotests.QMPTestCase): def do_test_stop(self, cmd, **args): """Test 'stop' while block job is running on a throttled drive. The 'stop' command shouldn't drain the job""" - result =3D self.vm.qmp(cmd, **args) - self.assert_qmp(result, 'return', {}) + self.vm.cmd(cmd, **args) =20 - result =3D self.vm.qmp("stop") - self.assert_qmp(result, 'return', {}) + self.vm.cmd("stop") result =3D self.vm.qmp("query-block-jobs") =20 self.assert_qmp(result, 'return[0]/status', 'running') @@ -87,7 +85,7 @@ class TestStopWithBlockJob(iotests.QMPTestCase): iotests.qemu_img('create', '-f', iotests.imgfmt, self.overlay_img, '1G') =20 - result =3D self.vm.qmp('blockdev-add', **{ + self.vm.cmd('blockdev-add', { 'node-name': 'overlay', 'driver': iotests.imgfmt, 'file': { @@ -95,11 +93,8 @@ class TestStopWithBlockJob(iotests.QMPTestCase): 'filename': self.overlay_img } }) - self.assert_qmp(result, 'return', {}) =20 - result =3D self.vm.qmp('blockdev-snapshot', - node=3D'source', overlay=3D'overlay') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-snapshot', node=3D'source', overlay=3D'overl= ay') =20 self.do_test_stop('block-commit', device=3D'drive0', top_node=3D's= ource') =20 diff --git a/tests/qemu-iotests/132 b/tests/qemu-iotests/132 index 367ea08036..12a64b3d95 100755 --- a/tests/qemu-iotests/132 +++ b/tests/qemu-iotests/132 @@ -47,9 +47,8 @@ class TestSingleDrive(iotests.QMPTestCase): pass =20 def test_mirror_discard(self): - result =3D self.vm.qmp('drive-mirror', device=3D'drive0', sync=3D'= full', - target=3Dtarget_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', device=3D'drive0', sync=3D'full', + target=3Dtarget_img) self.vm.hmp_qemu_io('drive0', 'discard 0 64k') self.complete_and_wait('drive0') self.vm.shutdown() diff --git a/tests/qemu-iotests/139 b/tests/qemu-iotests/139 index 178b1ee230..9fea38adff 100755 --- a/tests/qemu-iotests/139 +++ b/tests/qemu-iotests/139 @@ -58,8 +58,7 @@ class TestBlockdevDel(iotests.QMPTestCase): 'file': {'driver': 'file', 'node-name': file_node, 'filename': base_img}} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) self.checkBlockDriverState(node) self.checkBlockDriverState(file_node) =20 @@ -73,8 +72,7 @@ class TestBlockdevDel(iotests.QMPTestCase): 'backing': None, 'file': {'driver': 'file', 'filename': new_img}} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) self.checkBlockDriverState(node) =20 # Delete a BlockDriverState @@ -89,17 +87,14 @@ class TestBlockdevDel(iotests.QMPTestCase): =20 # Add a device model def addDeviceModel(self, device, backend, driver =3D 'virtio-blk'): - result =3D self.vm.qmp('device_add', id =3D device, - driver =3D driver, drive =3D backend) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', id =3D device, + driver =3D driver, drive =3D backend) =20 # Delete a device model def delDeviceModel(self, device, is_virtio_blk =3D True): - result =3D self.vm.qmp('device_del', id =3D device) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_del', id =3D device) =20 - result =3D self.vm.qmp('system_reset') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('system_reset') =20 if is_virtio_blk: device_path =3D '/machine/peripheral/%s/virtio-backend' % devi= ce @@ -126,9 +121,7 @@ class TestBlockdevDel(iotests.QMPTestCase): # Insert a BlockDriverState def insertDrive(self, device, node): self.checkBlockDriverState(node) - result =3D self.vm.qmp('blockdev-insert-medium', - id =3D device, node_name =3D node) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-insert-medium', id =3D device, node_name =3D= node) self.checkBlockDriverState(node) =20 # Create a snapshot using 'blockdev-snapshot-sync' @@ -139,8 +132,7 @@ class TestBlockdevDel(iotests.QMPTestCase): 'snapshot-file': new_img, 'snapshot-node-name': overlay, 'format': iotests.imgfmt} - result =3D self.vm.qmp('blockdev-snapshot-sync', conv_keys=3DFalse= , **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-snapshot-sync', conv_keys=3DFalse, **opts) self.checkBlockDriverState(node) self.checkBlockDriverState(overlay) =20 @@ -148,9 +140,7 @@ class TestBlockdevDel(iotests.QMPTestCase): def createSnapshot(self, node, overlay): self.checkBlockDriverState(node) self.checkBlockDriverState(overlay) - result =3D self.vm.qmp('blockdev-snapshot', - node =3D node, overlay =3D overlay) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-snapshot', node =3D node, overlay =3D overla= y) self.checkBlockDriverState(node) self.checkBlockDriverState(overlay) =20 @@ -163,14 +153,12 @@ class TestBlockdevDel(iotests.QMPTestCase): 'node-name': new_node, 'sync': 'top', 'format': iotests.imgfmt} - result =3D self.vm.qmp('drive-mirror', conv_keys=3DFalse, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('drive-mirror', conv_keys=3DFalse, **opts) self.checkBlockDriverState(new_node) =20 # Complete an existing block job def completeBlockJob(self, id, node_before, node_after): - result =3D self.vm.qmp('block-job-complete', device=3Did) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-complete', device=3Did) self.wait_until_completed(id) =20 # Add a BlkDebug node @@ -186,8 +174,7 @@ class TestBlockdevDel(iotests.QMPTestCase): opts =3D {'driver': 'blkdebug', 'node-name': debug, 'image': image} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) self.checkBlockDriverState(node) self.checkBlockDriverState(debug) =20 @@ -211,8 +198,7 @@ class TestBlockdevDel(iotests.QMPTestCase): 'node-name': blkverify, 'test': node_0, 'raw': node_1} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) self.checkBlockDriverState(test) self.checkBlockDriverState(raw) self.checkBlockDriverState(blkverify) @@ -235,8 +221,7 @@ class TestBlockdevDel(iotests.QMPTestCase): 'node-name': quorum, 'vote-threshold': 1, 'children': [ child_0, child_1 ]} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) self.checkBlockDriverState(child0) self.checkBlockDriverState(child1) self.checkBlockDriverState(quorum) diff --git a/tests/qemu-iotests/147 b/tests/qemu-iotests/147 index 47dfa62e6b..8c5e08bf2b 100755 --- a/tests/qemu-iotests/147 +++ b/tests/qemu-iotests/147 @@ -58,8 +58,7 @@ class NBDBlockdevAddBase(iotests.QMPTestCase): def client_test(self, filename, address, export=3DNone, node_name=3D'nbd-blockdev', delete=3DTrue): bao =3D self.blockdev_add_options(address, export, node_name) - result =3D self.vm.qmp('blockdev-add', **bao) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', **bao) =20 found =3D False result =3D self.vm.qmp('query-named-block-nodes') @@ -75,8 +74,7 @@ class NBDBlockdevAddBase(iotests.QMPTestCase): self.assertTrue(found) =20 if delete: - result =3D self.vm.qmp('blockdev-del', node_name=3Dnode_name) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3Dnode_name) =20 =20 class QemuNBD(NBDBlockdevAddBase): @@ -158,16 +156,14 @@ class BuiltinNBD(NBDBlockdevAddBase): self.assert_qmp(result, 'return', {}) =20 if export_name is None: - result =3D self.server.qmp('nbd-server-add', device=3D'nbd-exp= ort') + self.server.cmd('nbd-server-add', device=3D'nbd-export') else: - result =3D self.server.qmp('nbd-server-add', device=3D'nbd-exp= ort', - name=3Dexport_name) - self.assert_qmp(result, 'return', {}) + self.server.cmd('nbd-server-add', device=3D'nbd-export', + name=3Dexport_name) =20 if export_name2 is not None: - result =3D self.server.qmp('nbd-server-add', device=3D'nbd-exp= ort', - name=3Dexport_name2) - self.assert_qmp(result, 'return', {}) + self.server.cmd('nbd-server-add', device=3D'nbd-export', + name=3Dexport_name2) =20 return True =20 @@ -175,8 +171,7 @@ class BuiltinNBD(NBDBlockdevAddBase): self.assertTrue(self._try_server_up(address, export_name, export_n= ame2)) =20 def _server_down(self): - result =3D self.server.qmp('nbd-server-stop') - self.assert_qmp(result, 'return', {}) + self.server.cmd('nbd-server-stop') =20 def do_test_inet(self, export_name=3DNone): while True: @@ -218,10 +213,8 @@ class BuiltinNBD(NBDBlockdevAddBase): flatten_sock_addr(address), 'exp1', 'node1', Fals= e) self.client_test('nbd://localhost:%i/%s' % (nbd_port, 'exp2'), flatten_sock_addr(address), 'exp2', 'node2', Fals= e) - result =3D self.vm.qmp('blockdev-del', node_name=3D'node1') - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('blockdev-del', node_name=3D'node2') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'node1') + self.vm.cmd('blockdev-del', node_name=3D'node2') self._server_down() =20 def test_inet6(self): @@ -272,8 +265,7 @@ class BuiltinNBD(NBDBlockdevAddBase): result =3D self.vm.send_fd_scm(fd=3Dsockfd.fileno()) self.assertEqual(result, 0, 'Failed to send socket FD') =20 - result =3D self.vm.qmp('getfd', fdname=3D'nbd-fifo') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('getfd', fdname=3D'nbd-fifo') =20 address =3D { 'type': 'fd', 'data': { 'str': 'nbd-fifo' } } diff --git a/tests/qemu-iotests/151 b/tests/qemu-iotests/151 index 93d14193d0..26154d3d0c 100755 --- a/tests/qemu-iotests/151 +++ b/tests/qemu-iotests/151 @@ -75,14 +75,13 @@ class TestActiveMirror(iotests.QMPTestCase): self.vm.hmp_qemu_io('source', 'aio_write -z %i 1M' % offset) =20 # Start the block job - result =3D self.vm.qmp('blockdev-mirror', - job_id=3D'mirror', - filter_node_name=3D'mirror-node', - device=3D'source-node', - target=3D'target-node', - sync=3D'full', - copy_mode=3D'write-blocking') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-mirror', + job_id=3D'mirror', + filter_node_name=3D'mirror-node', + device=3D'source-node', + target=3D'target-node', + sync=3D'full', + copy_mode=3D'write-blocking') =20 # Start some more requests for offset in range(3 * self.image_len // 8, 5 * self.image_len //= 8, 1024 * 1024): @@ -121,23 +120,21 @@ class TestActiveMirror(iotests.QMPTestCase): result =3D self.vm.hmp_qemu_io('source', 'write -P 1 0 2M') =20 # Start the block job (very slowly) - result =3D self.vm.qmp('blockdev-mirror', - job_id=3D'mirror', - filter_node_name=3D'mirror-node', - device=3D'source-node', - target=3D'target-node', - sync=3D'full', - copy_mode=3D'write-blocking', - buf_size=3D(1048576 // 4), - speed=3D1) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-mirror', + job_id=3D'mirror', + filter_node_name=3D'mirror-node', + device=3D'source-node', + target=3D'target-node', + sync=3D'full', + copy_mode=3D'write-blocking', + buf_size=3D(1048576 // 4), + speed=3D1) =20 # Start an unaligned request to a dirty area result =3D self.vm.hmp_qemu_io('source', 'write -P 2 %i 1' % (1048= 576 + 42)) =20 # Let the job finish - result =3D self.vm.qmp('block-job-set-speed', device=3D'mirror', s= peed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'mirror', speed=3D0) self.complete_and_wait(drive=3D'mirror') =20 self.potential_writes_in_flight =3D False @@ -185,8 +182,7 @@ class TestActiveMirror(iotests.QMPTestCase): # After resuming 4, one of 2 and 3 goes first and set in_flight_bi= tmap, # so the other will wait for it. =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'mirror', s= peed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'mirror', speed=3D0) self.complete_and_wait(drive=3D'mirror') =20 self.potential_writes_in_flight =3D False diff --git a/tests/qemu-iotests/155 b/tests/qemu-iotests/155 index eadda52615..6b1625b8f3 100755 --- a/tests/qemu-iotests/155 +++ b/tests/qemu-iotests/155 @@ -110,8 +110,7 @@ class BaseClass(iotests.QMPTestCase): elif self.target_blockdev_backing: options['backing'] =3D self.target_blockdev_backing =20 - result =3D self.vm.qmp('blockdev-add', **options) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', **options) =20 def tearDown(self): self.vm.shutdown() @@ -178,20 +177,18 @@ class MirrorBaseClass(BaseClass): =20 def runMirror(self, sync): if self.cmd =3D=3D 'blockdev-mirror': - result =3D self.vm.qmp(self.cmd, job_id=3D'mirror-job', device= =3D'source', - sync=3Dsync, target=3D'target', - auto_finalize=3DFalse) + self.vm.cmd(self.cmd, job_id=3D'mirror-job', device=3D'source', + sync=3Dsync, target=3D'target', + auto_finalize=3DFalse) else: if self.existing: mode =3D 'existing' else: mode =3D 'absolute-paths' - result =3D self.vm.qmp(self.cmd, job_id=3D'mirror-job', device= =3D'source', - sync=3Dsync, target=3Dtarget_img, - format=3Diotests.imgfmt, mode=3Dmode, - node_name=3D'target', auto_finalize=3DFal= se) - - self.assert_qmp(result, 'return', {}) + self.vm.cmd(self.cmd, job_id=3D'mirror-job', device=3D'source', + sync=3Dsync, target=3Dtarget_img, + format=3Diotests.imgfmt, mode=3Dmode, + node_name=3D'target', auto_finalize=3DFalse) =20 self.vm.run_job('mirror-job', auto_finalize=3DFalse, pre_finalize=3Dself.openBacking, auto_dismiss=3DTr= ue) @@ -258,16 +255,14 @@ class TestBlockdevMirrorReopen(MirrorBaseClass): =20 def openBacking(self): if not self.target_open_with_backing: - result =3D self.vm.qmp('blockdev-add', node_name=3D"backing", - driver=3D"null-co") - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('blockdev-reopen', options=3D[{ - 'node-name': "target", - 'driver': iotests.imgfmt, - 'file': "target-file", - 'backing': "backing" - }]) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', node_name=3D"backing", + driver=3D"null-co") + self.vm.cmd('blockdev-reopen', options=3D[{ + 'node-name': "target", + 'driver': iotests.imgfmt, + 'file': "target-file", + 'backing': "backing" + }]) =20 class TestBlockdevMirrorReopenIothread(TestBlockdevMirrorReopen): use_iothread =3D True @@ -281,12 +276,8 @@ class TestBlockdevMirrorSnapshot(MirrorBaseClass): =20 def openBacking(self): if not self.target_open_with_backing: - result =3D self.vm.qmp('blockdev-add', node_name=3D"backing", - driver=3D"null-co") - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('blockdev-snapshot', node=3D"backing", - overlay=3D"target") - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', node_name=3D"backing", driver=3D"n= ull-co") + self.vm.cmd('blockdev-snapshot', node=3D"backing", overlay=3D"= target") =20 class TestBlockdevMirrorSnapshotIothread(TestBlockdevMirrorSnapshot): use_iothread =3D True @@ -295,14 +286,12 @@ class TestCommit(BaseClass): existing =3D False =20 def testCommit(self): - result =3D self.vm.qmp('block-commit', job_id=3D'commit-job', - device=3D'source', base=3Dback1_img) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', job_id=3D'commit-job', + device=3D'source', base=3Dback1_img) =20 self.vm.event_wait('BLOCK_JOB_READY') =20 - result =3D self.vm.qmp('block-job-complete', device=3D'commit-job') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-complete', device=3D'commit-job') =20 self.vm.event_wait('BLOCK_JOB_COMPLETED') =20 diff --git a/tests/qemu-iotests/165 b/tests/qemu-iotests/165 index e3ef28e2ee..028c6a15ca 100755 --- a/tests/qemu-iotests/165 +++ b/tests/qemu-iotests/165 @@ -116,9 +116,7 @@ class TestPersistentDirtyBitmap(iotests.QMPTestCase): sha256_2 =3D self.getSha256() assert sha256_1 !=3D sha256_2 # Otherwise, it's not very interesti= ng. =20 - result =3D self.vm.qmp('block-dirty-bitmap-clear', node=3D'drive0', - name=3D'bitmap0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-dirty-bitmap-clear', node=3D'drive0', name=3D'b= itmap0') =20 # Start with regions1 =20 @@ -137,7 +135,7 @@ class TestPersistentDirtyBitmap(iotests.QMPTestCase): assert sha256_1 =3D=3D self.getSha256() =20 # Reopen to RW - result =3D self.vm.qmp('blockdev-reopen', options=3D[{ + self.vm.cmd('blockdev-reopen', options=3D[{ 'node-name': 'node0', 'driver': iotests.imgfmt, 'file': { @@ -146,7 +144,6 @@ class TestPersistentDirtyBitmap(iotests.QMPTestCase): }, 'read-only': False }]) - self.assert_qmp(result, 'return', {}) =20 # Check that bitmap is reopened to RW and we can write to it. self.writeRegions(regions2) diff --git a/tests/qemu-iotests/196 b/tests/qemu-iotests/196 index 76509a5ad1..e5105b1354 100755 --- a/tests/qemu-iotests/196 +++ b/tests/qemu-iotests/196 @@ -45,8 +45,7 @@ class TestInvalidateAutoclear(iotests.QMPTestCase): self.vm_b.add_incoming("exec: cat '" + migfile + "'") =20 def test_migration(self): - result =3D self.vm_a.qmp('migrate', uri=3D'exec:cat>' + migfile) - self.assert_qmp(result, 'return', {}); + self.vm_a.cmd('migrate', uri=3D'exec:cat>' + migfile) self.assertNotEqual(self.vm_a.event_wait("STOP"), None) =20 with open(disk, 'r+b') as f: diff --git a/tests/qemu-iotests/205 b/tests/qemu-iotests/205 index c0e107328f..5f5e595ee5 100755 --- a/tests/qemu-iotests/205 +++ b/tests/qemu-iotests/205 @@ -44,10 +44,8 @@ class TestNbdServerRemove(iotests.QMPTestCase): } } =20 - result =3D self.vm.qmp('nbd-server-start', addr=3Daddress) - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('nbd-server-add', device=3D'drive0', name= =3D'exp') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('nbd-server-start', addr=3Daddress) + self.vm.cmd('nbd-server-add', device=3D'drive0', name=3D'exp') =20 def tearDown(self): self.vm.shutdown() diff --git a/tests/qemu-iotests/245 b/tests/qemu-iotests/245 index 8cbed7821b..6120cdd9ad 100755 --- a/tests/qemu-iotests/245 +++ b/tests/qemu-iotests/245 @@ -134,8 +134,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): def test_incorrect_parameters_single_file(self): # Open 'hd0' only (no backing files) opts =3D hd_opts(0) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) original_graph =3D self.vm.qmp('query-named-block-nodes') =20 # We can reopen the image passing the same options @@ -169,8 +168,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.check_node_graph(original_graph) =20 # Remove the node - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') =20 # This test opens an image with a backing file and tries to reopen # it with illegal / incorrect parameters. @@ -178,8 +176,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): # Open hd1 omitting the backing options (hd0 will be opened # with the default options) opts =3D hd_opts(1) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) original_graph =3D self.vm.qmp('query-named-block-nodes') =20 # We can't reopen the image passing the same options, 'backing' is= mandatory @@ -211,8 +208,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.check_node_graph(original_graph) =20 # Remove the node - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd1= ') =20 # Reopen an image several times changing some of its options def test_reopen(self): @@ -225,8 +221,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): # Open the hd1 image passing all backing options opts =3D hd_opts(1) opts['backing'] =3D hd_opts(0) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) original_graph =3D self.vm.qmp('query-named-block-nodes') =20 # We can reopen the image passing the same options @@ -301,8 +296,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.assert_qmp_absent(self.get_node('hd1'), 'image/backing-image') =20 # Open the 'hd0' image - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **hd_o= pts(0)) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **hd_opts(0)) =20 # Reopen the hd1 image setting 'hd0' as its backing image self.reopen(opts, {'backing': 'hd0'}) @@ -321,10 +315,8 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.assert_qmp(result, 'error/desc', "Node 'hd0' is busy: node is= used as backing hd of 'hd1'") =20 # But we can remove both nodes if done in the proper order - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd1') - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd1= ') + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') =20 # Reopen a raw image and see the effect of changing the 'offset' option def test_reopen_raw(self): @@ -340,8 +332,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): qemu_io('-f', 'raw', '-c', 'write -P 0xa1 1M 1M', hd_path[0]) =20 # Open the raw file with QEMU - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Read 1MB from offset 0 self.run_qemu_io("hd0", "read -P 0xa0 0 1M") @@ -357,8 +348,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.run_qemu_io("hd0", "read -P 0xa0 0 1M") =20 # Remove the block device - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') =20 # Omitting an option should reset it to the default value, but if # an option cannot be changed it shouldn't be possible to reset it @@ -372,8 +362,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): 'node-name': 'hd0-file' } } =20 # Open the file with QEMU - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # file.x-check-cache-dropped can be changed... self.reopen(opts, { 'file.x-check-cache-dropped': False }) @@ -389,8 +378,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.reopen(opts, { 'file.locking': 'off' }) =20 # Remove the block device - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') =20 # This test modifies the node graph a few times by changing the # 'backing' option on reopen and verifies that the guest data that @@ -402,8 +390,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): for i in range(3): opts.append(hd_opts(i)) opts[i]['backing'] =3D None - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **= opts[i]) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts[i]) =20 # hd0 self.run_qemu_io("hd0", "read -P 0xa0 0 1M") @@ -494,8 +481,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): for i in range(3): opts.append(hd_opts(i)) opts[i]['backing'] =3D None - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **= opts[i]) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts[i]) =20 # hd1 <- hd0, hd1 <- hd2 self.reopen(opts[0], {'backing': 'hd1'}) @@ -527,8 +513,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): 'node-name': 'bv', 'test': 'hd0', 'raw': 'hd1'} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **bvop= ts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **bvopts) =20 # blkverify doesn't currently allow reopening. TODO: implement this self.reopen(bvopts, {}, "Block format 'blkverify' used by node 'bv= '" + @@ -539,8 +524,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): "Making 'bv' a backing child of 'hd0' would create a c= ycle") =20 # Delete the blkverify node - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'bv') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'bv') =20 # Replace the protocol layer ('file' parameter) of a disk image def test_replace_file(self): @@ -551,16 +535,13 @@ class TestBlockdevReopen(iotests.QMPTestCase): hd0_opts =3D {'driver': 'file', 'node-name': 'hd0-file', 'filename= ': hd_path[0] } hd1_opts =3D {'driver': 'file', 'node-name': 'hd1-file', 'filename= ': hd_path[1] } =20 - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **hd0_= opts) - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **hd1_= opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **hd0_opts) + self.vm.cmd('blockdev-add', conv_keys =3D False, **hd1_opts) =20 # Add a raw format layer that uses hd0-file as its protocol layer opts =3D {'driver': 'raw', 'node-name': 'hd', 'file': 'hd0-file'} =20 - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Fill the image with data self.run_qemu_io("hd", "read -P 0 0 10k") @@ -583,21 +564,18 @@ class TestBlockdevReopen(iotests.QMPTestCase): def test_insert_throttle_filter(self): # Add an image to the VM hd0_opts =3D hd_opts(0) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **hd0_= opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **hd0_opts) =20 # Create a throttle-group object opts =3D { 'qom-type': 'throttle-group', 'id': 'group0', 'limits': { 'iops-total': 1000 } } - result =3D self.vm.qmp('object-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', conv_keys =3D False, **opts) =20 # Add a throttle filter with the group that we just created. # The filter is not used by anyone yet opts =3D { 'driver': 'throttle', 'node-name': 'throttle0', 'throttle-group': 'group0', 'file': 'hd0-file' } - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Insert the throttle filter between hd0 and hd0-file self.reopen(hd0_opts, {'file': 'throttle0'}) @@ -609,15 +587,13 @@ class TestBlockdevReopen(iotests.QMPTestCase): def test_insert_compress_filter(self): # Add an image to the VM: hd (raw) -> hd0 (qcow2) -> hd0-file (fil= e) opts =3D {'driver': 'raw', 'node-name': 'hd', 'file': hd_opts(0)} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Add a 'compress' filter filter_opts =3D {'driver': 'compress', 'node-name': 'compress0', 'file': 'hd0'} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **filt= er_opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **filter_opts) =20 # Unmap the beginning of the image (we cannot write compressed # data to an allocated cluster) @@ -653,12 +629,10 @@ class TestBlockdevReopen(iotests.QMPTestCase): def test_swap_files(self): # Add hd0 and hd2 (none of them with backing files) opts0 =3D hd_opts(0) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts= 0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts0) =20 opts2 =3D hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts= 2) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts2) =20 # Write different data to both block devices self.run_qemu_io("hd0", "write -P 0xa0 0 1k") @@ -706,15 +680,13 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts =3D hd_opts(i) # Open all three images without backing file opts['backing'] =3D None - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **= opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 opts =3D {'driver': 'quorum', 'node-name': 'quorum0', 'children': ['hd0', 'hd1', 'hd2'], 'vote-threshold': 2} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Quorum doesn't currently allow reopening. TODO: implement this self.reopen(opts, {}, "Block format 'quorum' used by node 'quorum0= '" + @@ -726,14 +698,12 @@ class TestBlockdevReopen(iotests.QMPTestCase): "Making 'quorum0' a backing child of 'hd0' would creat= e a cycle") =20 # Delete quorum0 - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'quorum0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'quo= rum0') =20 # Delete hd0, hd1 and hd2 for i in range(3): - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, - node_name =3D 'hd%d' % i) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, + node_name =3D 'hd%d' % i) =20 ###################### ###### blkdebug ###### @@ -742,8 +712,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): 'node-name': 'bd', 'config': '/dev/null', 'image': hd_opts(0)} - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # blkdebug allows reopening if we keep the same options self.reopen(opts) @@ -756,16 +725,14 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.reopen(opts, {}, "Option 'config' cannot be reset to its defa= ult value") =20 # Delete the blkdebug node - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'bd') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'bd') =20 ################## ###### null ###### ################## opts =3D {'driver': 'null-co', 'node-name': 'root', 'size': 1024} =20 - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # 1 << 30 is the default value, but we cannot change it explicitly self.reopen(opts, {'size': (1 << 30)}, "Cannot change the option '= size'") @@ -774,16 +741,14 @@ class TestBlockdevReopen(iotests.QMPTestCase): del opts['size'] self.reopen(opts, {}, "Option 'size' cannot be reset to its defaul= t value") =20 - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'root') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'roo= t') =20 ################## ###### file ###### ################## opts =3D hd_opts(0) opts['file']['locking'] =3D 'on' - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # 'locking' cannot be changed del opts['file']['locking'] @@ -797,27 +762,23 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.reopen(opts, {'locking': 'off'}, "Cannot change the option 'l= ocking'") self.reopen(opts, {}, "Option 'locking' cannot be reset to its def= ault value") =20 - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') =20 ###################### ###### throttle ###### ###################### opts =3D { 'qom-type': 'throttle-group', 'id': 'group0', 'limits': { 'iops-total': 1000 } } - result =3D self.vm.qmp('object-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', conv_keys =3D False, **opts) =20 opts =3D { 'qom-type': 'throttle-group', 'id': 'group1', 'limits': { 'iops-total': 2000 } } - result =3D self.vm.qmp('object-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', conv_keys =3D False, **opts) =20 # Add a throttle filter with group =3D group0 opts =3D { 'driver': 'throttle', 'node-name': 'throttle0', 'throttle-group': 'group0', 'file': hd_opts(0) } - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # We can reopen it if we keep the same options self.reopen(opts) @@ -845,16 +806,13 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.assert_qmp(result, 'error/desc', "object 'group0' is in use, = can not be deleted") =20 # But group1 is free this time, and it can be deleted - result =3D self.vm.qmp('object-del', id =3D 'group1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-del', id =3D 'group1') =20 # Let's delete the filter node - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'throttle0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'thr= ottle0') =20 # And we can finally get rid of group0 - result =3D self.vm.qmp('object-del', id =3D 'group0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-del', id =3D 'group0') =20 # If an image has a backing file then the 'backing' option must be # passed on reopen. We don't allow leaving the option out in this @@ -862,13 +820,11 @@ class TestBlockdevReopen(iotests.QMPTestCase): def test_missing_backing_options_1(self): # hd2 opts =3D hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # hd0 opts =3D hd_opts(0) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # hd0 has no backing file: we can omit the 'backing' option self.reopen(opts) @@ -891,11 +847,9 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.reopen(opts) =20 # Remove both hd0 and hd2 - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') =20 - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd2') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd2= ') =20 # If an image has default backing file (as part of its metadata) # then the 'backing' option must be passed on reopen. We don't @@ -905,8 +859,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): # hd0 <- hd1 # (hd0 is hd1's default backing file) opts =3D hd_opts(1) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # hd1 has a backing file: we can't omit the 'backing' option self.reopen(opts, {}, "backing is missing for 'hd1'") @@ -917,8 +870,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): # No backing file attached to hd1 now, but we still can't omit the= 'backing' option self.reopen(opts, {}, "backing is missing for 'hd1'") =20 - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd1= ') =20 # Test that making 'backing' a reference to an existing child # keeps its current options @@ -931,8 +883,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts['detect-zeroes'] =3D 'on' opts['backing']['detect-zeroes'] =3D 'on' opts['backing']['backing']['detect-zeroes'] =3D 'on' - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Reopen the chain passing the minimum amount of required options. # By making 'backing' a reference to hd1 (instead of a sub-dict) @@ -955,12 +906,10 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts =3D hd_opts(0) opts['backing'] =3D hd_opts(1) opts['backing']['backing'] =3D None - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Stream hd1 into hd0 and wait until it's done - result =3D self.vm.qmp('block-stream', conv_keys =3D True, job_id = =3D 'stream0', device =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', conv_keys =3D True, job_id =3D 'stream= 0', device =3D 'hd0') self.wait_until_completed(drive =3D 'stream0') =20 # Now we have only hd0 @@ -976,8 +925,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): # We can also reopen hd0 if we set 'backing' to null self.reopen(opts, {'backing': None}) =20 - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') =20 # Another block_stream test def test_block_stream_2(self): @@ -985,13 +933,11 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts =3D hd_opts(0) opts['backing'] =3D hd_opts(1) opts['backing']['backing'] =3D hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # Stream hd1 into hd0 and wait until it's done - result =3D self.vm.qmp('block-stream', conv_keys =3D True, job_id = =3D 'stream0', - device =3D 'hd0', base_node =3D 'hd2') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', conv_keys =3D True, job_id =3D 'stream= 0', + device =3D 'hd0', base_node =3D 'hd2') self.wait_until_completed(drive =3D 'stream0') =20 # The chain is hd2 <- hd0 now. hd1 is missing @@ -1013,8 +959,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.reopen(opts, {}, "backing is missing for 'hd0'") =20 # Now we can delete hd0 (and hd2) - result =3D self.vm.qmp('blockdev-del', conv_keys =3D True, node_na= me =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', conv_keys =3D True, node_name =3D 'hd0= ') self.assertEqual(self.get_node('hd2'), None) =20 # Reopen the chain during a block-stream job (from hd1 to hd0) @@ -1023,14 +968,12 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts =3D hd_opts(0) opts['backing'] =3D hd_opts(1) opts['backing']['backing'] =3D hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # hd2 <- hd0 - result =3D self.vm.qmp('block-stream', conv_keys =3D True, job_id = =3D 'stream0', - device =3D 'hd0', base_node =3D 'hd2', - auto_finalize =3D False) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', conv_keys =3D True, job_id =3D 'stream= 0', + device =3D 'hd0', base_node =3D 'hd2', + auto_finalize =3D False) =20 # We can remove hd2 while the stream job is ongoing opts['backing']['backing'] =3D None @@ -1048,14 +991,12 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts =3D hd_opts(0) opts['backing'] =3D hd_opts(1) opts['backing']['backing'] =3D hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 # hd1 <- hd0 - result =3D self.vm.qmp('block-stream', conv_keys =3D True, job_id = =3D 'stream0', - device =3D 'hd1', filter_node_name=3D'cor', - auto_finalize =3D False) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-stream', conv_keys =3D True, job_id =3D 'stream= 0', + device =3D 'hd1', filter_node_name=3D'cor', + auto_finalize =3D False) =20 # We can't reopen with the original options because there is a fil= ter # inserted by stream job above hd1. @@ -1084,12 +1025,10 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts =3D hd_opts(0) opts['backing'] =3D hd_opts(1) opts['backing']['backing'] =3D hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 - result =3D self.vm.qmp('block-commit', conv_keys =3D True, job_id = =3D 'commit0', - device =3D 'hd0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', conv_keys =3D True, job_id =3D 'commit= 0', + device =3D 'hd0') =20 # We can't remove hd2 while the commit job is ongoing opts['backing']['backing'] =3D None @@ -1104,8 +1043,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): self.assert_qmp(event, 'data/type', 'commit') self.assert_qmp_absent(event, 'data/error') =20 - result =3D self.vm.qmp('block-job-complete', device=3D'commit0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-complete', device=3D'commit0') =20 self.wait_until_completed(drive =3D 'commit0') =20 @@ -1115,13 +1053,11 @@ class TestBlockdevReopen(iotests.QMPTestCase): opts =3D hd_opts(0) opts['backing'] =3D hd_opts(1) opts['backing']['backing'] =3D hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 - result =3D self.vm.qmp('block-commit', conv_keys =3D True, job_id = =3D 'commit0', - device =3D 'hd0', top_node =3D 'hd1', - auto_finalize =3D False) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', conv_keys =3D True, job_id =3D 'commit= 0', + device =3D 'hd0', top_node =3D 'hd1', + auto_finalize =3D False) =20 # We can't remove hd2 while the commit job is ongoing opts['backing']['backing'] =3D None @@ -1141,36 +1077,28 @@ class TestBlockdevReopen(iotests.QMPTestCase): def run_test_iothreads(self, iothread_a, iothread_b, errmsg =3D None, opts_a =3D None, opts_b =3D None): opts =3D opts_a or hd_opts(0) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts) =20 opts2 =3D opts_b or hd_opts(2) - result =3D self.vm.qmp('blockdev-add', conv_keys =3D False, **opts= 2) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', conv_keys =3D False, **opts2) =20 - result =3D self.vm.qmp('object-add', qom_type=3D'iothread', id=3D'= iothread0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', qom_type=3D'iothread', id=3D'iothread0') =20 - result =3D self.vm.qmp('object-add', qom_type=3D'iothread', id=3D'= iothread1') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', qom_type=3D'iothread', id=3D'iothread1') =20 - result =3D self.vm.qmp('device_add', driver=3D'virtio-scsi', id=3D= 'scsi0', - iothread=3Diothread_a) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', driver=3D'virtio-scsi', id=3D'scsi0', + iothread=3Diothread_a) =20 - result =3D self.vm.qmp('device_add', driver=3D'virtio-scsi', id=3D= 'scsi1', - iothread=3Diothread_b) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', driver=3D'virtio-scsi', id=3D'scsi1', + iothread=3Diothread_b) =20 if iothread_a: - result =3D self.vm.qmp('device_add', driver=3D'scsi-hd', drive= =3D'hd0', - share_rw=3DTrue, bus=3D"scsi0.0") - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', driver=3D'scsi-hd', drive=3D'hd0', + share_rw=3DTrue, bus=3D"scsi0.0") =20 if iothread_b: - result =3D self.vm.qmp('device_add', driver=3D'scsi-hd', drive= =3D'hd2', - share_rw=3DTrue, bus=3D"scsi1.0") - self.assert_qmp(result, 'return', {}) + self.vm.cmd('device_add', driver=3D'scsi-hd', drive=3D'hd2', + share_rw=3DTrue, bus=3D"scsi1.0") =20 # Attaching the backing file may or may not work self.reopen(opts, {'backing': 'hd2'}, errmsg) @@ -1199,8 +1127,7 @@ class TestBlockdevReopen(iotests.QMPTestCase): # Create a throttle-group object opts =3D { 'qom-type': 'throttle-group', 'id': 'group0', 'limits': { 'iops-total': 1000 } } - result =3D self.vm.qmp('object-add', conv_keys =3D False, **opts) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('object-add', conv_keys =3D False, **opts) =20 # Options with a throttle filter between format and protocol opts =3D [ diff --git a/tests/qemu-iotests/264 b/tests/qemu-iotests/264 index bc431d1a19..67364a05e6 100755 --- a/tests/qemu-iotests/264 +++ b/tests/qemu-iotests/264 @@ -48,18 +48,16 @@ class TestNbdReconnect(iotests.QMPTestCase): """Stat job with nbd target and kill the server""" assert job in ('blockdev-backup', 'blockdev-mirror') with qemu_nbd_popen('-k', nbd_sock, '-f', iotests.imgfmt, disk_b): - result =3D self.vm.qmp('blockdev-add', - **{'node_name': 'backup0', - 'driver': 'raw', - 'file': {'driver': 'nbd', - 'server': {'type': 'unix', - 'path': nbd_sock}, - 'reconnect-delay': 10}}) - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp(job, device=3D'drive0', - sync=3D'full', target=3D'backup0', - speed=3D(1 * 1024 * 1024)) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', + {'node-name': 'backup0', + 'driver': 'raw', + 'file': {'driver': 'nbd', + 'server': {'type': 'unix', + 'path': nbd_sock}, + 'reconnect-delay': 10}}) + self.vm.cmd(job, device=3D'drive0', + sync=3D'full', target=3D'backup0', + speed=3D(1 * 1024 * 1024)) =20 # Wait for some progress t =3D 0.0 @@ -77,8 +75,7 @@ class TestNbdReconnect(iotests.QMPTestCase): self.assertTrue(jobs) self.assertTrue(jobs[0]['offset'] < jobs[0]['len']) =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', s= peed=3D0) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D0) =20 # Emulate server down time for 1 second time.sleep(1) @@ -91,12 +88,10 @@ class TestNbdReconnect(iotests.QMPTestCase): with qemu_nbd_popen('-k', nbd_sock, '-f', iotests.imgfmt, disk_b): e =3D self.vm.event_wait('BLOCK_JOB_COMPLETED') self.assertEqual(e['data']['offset'], size) - result =3D self.vm.qmp('blockdev-del', node_name=3D'backup0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'backup0') =20 def cancel_job(self): - result =3D self.vm.qmp('block-job-cancel', device=3D'drive0', forc= e=3DTrue) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-cancel', device=3D'drive0', force=3DTrue) =20 start_t =3D time.time() self.vm.event_wait('BLOCK_JOB_CANCELLED') diff --git a/tests/qemu-iotests/281 b/tests/qemu-iotests/281 index 5e1339bd75..f6746a12e8 100755 --- a/tests/qemu-iotests/281 +++ b/tests/qemu-iotests/281 @@ -56,15 +56,13 @@ class TestDirtyBitmapIOThread(iotests.QMPTestCase): os.remove(self.images[name]) =20 def test_add_dirty_bitmap(self): - result =3D self.vm.qmp( + self.vm.cmd( 'block-dirty-bitmap-add', node=3D'drive0', name=3D'bitmap1', persistent=3DTrue, ) =20 - self.assert_qmp(result, 'return', {}) - =20 # Test for RHBZ#1746217 & RHBZ#1773517 class TestNBDMirrorIOThread(iotests.QMPTestCase): @@ -105,23 +103,21 @@ class TestNBDMirrorIOThread(iotests.QMPTestCase): os.remove(self.images[name]) =20 def test_nbd_mirror(self): - result =3D self.vm_tgt.qmp( + self.vm_tgt.cmd( 'nbd-server-start', addr=3D{ 'type': 'unix', 'data': { 'path': self.nbd_sock } } ) - self.assert_qmp(result, 'return', {}) =20 - result =3D self.vm_tgt.qmp( + self.vm_tgt.cmd( 'nbd-server-add', device=3D'drive0', writable=3DTrue ) - self.assert_qmp(result, 'return', {}) =20 - result =3D self.vm_src.qmp( + self.vm_src.cmd( 'drive-mirror', device=3D'drive0', target=3D'nbd+unix:///drive0?socket=3D' + self.nbd_sock, @@ -130,7 +126,6 @@ class TestNBDMirrorIOThread(iotests.QMPTestCase): speed=3D64*1024*1024, job_id=3D'j1' ) - self.assert_qmp(result, 'return', {}) =20 self.vm_src.event_wait(name=3D"BLOCK_JOB_READY") =20 @@ -290,8 +285,7 @@ class TestYieldingAndTimers(iotests.QMPTestCase): # they will remain active, fire later, and then access freed data. # (Or, with "block/nbd: Assert there are no timers when closed" # applied, the assertions added in that patch will fail.) - result =3D self.vm.qmp('blockdev-del', node_name=3D'nbd') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', node_name=3D'nbd') =20 # Give the timers some time to fire (both have a timeout of 1 s). # (Sleeping in an iotest may ring some alarm bells, but note that = if @@ -303,9 +297,8 @@ class TestYieldingAndTimers(iotests.QMPTestCase): def test_yield_in_iothread(self): # Move the NBD node to the I/O thread; the NBD block driver should # attach the connection's QIOChannel to that thread's AioContext, = too - result =3D self.vm.qmp('x-blockdev-set-iothread', - node_name=3D'nbd', iothread=3D'iothr') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('x-blockdev-set-iothread', + node_name=3D'nbd', iothread=3D'iothr') =20 # Do some I/O that will be throttled by the QSD, so that the netwo= rk # connection hopefully will yield here. When it is resumed, it mu= st diff --git a/tests/qemu-iotests/295 b/tests/qemu-iotests/295 index 270ad3999f..2d472c9447 100755 --- a/tests/qemu-iotests/295 +++ b/tests/qemu-iotests/295 @@ -57,8 +57,7 @@ class EncryptionSetupTestCase(iotests.QMPTestCase): # create the secrets and load 'em into the VM self.secrets =3D [ Secret(i) for i in range(0, 6) ] for secret in self.secrets: - result =3D self.vm.qmp("object-add", **secret.to_qmp_object()) - self.assert_qmp(result, 'return', {}) + self.vm.cmd("object-add", **secret.to_qmp_object()) =20 if iotests.imgfmt =3D=3D "qcow2": self.pfx =3D "encrypt." @@ -102,26 +101,22 @@ class EncryptionSetupTestCase(iotests.QMPTestCase): } } =20 - result =3D self.vm.qmp('blockdev-add', ** - { - 'driver': iotests.imgfmt, - 'node-name': id, - 'read-only': read_only, + self.vm.cmd('blockdev-add', { + 'driver': iotests.imgfmt, + 'node-name': id, + 'read-only': read_only, =20 - **encrypt_options, + **encrypt_options, =20 - 'file': { - 'driver': 'file', - 'filename': test_img, - } + 'file': { + 'driver': 'file', + 'filename': test_img, } - ) - self.assert_qmp(result, 'return', {}) + }) =20 # close the encrypted block device def closeImageQmp(self, id): - result =3D self.vm.qmp('blockdev-del', **{ 'node-name': id }) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-del', **{ 'node-name': id }) =20 ######################################################################= ##### # add a key to an encrypted block device diff --git a/tests/qemu-iotests/296 b/tests/qemu-iotests/296 index 0d21b740a7..75a8695cfa 100755 --- a/tests/qemu-iotests/296 +++ b/tests/qemu-iotests/296 @@ -61,10 +61,8 @@ class EncryptionSetupTestCase(iotests.QMPTestCase): # create the secrets and load 'em into the VMs self.secrets =3D [ Secret(i) for i in range(0, 4) ] for secret in self.secrets: - result =3D self.vm1.qmp("object-add", **secret.to_qmp_object()) - self.assert_qmp(result, 'return', {}) - result =3D self.vm2.qmp("object-add", **secret.to_qmp_object()) - self.assert_qmp(result, 'return', {}) + self.vm1.cmd("object-add", **secret.to_qmp_object()) + self.vm2.cmd("object-add", **secret.to_qmp_object()) =20 # test case shutdown def tearDown(self): @@ -132,10 +130,9 @@ class EncryptionSetupTestCase(iotests.QMPTestCase): } =20 if reOpen: - result =3D vm.qmp(command, options=3D[opts]) + vm.cmd(command, options=3D[opts]) else: - result =3D vm.qmp(command, **opts) - self.assert_qmp(result, 'return', {}) + vm.cmd(command, **opts) =20 =20 ######################################################################= ##### @@ -154,8 +151,7 @@ class EncryptionSetupTestCase(iotests.QMPTestCase): =20 # close the encrypted block device def closeImageQmp(self, vm, id): - result =3D vm.qmp('blockdev-del', **{ 'node-name': id }) - self.assert_qmp(result, 'return', {}) + vm.cmd('blockdev-del', **{ 'node-name': id }) =20 ######################################################################= ##### =20 diff --git a/tests/qemu-iotests/298 b/tests/qemu-iotests/298 index fae72211b1..c43c6586d3 100755 --- a/tests/qemu-iotests/298 +++ b/tests/qemu-iotests/298 @@ -80,25 +80,23 @@ class TestPreallocateFilter(TestPreallocateBase): def test_external_snapshot(self): self.test_prealloc() =20 - result =3D self.vm.qmp('blockdev-snapshot-sync', node_name=3D'disk= ', - snapshot_file=3Doverlay, - snapshot_node_name=3D'overlay') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-snapshot-sync', node_name=3D'disk', + snapshot_file=3Doverlay, + snapshot_node_name=3D'overlay') =20 # on reopen to r-o base preallocation should be dropped self.check_small() =20 self.vm.hmp_qemu_io('drive0', 'write 1M 1M') =20 - result =3D self.vm.qmp('block-commit', device=3D'overlay') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-commit', device=3D'overlay') self.complete_and_wait() =20 # commit of new megabyte should trigger preallocation self.check_big() =20 def test_reopen_opts(self): - result =3D self.vm.qmp('blockdev-reopen', options=3D[{ + self.vm.cmd('blockdev-reopen', options=3D[{ 'node-name': 'disk', 'driver': iotests.imgfmt, 'file': { @@ -113,7 +111,6 @@ class TestPreallocateFilter(TestPreallocateBase): } } }]) - self.assert_qmp(result, 'return', {}) =20 self.vm.hmp_qemu_io('drive0', 'write 0 1M') self.assertTrue(os.path.getsize(disk) =3D=3D 25 * MiB) diff --git a/tests/qemu-iotests/300 b/tests/qemu-iotests/300 index dbd28384ec..7b479ab01e 100755 --- a/tests/qemu-iotests/300 +++ b/tests/qemu-iotests/300 @@ -50,10 +50,9 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): self.vm_b.add_incoming(f'unix:{mig_sock}') self.vm_b.launch() =20 - result =3D self.vm_a.qmp('block-dirty-bitmap-add', - node=3Dself.src_node_name, - name=3Dself.src_bmap_name) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('block-dirty-bitmap-add', + node=3Dself.src_node_name, + name=3Dself.src_bmap_name) =20 # Dirty some random megabytes for _ in range(9): @@ -69,8 +68,7 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): for name in ('dirty-bitmaps', 'events')] =20 for vm in (self.vm_a, self.vm_b): - result =3D vm.qmp('migrate-set-capabilities', capabilities=3Dc= aps) - self.assert_qmp(result, 'return', {}) + vm.cmd('migrate-set-capabilities', capabilities=3Dcaps) =20 def tearDown(self) -> None: self.vm_a.shutdown() @@ -93,8 +91,7 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): =20 def migrate(self, bitmap_name_valid: bool =3D True, migration_success: bool =3D True) -> None: - result =3D self.vm_a.qmp('migrate', uri=3Df'unix:{mig_sock}') - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('migrate', uri=3Df'unix:{mig_sock}') =20 with iotests.Timeout(5, 'Timeout waiting for migration to complete= '): self.assertEqual(self.vm_a.wait_migration('postmigrate'), @@ -442,10 +439,9 @@ class TestBlockBitmapMappingErrors(TestDirtyBitmapMigr= ation): def test_bitmap_name_too_long(self) -> None: name =3D 'a' * 256 =20 - result =3D self.vm_a.qmp('block-dirty-bitmap-add', - node=3Dself.src_node_name, - name=3Dname) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('block-dirty-bitmap-add', + node=3Dself.src_node_name, + name=3Dname) =20 self.migrate(False, False) =20 @@ -517,22 +513,16 @@ class TestCrossAliasMigration(TestDirtyBitmapMigratio= n): TestDirtyBitmapMigration.setUp(self) =20 # Now create another block device and let both have two bitmaps ea= ch - result =3D self.vm_a.qmp('blockdev-add', - node_name=3D'node-b', driver=3D'null-co') - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('blockdev-add', node_name=3D'node-b', driver=3D'null= -co') =20 - result =3D self.vm_b.qmp('blockdev-add', - node_name=3D'node-a', driver=3D'null-co') - self.assert_qmp(result, 'return', {}) + self.vm_b.cmd('blockdev-add', node_name=3D'node-a', driver=3D'null= -co') =20 bmaps_to_add =3D (('node-a', 'bmap-b'), ('node-b', 'bmap-a'), ('node-b', 'bmap-b')) =20 for (node, bmap) in bmaps_to_add: - result =3D self.vm_a.qmp('block-dirty-bitmap-add', - node=3Dnode, name=3Dbmap) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('block-dirty-bitmap-add', node=3Dnode, name=3Dbm= ap) =20 @staticmethod def cross_mapping() -> BlockBitmapMapping: @@ -611,24 +601,20 @@ class TestAliasTransformMigration(TestDirtyBitmapMigr= ation): TestDirtyBitmapMigration.setUp(self) =20 # Now create another block device and let both have two bitmaps ea= ch - result =3D self.vm_a.qmp('blockdev-add', - node_name=3D'node-b', driver=3D'null-co', - read_zeroes=3DFalse) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('blockdev-add', + node_name=3D'node-b', driver=3D'null-co', + read_zeroes=3DFalse) =20 - result =3D self.vm_b.qmp('blockdev-add', - node_name=3D'node-a', driver=3D'null-co', - read_zeroes=3DFalse) - self.assert_qmp(result, 'return', {}) + self.vm_b.cmd('blockdev-add', + node_name=3D'node-a', driver=3D'null-co', + read_zeroes=3DFalse) =20 bmaps_to_add =3D (('node-a', 'bmap-b'), ('node-b', 'bmap-a'), ('node-b', 'bmap-b')) =20 for (node, bmap) in bmaps_to_add: - result =3D self.vm_a.qmp('block-dirty-bitmap-add', - node=3Dnode, name=3Dbmap) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('block-dirty-bitmap-add', node=3Dnode, name=3Dbm= ap) =20 @staticmethod def transform_mapping() -> BlockBitmapMapping: diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py index 87eb5f542f..e4777dcd9f 100644 --- a/tests/qemu-iotests/iotests.py +++ b/tests/qemu-iotests/iotests.py @@ -1232,8 +1232,7 @@ def assert_json_filename_equal(self, json_filename, r= eference): def cancel_and_wait(self, drive=3D'drive0', force=3DFalse, resume=3DFalse, wait=3D60.0): '''Cancel a block job and wait for it to finish, returning the eve= nt''' - result =3D self.vm.qmp('block-job-cancel', device=3Ddrive, force= =3Dforce) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-cancel', device=3Ddrive, force=3Dforce) =20 if resume: self.vm.resume_drive(drive) @@ -1295,8 +1294,7 @@ def complete_and_wait(self, drive=3D'drive0', wait_re= ady=3DTrue, if wait_ready: self.wait_ready(drive=3Ddrive) =20 - result =3D self.vm.qmp('block-job-complete', device=3Ddrive) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-complete', device=3Ddrive) =20 event =3D self.wait_until_completed(drive=3Ddrive, error=3Dcomplet= ion_error) self.assertTrue(event['data']['type'] in ['mirror', 'commit']) diff --git a/tests/qemu-iotests/tests/migrate-bitmaps-postcopy-test b/tests= /qemu-iotests/tests/migrate-bitmaps-postcopy-test index fc9c4b4ef4..2c33420151 100755 --- a/tests/qemu-iotests/tests/migrate-bitmaps-postcopy-test +++ b/tests/qemu-iotests/tests/migrate-bitmaps-postcopy-test @@ -118,11 +118,10 @@ class TestDirtyBitmapPostcopyMigration(iotests.QMPTes= tCase): def start_postcopy(self): """ Run migration until RESUME event on target. Return this event.= """ for i in range(nb_bitmaps): - result =3D self.vm_a.qmp('block-dirty-bitmap-add', node=3D'dri= ve0', - name=3D'bitmap{}'.format(i), - granularity=3Dgranularity, - persistent=3DTrue) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('block-dirty-bitmap-add', node=3D'drive0', + name=3D'bitmap{}'.format(i), + granularity=3Dgranularity, + persistent=3DTrue) =20 result =3D self.vm_a.qmp('x-debug-block-dirty-bitmap-sha256', node=3D'drive0', name=3D'bitmap0') @@ -140,9 +139,8 @@ class TestDirtyBitmapPostcopyMigration(iotests.QMPTestC= ase): # We want to calculate resulting sha256. Do it in bitmap0, so, dis= able # other bitmaps for i in range(1, nb_bitmaps): - result =3D self.vm_a.qmp('block-dirty-bitmap-disable', node=3D= 'drive0', - name=3D'bitmap{}'.format(i)) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('block-dirty-bitmap-disable', node=3D'drive0', + name=3D'bitmap{}'.format(i)) =20 apply_discards(self.vm_a, discards2) =20 @@ -152,24 +150,19 @@ class TestDirtyBitmapPostcopyMigration(iotests.QMPTes= tCase): =20 # Now, enable some bitmaps, to be updated during migration for i in range(2, nb_bitmaps, 2): - result =3D self.vm_a.qmp('block-dirty-bitmap-enable', node=3D'= drive0', - name=3D'bitmap{}'.format(i)) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('block-dirty-bitmap-enable', node=3D'drive0', + name=3D'bitmap{}'.format(i)) =20 caps =3D [{'capability': 'dirty-bitmaps', 'state': True}, {'capability': 'events', 'state': True}] =20 - result =3D self.vm_a.qmp('migrate-set-capabilities', capabilities= =3Dcaps) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('migrate-set-capabilities', capabilities=3Dcaps) =20 - result =3D self.vm_b.qmp('migrate-set-capabilities', capabilities= =3Dcaps) - self.assert_qmp(result, 'return', {}) + self.vm_b.cmd('migrate-set-capabilities', capabilities=3Dcaps) =20 - result =3D self.vm_a.qmp('migrate', uri=3D'exec:cat>' + fifo) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('migrate', uri=3D'exec:cat>' + fifo) =20 - result =3D self.vm_a.qmp('migrate-start-postcopy') - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('migrate-start-postcopy') =20 event_resume =3D self.vm_b.event_wait('RESUME') self.vm_b_events.append(event_resume) diff --git a/tests/qemu-iotests/tests/migrate-bitmaps-test b/tests/qemu-iot= ests/tests/migrate-bitmaps-test index 59f3357580..b5b03c0377 100755 --- a/tests/qemu-iotests/tests/migrate-bitmaps-test +++ b/tests/qemu-iotests/tests/migrate-bitmaps-test @@ -67,8 +67,7 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): if persistent: params['persistent'] =3D True =20 - result =3D vm.qmp('block-dirty-bitmap-add', **params) - self.assert_qmp(result, 'return', {}) + vm.cmd('block-dirty-bitmap-add', **params) =20 def check_bitmap(self, vm, sha256): result =3D vm.qmp('x-debug-block-dirty-bitmap-sha256', @@ -91,9 +90,7 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): if migrate_bitmaps: mig_caps.append({'capability': 'dirty-bitmaps', 'state': True}) =20 - result =3D self.vm_a.qmp('migrate-set-capabilities', - capabilities=3Dmig_caps) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('migrate-set-capabilities', capabilities=3Dmig_caps) =20 self.add_bitmap(self.vm_a, granularity, persistent) for r in regions: @@ -114,8 +111,7 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): removed =3D (not migrate_bitmaps) and persistent self.check_bitmap(self.vm_a, False if removed else sha256) =20 - result =3D self.vm_a.qmp('cont') - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('cont') =20 # test that bitmap is still here after invalidation self.check_bitmap(self.vm_a, sha256) @@ -158,9 +154,7 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): if online: os.mkfifo(mig_file) self.vm_b.launch() - result =3D self.vm_b.qmp('migrate-set-capabilities', - capabilities=3Dmig_caps) - self.assert_qmp(result, 'return', {}) + self.vm_b.cmd('migrate-set-capabilities', capabilities=3Dmig_c= aps) =20 self.add_bitmap(self.vm_a, granularity, persistent) for r in regions: @@ -171,9 +165,7 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): self.vm_a.shutdown() self.vm_a.launch() =20 - result =3D self.vm_a.qmp('migrate-set-capabilities', - capabilities=3Dmig_caps) - self.assert_qmp(result, 'return', {}) + self.vm_a.cmd('migrate-set-capabilities', capabilities=3Dmig_caps) =20 result =3D self.vm_a.qmp('migrate', uri=3Dmig_cmd) while True: @@ -184,11 +176,8 @@ class TestDirtyBitmapMigration(iotests.QMPTestCase): if not online: self.vm_a.shutdown() self.vm_b.launch() - result =3D self.vm_b.qmp('migrate-set-capabilities', - capabilities=3Dmig_caps) - self.assert_qmp(result, 'return', {}) - result =3D self.vm_b.qmp('migrate-incoming', uri=3Dincoming_cm= d) - self.assert_qmp(result, 'return', {}) + self.vm_b.cmd('migrate-set-capabilities', capabilities=3Dmig_c= aps) + self.vm_b.cmd('migrate-incoming', uri=3Dincoming_cmd) =20 while True: event =3D self.vm_b.event_wait('MIGRATION') @@ -254,8 +243,7 @@ class TestDirtyBitmapBackingMigration(iotests.QMPTestCa= se): self.vm =3D iotests.VM() self.vm.launch() =20 - result =3D self.vm.qmp('blockdev-add', **blockdev) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-add', **blockdev) =20 # Check that the bitmaps are there nodes =3D self.vm.qmp('query-named-block-nodes', flat=3DTrue)['ret= urn'] @@ -264,8 +252,7 @@ class TestDirtyBitmapBackingMigration(iotests.QMPTestCa= se): self.assert_qmp(node, 'dirty-bitmaps[0]/name', 'bmap0') =20 caps =3D [{'capability': 'events', 'state': True}] - result =3D self.vm.qmp('migrate-set-capabilities', capabilities=3D= caps) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('migrate-set-capabilities', capabilities=3Dcaps) =20 def tearDown(self): self.vm.shutdown() @@ -276,14 +263,12 @@ class TestDirtyBitmapBackingMigration(iotests.QMPTest= Case): """ Continue the source after migration. """ - result =3D self.vm.qmp('migrate', uri=3D'exec: cat > /dev/null') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('migrate', uri=3D'exec: cat > /dev/null') =20 with Timeout(10, 'Migration timeout'): self.vm.wait_migration('postmigrate') =20 - result =3D self.vm.qmp('cont') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('cont') =20 =20 def main() -> None: diff --git a/tests/qemu-iotests/tests/migrate-during-backup b/tests/qemu-io= tests/tests/migrate-during-backup index 34103229ee..0645b8a718 100755 --- a/tests/qemu-iotests/tests/migrate-during-backup +++ b/tests/qemu-iotests/tests/migrate-during-backup @@ -43,7 +43,7 @@ class TestMigrateDuringBackup(iotests.QMPTestCase): =20 self.vm =3D iotests.VM().add_drive(disk_a) self.vm.launch() - result =3D self.vm.qmp('blockdev-add', { + self.vm.cmd('blockdev-add', { 'node-name': 'target', 'driver': iotests.imgfmt, 'file': { @@ -51,26 +51,21 @@ class TestMigrateDuringBackup(iotests.QMPTestCase): 'filename': disk_b } }) - self.assert_qmp(result, 'return', {}) =20 def test_migrate(self): - result =3D self.vm.qmp('blockdev-backup', device=3D'drive0', - target=3D'target', sync=3D'full', - speed=3D1, x_perf=3D{ - 'max-workers': 1, - 'max-chunk': 64 * 1024 - }) - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('job-pause', id=3D'drive0') - self.assert_qmp(result, 'return', {}) - - result =3D self.vm.qmp('migrate-set-capabilities', - capabilities=3D[{'capability': 'events', - 'state': True}]) - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('migrate', uri=3Dmig_cmd) - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-backup', device=3D'drive0', + target=3D'target', sync=3D'full', + speed=3D1, x_perf=3D{ + 'max-workers': 1, + 'max-chunk': 64 * 1024 + }) + + self.vm.cmd('job-pause', id=3D'drive0') + + self.vm.cmd('migrate-set-capabilities', + capabilities=3D[{'capability': 'events', + 'state': True}]) + self.vm.cmd('migrate', uri=3Dmig_cmd) =20 e =3D self.vm.events_wait((('MIGRATION', {'data': {'status': 'completed'}}), @@ -80,11 +75,8 @@ class TestMigrateDuringBackup(iotests.QMPTestCase): # Don't assert that e is 'failed' now: this way we'll miss # possible crash when backup continues :) =20 - result =3D self.vm.qmp('block-job-set-speed', device=3D'drive0', - speed=3D0) - self.assert_qmp(result, 'return', {}) - result =3D self.vm.qmp('job-resume', id=3D'drive0') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-set-speed', device=3D'drive0', speed=3D0) + self.vm.cmd('job-resume', id=3D'drive0') =20 # For future: if something changes so that both migration # and backup pass, let's not miss that moment, as it may diff --git a/tests/qemu-iotests/tests/migration-permissions b/tests/qemu-io= tests/tests/migration-permissions index 6be02581c7..04212e8cc8 100755 --- a/tests/qemu-iotests/tests/migration-permissions +++ b/tests/qemu-iotests/tests/migration-permissions @@ -45,11 +45,10 @@ class TestMigrationPermissions(iotests.QMPTestCase): =20 vms[i].launch() =20 - result =3D vms[i].qmp('migrate-set-capabilities', - capabilities=3D[ - {'capability': 'events', 'state': True} - ]) - self.assert_qmp(result, 'return', {}) + vms[i].cmd('migrate-set-capabilities', + capabilities=3D[ + {'capability': 'events', 'state': True} + ]) =20 self.vm_s =3D vms[0] self.vm_d =3D vms[1] diff --git a/tests/qemu-iotests/tests/mirror-top-perms b/tests/qemu-iotests= /tests/mirror-top-perms index 8bca592708..0e41504163 100755 --- a/tests/qemu-iotests/tests/mirror-top-perms +++ b/tests/qemu-iotests/tests/mirror-top-perms @@ -78,12 +78,11 @@ class TestMirrorTopPerms(iotests.QMPTestCase): difficult to let some other qemu process open the image.) """ =20 - result =3D self.vm.qmp('blockdev-mirror', - job_id=3D'mirror', - device=3D'drive0', - target=3D'null', - sync=3D'full') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('blockdev-mirror', + job_id=3D'mirror', + device=3D'drive0', + target=3D'null', + sync=3D'full') =20 self.vm.event_wait('BLOCK_JOB_READY') =20 @@ -105,9 +104,7 @@ class TestMirrorTopPerms(iotests.QMPTestCase): except machine.VMLaunchFailure as exc: assert 'Is another process using the image' in exc.output =20 - result =3D self.vm.qmp('block-job-cancel', - device=3D'mirror') - self.assert_qmp(result, 'return', {}) + self.vm.cmd('block-job-cancel', device=3D'mirror') =20 self.vm.event_wait('BLOCK_JOB_COMPLETED') =20 --=20 2.35.1