glusterfs.git/cli/src, branch v3.7.8

glusterd/cli: mask out inaccurate scrub statistics

2016-01-27T11:49:58+00:00

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

Some of the scrub statistics are inaccurate at the moment and would
successfully fool users at times. It's best to not display these
stats and stick to what is accurate.

Note that the change is only in the CLI part - glusterd (and brick)
supplies _all_ available scrub statistics, but CLI just selectively
sticks to what needs to be displayed.

Change-Id: Ia99f505a2cb10d5406cd37035d082e593f7e9a8c
BUG: 1291546
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.org/13223
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Atin Mukherjee

quota: start aux mount from glusterd with inet address

2016-01-22T11:27:14+00:00

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

With below patches, quota aux mount now
uses unix domain socket to connect to
glusterd
http://review.gluster.org/#/c/12645/
http://review.gluster.org/#/c/12819/

When USS is enabled, snapd protocol client tries to
connect to glusterd with inet and fails,
because remote-host option by client process
is set to UDS file

This patch starts the aux client process from glusterd
with inet address

> Change-Id: I6967043bfd8824658ea39bfd2842591fcc3280fd
> BUG: 1299497
> Signed-off-by: vmallika 

Change-Id: Ica512aaf5c2b25f86506bd1e0d7810a8ff1f7632
BUG: 1300243
Signed-off-by: vmallika 
Reviewed-on: http://review.gluster.org/13267
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Atin Mukherjee

cli: Add arbiter details to volinfo xml output

2016-01-22T05:28:08+00:00

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

The followig are added:
1. "1" and
"1"

2. "0" on the brick info, like so:
127.0.0.2:/home/ravi/bricks/brick1
127.0.0.2:/home/ravi/bricks/brick1
cafa8612-d7d4-4007-beea-72ae7477f3bb
0


Also fix a bug in gluster vol info where the abiter brick was shown the
wrong brick of the cold tier after performing a tier-attach.

Change-Id: Id978325d02b04f1a08856427827320e169169810
BUG: 1300174
Signed-off-by: Ravishankar N 
Reviewed-on: http://review.gluster.org/13263
Smoke: Gluster Build System 
Reviewed-by: Anuradha Talur 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri

all: reduce "inline" usage

2016-01-18T09:02:34+00:00

There are three kinds of inline functions: plain inline, extern inline,
and static inline.  All three have been removed from .c files, except
those in "contrib" which aren't our problem.  Inlines in .h files, which
are overwhelmingly "static inline" already, have generally been left
alone.  Over time we should be able to "lower" these into .c files, but
that has to be done in a case-by-case fashion requiring more manual
effort.  This part was easy to do automatically without (as far as I can
tell) any ill effect.

In the process, several pieces of dead code were flagged by the
compiler, and were removed.

backport of Change-Id: I56a5e614735c9e0a6ee420dab949eac22e25c155,
http://review.gluster.org/11769, BUG: 1245331

Change-Id: Iba1efb0bc578ea4a5e9bf76b7bd93dc1be9eba44
BUG: 1283302
Signed-off-by: Kaleb S KEITHLEY 
Reviewed-on: http://review.gluster.org/12646
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Niels de Vos

Tier : typo in tier help

2016-01-08T13:27:47+00:00

        backport of : http://review.gluster.org/#/c/13069/

>Change-Id: I75bca55901849cf725e02c782f75ff1e6054fddd
>BUG: 1294448
>Signed-off-by: hari gowtham 
>Reviewed-on: http://review.gluster.org/13097
>Tested-by: Gluster Build System 
>Reviewed-by: mohammed rafi  kc 
>Reviewed-by: Dan Lambright 
>Tested-by: Dan Lambright 

Change-Id: I95504ffd521de9232656f3e8ac2ca935de3bd3e6
BUG: 1294942
Signed-off-by: hari gowtham 
Reviewed-on: http://review.gluster.org/13124
Tested-by: Gluster Build System 
Reviewed-by: Dan Lambright 
Tested-by: Dan Lambright

Tier: "tier start force" command implementation

2015-12-23T01:45:00+00:00

	back port of : http://review.gluster.org/#/c/12983/

The start command doesnt restart the tier deamon if the deamon
is running at one node. hence to bring up the tierd on the nodes
where the deamon is down, the force command is implemented.
It skips the check for tierd running.

>Change-Id: I0037d3e5ecfe56637d0da201a97903c435d26436
>BUG: 1292112
>Signed-off-by: hari gowtham 

Change-Id: Idaca442c1a41ded8bf555a6e34eed0ebb9ea4034
BUG: 1293698
Signed-off-by: hari 
Reviewed-on: http://review.gluster.org/13069
Tested-by: Gluster Build System 
Reviewed-by: Dan Lambright 
Tested-by: Dan Lambright

cli/xml: display correct xml output of tier volume

2015-12-23T00:29:14+00:00

This patch is backport of: http://review.gluster.org/#/c/12982/

Currently When hot tier type is distributed-replicate and cold tier
type is disperse volume then #gluster volume info --xml command is
not giving its correct output. In case of HOT tier case its displaying
wrong volume type.

With this fix it will show correct xml output for tier volume
irrespective of all the type of the volume's.

  >> Change-Id: If1de8d52d1e0ef3d0523163abed37b2b571715e8
  >> BUG: 1293309
  >> Signed-off-by: Gaurav Kumar Garg 
  >> Reviewed-on: http://review.gluster.org/12982
  >> Tested-by: NetBSD Build System 
  >> Reviewed-by: mohammed rafi  kc 
  >> Tested-by: Gluster Build System 
  >> Reviewed-by: Kotresh HR 
  >> Reviewed-by: Atin Mukherjee 

Change-Id: If1de8d52d1e0ef3d0523163abed37b2b571715e8
BUG: 1293309
Signed-off-by: Gaurav Kumar Garg 
(cherry picked from commit b0e126d0edf10946701c2fd4f0f1cf8c7b07eda1)
Reviewed-on: http://review.gluster.org/13042
Reviewed-by: hari gowtham 
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Dan Lambright 
Tested-by: Dan Lambright

bitrot: getting correct value of scrub stat's

2015-12-18T03:29:00+00:00

This patch is backport of: http://review.gluster.org/#/c/12776/

When user execute bitrot scrub status command then gluster
is not giving correct value of Number of Scrubbed files,
Number of Unsigned files, Last completed scrub time,
Duration of last scrub.

With this patch scrub status will give correct value for
all the above fields.

  >> Change-Id: Ic966f76d22db5b0c889e6386a1c2219afbda1f49
  >> BUG: 1285989
  >> Signed-off-by: Gaurav Kumar Garg 
  >> Signed-off-by: Kotresh HR 
  >> Reviewed-on: http://review.gluster.org/12776
  >> Tested-by: NetBSD Build System 
  >> Tested-by: Gluster Build System 
  >> Reviewed-by: Venky Shankar 

Change-Id: Ic966f76d22db5b0c889e6386a1c2219afbda1f49
BUG: 1291546
Signed-off-by: Gaurav Kumar Garg 


(cherry picked from commit 22827d51c232c44a8f5ac003529d907d93baf7b0)

Change-Id: Icef24cce35c8d54ffdfa5282491338318e78780b
Reviewed-on: http://review.gluster.org/12966
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Venky Shankar

tier/glusterd : making new tier detach command throw warning

2015-12-16T17:28:19+00:00

        back port of : http://review.gluster.org/#/c/12883/

For detach tier, the validation was done using the string "detach-tier"
but the new commands used has the string "tier". Making the string use
"tier" to compare, creates problem as the tier status and tier detach
have the keyword "tier". So tier detach and tier status were separated.
and strtok was used to prevent the condition from passing when the
volume name has a substring of "tier". (only the second word from the
string is got and checked if the feature is tier).

Problem: new detach tier command doesnt throw warnings like
"not a tier volume" or " detach tier not started" respectively
instead it prints empty output.

Fix: while validate the volume is checked if its a tiered volume
if yes it is checked if the detach tier is started, else a warning
is thrown respectively.

>Change-Id: I94246d53b18ab0e9406beaf459eaddb7c5b766c2
>BUG: 1288517
>Signed-off-by: hari gowtham 
>Reviewed-on: http://review.gluster.org/12883
>Tested-by: NetBSD Build System 
>Tested-by: Gluster Build System 
>Reviewed-by: Atin Mukherjee 

Change-Id: I1ac3b6baaec644dbc2025085a7f17abd56ba169d
BUG: 1291970
Signed-off-by: hari gowtham 
Reviewed-on: http://review.gluster.org/12976
Tested-by: NetBSD Build System 
Reviewed-by: Dan Lambright 
Tested-by: Dan Lambright

quota: show used_space,(file,dir) count even when quota limit is not set

2015-12-10T14:14:53+00:00

Problem:
As of now quota 'list/list-objects' will list the usage only if limit is
set for every directory else it will fail with ENOATTR(If inode/inode-quota
is already configured for the first time).

Feature:
With the patch we are enhancing this command to list the usage even
if quota limit is not set but still the user has to configure
inode/inode-quota for the first time.

Example:
Consider we have /client/dir and /client1(absolute path from mount point):
Quota limit is set only on /client. when we try listing /client/dir or /client1,
it shows "Limit not set".

Fix:
The patch fixes this by showing "used space" in case of list command and
shows "file_count" & "dir_count" in case of list-objects command. This works
fine with xml output as well.

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

Cherry-picked from commit 608c646cc00e2c1c0b9c87e9fd7cb9e1eb271db2
> Change-Id: I68b08ec77a583b3c7f39fe4d6b15d3d77adb095a
> BUG: 1284752
> Signed-off-by: Manikandan Selvaganesh 

Change-Id: I68b08ec77a583b3c7f39fe4d6b15d3d77adb095a
BUG: 1289063
Signed-off-by: Manikandan Selvaganesh 
Reviewed-on: http://review.gluster.org/12893
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Kaushal M