glusterfs.git/xlators/mgmt/glusterd/src/Makefile.am, branch v3.8rc2

Build fix: remove undefined -I${rpclibdir}

2016-04-19T03:14:51+00:00

The variable is not defined anywhere, remove it.

Change-Id: Iaefb349cceb4108ac22c44cd32e5ea3d3c8bc0e5
Signed-off-by: Emmanuel Dreyfus 
BUG: 1129939
Reviewed-on: http://review.gluster.org/13867
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Vijay Bellur

build: export minimum symbols from xlators for correct resolution

2015-12-22T17:15:01+00:00

Revisiting http://review.gluster.org/#/c/11814/, which unintentionally
introduced warnings from libtool about the xlator .so names.

According to [1], the -module option must appear in the Makefile.am
file(s); if -module is defined in a macro, e.g. in configure(.ac),
then libtool will not recognize that this is a module and will emit a
warning.

[1]
http://www.gnu.org/software/automake/manual/automake.html#Libtool-Modules

Change-Id: Ifa5f9327d18d139597791c305aa10cc4410fb078
BUG: 1248669
Signed-off-by: Kaleb S KEITHLEY 
Reviewed-on: http://review.gluster.org/13003
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: soumya k 
Reviewed-by: Niels de Vos

build: MKDIR_P is not defined for Makefiles, use mkdir_p instead

2015-11-10T08:57:44+00:00

Change-Id: Id6d5263eb7b1c53e72a7668e716e9cc4e34b82cd
Reported-by: Milind Changire 
BUG: 1198849
Signed-off-by: Niels de Vos 
Reviewed-on: http://review.gluster.org/12553
Tested-by: NetBSD Build System 
Reviewed-by: Milind Changire 
Tested-by: Gluster Build System

build: export minimum symbols from xlators for correct resolution

2015-09-24T14:37:42+00:00

We've been lucky that we haven't had any symbol collisions until now.
Now we have a collision between the snapview-client's svc_lookup() and
libntirpc's svc_lookup() with nfs-ganesha's FSAL_GLUSTER and libgfapi.

As a short term solution all the snapview-client's FOP methods were
changed to static scope. See http://review.gluster.org/11805. This
works in snapview-client because all the FOP methods are defined in
a single source file. This solution doesn't work for other xlators
with FOP methods defined in multiple source files.

To address this we link with libtool's '-export-symbols $symbol-file'
(a wrapper around `ld --version-script ...` --- on linux anyway) and
only export the minimum required symbols from the xlator sharedlib.

N.B. the libtool man page says that the symbol file should be named
foo.sym, thus the rename of *.exports to *.sym. While foo.exports
worked, we will follow the documentation.

Signed-off-by: Kaleb S. KEITHLEY 
BUG: 1248669
Change-Id: I1de68b3e3be58ae690d8bfb2168bfc019983627c
Reviewed-on: http://review.gluster.org/11814
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: soumya k 
Reviewed-by: Niels de Vos

build: xlators/mgmt/glusterd/src/glusterd-errno.h is not in dist tarball

2015-05-30T03:30:39+00:00

attempts to build RPMs, e.g. in tree with `make -C extras/LinuxRPM
glusterrpms` or from a dist tarball produced in tree will fail with:

glusterd-locks.c:19:28: fatal error: glusterd-errno.h: No such file
or directory compilation terminated

Change-Id: I365dbcd5b7bc3410f4a7752a05f570e4061461d0
BUG: 1226384
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/11007
Tested-by: Gluster Build System 
Reviewed-by: Shyamsundar Ranganathan

Fix case mistake for MKDIR_P in Makefiles

2015-04-20T11:29:26+00:00

Some makefiles used $(mkdir_p) instead of the corectly defined $(MKDIR_P).
The former is substituted as an empty string, leading to possible failures
depending of the user shell tolerance. NetBSD's /bin/sh seems to choke
more easily than Linux's /bin/bash, but if the later does not fail, it
does not created the intended directories anyway.

BUG: 1129939
Change-Id: I8caed4000f3c91cb3a685453848fb854793945ed
Signed-off-by: Emmanuel Dreyfus 
Reviewed-on: http://review.gluster.org/10276
Tested-by: NetBSD Build System
Tested-by: Gluster Build System 
Reviewed-by: Niels de Vos

build: make contrib/uuid dependency optional

2015-04-10T11:39:52+00:00

On Linux systems we should use the libuuid from the distribution and not
bundle and statically link the contrib/uuid/ bits.

libglusterfs/src/compat-uuid.h has been introduced and should become an
abstraction layer for different UUID APIs. Non-Linux operating systems
should implement their compatibility layer there.

Once all operating systems have an implementation in compat-uuid.h, we
can remove contrib/uuid/ from the repository completely.

Change-Id: I345e5357644be2521685e00358bb8c83c4ea0577
BUG: 1206587
Signed-off-by: Niels de Vos 
Reviewed-on: http://review.gluster.org/10129
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

glusterd: group server-quorum related code together

2015-04-01T13:07:14+00:00

Server-quorum implementation was spread in many files.  This patch
brings them all together into a single file, namely
glusterd-server-quorum.c. All exported functions are available via
glusterd-server-quorum.h

Change-Id: I8fd77114b5bc6b05127cb8a6a641e0295f0be7bb
BUG: 1205592
Signed-off-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/9492
Reviewed-by: Atin Mukherjee 
Tested-by: Gluster Build System 
Reviewed-by: Kaushal M

cli/glusterd: CLI interface for BitRot detection

2015-03-24T17:26:15+00:00

* BitRot enable/disable CLI per volume
* Volfile generation for Scrubber
* Relevant glusterd infrastructure

Change-Id: I1212af63f93ecc52b22ee6da920e1664f66a1e39
BUG: 1170075
Original-Author: Raghavendra Bhat 
Original-Author: Venky Shankar 
Original-Author: Gaurav Kumar Garg 
Original-Author: Anand Nekkunti 
Reviewed-on: http://review.gluster.org/9986
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

mgmt/glusterd: generate volfile for BitD

2015-03-24T14:56:32+00:00

* Implement the skeleton of bit-rot xlator.

Original-Author: Raghavendra Bhat 
Signed-off-by: Venky Shankar 
Signed-off-by: Gaurav Kumar Garg 
Signed-off-by: Anand Nekkunti 
Change-Id: If33218bdc694f5f09cb7b8097c4fdb74d7a23b2d
BUG: 1170075
Reviewed-on: http://review.gluster.org/9710
Reviewed-by: Vijay Bellur 
Tested-by: Vijay Bellur