Bug 1174765

Summary:	Hook scripts are not installed after make install
Product:	[Community] GlusterFS	Reporter:	Niels de Vos <ndevos>
Component:	build	Assignee:	Niels de Vos <ndevos>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	bugs, kkeithle, lmohanty
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	glusterfs-3.8rc2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1171477
Clones:	1302955 (view as bug list)		Environment:
Last Closed:	2016-06-16 12:40:24 UTC	Type:	Bug
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:	1171477, 1293708, 1302955

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