Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package virt-manager for openSUSE:Factory
checked in at 2021-06-25 15:00:31
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/virt-manager (Old)
and /work/SRC/openSUSE:Factory/.virt-manager.new.2625 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "virt-manager"
Fri Jun 25 15:00:31 2021 rev:223 rq:901224 version:3.2.0
Changes:
--------
--- /work/SRC/openSUSE:Factory/virt-manager/virt-manager.changes
2021-04-23 17:49:39.534694446 +0200
+++ /work/SRC/openSUSE:Factory/.virt-manager.new.2625/virt-manager.changes
2021-06-25 15:00:35.700097201 +0200
@@ -1,0 +2,10 @@
+Mon Jun 21 14:30:27 MDT 2021 - carn...@suse.com
+
+- Upstream bug fixes (bsc#1027942)
+ d3c627f1-volumeupload-Use-1MiB-read-size.patch
+ cf93e2db-console-fix-error-with-old-pygobject.patch
+ 143c6bef-virtinst-fix-error-message-format-string.patch
+ fe8722e7-createnet-Remove-some-unnecessary-max_length-annotations.patch
+ d9b5090e-Fix-forgetting-password-from-keyring.patch
+
+-------------------------------------------------------------------
New:
----
143c6bef-virtinst-fix-error-message-format-string.patch
cf93e2db-console-fix-error-with-old-pygobject.patch
d3c627f1-volumeupload-Use-1MiB-read-size.patch
d9b5090e-Fix-forgetting-password-from-keyring.patch
fe8722e7-createnet-Remove-some-unnecessary-max_length-annotations.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ virt-manager.spec ++++++
--- /var/tmp/diff_new_pack.adoPbj/_old 2021-06-25 15:00:36.628098333 +0200
+++ /var/tmp/diff_new_pack.adoPbj/_new 2021-06-25 15:00:36.632098338 +0200
@@ -34,6 +34,11 @@
# Upstream Patches
Patch1: e7222b50-addstorage-Dont-pass-None-to-widget.set_active.patch
Patch2: 4d0e3232-virtinst-Fix-TOCTOU-in-domain-enumeration.patch
+Patch3: d3c627f1-volumeupload-Use-1MiB-read-size.patch
+Patch4: cf93e2db-console-fix-error-with-old-pygobject.patch
+Patch5: 143c6bef-virtinst-fix-error-message-format-string.patch
+Patch6:
fe8722e7-createnet-Remove-some-unnecessary-max_length-annotations.patch
+Patch7: d9b5090e-Fix-forgetting-password-from-keyring.patch
# SUSE Only
Patch70: virtman-desktop.patch
Patch71: virtman-kvm.patch
@@ -167,6 +172,11 @@
# Upstream Patches
%patch1 -p1
%patch2 -p1
+%patch3 -p1
+%patch4 -p1
+%patch5 -p1
+%patch6 -p1
+%patch7 -p1
# SUSE Only
%patch70 -p1
%patch71 -p1
++++++ 143c6bef-virtinst-fix-error-message-format-string.patch ++++++
Subject: virtinst: fix error message format string
From: Roman Bogorodskiy bogorods...@gmail.com Wed Feb 10 17:54:10 2021 +0400
Date: Wed Feb 10 10:26:23 2021 -0500:
Git: 143c6befc33ee507379fd7eca8cf5e5bd1685799
Fix a regression introduced by commit 71f034d6b where
format string expects kwarg "domain", but "vm" is passed instead.
Reviewed-by: Cole Robinson <crobi...@redhat.com>
Signed-off-by: Roman Bogorodskiy <bogorods...@gmail.com>
diff --git a/virtinst/virtxml.py b/virtinst/virtxml.py
index bd2b4282..0c8da37e 100644
--- a/virtinst/virtxml.py
+++ b/virtinst/virtxml.py
@@ -251,7 +251,7 @@ def start_domain_transient(conn, xmlobj, devs, action,
confirm):
dom = conn.createXML(xmlobj.get_xml())
except libvirt.libvirtError as e:
fail(_("Failed starting domain '%(domain)s': %(error)s") % {
- "vm": xmlobj.name,
+ "domain": xmlobj.name,
"error": e,
})
else:
++++++ cf93e2db-console-fix-error-with-old-pygobject.patch ++++++
Subject: console: fix error with old pygobject
From: Pavel Hrdina phrd...@redhat.com Fri Feb 5 12:15:46 2021 +0100
Date: Fri Feb 5 12:15:46 2021 +0100:
Git: cf93e2dbff28fe05d6d45364c579f923b157beb1
The code doesn't work as expected. From python documentation:
x and y
is the same as
x if not x or y
so in the code if for some reasone `dev` is None the value stored in
`sensitive` will be None as well.
No the code itself works with pygobject >= 3.31.3 where they allowed
None as a valid boolean value, but with older versions it will fail
with this error message:
TypeError: Argument 1 does not allow None as a value
Resolves: https://github.com/virt-manager/virt-manager/issues/226
Signed-off-by: Pavel Hrdina <phrd...@redhat.com>
diff --git a/virtManager/details/console.py b/virtManager/details/console.py
index c4ed478e..18f9ddd9 100644
--- a/virtManager/details/console.py
+++ b/virtManager/details/console.py
@@ -258,7 +258,7 @@ class _ConsoleMenu:
cb = toggled_cb
cbdata = dev
- sensitive = dev and not tooltip
+ sensitive = bool(dev and not tooltip)
active = False
if oldlabel is None and sensitive:
++++++ d3c627f1-volumeupload-Use-1MiB-read-size.patch ++++++
Subject: volumeupload: Use 1MiB read size
From: Cole Robinson crobi...@redhat.com Wed Feb 3 14:02:32 2021 -0500
Date: Wed Feb 3 14:15:57 2021 -0500:
Git: d3c627f189dc107f22d0b614537fd0a8937c65a9
Rather than 1K. This drastically speeds up the volumeupload case
for a local URI for the cost of some higher runtime memory but
I think that's worth it
Fixes: #221
Signed-off-by: Cole Robinson <crobi...@redhat.com>
diff --git a/virtinst/install/volumeupload.py b/virtinst/install/volumeupload.py
index 02634f95..431e848c 100644
--- a/virtinst/install/volumeupload.py
+++ b/virtinst/install/volumeupload.py
@@ -108,8 +108,7 @@ def _upload_file(conn, meter, destpool, src):
meter.start(size=size,
text=_("Transferring %s") % os.path.basename(src))
while True:
- # blocksize = (1024 ** 2)
- blocksize = 1024
+ blocksize = 1024 * 1024 # 1 MiB
data = fileobj.read(blocksize)
if not data:
break
++++++ d9b5090e-Fix-forgetting-password-from-keyring.patch ++++++
Subject: Fix forgetting password from keyring
From: WGH w...@torlan.ru Sun Mar 21 20:44:02 2021 +0300
Date: Tue Apr 6 18:30:08 2021 -0400:
Git: d9b5090e061e9fad9738359a8b1f86f16eac45cf
First, Item.Delete never accepted any arguments, so this code likely
never worked.
Second, Item.Delete might return a Prompt object, which client
is supposed to call if keyring wants to confirm deletion.
diff --git a/virtManager/lib/keyring.py b/virtManager/lib/keyring.py
index 203886dc..c0f50142 100644
--- a/virtManager/lib/keyring.py
+++ b/virtManager/lib/keyring.py
@@ -87,7 +87,12 @@ class vmmKeyring(vmmGObject):
iface = Gio.DBusProxy.new_sync(self._dbus, 0, None,
"org.freedesktop.secrets", path,
"org.freedesktop.Secret.Item", None)
- iface.Delete("(s)", "/")
+ prompt = iface.Delete()
+ if prompt != "/":
+ iface = Gio.DBusProxy.new_sync(self._dbus, 0, None,
+ "org.freedesktop.secrets",
prompt,
+
"org.freedesktop.Secret.Prompt", None)
+ iface.Prompt("(s)", "")
except Exception:
log.exception("Failed to delete keyring secret")
++++++ fe8722e7-createnet-Remove-some-unnecessary-max_length-annotations.patch
++++++
Subject: createnet: Remove some unnecessary max_length annotations
From: Cole Robinson crobi...@redhat.com Tue Apr 6 13:35:15 2021 -0400
Date: Tue Apr 6 13:35:15 2021 -0400:
Git: fe8722e763928980315467ef185a27dbf07d3d15
Fixes: #238
Signed-off-by: Cole Robinson <crobi...@redhat.com>
diff --git a/ui/createnet.ui b/ui/createnet.ui
index a70ec7e2..18a1795f 100644
--- a/ui/createnet.ui
+++ b/ui/createnet.ui
@@ -229,7 +229,6 @@
<object class="GtkEntry" id="net-name">
<property name="visible">True</property>
<property name="can_focus">True</property>
- <property name="max_length">16</property>
<property name="width_chars">25</property>
<child internal-child="accessible">
<object class="AtkObject"
id="net-name-atkobject">
@@ -730,7 +729,6 @@
<object class="GtkEntry"
id="net-domain-name">
<property name="visible">True</property>
<property name="can_focus">True</property>
- <property name="max_length">20</property>
<property
name="invisible_char">???</property>
<property name="width_chars">20</property>
<child internal-child="accessible">
