diff options
| author | Aravinda VK <avishwan@redhat.com> | 2016-10-18 14:57:47 +0530 |
|---|---|---|
| committer | Aravinda VK <avishwan@redhat.com> | 2016-10-20 22:34:08 -0700 |
| commit | ee4c768dafda992ab6bf3787b4efc8b0f330623e (patch) | |
| tree | 7adca00750786dd93a1b64befd96442d3ed428f7 /glusterfs.spec.in | |
| parent | 8f8e23b0142a21779648802664db5b3d5253e4e0 (diff) | |
eventsapi/packaging: Fix wrong usage of %post
%postun was used for events package instead of %post. eventsd
service should be restarted only after install/upgrade and not
during uninstallation(%postun)
BUG: 1386141
Change-Id: Iae3eab06d02c5f4134b3de09f040123bed053bb8
Signed-off-by: Aravinda VK <avishwan@redhat.com>
Reviewed-on: http://review.gluster.org/15670
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Niels de Vos <ndevos@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Diffstat (limited to 'glusterfs.spec.in')
| -rw-r--r-- | glusterfs.spec.in | 25 |
1 files changed, 18 insertions, 7 deletions
diff --git a/glusterfs.spec.in b/glusterfs.spec.in index e70ec0cdfb3..3753651df4a 100644 --- a/glusterfs.spec.in +++ b/glusterfs.spec.in @@ -111,6 +111,7 @@ # can't seem to make a generic macro that works %global _init_glusterd %{_unitdir}/glusterd.service %global _init_glusterfsd %{_unitdir}/glusterfsd.service +%global _init_glustereventsd %{_unitdir}/glustereventsd.service %else %global _init_enable() /sbin/chkconfig --add %1 ; %global _init_disable() /sbin/chkconfig --del %1 ; @@ -121,6 +122,7 @@ # can't seem to make a generic macro that works %global _init_glusterd %{_sysconfdir}/init.d/glusterd %global _init_glusterfsd %{_sysconfdir}/init.d/glusterfsd +%global _init_glustereventsd %{_sysconfdir}/init.d/glustereventsd %endif %if ( 0%{_for_fedora_koji_builds} ) @@ -775,6 +777,11 @@ exit 0 %post api /sbin/ldconfig +%if ( 0%{!?_without_events:1} ) +%post events +%_init_restart glustereventsd +%endif + %if ( 0%{?rhel} == 5 ) %post fuse modprobe fuse @@ -871,6 +878,17 @@ exit 0 ##----------------------------------------------------------------------------- ## All %%preun should be placed here and keep them sorted ## +%if ( 0%{!?_without_events:1} ) +%preun events +if [ $1 -eq 0 ]; then + if [ -f %_init_glustereventsd ]; then + %_init_stop glustereventsd + %_init_disable glustereventsd + fi +fi +exit 0 +%endif + %preun server if [ $1 -eq 0 ]; then if [ -f %_init_glusterfsd ]; then @@ -904,13 +922,6 @@ exit 0 %postun api /sbin/ldconfig -%postun events -%if ( 0%{!?_without_events:1} ) -%if ( 0%{?fedora} ) || ( 0%{?rhel} && 0%{?rhel} >= 6 ) -%_init_restart glustereventsd -%endif -%endif - %postun libs /sbin/ldconfig |