glusterfs.git/rpc/rpc-lib, branch v3.13.0rc0

gluster: IPv6 single stack support

2017-10-24T09:28:08+00:00

Summary:
- This diff changes all locations in the code to prefer inet6 family
  instead of inet.  This will allow change GlusterFS to operate
  via IPv6 instead of IPv4 for all internal operations while still
  being able to serve (FUSE or NFS) clients via IPv4.
- The changes apply to NFS as well.
- This diff ports D1892990, D1897341 & D1896522 to the 3.8 branch.

Test Plan: Prove tests!

Reviewers: dph, rwareing

Signed-off-by: Shreyas Siravara 

Change-Id: I34fdaaeb33c194782255625e00616faf75d60c33
BUG: 1406898
Reviewed-on-3.8-fb: http://review.gluster.org/16059
Reviewed-by: Shreyas Siravara 
Tested-by: Shreyas Siravara

rpc: free conn->name in rpc_clnt_destroy()

2017-10-11T04:57:38+00:00

Change-Id: Idf99908aa48718a7faf7f0bbc647a679ec548282
BUG: 1443145
Signed-off-by: Niels de Vos

rpc: free registered callback programs

2017-10-11T04:55:31+00:00

Change-Id: I8c6f6b642f025d1faf74015b8f7aaecd7ebfd4d5
BUG: 1443145
Signed-off-by: Niels de Vos

heal: New feature heal info summary to list the status of brick and count of entries to be healed

2017-09-15T12:21:36+00:00

Command output:
Brick 192.168.2.8:/brick/1
Status: Connected
Total Number of entries: 363
Number of entries in heal pending: 362
Number of entries in split-brain: 0
Number of entries possibly healing: 1



  
    
      
        192.168.2.8:/brick/1
        Connected
        363
        362
        0
        1
      
    
  
  0
  0
  


Change-Id: I40cb6f77a14131c9e41b292f4901b41a228863d7
BUG: 1261463
Signed-off-by: Mohamed Ashiq Liyazudeen 
Reviewed-on: https://review.gluster.org/12154
Smoke: Gluster Build System 
Tested-by: Karthik U S 
CentOS-regression: Gluster Build System 
Reviewed-by: Ravishankar N 
Reviewed-by: Atin Mukherjee

rpc: destroy transport after client_t

2017-08-31T03:45:09+00:00

Problem:
1. Ref counting increment on the client_t object is done in
   rpcsvc_request_init() which is incorrect.
2. Ref not taken when delegating to grace_time_handler()

Solution:
1. Only fop requests which require processing down the graph via
   stack 'frames' now ref count the request in get_frame_from_request()
2. Take ref on client_t object in server_rpc_notify() but avoid
   dropping in RPCSVC_EVENT_TRANSPORT_DESRTROY. Drop the ref
   unconditionally when exiting out of grace_time_handler().
   Also, avoid dropping ref on client_t in
   RPCSVC_EVENT_TRANSPORT_DESTROY when ref mangement as been
   delegated to grace_time_handler()

Change-Id: Ic16246bebc7ea4490545b26564658f4b081675e4
BUG: 1481600
Reported-by: Raghavendra G 
Signed-off-by: Milind Changire 
Reviewed-on: https://review.gluster.org/17982
Tested-by: Raghavendra G 
Reviewed-by: Raghavendra G 
CentOS-regression: Gluster Build System 
Smoke: Gluster Build System

Revert "rpc: disable client on disconnection from rebalance"

2017-08-24T18:47:38+00:00

This reverts commit 5b14c11d3cae38bc66006b02217ede485ae30dea.

BUG: 1484225
Change-Id: I3269d3fc64de3f3cc6f670ea564a87d7725e10fd
Signed-off-by: Milind Changire 
Reviewed-on: https://review.gluster.org/18113
Smoke: Gluster Build System 
Reviewed-by: Atin Mukherjee 
Tested-by: Atin Mukherjee 
Reviewed-by: Raghavendra G 
CentOS-regression: Gluster Build System

rpc: disable client on disconnection from rebalance

2017-08-23T10:34:33+00:00

Problem:
glusterd rpc code path attempts to reconnect to rebalance process
via the reconnect timer even after the rebalance process disconnection

Solution:
Set the clnt->disabled flag to 1 to avoid reconnection and cause
the clnt object to be unref'd

Change-Id: I4e38eaef45d2fdea86d25e9dff9f1af0cd29cf66
BUG: 1484225
Signed-off-by: Milind Changire 
Reviewed-on: https://review.gluster.org/18093
Smoke: Gluster Build System 
Tested-by: Raghavendra G 
Reviewed-by: Raghavendra G 
CentOS-regression: Gluster Build System

tier: separation of attach-tier from add-brick

2017-08-01T11:08:04+00:00

PROBLEM: Both attach tier and add brick have the same RPC
and set of code. This becomes a hurdle while tring to implement
add brick on a tiered volume.

FIX: This patch separates the add brick and attach tier
giving them separate RPCs.

Change-Id: Iec57e972be968a9ff00b15b507e56a4f6dc398a2
BUG: 1376326
Signed-off-by: hari gowtham 
Reviewed-on: https://review.gluster.org/15503
Smoke: Gluster Build System 
Tested-by: hari gowtham 
Reviewed-by: Samikshan Bairagya 
CentOS-regression: Gluster Build System

rpc: prevent logging null client [Invalid argument]

2017-07-25T12:47:02+00:00

The following log entry is observed during volume operations such
as volume create:

[2017-07-20 05:13:43.213797] E [client_t.c:321:gf_client_ref]
(-->/usr/local/lib/libgfrpc.so.0(rpcsvc_request_create+0x1a4) [0x7f987f66cd20]
-->/usr/local/lib/libgfrpc.so.0(rpcsvc_request_init+0xd0) [0x7f987f66ca23]
-->/usr/local/lib/libglusterfs.so.0(gf_client_ref+0x56) [0x7f987f91cbd5] )
0-client_t: null client [Invalid argument]

Change-Id: I49ba753e8d1a828bb275b0ccb1a181706774f387
BUG: 1193929
Signed-off-by: Prashanth Pai 
Reviewed-on: https://review.gluster.org/17848
Reviewed-by: Raghavendra G 
Reviewed-by: Amar Tumballi 
Tested-by: Amar Tumballi 
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System

rpc: include current second in timed out frame cleanup

2017-07-21T06:22:21+00:00

Problem:
frames which time out at current second are missed out

Solution:
change test to include frames timing out at current second
i.e. timeout <= current.tv_sec
instead of
timeout < current.tv_sec

Change-Id: I459d47856ade2b657a0289e49f7f63da29186d6e
BUG: 1468433
Signed-off-by: Milind Changire 
Reviewed-on: https://review.gluster.org/17722
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Raghavendra G