glusterfs.git/extras, branch v3.7.6

common-ha: Corrected refresh-config output parsing

2015-11-06T12:35:01+00:00

>>>> Sample program with the earlier changes -
output=$(dbus-send --print-reply --system \
--dest=org.ganesha.nfsd /org/ganesha/nfsd/ExportMgr \
org.ganesha.nfsd.exportmgr.RemoveExport uint16:5 2>&1\
  | grep -v "^method return")
ret=$?
echo "${output}"
echo $ret
sleep 1
output=$(dbus-send --system --dest=org.ganesha.nfsd \
/org/ganesha/nfsd/ExportMgr org.ganesha.nfsd.exportmgr.AddExport \
string:/usr/etc/ganesha/exports/export.vol3.conf \
string:"EXPORT(Path=/vol3)" 2>&1  | grep -v "^method return")
ret=$?
echo "${output}"
echo $ret

Output:

1

1

Even if the command was successfully executed, 'grep -v' has
filtered out the output.

>>>> Sample program with the current changes -

output=$(dbus-send --print-reply --system --dest=org.ganesha.nfsd \
/org/ganesha/nfsd/ExportMgr org.ganesha.nfsd.exportmgr.RemoveExport\
 uint16:5 2>&1)
ret=$?
echo "${output}"
echo $ret
sleep 1
output=$(dbus-send --print-reply --system --dest=org.ganesha.nfsd \
/org/ganesha/nfsd/ExportMgr org.ganesha.nfsd.exportmgr.AddExport \
string:/usr/etc/ganesha/exports/export.vol3.conf \
string:"EXPORT(Path=/vol3)" 2>&1)
ret=$?
echo "${output}"
echo $ret

Output:
method return sender=:1.155 -> dest=:1.174 reply_serial=2
0
method return sender=:1.155 -> dest=:1.175 reply_serial=2
   string "1 exports added"
0

Change-Id: If9a38e825b2c1a87101de303f9494a0769a9e897
BUG: 1254494
Signed-off-by: Soumya Koduri 
Reviewed-on: http://review.gluster.org/12439
Tested-by: NetBSD Build System 
Reviewed-by: jiffin tony Thottan 
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY

firewall/spec: Create glusterfs firewall service if firewalld installed.

2015-10-15T22:53:03+00:00

It creates glusterfs firewall service during installation.

glusterfs service : It contains all default ports which needs to be opened.
During installation  glusterfs.xml is copied into firewall service directory(/usr/lib/firewalld/services/).

Note:
1.For bricks: It opens the 512 ports, if brick is running out side this range(>49664) then admin need to open the port
for that brick.
2.By default this service is not enabled in any of zone.

To enable this service(glusterfs) in firewall:

1. Get active zone(s) in node
     firewall-cmd  --get-active-zones

2. Attached this service(glusterfs) to zone(s)
   firewall-cmd  --zone=  --add-service=glusterfs               --To apply runtime
   firewall-cmd  --permanent  --zone=  --add-service=glusterfs  --To apply permanent

Note:
  we can also use firewall-config which gives GUI to configure firewall.

Backport of:
>Change-Id: Id97fe620c560fd10599511d751aed11a99ba4da5
>BUG: 1253967
>Signed-off-by: anand 
>Reviewed-on: http://review.gluster.org/11989
>Reviewed-by: Niels de Vos 
>Tested-by: NetBSD Build System 
>Tested-by: Gluster Build System 
>(cherry picked from commit 7f327d3b4f9222995d2ee78862e48ca44c28411c)

Change-Id: Iacf44b15ffb176c965c7f3b074065a54cf785dc7
BUG: 1057295
Signed-off-by: anand ;
Reviewed-on: http://review.gluster.org/12357
Reviewed-by: Niels de Vos 
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System

systemd/spec/glusterd: Adding EnvironmentFile in glusterd systemctrl service file.

2015-10-15T05:38:01+00:00

Issue: glusterd was not running in debug mode, this is because of
"systemctrl start glusterd" was not reading configuration from glusterd-sysconfig file.

Fix: Set glusterd-sysconfig file as "EnvironmentFile"  and pass args to glusterd.

Backport of:
>Change-Id: I6a032a2e86e5a25c39199ddf953bfbd6590d53b6
>BUG: 1263087
>Signed-off-by: anand 
>Reviewed-on: http://review.gluster.org/12175
>Reviewed-by: Anoop C S 
>Reviewed-by: Niels de Vos 
>Tested-by: NetBSD Build System 
>Tested-by: Gluster Build System 
>(cherry picked from commit 63e3806ba3c4d5f196ad2254f63d85b79602705a)

Change-Id: Ib081a2a4bcfbfc1fe4334da1ec8c1dbab0760657
BUG: 1271540
Signed-off-by: anand 
Reviewed-on: http://review.gluster.org/12358
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Atin Mukherjee

common-ha: distribution neutral location of config files

2015-09-29T18:18:59+00:00

work on systems with config files in directories other than
/etc/sysconfig.

BUG: 1251821
Change-Id: Ie8e0a330810430cbc92499181ab60d79ea43c8f1
Signed-off-by: Joe Julian 
Reviewed-on: http://review.gluster.org/12132
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: soumya k 
Reviewed-by: Kaleb KEITHLEY

common-ha: refresh-config output includes dbus "method return" msg

2015-09-24T23:36:01+00:00

filter out extraneous text from dbus-send

Change-Id: If8467f8e756584027d36559960fd691dcaf98993
BUG: 1262881
Signed-off-by: Kaleb S. KEITHLEY 
Reviewed-on: http://review.gluster.org/12174
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: jiffin tony Thottan 
Reviewed-by: soumya k

extras/hookscripts : introducing additional check in S31ganesha-start.sh

2015-09-24T08:53:41+00:00

New export file with default configuration will be created for a volume
when it is started again. This patch will create new export file only
when it is not present. This change is required for scenarios such as
snapshot restore , node reboot etc.

Change-Id: I34123911f176dcb29d5c016aa097af3a3b2c727b
BUG: 1261444
Signed-off-by: Jiffin Tony Thottan 
Reviewed-on: http://review.gluster.org/12219
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Niels de Vos

CommonHA: Fix the path of 'systemctl' cmd

2015-09-03T08:47:30+00:00

'systemctl' command path should be '/usr/bin/sytemctl'.
Fixed the typo in the HA-script.

Change-Id: Ied9fb85e25646ef00b303e361966e9d1354fe440
BUG: 1259225
Signed-off-by: Soumya Koduri 
Reviewed-on: http://review.gluster.org/12089
Reviewed-by: Kaleb KEITHLEY 
Reviewed-by: Niels de Vos 
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System

CommonHA: Avoid scp of the config state to the same host

2015-09-03T00:31:35+00:00

During add-node, have seen an issue where in scp doesn't work
if the source and destination host are same. Fixed the same.

Change-Id: I33fc18ff39e54ced277f94afef7a11a22c9ccce0
BUG: 1259225
Signed-off-by: Soumya Koduri 
Reviewed-on: http://review.gluster.org/12091
Reviewed-by: Kaleb KEITHLEY 
Tested-by: Gluster Build System

rpm: include required directory for glusterfind

2015-08-25T15:50:55+00:00

The directory was marked as %ghost, which causes the following
installation failure:

  Error unpacking rpm package glusterfs-server-3.8dev-0.446.git45e13fe.el7.centos.x86_64
  error: unpacking of archive failed on file /var/lib/glusterd/hooks/1/delete/post/S57glusterfind-delete-post.py;5581f20e: cpio: open

Also, *all* Python files should be part of the RPM package. This
includes generated .pyc and .pyo files.

BUG: 1256307
Change-Id: Iee74905b101912c4a845257742c470c3fe42ce2a
Signed-off-by: Niels de Vos 
Signed-off-by: Aravinda VK 
Reviewed-on: http://review.gluster.org/11298
Reviewed-on: http://review.gluster.org/12000
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System

common-ha : refresh-config should print sensible output

2015-08-25T03:47:09+00:00

ganesha-ha.sh --refresh-config printed the dbus-send output
messages as is on the console. Improving the output
of the operation by redirecting the messages to /var/log/messages
and by checking the exit code of the command executed.

The behaviour is also changed a litlle by exiting
when refresh-config fails on any of the nodes.
We don't want to continue changing config files
on other nodes when refresh-config has already failed
on one of the nodes.

Change-Id: Ic2e4e26bc09b1ba23cc6b62a1590c6083a0bde80
BUG: 1254494
Signed-off-by: Meghana M 
Reviewed-on: http://review.gluster.org/11949
Reviewed-by: soumya k 
Tested-by: Gluster Build System 
Reviewed-by: Kaleb KEITHLEY 
Tested-by: NetBSD Build System