Bug 763302 (GLUSTER-1570) - geosync related changes
Summary: geosync related changes
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-1570
Product: GlusterFS
Classification: Community
Component: geo-replication
Version: mainline
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Csaba Henk
QA Contact:
URL:
Whiteboard:
: GLUSTER-2310 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-09-08 13:48 UTC by Csaba Henk
Modified: 2015-12-01 16:45 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:


Attachments (Terms of Use)

Description Csaba Henk 2010-09-08 13:48:33 UTC

Comment 1 Vijay Bellur 2010-09-17 03:07:30 UTC
PATCH: http://patches.gluster.com/patch/4807 in master (cli: add a simplistic option parser, convert "mode script" to "--mode=script")

Comment 2 Vijay Bellur 2010-09-17 03:07:37 UTC
PATCH: http://patches.gluster.com/patch/4808 in master (cli: add "--remote-host=<hostname>" option)

Comment 3 Vijay Bellur 2010-09-17 03:07:41 UTC
PATCH: http://patches.gluster.com/patch/4809 in master (cli: make cli behave nicely with batch commands piped in in stdin)

Comment 4 Vijay Bellur 2010-09-17 03:07:47 UTC
PATCH: http://patches.gluster.com/patch/4810 in master (cli: add system namespace)

Comment 5 Vijay Bellur 2010-09-17 03:07:51 UTC
PATCH: http://patches.gluster.com/patch/4811 in master (cli: added system:: getspec)

Comment 6 Vijay Bellur 2010-09-17 03:07:56 UTC
PATCH: http://patches.gluster.com/patch/4812 in master (cli: added system:: portmap brick2port)

Comment 7 Vijay Bellur 2010-09-17 03:08:00 UTC
PATCH: http://patches.gluster.com/patch/4813 in master (portmap: make grounds for typed service mapping)

Comment 8 Vijay Bellur 2010-09-17 03:08:05 UTC
PATCH: http://patches.gluster.com/patch/4814 in master (portmap: add support for registering multiple bricks on one port)

Comment 9 Vijay Bellur 2010-09-17 08:40:17 UTC
PATCH: http://patches.gluster.com/patch/4832 in master (cli: fix incorrect return value for system:: getspec)

Comment 10 Vijay Bellur 2010-09-18 03:34:59 UTC
PATCH: http://patches.gluster.com/patch/4844 in master (glusterd: refactor service termination)

Comment 11 Csaba Henk 2011-02-03 13:32:49 UTC
*** Bug 2310 has been marked as a duplicate of this bug. ***

Comment 12 Anand Avati 2011-02-11 03:17:41 UTC
PATCH: http://patches.gluster.com/patch/6150 in master (syncdaemon: Removing the simplecfg.py entry from the Makefile.am, since it is removed.)

Comment 13 Anand Avati 2011-02-11 03:17:46 UTC
PATCH: http://patches.gluster.com/patch/6145 in master (syncdaemon: config revamp #1: drop simplecfg, switch to stdlib's ConfigParser)

Comment 14 Anand Avati 2011-02-11 03:17:53 UTC
PATCH: http://patches.gluster.com/patch/6144 in master (syncdaemon: config revamp #2: implement canonicalized urls, use them in config file sections)

Comment 15 Anand Avati 2011-02-11 03:17:58 UTC
PATCH: http://patches.gluster.com/patch/6146 in master (syncdaemon: config revamp #3: implement command line get/set/del interface to configuration)

Comment 16 Anand Avati 2011-02-11 03:18:10 UTC
PATCH: http://patches.gluster.com/patch/6160 in master (gsync: cli support for gsyncd.)

Comment 17 Anand Avati 2011-02-11 13:35:47 UTC
PATCH: http://patches.gluster.com/patch/6161 in master (syncdaemon: configinterface.py file moved under xlators/features/marker/utils/syncdaemon/ directory.)

Comment 18 Anand Avati 2011-02-13 19:39:47 UTC
PATCH: http://patches.gluster.com/patch/6167 in master (syncdaemon: Create conf directory if not preset)

Comment 19 Anand Avati 2011-02-13 19:39:53 UTC
PATCH: http://patches.gluster.com/patch/6168 in master (syncdaemon: conf file to be read properly)

Comment 20 Anand Avati 2011-02-13 19:39:57 UTC
PATCH: http://patches.gluster.com/patch/6169 in master (syncdaemon: Using /usr/local/libexec/python/syncdaemon as the default directory where the python scripts are installed.)

Comment 21 Lakshmipathi G 2011-02-14 09:39:17 UTC
following two  issues found with gsyncqa9 and qa10
=========================
1) RawConfigparser creation issue where dict fails 
---------
[2011-02-14 23:44:11.458415] E [gsyncd:140:exception] <top>: FAIL: 
Traceback (most recent call last):
  File "./gsyncd.py", line 136, in main
    main_i()
  File "./gsyncd.py", line 222, in main_i
    gcnf = GConffile(rconf['config_file'], canon_peers)
  File "/usr/local/libexec/python/syncdaemon/configinterface.py", line 13, in __init__
    self.config = ConfigParser.RawConfigParser({},dict)
TypeError: __init__() takes at most 2 arguments (3 given)
failed with TypeError.
=========================
2)when starting gsyncd with -LDEBUG  -l /tmp/master.log  master fails with  log message
===
[2011-02-14 23:27:20.126439] E [gsyncd:140:exception] <top>: FAIL: 
Traceback (most recent call last):
  File "./gsyncd.py", line 136, in main
    main_i()
  File "./gsyncd.py", line 256, in main_i
    remote.connect_remote(go_daemon='done')
  File "/usr/local/libexec/python/syncdaemon/resource.py", line 417, in connect_remote
    return self.start_fd_client(*self.fd_pair)
  File "/usr/local/libexec/python/syncdaemon/resource.py", line 403, in start_fd_client
    if opts['deferred']:
KeyError: 'deferred'
=========

Comment 22 Anand Avati 2011-02-14 19:02:28 UTC
PATCH: http://patches.gluster.com/patch/6176 in master (syncdaemon: make configparser code work with all supported python versions)

Comment 23 Anand Avati 2011-02-14 19:02:34 UTC
PATCH: http://patches.gluster.com/patch/6177 in master (syncdaemon: fix KeyError)

Comment 24 Anand Avati 2011-02-14 19:02:40 UTC
PATCH: http://patches.gluster.com/patch/6173 in master (syncdaemon: config-get-all option add to gsync cli.)

Comment 25 Anand Avati 2011-02-14 19:32:06 UTC
PATCH: http://patches.gluster.com/patch/6182 in master (syncdaemon: fix some pidfile issues)

Comment 26 Anand Avati 2011-02-14 19:32:11 UTC
PATCH: http://patches.gluster.com/patch/6183 in master (Changes to include geo-sync in spec)

Comment 27 Anand Avati 2011-02-16 02:44:31 UTC
PATCH: http://patches.gluster.com/patch/6184 in master (syncdaemon: Moved the gsync start code to cli.)

Comment 28 Anand Avati 2011-02-16 02:44:37 UTC
PATCH: http://patches.gluster.com/patch/6188 in master (syncdaemon: set the default value of timeout to 120sec)

Comment 29 Anand Avati 2011-02-16 02:44:42 UTC
PATCH: http://patches.gluster.com/patch/6191 in master (syncdaemon: Check for ctx context in the glusterd_op_stage_gsyc_set().)

Comment 30 Anand Avati 2011-02-16 02:44:47 UTC
PATCH: http://patches.gluster.com/patch/6192 in master (syncdaemon: change pidfile handling approach)

Comment 31 Anand Avati 2011-02-16 02:44:54 UTC
PATCH: http://patches.gluster.com/patch/6194 in master (syncdaemon: Checking for the existence of gsyncd by trying to aquire a lock on the pidfile.)

Comment 32 Anand Avati 2011-03-01 07:06:13 UTC
PATCH: http://patches.gluster.com/patch/6254 in master (Gsync : Do not expose volume gsync, when geo-replication is disabled)

Comment 33 Vijay Bellur 2011-03-10 12:39:20 UTC
PATCH: http://patches.gluster.com/patch/6368 in master (syncdaemon: add support from dumping urls in canonical and escaped canonical form)

Comment 34 Vijay Bellur 2011-03-10 12:39:32 UTC
PATCH: http://patches.gluster.com/patch/6370 in master (syncdaemon: improve loglevel handling)

Comment 35 Vijay Bellur 2011-03-10 12:39:36 UTC
PATCH: http://patches.gluster.com/patch/6371 in master (Revert "syncdaemon: Create conf directory if not preset")

Comment 36 Vijay Bellur 2011-03-10 12:39:49 UTC
PATCH: http://patches.gluster.com/patch/6374 in master (syncademon: fortify handling of failed entries queries)

Comment 37 Vijay Bellur 2011-03-10 12:39:58 UTC
PATCH: http://patches.gluster.com/patch/6376 in master (syncdaemon: don't follow symlinks on entries query)

Comment 38 Vijay Bellur 2011-03-14 09:46:30 UTC
PATCH: http://patches.gluster.com/patch/6422 in release-3.1 (Disable geo-replication build)

Comment 39 Joe Julian 2011-03-15 20:04:56 UTC
You missed removing %{_libexecdir}/gsyncd and %{_libexecdir}/python/syncdaemon/*  (lines 186 & 187) from the spec.in in 3.1.3.

Comment 40 Vijay Bellur 2011-03-18 03:27:25 UTC
PATCH: http://patches.gluster.com/patch/6493 in master (syncdaemon: don't try to set log level to None)

Comment 41 Vijay Bellur 2011-03-25 08:12:02 UTC
PATCH: http://patches.gluster.com/patch/6563 in master (Gsyncd: fix a typo for the command-line arg of gluster process)

Comment 42 Anand Avati 2011-04-11 07:52:31 UTC
PATCH: http://patches.gluster.com/patch/6785 in master (syncdaemon: minor fixes)

Comment 43 Anand Avati 2011-04-13 08:42:46 UTC
PATCH: http://patches.gluster.com/patch/6847 in master (syncdaemon: minor fixes)

Comment 44 Anand Avati 2011-04-13 08:42:51 UTC
PATCH: http://patches.gluster.com/patch/6848 in master (syncdaemon: fix pidfile deletion on failed pidfile acquisition)

Comment 45 Anand Avati 2011-04-13 08:42:56 UTC
PATCH: http://patches.gluster.com/patch/6849 in master (syncdaemon: return exit code properly)

Comment 46 Anand Avati 2011-04-15 04:30:55 UTC
PATCH: http://patches.gluster.com/patch/6880 in master (syncdaemon: work around python feature of allowing double leading slashes in canonicalized paths)

Comment 47 Anand Avati 2011-04-15 04:31:00 UTC
PATCH: http://patches.gluster.com/patch/6881 in master (syncdaemon: split gluster_command to make it possible to set options for glusterfs)

Comment 48 Anand Avati 2011-04-15 07:58:40 UTC
PATCH: http://patches.gluster.com/patch/6893 in master (syncdaemon: name tempdirs used by gsyncd more characteristically to ease debugging)

Comment 49 Anand Avati 2011-04-16 08:10:30 UTC
PATCH: http://patches.gluster.com/patch/6905 in master (syncdaemon: when getting a single config option, don't display the option name, just the value)

Comment 50 Anand Avati 2011-04-18 08:31:54 UTC
PATCH: http://patches.gluster.com/patch/6936 in master (mgmt/glusterd: change the cli output of geo-rep start/stop to resemble the output of volume start/stop)

Comment 51 Anand Avati 2011-09-08 04:06:59 UTC
CHANGE: http://review.gluster.com/319 (Change-Id: I559e6a0709b8064cfd54c693e289c741f9c4c4ab) merged in master by Anand Avati (avati)

Comment 52 Anand Avati 2011-09-12 10:20:44 UTC
CHANGE: http://review.gluster.com/320 (Also add __codecheck script which can verify if source is OK at the) merged in master by Vijay Bellur (vijay)

Comment 53 Anand Avati 2012-05-19 12:27:49 UTC
CHANGE: http://review.gluster.com/3375 (geo-rep / gsyncd: typo fix) merged in master by Vijay Bellur (vijay)


Note You need to log in before you can comment on or make changes to this bug.