glusterfs.git/cli/src, branch release-3.9

cli: keep 'gluster volume status detail' consistent

2017-01-25T04:50:39+00:00

The output of the command 'gluster volume status  detail' is
not consistent between operating systems. On linux hosts it shows the
file system type, the device name, mount options and inode size of each
brick. However the same command executed on a FreeBSD host doesn't show
all this information, even for bricks stored on a linux.

Additionally, for hosts other than linux, this information is shown as
'N/A' many times. This has been fixed to show as much information as it
can be retrieved from the operating system.

The file contrib/mount/mntent.c has been mostly rewriten because it
contained many errors that caused mount information to not be retrieved
on some operating systems.

> Change-Id: Icb6e19e8af6ec82255e7792ad71914ef679fc316
> BUG: 1411334
> Signed-off-by: Xavier Hernandez 
> Reviewed-on: http://review.gluster.org/16371
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
> Reviewed-by: Atin Mukherjee 
> Reviewed-by: Kaleb KEITHLEY 

Change-Id: I1976e4765df9204aea6ca923a6d2b2bed06ed3b9
BUG: 1415131
Signed-off-by: Xavier Hernandez 
Reviewed-on: https://review.gluster.org/16441
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY

cli: glusterfs_ctx_defaults_init should not re-initialize ctx->locks

2016-12-14T15:30:45+00:00

glusterfs_ctx_new already initialize ctx->locks therefore the second
initialization in glusterfs_ctx_defaults_init does not make sense.

> Reviewed-on: http://review.gluster.org/15904
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
> Reviewed-by: Atin Mukherjee 
(cherry picked from commit 8df8a2308fcf5efda638f160428158127930d00f)

Change-Id: I8a44bed13af20e60e8bafb05360e88c0b237056c
BUG: 1400803
Signed-off-by: Rajesh Joseph 
Reviewed-on: http://review.gluster.org/16004
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Atin Mukherjee

geo-rep/cli: Validate Checkpoint label

2016-12-01T05:55:41+00:00

Checkpoint command accepts "now" or any other Time
in "%Y-%m-%d %H:%M:%S" format as label.

Validation added with this patch for the input label. Checkpoint set
will fail for invalid label.

> Reviewed-on: http://review.gluster.org/15721
> Smoke: Gluster Build System 
> CentOS-regression: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> Reviewed-by: Saravanakumar Arumugam 
> Reviewed-by: Kotresh HR 

BUG: 1395626
Change-Id: I23518c151ab4b294f64cae3b78baaacb3d8f7b82
Signed-off-by: Aravinda VK 
(cherry picked from commit 8a1993b32f476765f9f5c9294e7c3f2ae75198a0)
Reviewed-on: http://review.gluster.org/15854
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Kotresh HR 
Reviewed-by: Saravanakumar Arumugam

eventsapi: Fix sending event during volume set help

2016-11-30T10:17:47+00:00

Event sent when `gluster volume set help` command is run
with Volume name as "help" and empty options list

With this patch, event sent only when volume set on a real volume

> Reviewed-on: http://review.gluster.org/15685
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
> Reviewed-by: Atin Mukherjee 

BUG: 1388070
Change-Id: Ia8785d6108cb86f7d89ecf9ea552df0334b41398
Signed-off-by: Aravinda VK 
(cherry picked from commit f31b3213e2a97259faa7dcae2354d2535732068b)
Reviewed-on: http://review.gluster.org/15714
CentOS-regression: Gluster Build System 
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
Reviewed-by: Atin Mukherjee

cluster/afr: CLI for granular entry heal enablement/disablement

2016-11-29T10:52:15+00:00

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

When there are already existing non-granular indices created that are
yet to be healed, if granular-entry-heal option is toggled from 'off' to
'on', AFR self-heal whenever it kicks in, will try to look for granular
indices in 'entry-changes'. Because of the absence of name indices,
granular entry healing logic will fail to heal these directories, and
worse yet unset pending extended attributes with the assumption that
are no entries that need heal.

To get around this, a new CLI is introduced which will invoke glfsheal
program to figure whether at the time an attempt is made to enable
granular entry heal, there are pending heals on the volume OR there
are one or more bricks that are down. If either of them is true, the
command will be failed with the appropriate error.

New CLI: gluster volume heal  granular-entry-heal {enable,disable}

Change-Id: Ic79519468a087cd337df664b968188c4adcba43a
BUG: 1398500
Signed-off-by: Krutika Dhananjay 
Reviewed-on: http://review.gluster.org/15941
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Pranith Kumar Karampuri

events: Add FMT_WARN for gf_event

2016-11-24T05:44:51+00:00

Raghavendra G found that posix is trying to print %s
but passing an int when HEALTH_CHECK fails in posix.
These are the kind of bugs that should be caught
at compilation itself.
Also fixed the problematic gf_event() callers.

 >BUG: 1386097
 >Change-Id: Id7bd6d9a9690237cec3ca1aefa2aac085e8a1270
 >Signed-off-by: Pranith Kumar K 
 >Reviewed-on: http://review.gluster.org/15671
 >Smoke: Gluster Build System 
 >NetBSD-regression: NetBSD Build System 
 >Reviewed-by: Atin Mukherjee 
 >CentOS-regression: Gluster Build System 

BUG: 1396778
Change-Id: Idf8e1f427578d02dccd2a8165884a5cf086eb07e
Signed-off-by: Pranith Kumar K 
Reviewed-on: http://review.gluster.org/15884
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Atin Mukherjee

cli/rebalance: remove brick status is incorrect

2016-11-17T18:15:53+00:00

If a remove brick operation is preceded by a fix-layout,
running remove-brick status on a node which does not
contain any of the bricks that were removed displays
fix-layout status.

The defrag_cmd variable was not updated in glusterd
for the nodes not hosting removed bricks causing the
status parsing to go wrong. This is now updated.
Also made minor modifications to the spacing in
the fix-layout status output.

> Change-Id: Ib735ce26be7434cd71b76e4c33d9b0648d0530db
> BUG: 1389697
> Signed-off-by: N Balachandran 
> Reviewed-on: http://review.gluster.org/15749
> Smoke: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> CentOS-regression: Gluster Build System 
> Reviewed-by: Atin Mukherjee 
(cherry picked from commit 35b085ba345cafb2b0ee978a4c4475ab0dcba5a6)

Change-Id: I3da89c61da07bc5e037527aafc84d184dcd1f764
BUG: 1396109
Signed-off-by: N Balachandran 
Reviewed-on: http://review.gluster.org/15870
Tested-by: Atin Mukherjee 
Smoke: Gluster Build System 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Atin Mukherjee

nfs/cli : add warning message while enabling gluster nfs

2016-11-17T04:31:33+00:00

>Change-Id: Ice70003f7295d2d8af9877e1ecf6f3c81422b30c
>BUG: 1376693
>Signed-off-by: Jiffin Tony Thottan 
>Reviewed-on: http://review.gluster.org/15805
>Smoke: Gluster Build System 
>NetBSD-regression: NetBSD Build System 
>CentOS-regression: Gluster Build System 
>Reviewed-by: Atin Mukherjee 
>Reviewed-by: soumya k 
>Reviewed-by: Kaleb KEITHLEY 
>(cherry picked from commit a35d63050ec7e745a66c1aef8e1c8eb42d7b67a1)

Change-Id: Ice70003f7295d2d8af9877e1ecf6f3c81422b30c
BUG: 1395526
Signed-off-by: Jiffin Tony Thottan 
Reviewed-on: http://review.gluster.org/15852
Smoke: Gluster Build System 
Reviewed-by: soumya k 
NetBSD-regression: NetBSD Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Atin Mukherjee

CLI/TIER: throw warning regarding the removal of the older commands.

2016-10-26T07:04:29+00:00

The older tier commands for attach tier and detach tier have to be
removed from code. This patch sends a warning asking to use new
command as older ones are depricated and will be removed.

>Change-Id: Ie1c62947bad6ff106f40331ff6134838a6c72a7a
>BUG: 1388062
>Signed-off-by: hari 
>Reviewed-on: http://review.gluster.org/15713
>Tested-by: hari gowtham 
>Smoke: Gluster Build System 
>NetBSD-regression: NetBSD Build System 
>CentOS-regression: Gluster Build System 
>Reviewed-by: Dan Lambright 

Change-Id: I28bcb0b0d2f362fbdd28aab9bea95e3a2947caa6
BUG: 1388470
Signed-off-by: hari 
Reviewed-on: http://review.gluster.org/15723
Tested-by: hari gowtham 
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
NetBSD-regression: NetBSD Build System 
Reviewed-by: Atin Mukherjee

events: add TIER_START and TIER_START_FORCE events

2016-10-26T05:11:54+00:00

Add TIER_START and TIER_START_FORCE events
Conditionally generate DETACH events as per user confirmation.

> Reviewed-on: http://review.gluster.org/15675
> Smoke: Gluster Build System 
> CentOS-regression: Gluster Build System 
> NetBSD-regression: NetBSD Build System 
> Reviewed-by: Dan Lambright 
(cherry picked from commit 8f8e23b0142a21779648802664db5b3d5253e4e0)

Change-Id: I205dc14884d707087edce42e8cf4208bd89d31dc
BUG: 1387981
Signed-off-by: Milind Changire 
Reviewed-on: http://review.gluster.org/15708
NetBSD-regression: NetBSD Build System 
Smoke: Gluster Build System 
CentOS-regression: Gluster Build System 
Reviewed-by: Aravinda VK