Change in vdsm[ovirt-3.3]: spec: postun script after upgrade runs with the updated code

ybronhei at redhat.com ybronhei at redhat.com
Wed Jan 15 14:57:54 UTC 2014


Yaniv Bronhaim has uploaded a new change for review.

Change subject: spec: postun script after upgrade runs with the updated code
......................................................................

spec: postun script after upgrade runs with the updated code

During an upgrade we reconfigure libvirt service. This reconfigure process uses
vdsm-tool api. The interface has been changed between vdsm 3.3 to 3.4
and the post upgrade script should work also with the new 3.4 vdsm-tool
api. Regards to this, this patch changes the implementation of the
script to use 'service vdsmd reconfigure' which exists in both versions
and call the right vdsm-tool api.

Change-Id: I429f706d47630135a5488f718b435421d64e47a1
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1040022
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm.spec.in
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/82/23282/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index 9aeb607..3a2032f 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -752,7 +752,10 @@
         supervdsmd_start_required='yes'
     fi
 
-    %{_bindir}/vdsm-tool libvirt-configure >/dev/null 2>&1
+    # this part runs after upgrading vdsm code. To allow reconfigure
+    # with vdsm 3.4 api and 3.3 api we use vdsmd reconfigure verb instead of
+    # using vdsm-tool directly which changed its api between those versions.
+    /sbin/service vdsmd reconfigure >/dev/null 2>&1
 
     if [ "${supervdsmd_start_required}" = 'yes' ]; then
         %{_bindir}/vdsm-tool service-start supervdsmd >/dev/null 2>&1


-- 
To view, visit http://gerrit.ovirt.org/23282
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I429f706d47630135a5488f718b435421d64e47a1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list