Hello community, here is the log from the commit of package virt-manager for openSUSE:Factory checked in at 2019-12-10 22:46:41 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/virt-manager (Old) and /work/SRC/openSUSE:Factory/.virt-manager.new.4691 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "virt-manager" Tue Dec 10 22:46:41 2019 rev:204 rq:755653 version:2.2.1 Changes: -------- --- /work/SRC/openSUSE:Factory/virt-manager/virt-manager.changes 2019-11-28 10:17:11.659639035 +0100 +++ /work/SRC/openSUSE:Factory/.virt-manager.new.4691/virt-manager.changes 2019-12-10 22:46:59.713707347 +0100 @@ -1,0 +2,20 @@ +Mon Dec 2 14:03:00 MST 2019 - carn...@suse.com + +- bsc#1157144 - [s390][virt-manager] There was the black screen + from guest graphical console during guest installation + virtinst-s390x-disable-graphics.patch +- bsc#1158227 - virt-manager: Fix duplicate entries in the + operating system URL drop down menu + virtman-show-suse-install-repos.patch +- Upstream bug fix (bsc#1027942) + 29f9f5f2-virt-xml-fix-defined_xml_is_unchanged.patch +- Drop virtman-default-to-xen-pv.patch +- Refreshed + virtinst-add-pvh-support.patch + virtinst-modify-gui-defaults.patch + virt-manager.changes + virt-manager.spec + virtman-allow-creating-i686-vm.patch + virtman-python2-to-python3-conversion.patch + +------------------------------------------------------------------- Old: ---- virtman-default-to-xen-pv.patch New: ---- 29f9f5f2-virt-xml-fix-defined_xml_is_unchanged.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ virt-manager.spec ++++++ --- /var/tmp/diff_new_pack.Z8LzDW/_old 2019-12-10 22:47:02.757706218 +0100 +++ /var/tmp/diff_new_pack.Z8LzDW/_new 2019-12-10 22:47:02.761706217 +0100 @@ -12,7 +12,7 @@ # license that conforms to the Open Source Definition (Version 1.9) # published by the Open Source Initiative. -# Please submit bugfixes or comments via http://bugs.opensuse.org/ +# Please submit bugfixes or comments via https://bugs.opensuse.org/ # @@ -46,6 +46,7 @@ Patch12: ae19d6d6-fix-resizing-of-spice-clients-with-guest-resizing-enabled.patch Patch13: 51d84c54-connection-Avoid-repeated-default-pool-creation-attempts.patch Patch14: d934d6f2-domcaps-Fix-check-for-uncached-security-features.patch +Patch15: 29f9f5f2-virt-xml-fix-defined_xml_is_unchanged.patch # SUSE Only Patch70: virtman-desktop.patch Patch71: virtman-kvm.patch @@ -57,7 +58,6 @@ Patch77: virtman-fix-env-script-interpreter.patch Patch78: virtinst-set-qemu-emulator.patch # Features or Enhancements -Patch102: virtman-default-to-xen-pv.patch Patch103: virtman-load-stored-uris.patch Patch120: virtinst-default-xen-to-qcow2-format.patch Patch121: virtinst-detect-oes-distros.patch @@ -192,6 +192,7 @@ %patch12 -p1 %patch13 -p1 %patch14 -p1 +%patch15 -p1 # SUSE Only %patch70 -p1 %patch71 -p1 @@ -203,7 +204,6 @@ %patch77 -p1 %patch78 -p1 # Enhancements -%patch102 -p1 %patch103 -p1 %patch120 -p1 %patch121 -p1 ++++++ 29f9f5f2-virt-xml-fix-defined_xml_is_unchanged.patch ++++++ Subject: virt-xml: fix defined_xml_is_unchanged From: Pavel Hrdina phrd...@redhat.com Tue Dec 3 13:02:21 2019 +0100 Date: Tue Dec 3 13:04:37 2019 +0100: Git: 29f9f5f2d7f47fe6cc3333f2a9e6c6209db5a8f3 Commit <53f075ab76e1c372474ae0d88f202e487d9f213f> added a warning if the VM XML is not changed after removing default devices but the code was incorrect. We have to compare strings instead of string vs Guest object and also the condition was inverted. Signed-off-by: Pavel Hrdina <phrd...@redhat.com> diff --git a/virt-xml b/virt-xml index 71445c9f..7b0174c9 100755 --- a/virt-xml +++ b/virt-xml @@ -105,8 +105,8 @@ def get_domain_and_guest(conn, domstr): def defined_xml_is_unchanged(conn, domain, original_xml): rawxml = get_xmldesc(domain, inactive=True) - new_xml = virtinst.Guest(conn, parsexml=rawxml) - return new_xml != original_xml + new_xml = virtinst.Guest(conn, parsexml=rawxml).get_xml() + return new_xml == original_xml ################ ++++++ virtinst-add-pvh-support.patch ++++++ --- /var/tmp/diff_new_pack.Z8LzDW/_old 2019-12-10 22:47:02.949706147 +0100 +++ /var/tmp/diff_new_pack.Z8LzDW/_new 2019-12-10 22:47:02.949706147 +0100 @@ -5,7 +5,7 @@ =================================================================== --- virt-manager-2.2.1.orig/virtManager/createvm.py +++ virt-manager-2.2.1/virtManager/createvm.py -@@ -796,6 +796,9 @@ class vmmCreateVM(vmmGObjectUI): +@@ -792,6 +792,9 @@ class vmmCreateVM(vmmGObjectUI): for guest in guests: if not guest.domains: continue ++++++ virtinst-modify-gui-defaults.patch ++++++ --- /var/tmp/diff_new_pack.Z8LzDW/_old 2019-12-10 22:47:03.177706063 +0100 +++ /var/tmp/diff_new_pack.Z8LzDW/_new 2019-12-10 22:47:03.201706053 +0100 @@ -1,11 +1,11 @@ Enhancement for the following GUI wizard installation option. 1) If Xen, leave 'Architecture options' expanded so users know PV is the default -Index: virt-manager-2.2.0/virtManager/createvm.py +Index: virt-manager-2.2.1/virtManager/createvm.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/createvm.py -+++ virt-manager-2.2.0/virtManager/createvm.py -@@ -403,8 +403,20 @@ class vmmCreateVM(vmmGObjectUI): +--- virt-manager-2.2.1.orig/virtManager/createvm.py ++++ virt-manager-2.2.1/virtManager/createvm.py +@@ -404,8 +404,20 @@ class vmmCreateVM(vmmGObjectUI): self.widget("method-local").set_active(True) self.widget("create-conn").set_active(-1) activeconn = self._populate_conn_list(urihint) ++++++ virtinst-s390x-disable-graphics.patch ++++++ --- /var/tmp/diff_new_pack.Z8LzDW/_old 2019-12-10 22:47:03.369705991 +0100 +++ /var/tmp/diff_new_pack.Z8LzDW/_new 2019-12-10 22:47:03.369705991 +0100 @@ -34,12 +34,3 @@ return if self.devices.video: return -@@ -931,7 +934,7 @@ class Guest(XMLBuilder): - return - if self.os.is_container() and not self.conn.is_vz(): - return -- if self.os.arch not in ["x86_64", "i686", "ppc64", "ppc64le"]: -+ if self.os.arch not in ["x86_64", "i686", "ppc64", "ppc64le", "s390x"]: - return - self.add_device(DeviceGraphics(self.conn)) - ++++++ virtman-allow-creating-i686-vm.patch ++++++ --- /var/tmp/diff_new_pack.Z8LzDW/_old 2019-12-10 22:47:03.541705927 +0100 +++ /var/tmp/diff_new_pack.Z8LzDW/_new 2019-12-10 22:47:03.541705927 +0100 @@ -1,11 +1,11 @@ References: bsc#919692 Because openSUSE repos combine 32 and 64 bit sources we need to continue showing the 'Architecture' pop-up. -Index: virt-manager-2.2.0/virtManager/createvm.py +Index: virt-manager-2.2.1/virtManager/createvm.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/createvm.py -+++ virt-manager-2.2.0/virtManager/createvm.py -@@ -834,11 +834,6 @@ class vmmCreateVM(vmmGObjectUI): +--- virt-manager-2.2.1.orig/virtManager/createvm.py ++++ virt-manager-2.2.1/virtManager/createvm.py +@@ -830,11 +830,6 @@ class vmmCreateVM(vmmGObjectUI): for guest in self.conn.caps.guests: if guest.os_type == self._capsinfo.os_type: archs.append(guest.arch) ++++++ virtman-python2-to-python3-conversion.patch ++++++ --- /var/tmp/diff_new_pack.Z8LzDW/_old 2019-12-10 22:47:03.737705855 +0100 +++ /var/tmp/diff_new_pack.Z8LzDW/_new 2019-12-10 22:47:03.737705855 +0100 @@ -1,7 +1,7 @@ -Index: virt-manager-2.2.0/virt-manager +Index: virt-manager-2.2.1/virt-manager =================================================================== ---- virt-manager-2.2.0.orig/virt-manager -+++ virt-manager-2.2.0/virt-manager +--- virt-manager-2.2.1.orig/virt-manager ++++ virt-manager-2.2.1/virt-manager @@ -66,7 +66,7 @@ def _import_gtk(leftovers): print("gtk3 3.22.0 or later is required.") sys.exit(1) @@ -11,10 +11,10 @@ # This will error if Gtk wasn't correctly initialized Gtk.Window() else: -Index: virt-manager-2.2.0/virtManager/details/console.py +Index: virt-manager-2.2.1/virtManager/details/console.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/details/console.py -+++ virt-manager-2.2.0/virtManager/details/console.py +--- virt-manager-2.2.1.orig/virtManager/details/console.py ++++ virt-manager-2.2.1/virtManager/details/console.py @@ -21,7 +21,7 @@ from ..vmwindow import DETAILS_PAGE_CONS (_CONSOLE_PAGE_UNAVAILABLE, _CONSOLE_PAGE_AUTHENTICATE, @@ -24,10 +24,10 @@ class _TimedRevealer(vmmGObject): -Index: virt-manager-2.2.0/virtManager/connection.py +Index: virt-manager-2.2.1/virtManager/connection.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/connection.py -+++ virt-manager-2.2.0/virtManager/connection.py +--- virt-manager-2.2.1.orig/virtManager/connection.py ++++ virt-manager-2.2.1/virtManager/connection.py @@ -159,7 +159,7 @@ class vmmConnection(vmmGObject): (_STATE_DISCONNECTED, @@ -37,10 +37,10 @@ def __init__(self, uri): self._uri = uri -Index: virt-manager-2.2.0/virtManager/addhardware.py +Index: virt-manager-2.2.1/virtManager/addhardware.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/addhardware.py -+++ virt-manager-2.2.0/virtManager/addhardware.py +--- virt-manager-2.2.1.orig/virtManager/addhardware.py ++++ virt-manager-2.2.1/virtManager/addhardware.py @@ -44,7 +44,7 @@ from .xmleditor import vmmXMLEditor PAGE_TPM, PAGE_RNG, @@ -50,10 +50,10 @@ def _build_combo(combo, values, default_value=None, sort=True): -Index: virt-manager-2.2.0/virtManager/details/snapshots.py +Index: virt-manager-2.2.1/virtManager/details/snapshots.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/details/snapshots.py -+++ virt-manager-2.2.0/virtManager/details/snapshots.py +--- virt-manager-2.2.1.orig/virtManager/details/snapshots.py ++++ virt-manager-2.2.1/virtManager/details/snapshots.py @@ -55,7 +55,7 @@ def _make_screenshot_pixbuf(mime, sdata) @@ -63,10 +63,10 @@ if val == m and not reverse: return e if val == e and reverse: -Index: virt-manager-2.2.0/virtManager/engine.py +Index: virt-manager-2.2.1/virtManager/engine.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/engine.py -+++ virt-manager-2.2.0/virtManager/engine.py +--- virt-manager-2.2.1.orig/virtManager/engine.py ++++ virt-manager-2.2.1/virtManager/engine.py @@ -22,7 +22,7 @@ from .lib.inspection import vmmInspectio from .systray import vmmSystray @@ -76,10 +76,10 @@ def _show_startup_error(fn): -Index: virt-manager-2.2.0/virtManager/device/mediacombo.py +Index: virt-manager-2.2.1/virtManager/device/mediacombo.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/device/mediacombo.py -+++ virt-manager-2.2.0/virtManager/device/mediacombo.py +--- virt-manager-2.2.1.orig/virtManager/device/mediacombo.py ++++ virt-manager-2.2.1/virtManager/device/mediacombo.py @@ -22,7 +22,7 @@ class vmmMediaCombo(vmmGObjectUI): (MEDIA_FIELD_PATH, MEDIA_FIELD_LABEL, @@ -89,10 +89,10 @@ def __init__(self, conn, builder, topwin): vmmGObjectUI.__init__(self, None, None, builder=builder, topwin=topwin) -Index: virt-manager-2.2.0/virtManager/manager.py +Index: virt-manager-2.2.1/virtManager/manager.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/manager.py -+++ virt-manager-2.2.0/virtManager/manager.py +--- virt-manager-2.2.1.orig/virtManager/manager.py ++++ virt-manager-2.2.1/virtManager/manager.py @@ -33,7 +33,7 @@ ROW_IS_CONN_CONNECTED, ROW_IS_VM, ROW_IS_VM_RUNNING, @@ -111,10 +111,10 @@ def _style_get_prop(widget, propname): -Index: virt-manager-2.2.0/virtManager/device/addstorage.py +Index: virt-manager-2.2.1/virtManager/device/addstorage.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/device/addstorage.py -+++ virt-manager-2.2.0/virtManager/device/addstorage.py +--- virt-manager-2.2.1.orig/virtManager/device/addstorage.py ++++ virt-manager-2.2.1/virtManager/device/addstorage.py @@ -108,7 +108,7 @@ class vmmAddStorage(vmmGObjectUI): errmsg = _("Errors were encountered changing permissions for the " "following directories:") @@ -124,10 +124,10 @@ if p not in broken_paths: continue details += "%s : %s\n" % (p, error) -Index: virt-manager-2.2.0/virtManager/details/details.py +Index: virt-manager-2.2.1/virtManager/details/details.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/details/details.py -+++ virt-manager-2.2.0/virtManager/details/details.py +--- virt-manager-2.2.1.orig/virtManager/details/details.py ++++ virt-manager-2.2.1/virtManager/details/details.py @@ -104,7 +104,7 @@ from ..xmleditor import vmmXMLEditor EDIT_FS, @@ -164,10 +164,10 @@ def _calculate_disk_bus_index(disklist): -Index: virt-manager-2.2.0/virtManager/createvm.py +Index: virt-manager-2.2.1/virtManager/createvm.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/createvm.py -+++ virt-manager-2.2.0/virtManager/createvm.py +--- virt-manager-2.2.1.orig/virtManager/createvm.py ++++ virt-manager-2.2.1/virtManager/createvm.py @@ -40,7 +40,7 @@ DEFAULT_MEM = 1024 PAGE_INSTALL, PAGE_MEM, @@ -193,7 +193,7 @@ ##################### -@@ -2127,7 +2127,7 @@ class vmmCreateVM(vmmGObjectUI): +@@ -2123,7 +2123,7 @@ class vmmCreateVM(vmmGObjectUI): 'insecure': self._get_config_oscontainer_isecure, 'root_password': self._get_config_oscontainer_root_password, } @@ -202,10 +202,10 @@ bootstrap_args[key] = getter() parentobj = self._customize_window or self -Index: virt-manager-2.2.0/virtManager/preferences.py +Index: virt-manager-2.2.1/virtManager/preferences.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/preferences.py -+++ virt-manager-2.2.0/virtManager/preferences.py +--- virt-manager-2.2.1.orig/virtManager/preferences.py ++++ virt-manager-2.2.1/virtManager/preferences.py @@ -129,7 +129,7 @@ class vmmPreferences(vmmGObjectUI): } model.append([-1, _("System default (%s)") % @@ -215,10 +215,10 @@ model.append([key, val]) combo.set_model(model) uiutil.init_combo_text_column(combo, 1) -Index: virt-manager-2.2.0/virtManager/migrate.py +Index: virt-manager-2.2.1/virtManager/migrate.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/migrate.py -+++ virt-manager-2.2.0/virtManager/migrate.py +--- virt-manager-2.2.1.orig/virtManager/migrate.py ++++ virt-manager-2.2.1/virtManager/migrate.py @@ -23,7 +23,7 @@ from .object.domain import vmmDomain NUM_COLS = 3 (COL_LABEL, @@ -228,10 +228,10 @@ class vmmMigrateDialog(vmmGObjectUI): -Index: virt-manager-2.2.0/virtManager/hoststorage.py +Index: virt-manager-2.2.1/virtManager/hoststorage.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/hoststorage.py -+++ virt-manager-2.2.0/virtManager/hoststorage.py +--- virt-manager-2.2.1.orig/virtManager/hoststorage.py ++++ virt-manager-2.2.1/virtManager/hoststorage.py @@ -32,13 +32,13 @@ VOL_NUM_COLUMNS = 7 VOL_COLUMN_SIZESTR, VOL_COLUMN_FORMAT, ++++++ virtman-show-suse-install-repos.patch ++++++ --- /var/tmp/diff_new_pack.Z8LzDW/_old 2019-12-10 22:47:03.765705844 +0100 +++ /var/tmp/diff_new_pack.Z8LzDW/_new 2019-12-10 22:47:03.765705844 +0100 @@ -1,11 +1,11 @@ Enhancement that gets installation repos from zypper. These locations are then presented as potential installation sources when creating a VM. -Index: virt-manager-2.2.0/virtManager/createvm.py +Index: virt-manager-2.2.1/virtManager/createvm.py =================================================================== ---- virt-manager-2.2.0.orig/virtManager/createvm.py -+++ virt-manager-2.2.0/virtManager/createvm.py -@@ -99,6 +99,69 @@ def is_virt_bootstrap_installed(): +--- virt-manager-2.2.1.orig/virtManager/createvm.py ++++ virt-manager-2.2.1/virtManager/createvm.py +@@ -99,6 +99,70 @@ def is_virt_bootstrap_installed(): return pkgutil.find_loader('virtBootstrap') is not None @@ -56,10 +56,11 @@ + if len(repo) >= uri_index: + str = repo[uri_index] + if str.startswith('ftp://') or str.startswith('http://') or str.startswith('nfs://') or str.startswith('smb://'): -+ zypper_output.append(str) -+ if dom0_inst_source is not None and str == dom0_inst_source: -+ index_dom0 = number_of_sources -+ number_of_sources += 1 ++ if str not in zypper_output: ++ zypper_output.append(str) ++ if dom0_inst_source is not None and str == dom0_inst_source: ++ index_dom0 = number_of_sources ++ number_of_sources += 1 + + if index_dom0 == -1 and dom0_inst_source: + index_dom0 = 0 @@ -75,7 +76,7 @@ ############## # Main class # ############## -@@ -368,7 +431,13 @@ class vmmCreateVM(vmmGObjectUI): +@@ -368,7 +432,13 @@ class vmmCreateVM(vmmGObjectUI): self.widget("install-url-entry").set_text("") self.widget("install-url-options").set_expanded(False) urlmodel = self.widget("install-url-combo").get_model()