From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 85DBD13835A for ; Fri, 20 Nov 2020 13:26:15 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id ACE2BE0825; Fri, 20 Nov 2020 13:26:14 +0000 (UTC) Received: from smtp.gentoo.org (dev.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 6839EE069C for ; Fri, 20 Nov 2020 13:26:14 +0000 (UTC) Received: from oystercatcher.gentoo.org (unknown [IPv6:2a01:4f8:202:4333:225:90ff:fed9:fc84]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id B6B2533BE0D for ; Fri, 20 Nov 2020 13:26:12 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 2262E43B for ; Fri, 20 Nov 2020 13:26:11 +0000 (UTC) From: "Aaron Bauman" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Aaron Bauman" Message-ID: <1605878707.453d6512bf17a5698f3540c2563b00a52a312bc8.bman@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: sys-cluster/nova/files/ X-VCS-Repository: repo/gentoo X-VCS-Files: sys-cluster/nova/files/CVE-2020-17376.patch X-VCS-Directories: sys-cluster/nova/files/ X-VCS-Committer: bman X-VCS-Committer-Name: Aaron Bauman X-VCS-Revision: 453d6512bf17a5698f3540c2563b00a52a312bc8 X-VCS-Branch: master Date: Fri, 20 Nov 2020 13:26:11 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: cb5a44b6-535b-4c96-88a4-3b9baded6041 X-Archives-Hash: be76ce3f2c4cdb6d8370f7e70a8d4bd8 commit: 453d6512bf17a5698f3540c2563b00a52a312bc8 Author: Michael Mair-Keimberger gmail com> AuthorDate: Wed Nov 18 18:40:01 2020 +0000 Commit: Aaron Bauman gentoo org> CommitDate: Fri Nov 20 13:25:07 2020 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=453d6512 sys-cluster/nova: remove unused patch Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Michael Mair-Keimberger gmail.com> Closes: https://github.com/gentoo/gentoo/pull/18318 Signed-off-by: Aaron Bauman gentoo.org> sys-cluster/nova/files/CVE-2020-17376.patch | 141 ---------------------------- 1 file changed, 141 deletions(-) diff --git a/sys-cluster/nova/files/CVE-2020-17376.patch b/sys-cluster/nova/files/CVE-2020-17376.patch deleted file mode 100644 index 8cb2d4423f7..00000000000 --- a/sys-cluster/nova/files/CVE-2020-17376.patch +++ /dev/null @@ -1,141 +0,0 @@ -From bbf9d1de06e9991acd968fceee899a8df3776d60 Mon Sep 17 00:00:00 2001 -From: Lee Yarwood -Date: Wed, 5 Aug 2020 23:00:06 +0100 -Subject: [PATCH] libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live - migration - -The VIR_MIGRATE_PARAM_PERSIST_XML parameter was introduced in libvirt -v1.3.4 and is used to provide the new persistent configuration for the -destination during a live migration: - -https://libvirt.org/html/libvirt-libvirt-domain.html#VIR_MIGRATE_PARAM_PERSIST_XML - -Without this parameter the persistent configuration on the destination -will be the same as the original persistent configuration on the source -when the VIR_MIGRATE_PERSIST_DEST flag is provided. - -As Nova does not currently provide the VIR_MIGRATE_PARAM_PERSIST_XML -param but does provide the VIR_MIGRATE_PERSIST_DEST flag this means that -a soft reboot by Nova of the instance after a live migration can revert -the domain back to the original persistent configuration from the -source. - -Note that this is only possible in Nova as a soft reboot actually -results in the virDomainShutdown and virDomainLaunch libvirt APIs being -called that recreate the domain using the persistent configuration. -virDomainReboot does not result in this but is not called at this time. - -The impact of this on the instance after the soft reboot is pretty -severe, host devices referenced in the original persistent configuration -on the source may not exist or could even be used by other users on the -destination. CPU and NUMA affinity could also differ drastically between -the two hosts resulting in the instance being unable to start etc. - -As MIN_LIBVIRT_VERSION is now > v1.3.4 this change simply includes the -VIR_MIGRATE_PARAM_PERSIST_XML param using the same updated XML for the -destination as is already provided to VIR_MIGRATE_PARAM_DEST_XML. - -NOTE(lyarwood): A simple change to test_migrate_v3_unicode is included -as Iccce0ab50eee515e533ab36c8e7adc10cb3f7019 had removed this from -master. - -Co-authored-by: Tadayoshi Hosoya -Closes-Bug: #1890501 -Change-Id: Ia3f1d8e83cbc574ce5cb440032e12bbcb1e10e98 -(cherry picked from commit 1bb8ee95d4c3ddc3f607ac57526b75af1b7fbcff) -Signed-off-by: Matthew Thode ---- - nova/tests/unit/virt/libvirt/test_driver.py | 8 +++++++- - nova/tests/unit/virt/libvirt/test_guest.py | 2 ++ - nova/virt/libvirt/guest.py | 1 + - 3 files changed, 10 insertions(+), 1 deletion(-) - -diff --git a/nova/tests/unit/virt/libvirt/test_driver.py b/nova/tests/unit/virt/libvirt/test_driver.py -index b416641d362..99ce85a870b 100644 ---- a/nova/tests/unit/virt/libvirt/test_driver.py -+++ b/nova/tests/unit/virt/libvirt/test_driver.py -@@ -11196,6 +11196,7 @@ class LibvirtConnTestCase(test.NoDBTestCase, - 'migrate_disks': disk_paths, - 'bandwidth': _bandwidth, - 'destination_xml': target_xml, -+ 'persistent_xml': target_xml, - } - - # start test -@@ -11303,7 +11304,8 @@ class LibvirtConnTestCase(test.NoDBTestCase, - 'migrate_disks': disk_paths, - 'migrate_uri': 'tcp://127.0.0.2', - 'bandwidth': CONF.libvirt.live_migration_bandwidth, -- 'destination_xml': target_xml -+ 'destination_xml': target_xml, -+ 'persistent_xml': target_xml, - } - - # Start test -@@ -11462,6 +11464,7 @@ class LibvirtConnTestCase(test.NoDBTestCase, - 'migrate_uri': 'tcp://127.0.0.2', - 'bandwidth': CONF.libvirt.live_migration_bandwidth, - 'destination_xml': target_xml, -+ 'persistent_xml': target_xml, - } - - # start test -@@ -11813,6 +11816,7 @@ class LibvirtConnTestCase(test.NoDBTestCase, - 'migrate_disks': ['vda', 'vdb'], - 'bandwidth': CONF.libvirt.live_migration_bandwidth, - 'destination_xml': target_xml, -+ 'persistent_xml': target_xml, - } - - # start test -@@ -11939,6 +11943,7 @@ class LibvirtConnTestCase(test.NoDBTestCase, - 'migrate_disks': device_names, - 'bandwidth': CONF.libvirt.live_migration_bandwidth, - 'destination_xml': '', -+ 'persistent_xml': '', - } - if not params['migrate_disks']: - del params['migrate_disks'] -@@ -12078,6 +12083,7 @@ class LibvirtConnTestCase(test.NoDBTestCase, - 'migrate_disks': disk_paths, - 'bandwidth': CONF.libvirt.live_migration_bandwidth, - 'destination_xml': '', -+ 'persistent_xml': '', - } - - # Prepare mocks -diff --git a/nova/tests/unit/virt/libvirt/test_guest.py b/nova/tests/unit/virt/libvirt/test_guest.py -index 55642c66f66..51899b730b3 100644 ---- a/nova/tests/unit/virt/libvirt/test_guest.py -+++ b/nova/tests/unit/virt/libvirt/test_guest.py -@@ -682,6 +682,7 @@ class GuestTestCase(test.NoDBTestCase): - 'an-uri', flags=1, params={'migrate_uri': 'dest-uri', - 'migrate_disks': 'disk1', - 'destination_xml': '', -+ 'persistent_xml': '', - 'bandwidth': 2}) - - @testtools.skipIf(not six.PY2, 'libvirt python3 bindings accept unicode') -@@ -699,6 +700,7 @@ class GuestTestCase(test.NoDBTestCase): - 'migrate_disks': ['disk1', - 'disk2'], - 'destination_xml': expect_dest_xml, -+ 'persistent_xml': expect_dest_xml, - 'bandwidth': 2}) - - def test_abort_job(self): -diff --git a/nova/virt/libvirt/guest.py b/nova/virt/libvirt/guest.py -index 0d485eb86d9..46593247303 100644 ---- a/nova/virt/libvirt/guest.py -+++ b/nova/virt/libvirt/guest.py -@@ -638,6 +638,7 @@ class Guest(object): - - if destination_xml: - params['destination_xml'] = destination_xml -+ params['persistent_xml'] = destination_xml - if migrate_disks: - params['migrate_disks'] = migrate_disks - if migrate_uri: --- -2.26.2 -