glusterfs.git/xlators/features/snapview-client, branch release-3.7

snapview-client: remove check for parent inode type

2016-02-23T05:33:29+00:00

In lookup call back path, we are checking for parent inode
type to make sure parent inode was set properly, so that
next op on the inode can be successfuly completed.

More info can be found at bugid 1297311

Back port of>
>Change-Id: Ifb1b17f472f855f9b12088c3dd8328389d895e77
>BUG: 1297311
>Signed-off-by: Mohammed Rafi KC 
>Reviewed-on: http://review.gluster.org/13227
>Reviewed-by: Avra Sengupta 
>Tested-by: NetBSD Build System 
>Reviewed-by: Rajesh Joseph 

Change-Id: I347aa0e42ae35491246747dee50724fc91964611
BUG: 1306131
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/13412
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Smoke: Gluster Build System 
Reviewed-by: Rajesh Joseph

USS: pre-existing .snaps should not be listed with 'ls -a' with USS

2016-02-15T08:23:20+00:00

enabled

This is a backport of http://review.gluster.org/#/c/13330/

If there is a .snaps directory pre-exists and when USS is enabled,
it should not be listed with 'ls -a'

> Change-Id: I1c43e2decc0bbbd3213b190b675e3a32d04b22d3
> BUG: 1303828
> Signed-off-by: vmallika 

Change-Id: I76c632239b45f7eb7296cbfe3b73bc68e5bd1655
BUG: 1306163
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/13419
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Rajesh Joseph

features/snap : cleanup the root loc in statfs

2015-10-20T08:39:38+00:00

Problem : In svc_statfs function, wipe_loc is getting called on loc
          passed by nfs. This loc is being used by svc_stat which
          throws erro if loc->inode is NULL.

Solution : wipe_loc should be called on local root_loc.
Back port of http://review.gluster.org/#/c/12123/

Change-Id: I9cc5ee3b1bd9f352f2362a6d997b7b09051c0f68
BUG: 1261732
Signed-off-by: Ashish Pandey 
Reviewed-on: http://review.gluster.org/12147
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Pranith Kumar Karampuri

snapshot: Make fops static for correct resolution of symbols

2015-08-25T18:36:20+00:00

Few of the snapshot fops (like 'svc_lookup') may not get resolved
while using dynamic loading as there could be other libraries(like libntirpc)
with same routine names. Making them static to resolve the same.

This is backport of the below fix -
http://review.gluster.org/#/c/11805/

Change-Id: I6577bf3705864f5583425c94427b4e1025a59bcd
BUG: 1256485
Signed-off-by: Soumya Koduri 
Reviewed-on: http://review.gluster.org/11805
Reviewed-by: Niels de Vos 
Reviewed-by: Rajesh Joseph 
Reviewed-on: http://review.gluster.org/12008
Reviewed-by: Kaleb KEITHLEY 
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System

uss: Take ref on root inode

2015-08-14T06:44:45+00:00

If we recieve a statfs call on snap directory, we will redirect
the call into the root, by creating a new root loc. So it is better to
take a ref on the root inode
(http://review.gluster.org/#/c/10358/5/xlators/features/
snapview-client/src/snapview-client.c) 


Back port of :
>Change-Id: I5649addac442d391b2550346b115dec58fed5b86
>Signed-off-by: Mohammed Rafi KC 
>Reviewed-on: http://review.gluster.org/10750
>Tested-by: NetBSD Build System 
>Reviewed-by: Raghavendra Bhat 

Change-Id: Idd575a5313bcbc3d7bfdd954c7bfecce371f3add
BUG: 1228521
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/11094
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Avra Sengupta 
Reviewed-by: Rajesh Joseph

snapview-client: Allocate memory using GF_CALLOC

2015-07-26T15:11:09+00:00

loc->path memory should allocate through
GF_CALLOC/GF_MALLOC, since it uses GF_FREE
from loc_wipe.

Back port of :
>Change-Id: If3030175dfb07b0e0b6c0a30234857f8a0780035
>BUG: 1240184
>Signed-off-by: Mohammed Rafi KC 
>Reviewed-on: http://review.gluster.org/11543
>Tested-by: NetBSD Build System 
>Reviewed-by: Avra Sengupta 
>Reviewed-by: Vijay Bellur 

(cherry picked from commit 32207db25eea821dfb1abd033df1013d4b8fc622)

Change-Id: I69f9338dc5ae9950b5344000259558423c3f8618
BUG: 1245908
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/11743
Reviewed-by: Niels de Vos 
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System

uss : implement statfs fop for snapd

2015-05-10T15:51:54+00:00

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

snapview-client and snapview-server doesnot have statfs
fop implemented

Change-Id: I2cdd4c5784414b0549a01af9a28dbc723b7cdc67
BUG: 1218741
Signed-off-by: Mohammed Rafi KC 
Reviewed-on: http://review.gluster.org/10740
Tested-by: NetBSD Build System
Reviewed-by: Niels de Vos 
Reviewed-by: Raghavendra Bhat 
Tested-by: Gluster Build System

snapview: Fixing the coverity issues

2015-04-04T18:00:47+00:00

Coverity CIDs:
1228601
1220056
1238185
1257624

Change-Id: Idb4cc3b44bc3ccb8b497ce355e1e3bacfb176db9
BUG: 789278
Signed-off-by: Nandaja Varma 
Reviewed-on: http://review.gluster.org/9615
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

features/snapview-client: Don't free un-allocated memory

2015-03-27T12:19:42+00:00

Change-Id: I8636ced27448dde4f2c11370fe2026067d4a7e74
BUG: 1203637
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/10004
Reviewed-by: Raghavendra Bhat 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur