glusterfs-nsr.git/geo-replication, branch master

geo-rep: fix the code bug introduced due to flake8 refactoring

2014-04-08T17:05:41+00:00

Sorry for the bug, which got introduced due to code refactoring.
http://review.gluster.org/#/c/7311/

Change-Id: Ide519ca114aa8a7d7624d7af99945c857f069069
Signed-off-by: Aravinda VK 
Reviewed-on: http://review.gluster.org/7417
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

geo-rep: code pep8/flake8 fixes

2014-04-08T04:56:55+00:00

pep8 is a style guide for python.
http://legacy.python.org/dev/peps/pep-0008/

pep8 can be installed using, `pip install pep8`

Usage: `pep8 `, For example, `pep8 master.py`
will display all the coding standard errors.

flake8 is used to identify unused imports and other issues
in code.

pip install flake8

cd $GLUSTER_REPO/geo-replication/
flake8 syncdaemon

Updated license headers to each source file.

Change-Id: I01c7d0a6091d21bfa48720e9fb5624b77fa3db4a
Signed-off-by: Aravinda VK 
Reviewed-on: http://review.gluster.org/7311
Reviewed-by: Kotresh HR 
Reviewed-by: Prashanth Pai 
Tested-by: Gluster Build System

geo-rep: Fix ValueError - signal only works in main thread

2014-03-21T06:25:43+00:00

When a worker process not confirmed within 60 seconds of start
then monitor thread was terminated instead of stopping and restarting
the worker thread.
Before terminate monitor thread tries to add a signal handler for
SIGTERM to cleanup the stuff before terminate. Signal handling will
not work inside thread, so ValueError was raised.

This patch will not terminate monitor thread, instead only kills and
restarts the worker.

Change-Id: I14df26c0cc3097af29293c81536c13b86075e28f
BUG: 1078068
Signed-off-by: Aravinda VK 
Reviewed-on: http://review.gluster.org/7294
Reviewed-by: Venky Shankar 
Reviewed-by: Vijay Bellur 
Tested-by: Vijay Bellur

geo-rep: quick-fix for remote xtime set failed

2014-03-08T04:14:04+00:00

Remote xtime is required for failover/failback,
this patch is quick fix to avoid the OSError.

Code is masked out, this need to be resolved when
failover/failback is worked on.

Change-Id: If339d88a2ccd8ef18a3b3c015df765c93dcb020c
BUG: 1073844
Signed-off-by: Kotresh H R 
Reviewed-on: http://review.gluster.org/7206
Reviewed-by: Aravinda VK 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

geo-rep: Config file upgrade

2014-02-11T01:05:44+00:00

When old config file is used with new geo-rep, config item like
'georep_session_working_dir' was missing in old config file.

With this patch geo-rep sets the default value for new items.

Following config options supported:
  - georep_session_working_dir
  - gluster_params
  - ssh_command_tar

BUG: 1036539
Change-Id: I389c62e749f3b567f9ecf96d4b41367ef962c025
Signed-off-by: Aravinda VK 
Reviewed-on: http://review.gluster.org/6934
Reviewed-by: Venky Shankar 
Tested-by: Gluster Build System

gverify: Correcting variable names

2014-02-08T02:59:33+00:00

Change-Id: I851a50e0ebb7554b821fb591375bb6ebd6240ade
BUG: 1036539
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/6939
Reviewed-by: Venky Shankar 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

geo-replication: gverify should use disk_size and available_size

2014-02-06T15:31:23+00:00

"commit ef4bb42382a5c9c3ca0907136669a659c2a20311" was incomplete since
reduction in sizes doesn't really have any affect on the problem when
disks sizes are similar.

Actual fix is to verify disk_size and then verify available_size

Change-Id: Idb57f606df229c0bdaeda8852c74e84d29109ffc
BUG: 1020154
Signed-off-by: Harshavardhana 
Reviewed-on: http://review.gluster.org/6844
Tested-by: Gluster Build System 
Reviewed-by: Venky Shankar

gsyncd / geo-rep: invoke changelog process() on non-empty change list

2014-02-06T09:55:58+00:00

Change-Id: Ida4890abdc90d683a4a83127a1573bbb3829ea23
BUG: 1036539
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.org/6793
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

gsyncd / geo-rep: ignore DHTs sticky bit file during crawl

2014-02-06T09:54:52+00:00

Change-Id: Ide927759c6a3d5301475eac9f6e785aa901d426e
BUG: 1036539
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.org/6792
Reviewed-by: Kotresh HR 
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur

gsyncd / geo-rep: "patch" up missing stime

2014-02-06T09:53:08+00:00

In cases (mostly upgrade) of unavailability of "stime" key and
availability of "xtime" (slave's xtime), introduce "stime" key
on the fly by setting it to the value to "xtime".

Change-Id: Iaa424662d838154c8abc2cf00830c7f9d6be45ac
BUG: 1036539
Signed-off-by: Venky Shankar 
Reviewed-on: http://review.gluster.org/6791
Tested-by: Gluster Build System 
Reviewed-by: Vijay Bellur