Change in vdsm[master]: Signs vdsm upgraded version for special treatment if required
mtayer at redhat.com
mtayer at redhat.com
Sun Oct 5 14:46:51 UTC 2014
mooli tayer has posted comments on this change.
Change subject: Signs vdsm upgraded version for special treatment if required
......................................................................
Patch Set 15:
(1 comment)
http://gerrit.ovirt.org/#/c/31561/15/vdsm.spec.in
File vdsm.spec.in:
Line 750: /usr/sbin/usermod -a -G %{cdrom_group} %{qemu_user}
Line 751:
Line 752: # We keep the previous rpm version number in a file for managing upgrade flow
Line 753: # in vdsmd_init_script upgraded_version_check task
Line 754: if [ "$1" -ge 1 ]; then
Jenkins failure:
/var/lib/vdsm/upgraded_version: No such file or directory
I suppose you need:
mkdir -p "%{_localstatedir}/lib/%{vdsm_name}"
?
[1]
http://jenkins.ovirt.org/job/vdsm_master_install-rpm-sanity-el6_created/425/console
16:03:04 Error in PREIN scriptlet in rpm package vdsm-4.16.0-357.gitf27c81d.el6.x86_64
16:03:04 /var/tmp/rpm-tmp.swfkun: line 13: /var/lib/vdsm/upgraded_version: No such file or directory
16:03:04 error: %pre(vdsm-4.16.0-357.gitf27c81d.el6.x86_64) scriptlet failed, exit status 1
16:03:04 error: install: %pre scriptlet failed (2), skipping vdsm-4.16.0-357.gitf27c81d.el6
Line 755: rpm -q %{vdsm_name} > "%{_localstatedir}/lib/%{vdsm_name}/upgraded_version"
Line 756: fi
Line 757:
Line 758: %post
--
To view, visit http://gerrit.ovirt.org/31561
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I412dc9992675acbb9d03dfa645468789cfb6e65e
Gerrit-PatchSet: 15
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list