Bug 1174765 - Hook scripts are not installed after make install
Summary: Hook scripts are not installed after make install
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: build
Version: mainline
Hardware: All
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Niels de Vos
QA Contact:
URL:
Whiteboard:
: 1293706 1293708 (view as bug list)
Depends On:
Blocks: 1171477 1293708 1302955
TreeView+ depends on / blocked
 
Reported: 2014-12-16 12:49 UTC by Niels de Vos
Modified: 2016-06-16 12:40 UTC (History)
3 users (show)

Fixed In Version: glusterfs-3.8rc2
Doc Type: Bug Fix
Doc Text:
Clone Of: 1171477
: 1302955 (view as bug list)
Environment:
Last Closed: 2016-06-16 12:40:24 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1293706 0 unspecified CLOSED hooks: Ganesha and Samba hooks are always installed 2021-02-22 00:41:40 UTC

Internal Links: 1293706

Description Niels de Vos 2014-12-16 12:49:54 UTC
+++ This bug was initially created as a clone of Bug #1171477 +++

Description of problem:
When GLusterFS is built from source, the hook scripts are not copied over to /var/lib/glusterd/hooks as part of the install make target.

Version-Release number of selected component (if applicable):
3.5.3

How reproducible:
5 times on 5 makes of Gluster on CentOS 6.4

Steps to Reproduce:
1. Download source and build
2. Observe that directories are created in /var/lib/glusterd/hooks, but hook scripts are not copied over

Actual results:
Hook scripts are not present in hooks directories

Expected results:
Hook scripts should be present in hooks directories after running make install

Additional info:
Attaching directory listings for the hooks directory post-install and in the source folder.

--- Additional comment from David Gibbons on 2014-12-07 16:24:06 CET ---



--- Additional comment from Niels de Vos on 2014-12-16 13:49:13 CET ---

The hook scripts are manually installed in the glusterfs.spec. It is much better to install these scripts when "make install" is run. This would make it much more common user experience on all distributions and manual installations.

Comment 1 Vijay Bellur 2015-12-23 13:05:00 UTC
REVIEW: http://review.gluster.org/13072 (build: use 'make install' to install the hook scripts) posted (#1) for review on master by Niels de Vos (ndevos)

Comment 2 Vijay Bellur 2015-12-26 17:31:39 UTC
COMMIT: http://review.gluster.org/13072 committed in master by Kaleb KEITHLEY (kkeithle) 
------
commit 7ab8a1a3f4510a284b1686302624ec4252c95a6f
Author: Niels de Vos <ndevos>
Date:   Wed Dec 23 13:57:14 2015 +0100

    build: use 'make install' to install the hook scripts
    
    The installation should be the same on all distributions, and doing
    manual installation of files in the .spec is very ugly. This change adds
    the rules so that 'make install' places the hook scripts in the right
    location.
    
    Also, the hook script(s) for NFS-Ganesha should be part of the
    glusterfs-ganesha sub-package and got moved there.
    
    BUG: 1174765
    Change-Id: Iba25a7a5112c7d40db4c10ff4a5ac7a5fb4f7c4e
    Signed-off-by: Niels de Vos <ndevos>
    Reviewed-on: http://review.gluster.org/13072
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Kaleb KEITHLEY <kkeithle>

Comment 3 Kaleb KEITHLEY 2015-12-28 12:23:56 UTC
*** Bug 1293706 has been marked as a duplicate of this bug. ***

Comment 4 Kaleb KEITHLEY 2015-12-28 12:25:22 UTC
*** Bug 1293708 has been marked as a duplicate of this bug. ***

Comment 5 Niels de Vos 2016-06-16 12:40:24 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


Note You need to log in before you can comment on or make changes to this bug.