Hello community,
here is the log from the commit of package vhostmd for openSUSE:Factory checked
in at 2019-10-10 11:53:12
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/vhostmd (Old)
and /work/SRC/openSUSE:Factory/.vhostmd.new.2352 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "vhostmd"
Thu Oct 10 11:53:12 2019 rev:29 rq:736725 version:1.1
Changes:
--------
--- /work/SRC/openSUSE:Factory/vhostmd/vhostmd.changes 2019-04-14
12:23:19.659825625 +0200
+++ /work/SRC/openSUSE:Factory/.vhostmd.new.2352/vhostmd.changes
2019-10-10 11:53:13.495356324 +0200
@@ -1,0 +2,7 @@
+Wed Oct 9 17:04:14 UTC 2019 - James Fehlig <jfeh...@suse.com>
+
+- Relax virtio requirement in config file
+ relax-virtio-config-requirement.patch
+ bsc#1152803
+
+-------------------------------------------------------------------
New:
----
relax-virtio-config-requirement.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ vhostmd.spec ++++++
--- /var/tmp/diff_new_pack.O37r64/_old 2019-10-10 11:53:14.347354063 +0200
+++ /var/tmp/diff_new_pack.O37r64/_new 2019-10-10 11:53:14.351354052 +0200
@@ -36,6 +36,7 @@
Patch1: manpage.patch
Patch2: value-newline.patch
Patch3: libmetrics-link.patch
+Patch4: relax-virtio-config-requirement.patch
BuildRequires: libtool
BuildRequires: libvirt-devel
BuildRequires: libxml2
@@ -87,6 +88,7 @@
%patch1 -p1
%patch2 -p1
%patch3 -p1
+%patch4 -p1
%build
%if ! %{with_xen}
++++++ relax-virtio-config-requirement.patch ++++++
commit 83cc269f6892852be94467cea771b3ad1da8a369
Author: Jim Fehlig <jfeh...@suse.com>
Date: Tue Oct 8 20:56:18 2019 -0600
Relax virtio requirement in config file
When the virtio transport was introduced the schema was changed to
require a <virtio> transport in vhostmd.conf. When updating existing
deployments without a virtio transport specified in vhostmd.conf,
vhostmd fails to start
/usr/sbin/vhostmd -d
/etc/vhostmd/vhostmd.conf:41: element globals: validity error : Element
globals content does not follow the DTD, expecting (disk , virtio ,
update_period , path , transport+), got (disk update_period path transport )
validate_config_file(): Failed to validate :/etc/vhostmd/vhostmd.conf
Config file: /etc/vhostmd/vhostmd.conf, fails DTD validation
Relax the requirement for virtio transport in the schema. With the
introduction of multiple transports perhaps the others shoud be optional
as well, but requiring virtio is clearly a regression.
Signed-off-by: Jim Fehlig <jfeh...@suse.com>
Index: vhostmd-1.1/vhostmd.dtd
===================================================================
--- vhostmd-1.1.orig/vhostmd.dtd
+++ vhostmd-1.1/vhostmd.dtd
@@ -9,7 +9,7 @@ Virtual Host Metrics Daemon (vhostmd). C
-->
<!ELEMENT vhostmd (globals,metrics)>
-<!ELEMENT globals (disk,virtio,update_period,path,transport+)>
+<!ELEMENT globals (disk,virtio*,update_period,path,transport+)>
<!ELEMENT disk (name,path,size)>
<!ELEMENT name (#PCDATA)>
