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