glusterfs.git/xlators, branch heal-info

glusterd: Add warning and abort in case of failures in migration during remove-brick commit

2019-08-25T05:19:11+00:00

Problem -
Currently remove-brick commit goes through even though there were files
that failed to migrate or were skipped. There is no warning raised to the user.
Solution-
Add a check in the remove brick staging phase to verify if the status of the
rebalnce process is complete but there has been failures or some skipped files
while migration, In this case user will be given a warning and remove-brick
commit. User will need to use the force option to remove the bricks.

Fixes: bz#1514683
Signed-offby- Vishal Pandey 

Change-Id: I014d0f0afb4b2fac35ab0de52227f98dbae079d5

cluster/afr - Unused variables

2019-08-24T04:28:00+00:00

-Minor change to if-else structure to avoid code duplication.
-Added logging in case method calls fails

CID: 1394654
Updates: bz#789278

Change-Id: Ibef4450dc89ddd3bf951303d5b87f503924fd250
Signed-off-by: Barak Sason

Revert "glusterd: (storhaug) remove ganesha (843e1b0)"

2019-08-24T02:08:57+00:00

please note as an additional change, macro GLUSTERD_GET_SNAP_DIR
moved from glusterd-store.c to glusterd-snapshot-utils.h

Change-Id: I811efefc148453fe32e4f0d322e80455447cec71
updates: #663
Signed-off-by: Jiffin Tony Thottan

posix: log aio_error return codes in posix_fs_health_check

2019-08-22T10:29:35+00:00

Problem: Sometime brick is going down to health check thread is
         failed without logging error codes return by aio system calls.
         As per aio_error man page it returns a positive error number
         if the asynchronous I/O operation failed.

Solution: log aio_error return codes in error message

Change-Id: I2496b1bc16e602b0fd3ad53e211de11ec8c641ef
Fixes: bz#1744519
Signed-off-by: Mohit Agrawal

ctime: Fix incorrect realtime passed to frame->root->ctime

2019-08-22T05:35:15+00:00

On systems that don't support "timespec_get"(e.g., centos6), it
was using "clock_gettime" with "CLOCK_MONOTONIC" to get unix epoch
time which is incorrect. This patch introduces "timespec_now_realtime"
which uses "clock_gettime" with "CLOCK_REALTIME" which fixes
the issue.

Change-Id: I57be35ce442d7e05319e82112b687eb4f28d7612
Signed-off-by: Kotresh HR 
fixes: bz#1743652

nlm: check if nlm4 is initialized in nlm_priv

2019-08-22T04:54:11+00:00

Otherwise, gnfs will crash in following steps.
1) gluster v set  nfs.disable off
2) gluster v set  nfs.nlm off
3) kill -SIGUSR1 
4) gnfs crash with SIGSEGV as follows:
nlm_priv (this=this@entry=0x7f1ad00173b0) at nlm4.c:2742
0x00007f1acf89d29d in nfs_priv (this=0x7f1ad00173b0) at nfs.c:1662
0x00007f1ae2941085 in gf_proc_dump_single_xlator_info (trav=trav@entry=0x7f1ad00173b0) at statedump.c:502
0x00007f1ae29410b8 in gf_proc_dump_per_xlator_info (top=top@entry=0x7f1ad00173b0) at statedump.c:519

fixes: bz#1739360
Change-Id: Ib9b207a4ccb3226dbc2c449b77de348cbc9a3d3c
Signed-off-by: Xie Changlong

storage/posix - Fixing a coverity issue

2019-08-21T14:01:40+00:00

Fixed a resource leak of variable 'pfd'

CID: 1400673
Updates: bz#789278

Change-Id: I78e1e8a89e0604b56e35a75c25d436b35db096c3
Signed-off-by: Barak Sason

features/utime - fixing a coverity issue

2019-08-21T14:01:02+00:00

-Modified op_errno init value to a non-negative value in order to avoid
using a negative value where it's not allowed.
-In the metod "STACK_UNWIND_STRICT" modified 3rd argument in order to
represnt the correct value to use (changed from -1 to ret).

CID: 1403650
Updates: bz#789278

Change-Id: I608031d5af13832e05e180e746b1b5280b54f559
Signed-off-by: Barak Sason

features/cloudsync - fix a coverity issue

2019-08-21T05:16:30+00:00

All assigns to op_errno in this mehod were to the same vlaue - ENOMEM.
Removed repeted assignments and assigned as init value.
This also prevents the problem of sending a negatve value of op_errno to
CS_STACK_UNWIND method

CID: 1394645 - https://scan6.coverity.com/reports.htm#v44018/p10714/fileInstanceId=92065749&defectInstanceId=28018364&mergedDefectId=1394645
Updates: bz#789278

Change-Id: If765a9216500a38f9392617aaf06583ce36e3262
Signed-off-by: Barak Sason

storage/posix - fixing a coverity issue

2019-08-21T04:42:57+00:00

CID: 1394644 & 1394639
Updates: bz#789278

Added logging in case method calls fails

Change-Id: Ib833a5f68d37b98287b84c325637bc688937f647
Signed-off-by: Barak Sason