Bug 764042 (GLUSTER-2310)

Summary: georeplication
Product: [Community] GlusterFS Reporter: Csaba Henk <csaba>
Component: geo-replicationAssignee: Csaba Henk <csaba>
Status: CLOSED DUPLICATE QA Contact:
Severity: low Docs Contact:
Priority: low    
Version: mainlineCC: amarts, gluster-bugs, vijay
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Csaba Henk 2011-01-21 16:44:39 UTC 
A container for georeplication related commits. As of now, this boils down to:

- marker xlator
- aggregation of marks on client side
- glusterd's needed management extensions
- syncdaemon
Comment 1 Anand Avati 2011-01-27 08:16:55 UTC 
PATCH: http://patches.gluster.com/patch/6060 in master (adding libxlator, to ensure proper client side aggregation of marks by clustering translators)
Comment 2 Anand Avati 2011-01-27 08:17:01 UTC 
PATCH: http://patches.gluster.com/patch/6061 in master (adding marker translator)
Comment 3 Anand Avati 2011-01-27 08:17:07 UTC 
PATCH: http://patches.gluster.com/patch/6064 in master (implement "--client-pid" option which can forcibly set the pid value in messages to a predefined value)
Comment 4 Anand Avati 2011-01-27 08:17:12 UTC 
PATCH: http://patches.gluster.com/patch/6062 in master (volgen: add marker to brick volfiles)
Comment 5 Anand Avati 2011-01-27 08:17:17 UTC 
PATCH: http://patches.gluster.com/patch/6063 in master (mgmt/glusterd: restart bricks when syncdaemon is enabled)
Comment 6 Anand Avati 2011-01-27 08:17:25 UTC 
PATCH: http://patches.gluster.com/patch/6065 in master (adding syncdaemon)
Comment 7 Amar Tumballi 2011-01-28 04:03:26 UTC 
Got this error while building..


In file included from /usr/include/fcntl.h:205,
                 from ../../../../../libglusterfs/src/glusterfs.h:40,
                 from ../../../../../libglusterfs/src/common-utils.h:47,
                 from ../../../../../libglusterfs/src/xlator.h:34,
                 from ../../../../../xlators/mgmt/glusterd/src/glusterd-volgen.c:28:
In function ‘open’,
    inlined from ‘generate_brick_volfiles’ at ../../../../../xlators/mgmt/glusterd/src/glusterd-volgen.c:1477:
/usr/include/bits/fcntl2.h:51: error: call to ‘__open_missing_mode’ declared with attribute error: open with O_CREAT in second argument needs 3 arguments
make[4]: *** [glusterd-volgen.lo] Error 1



Git blame on glusterd-volgen.c:1477 says recent commits in syncdaemon has caused this.


66808736 (Csaba Henk         2011-01-27 05:23:33 +0000 1476)         if (marker) {
66808736 (Csaba Henk         2011-01-27 05:23:33 +0000 1477)                 ret = open (tstamp_file, O_WRONLY|O_CREAT|O_EXCL);
66808736 (Csaba Henk         2011-01-27 05:23:33 +0000 1478)                 if (ret == -1 && errno == EEXIST)
Comment 8 Amar Tumballi 2011-01-28 04:06:51 UTC 
Noticed that error didn't come when I did 'make install CFLAGS="-g -O0"' but it comes when you do just 'make install' or with flags what rpmbuild uses:

make install CFLAGS="-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic"
Comment 9 Csaba Henk 2011-02-03 13:32:49 UTC 

*** This bug has been marked as a duplicate of bug 1570 ***
Comment 10 Anand Avati 2011-02-11 03:17:22 UTC 
PATCH: http://patches.gluster.com/patch/6066 in master (volgen: specify a mode argument for open(2))