d1681e
From 1ce0b65090c888b0e2b28cab03731674f4988aeb Mon Sep 17 00:00:00 2001
d1681e
From: Milind Changire <mchangir@redhat.com>
d1681e
Date: Tue, 10 Oct 2017 09:58:24 +0530
d1681e
Subject: [PATCH 33/74] build: launch glusterd upgrade after all new bits are
d1681e
 installed
d1681e
d1681e
Problem:
d1681e
glusterd upgrade mode needs new bits from glusterfs-rdma which
d1681e
optional and causes the dependency graph to break since it is
d1681e
not tied into glusterfs-server requirements
d1681e
d1681e
Solution:
d1681e
Run glusterd upgrade mode after all new bits are installed
d1681e
i.e. in %posttrans server section
d1681e
d1681e
Label: DOWNSTREAM ONLY
d1681e
d1681e
Change-Id: I356e02d0bf0eaaef43c20ce07b388262f63093a4
d1681e
Signed-off-by: Milind Changire <mchangir@redhat.com>
d1681e
Reviewed-on: https://code.engineering.redhat.com/gerrit/120094
d1681e
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
d1681e
Tested-by: RHGS Build Bot <nigelb@redhat.com>
d1681e
Reviewed-by: Raghavendra Talur <rtalur@redhat.com>
d1681e
---
d1681e
 glusterfs.spec.in | 56 ++++++++++++++++++++++++++++++++++---------------------
d1681e
 1 file changed, 35 insertions(+), 21 deletions(-)
d1681e
d1681e
diff --git a/glusterfs.spec.in b/glusterfs.spec.in
d1681e
index 600fa6e..f4386de 100644
d1681e
--- a/glusterfs.spec.in
d1681e
+++ b/glusterfs.spec.in
d1681e
@@ -963,27 +963,6 @@ fi
d1681e
     %firewalld_reload
d1681e
 %endif
d1681e
 
d1681e
-pidof -c -o %PPID -x glusterd &> /dev/null
d1681e
-if [ $? -eq 0 ]; then
d1681e
-    kill -9 `pgrep -f gsyncd.py` &> /dev/null
d1681e
-
d1681e
-    killall --wait glusterd &> /dev/null
d1681e
-    glusterd --xlator-option *.upgrade=on -N
d1681e
-
d1681e
-    #Cleaning leftover glusterd socket file which is created by glusterd in
d1681e
-    #rpm_script_t context.
d1681e
-    rm -f %{_rundir}/glusterd.socket
d1681e
-
d1681e
-    # glusterd _was_ running, we killed it, it exited after *.upgrade=on,
d1681e
-    # so start it again
d1681e
-    %_init_start glusterd
d1681e
-else
d1681e
-    glusterd --xlator-option *.upgrade=on -N
d1681e
-
d1681e
-    #Cleaning leftover glusterd socket file which is created by glusterd in
d1681e
-    #rpm_script_t context.
d1681e
-    rm -f %{_rundir}/glusterd.socket
d1681e
-fi
d1681e
 %endif
d1681e
 
d1681e
 ##-----------------------------------------------------------------------------
d1681e
@@ -2166,6 +2145,35 @@ os.remove(tmpname)
d1681e
 if not (ok == 0) then
d1681e
    error("Detected running glusterfs processes", ok)
d1681e
 end
d1681e
+
d1681e
+%posttrans server
d1681e
+pidof -c -o %PPID -x glusterd &> /dev/null
d1681e
+if [ $? -eq 0 ]; then
d1681e
+    kill -9 `pgrep -f gsyncd.py` &> /dev/null
d1681e
+
d1681e
+    killall --wait -SIGTERM glusterd &> /dev/null
d1681e
+
d1681e
+    if [ "$?" != "0" ]; then
d1681e
+        echo "killall failed while killing glusterd"
d1681e
+    fi
d1681e
+
d1681e
+    glusterd --xlator-option *.upgrade=on -N
d1681e
+
d1681e
+    #Cleaning leftover glusterd socket file which is created by glusterd in
d1681e
+    #rpm_script_t context.
d1681e
+    rm -rf /var/run/glusterd.socket
d1681e
+
d1681e
+    # glusterd _was_ running, we killed it, it exited after *.upgrade=on,
d1681e
+    # so start it again
d1681e
+    %_init_start glusterd
d1681e
+else
d1681e
+    glusterd --xlator-option *.upgrade=on -N
d1681e
+
d1681e
+    #Cleaning leftover glusterd socket file which is created by glusterd in
d1681e
+    #rpm_script_t context.
d1681e
+    rm -rf /var/run/glusterd.socket
d1681e
+fi
d1681e
+
d1681e
 %endif
d1681e
 
d1681e
 # Events
d1681e
@@ -2190,9 +2198,15 @@ end
d1681e
 %endif
d1681e
 
d1681e
 %changelog
d1681e
+* Tue Oct 10 2017 Milind Changire <mchangir@redhat.com>
d1681e
+- DOWNSTREAM ONLY patch - launch glusterd in upgrade mode after all new bits have been installed
d1681e
+
d1681e
 * Tue Aug 22 2017 Kaleb S. KEITHLEY <kkeithle@redhat.com>
d1681e
 - libibverbs-devel, librdmacm-devel -> rdma-core-devel #1483996
d1681e
 
d1681e
+* Fri Aug 04 2017 Kaleb S. KEITHLEY <kkeithle@rehat.com>
d1681e
+- /var/lib/glusterd/options made config(noreplace) to avoid losing shared state info
d1681e
+
d1681e
 * Thu Jul 20 2017 Aravinda VK <avishwan@redhat.com>
d1681e
 - Added new tool/binary to set the gfid2path xattr on files
d1681e
 
d1681e
-- 
d1681e
1.8.3.1
d1681e