glusterfs.git/extras, branch v3.6.6

Fix case mistake for MKDIR_P in Makefiles

2015-05-19T13:23:59+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.

Backport of I8caed4000f3c91cb3a685453848fb854793945ed

BUG: 1138897
Change-Id: I48a24231ba90aa3e84bf433b63ae5cb50307f042
Signed-off-by: Emmanuel Dreyfus 
Reviewed-on: http://review.gluster.org/10278
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Raghavendra Bhat

Stop logrotate from complaining about missing files.

2014-12-13T06:05:36+00:00

Previously logrotate reported errors for files missing in
"/var/log/glusterfs/geo-replication-slaves/*/*.log". Fixed by reordering
directives in the example config.

Change-Id: I7cdb9e0d82f08aa27d5e26664660204fdeb8e98e
BUG: 1165133
Signed-off-by: Jan-Hendrik Zab 
Reviewed-on: http://review.gluster.org/9239
Reviewed-by: Lalatendu Mohanty 
Tested-by: Gluster Build System 
Reviewed-by: Raghavendra Bhat 
Tested-by: Raghavendra Bhat

build: only install geo-rep scripts when enabled

2014-12-10T14:00:22+00:00

When geo-replication is not enabled, the related scripts should not get
installed. EPEL-5 does not support geo-replication, and the installed
but unpackaged scripts cause rebuilding for .el5 to fail.

Cherry picked from commit fcdac758ad7b3dda9843f92597a15e4a30a3b746:
> BUG: 1169005
> Change-Id: I1b6a352b73bd1ed1c5b330f54e26e81aed9a5a4e
> Reported-by: Lalatendu Mohanty 
> Signed-off-by: Niels de Vos 
> Reviewed-on: http://review.gluster.org/9221
> Reviewed-by: Humble Devassy Chirammal 
> Tested-by: Gluster Build System 

Change-Id: I1b6a352b73bd1ed1c5b330f54e26e81aed9a5a4e
BUG: 1169004
Signed-off-by: Niels de Vos 
Reviewed-on: http://review.gluster.org/9248
Tested-by: Gluster Build System 
Reviewed-by: Humble Devassy Chirammal 
Reviewed-by: Raghavendra Bhat 
Tested-by: Raghavendra Bhat

logrotate: gluster logrotate config should not be global

2014-10-30T08:54:07+00:00

Issue : Previously glusterfs logrotate config file pollutes
global config. So moved the directives inside the curly braces,
 so they don't pollute the global config state.

Change-Id: I8836893dfcdf457d9c5d766612d687bfce64e2ae
BUG: 1158456
"Signed-off-by: Lalatendu Mohanty "
"Reviewed-on: http://review.gluster.org/8994"
"Reviewed-by: Niels de Vos "
"Tested-by: Gluster Build System "
(cherry picked from commit a5d73daabf6df95bc73b186d92f3e2d1239a6f8a)
Signed-off-by: Lalatendu Mohanty 
Reviewed-on: http://review.gluster.org/9000
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

Hooks : Infinite while loop introduced by another change.

2014-10-28T10:21:28+00:00

A change made to all the hook scripts introduced an
infinite while loop in the script S31ganesha-reset.sh.
It resulted in 100% CPU usage by this script.

Change-Id: I8133c92d46616b6534fc88158d6dbfca49875f4a
BUG: 1157374
Reviewed-on: http://review.gluster.org/8974
Reviewed-by: Raghavendra Talur 
Reviewed-by: soumya k 
Reviewed-by: Vijay Bellur 
Tested-by: Vijay Bellur

osx: Remove legacy extras/MacOSX directory

2014-09-18T19:00:18+00:00

Backport of http://review.gluster.org/8736

BUG: 1141683
Change-Id: Ic37df769db856196727d2799396c1dbaf4bcdd1a
Signed-off-by: Justin Clift 
Reviewed-on: http://review.gluster.org/8737
Reviewed-by: Humble Devassy Chirammal 
Tested-by: Gluster Build System 
Reviewed-by: Niels de Vos 
Reviewed-by: Vijay Bellur

osx: LaunchDaemon plist filename should start with org instead of com

2014-09-15T17:03:43+00:00

Backport of http://review.gluster.org/8734

BUG: 1141665
Change-Id: Ia41b00f3c49644f6a3c12e76c2395fce117c1bee
Signed-off-by: Justin Clift 
Reviewed-on: http://review.gluster.org/8735
Reviewed-by: Harshavardhana 
Tested-by: Harshavardhana 
Tested-by: Gluster Build System

gluster: Fix the recursive goto outs in the source code.

2014-09-08T06:35:36+00:00

Added a script check_goto.pl, that when run from
the source code root, will scan all .c files to match
the following pattern:

label:
        if (condition)
            goto label;

On finding such a pattern the script will print the file name
and the line number. There are certain cases where the above
recursive pattern is intended. Hence adding those labels to
ignore-labels. Thanks Vijaikumar Mallikarjuna for the perl
script.

Also fixed all such existing errors

BUG: 1138952
Change-Id: Ie6b75621711736e7e30f2f9d25e50435d58fc1e2
Signed-off-by: Vijaikumar Mallikarjuna 
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/8307
Tested-by: Gluster Build System 
Reviewed-by: Jeff Darcy 
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/8637
Reviewed-by: Vijay Bellur

build: make GLUSTERD_WORKDIR rely on localstatedir

2014-09-03T18:01:45+00:00

Backport from master branch - http://review.gluster.org/#/c/8246/

- Break-way from '/var/lib/glusterd' hard-coded previously,
  instead rely on 'configure' value from 'localstatedir'
- Provide 's/lib/db' as default working directory for gluster
  management daemon for BSD and Darwin based installations
- loff_t is really off_t on Darwin
- fix-off the warnings generated by clang on FreeBSD/Darwin
- Now 'tests/*' use GLUSTERD_WORKDIR a common variable for all
  platforms.
- Define proper environment for running tests, define correct PATH
  and LD_LIBRARY_PATH when running tests, so that the desired version
  of glusterfs is used, regardless where it is installed.
  (Thanks to manu@netbsd.org for this additional work)

Change-Id: I06e684ac4c26d1e74c9daf76753403ad15f79276
BUG: 1130308
Signed-off-by: Harshavardhana 
Reviewed-on: http://review.gluster.org/8486
Tested-by: Gluster Build System

Changing the position of nfs-ganesha.log

2014-07-10T06:38:04+00:00

Changing the position of ganesha.log from /tmp/ganesha.log
to /var/log/nfs-ganesha/ganesha.nfsd.log

Change-Id: Iff95fbc7fc690f4fdbc1789d84f3e41aaca9045b
BUG: 1116723
Signed-off-by: Meghana Madhusudhan 
Reviewed-on: http://review.gluster.org/8249
Reviewed-by: soumya k 
Reviewed-by: Humble Devassy Chirammal 
Reviewed-by: Raghavendra Talur 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur