Bug 766640
Summary: | glusterfs-geo-replication unconditionally restarts glusterd in %post | |||
---|---|---|---|---|
Product: | [Red Hat Storage] Red Hat Gluster Storage | Reporter: | Anthony Towns <atowns> | |
Component: | build | Assignee: | Vijay Bellur <vbellur> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Rejy M Cyriac <rcyriac> | |
Severity: | high | Docs Contact: | ||
Priority: | medium | |||
Version: | 1.0 | CC: | amarts, flanagan, gluster-bugs, grajaiya, ndevos, psharma, rcyriac, rfortier, sdharane, vagarwal, vbellur | |
Target Milestone: | --- | Keywords: | Patch, ZStream | |
Target Release: | --- | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | glusterfs-3.4.0.33rhs-1 | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 858411 (view as bug list) | Environment: | ||
Last Closed: | 2013-12-09 16:45:14 UTC | Type: | --- | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 858411 |
Description
Anthony Towns
2011-12-12 14:01:07 UTC
The .spec contains %post geo-replication #restart glusterd. %{_sysconfdir}/init.d/glusterd restart &> /dev/null This should be replaced with if [ $1 -ge 1 ]; then /sbin/service glusterd condrestart &>/dev/null || : fi This will only restart glusterd on an update ($1 -ge 1) and only when it is running (condrestart). *** Bug 826855 has been marked as a duplicate of this bug. *** Review available at http://review.gluster.org/4063 Tested successfully like this: $ git clone http://review.gluster.org/p/glusterfs $ cd glusterfs $ git pull http://review.gluster.org/p/glusterfs refs/changes/63/4063/1 $ ./autogen.sh && ./configure --enable-fusermount && make dist && rpmbuild -ta glusterfs-3git.tar.gz Install the just build packages, but not glusterfs-geo-replication. Make sure to disable the glusterd service # service glusterd stop # chkconfig glusterd off Install the just built glusterfs-geo-replication. -> Confirm that glusterd is not running CHANGE: http://review.gluster.org/4063 (build: do not unconditionally restart glusterd in glusterfs-geo-replication %post) merged in master by Anand Avati (avati) Assigning to Vijay as the the change needs to be made in the glusterfs.spec in dist-git too. The current change is only in the glusterfs.spec.in which is fine for upstream. *** Bug 889248 has been marked as a duplicate of this bug. *** Verified with glusterfs-geo-replication-3.4.0.33rhs-1.el6rhs that it does not unconditionally start glusterd on install. |