glusterfs.git/heal, branch v3.7.6

heal: remove glfsh_print_brick()

2015-09-28T06:19:30+00:00

Backport of http://review.gluster.org/#/c/12212/

Use glfsh_print_brick_from_xl() instead so that the hostname:brickpath
displayed when heal info is run is consistent with other gluster cli
commands like `gluster volume info`.

Change-Id: I30ee3d76d0f68991a25bd678d40ec3bf7e0538c7
BUG: 1265633
Signed-off-by: Ravishankar N 
(cherry picked from commit 98fe4297131361d62d6579d0bde10a8485e4748a)
Reviewed-on: http://review.gluster.org/12220
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri

heal: Add check for healing directories in split-brain

2015-08-27T08:20:12+00:00

Backport of http://review.gluster.org/#/c/11937/

1. Healing using  `bigger-file' option does not make much sense for directories,
hence disallow it and notify the user.
2. Healing using 'source-brick' option for directories performs a conservative
merge. Notify the user about it.

Change-Id: I0acff7b438bdb256615ed0f40d45694f37a77dd5
BUG: 1256605
Signed-off-by: Ravishankar N 
(cherry picked from commit 97bae6371b5e841f03a61d6700033f40afd5cdc7)
Reviewed-on: http://review.gluster.org/12007
Reviewed-by: Pranith Kumar Karampuri 
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System

heal : Do not invoke glfs_fini for glfs-heal commands

2015-06-09T05:07:13+00:00

backport of http://review.gluster.org/11103

Change-Id: I2fa6789488c4ecebbed049f46c189441fba48535
BUG: 1229100
Signed-off-by: Anuradha 
Reviewed-on: http://review.gluster.org/11103
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur 
Reviewed-on: http://review.gluster.org/11115
Tested-by: NetBSD Build System

heal: Do not call glfs_fini in final builds

2015-06-04T12:55:29+00:00

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

BUG: 1226272
Change-Id: I213a2855ab44a19e0b5cd36bb0ac180015b8a21f
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/11048
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Niels de Vos

libgfapi: Store and restore THIS in every API exposed by libgfapi

2015-05-09T13:52:46+00:00

        Backport of http://review.gluster.org/#/c/9797

Storing and restoring THIS:
When the APIs exposed by libgfapi are called by other xlators like
snapview server etc. the THIS value is overwritten to contain the
THIS of libgfapi(viz libgfapi master xlator). Hence using 'THIS'
in any xlator after calling libgfapi API will lead to issues.
One such issue was uncovered in snapview and the patch
http://review.gluster.org/#/c/9469/ was sent to workaround this issue.

Hence, storing and restoring THIS, at the entry and exit of every API
exposed by libgfapi.

Change-Id: I6f330dde25e7700fb26339d667a7ccd193ec6ba0
BUG: 1210934
Signed-off-by: Poornima G 
Reviewed-on: http://review.gluster.org/9797
Reviewed-by: Raghavendra Talur 
Reviewed-by: soumya k 
Reviewed-by: Niels de Vos 
Tested-by: Gluster Build System 
(cherry picked from commit 1162bb36108ab8dba8303b86927a99835b791d79)
Reviewed-on: http://review.gluster.org/10414
Reviewed-by: Shyamsundar Ranganathan

build: glfsheal should link against $(UUID_LIBS)

2015-04-30T06:26:28+00:00

Without this, building fails on Debian Jessie:

    /usr/bin/ld: glfs-heal.o: undefined reference to symbol 'uuid_clear@@UUID_1.0'
    /lib/x86_64-linux-gnu/libuuid.so.1: error adding symbols: DSO missing from command line
    collect2: error: ld returned 1 exit status
    Makefile:503: recipe for target 'glfsheal' failed

While at it, clean some of the unused defines too.

Backport of:
> BUG: 1206587
> Change-Id: I2e513b2a806e6241542e60007d720cca16dabb06
> Reviewed-on: http://review.gluster.org/10453
> Signed-off-by: Niels de Vos 

BUG: 1217176
Change-Id: I2e513b2a806e6241542e60007d720cca16dabb06
Signed-off-by: Niels de Vos 
Reviewed-on: http://review.gluster.org/10454
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY

libglusterfs/syncop: Add xdata to all syncop calls

2015-04-08T15:14:59+00:00

This patch adds support for xdata in both the
request and response path of syncops.

Few calls like lookup already had the support;
have renamed variables in few places to maintain
uniformity.

xdata passed downwards is known as xdata_in
and xdata passed upwards is known as xdata_out.

There is an old patch by Jeff Darcy at
http://review.gluster.org/#/c/8769/3 which does the
same for some selected calls. It also brings in
xdata support at gfapi level.

xdata support at gfapi level would be introduced
in subsequent patches.

Change-Id: I340e94ebaf2a38e160e65bc30732e8fe1c532dcc
BUG: 1158621
Signed-off-by: Raghavendra Talur 
Reviewed-on: http://review.gluster.org/9859
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

Avoid conflict between contrib/uuid and system uuid

2015-04-04T17:48:35+00:00

glusterfs relies on Linux uuid implementation, which
API is incompatible with most other systems's uuid. As
a result, libglusterfs has to embed contrib/uuid,
which is the Linux implementation, on non Linux systems.
This implementation is incompatible with systtem's
built in, but the symbols have the same names.

Usually this is not a problem because when we link
with -lglusterfs, libc's symbols are trumped. However
there is a problem when a program not linked with
-lglusterfs will dlopen() glusterfs component. In
such a case, libc's uuid implementation is already
loaded in the calling program, and it will be used
instead of libglusterfs's implementation, causing
crashes.

A possible workaround is to use pre-load libglusterfs
in the calling program (using LD_PRELOAD on NetBSD for
instance), but such a mechanism is not portable, nor
is it flexible. A much better approach is to rename
libglusterfs's uuid_* functions to gf_uuid_* to avoid
any possible conflict. This is what this change attempts.

BUG: 1206587
Change-Id: I9ccd3e13afed1c7fc18508e92c7beb0f5d49f31a
Signed-off-by: Emmanuel Dreyfus 
Reviewed-on: http://review.gluster.org/10017
Tested-by: Gluster Build System 
Reviewed-by: Niels de Vos

cluster/ec: Implement heal info for ec

2015-03-31T06:38:49+00:00

This also lists the files that are on-going I/O, which
will be fixed later.

Change-Id: Ib3f60a8b7e8798d068658cf38eaef2a904f9e327
BUG: 1203581
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/10020
Tested-by: Gluster Build System 
Reviewed-by: Dan Lambright

glfsheal: Do not print pending heals for non-afr xls

2015-03-26T11:12:20+00:00

Change-Id: Ib9a1998b964af1435e86de2c861ecb1c0e9026c9
BUG: 1203637
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/10005
Reviewed-by: Krutika Dhananjay 
Reviewed-by: Anuradha Talur 
Reviewed-by: Ravishankar N 
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY