summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* features/marker: Replacing -1 with GF_CLIENT_PID_GSYNCD as part of code cleanup.release-3.2Mohammed Junaid2013-02-072-3/+7
| | | | | | | | | | | To Keep the code(rel-3.2) consistent with the master(rel-3.3) branch. Change-Id: I6a052a35abf022d5551ec9b59db0ae19f81f326d BUG: 769494 Signed-off-by: Mohammed Junaid <junaid@redhat.com> Reviewed-on: http://review.gluster.org/2969 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Csaba Henk <csaba@redhat.com>
* Merged in the fedora spec file to bring glusterfs into complianceJoe Julian2013-02-072-119/+404
| | | | | | | | | | | | | | | | | | with FSH and Redhat/Fedora requirements. Created a single spec that works for both 3.1 and 3.2 versions. Releases will now require changing the source url in configure.ac at the same time the release version is set. OLDBUG: 2970 BUG: 764702 Change-Id: If9067d673f34b70ccfbc9b723019cca31044347a Signed-off-by: Joe Julian <me@joejulian.name> Reviewed-on: http://review.gluster.org/702 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com>
* mgmt/glusterd: check brickinfo for NULL before dereferencing itRaghavendra Bhat2013-01-071-2/+4
| | | | | | | | | Change-Id: I59fed6adf13d6d72d79c986bf3578ca79b9756ac BUG: 781953 Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Reviewed-on: http://review.gluster.org/2648 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
* Most commands cause glusterd 3.2.7 to crash on Debian WheezyKaleb S. KEITHLEY2012-07-201-173/+118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When xlators/mgmt/glusterd/src/glusterd-rpc-ops.c is compiled with -O2 by gcc-4.7.1, the compiler optimizes away all the rsp.foo = bar statements because rsp goes out of scope at the end of each of the blocks in the switch statement in glusterd-op_send_cli_response(). That is: void *cli_rsp = NULL; switch(foo) { case BAR: { struct bar rsp = {0,}; /* init statements here are optimized away */ rsp.x = 0; rsp.y = 1; cli_rsp = &rsp; } ... } /* don't expect cli_rsp to point at anything meaningful here */ This particular idiom hasn't been a problem thus far, e.g. even with gcc-4.7.0 in Fedora17 also compiling with -O2. That not withstanding, using this idiom has probably always been on shaky ground; semantically it is correct that rsp goes out of scope at the end of the block. Note: I have not surveyed the source to see whether this idiom appears anywhere else. Note also that glusterd-op_send_cli_response() has been rewritten around the refactored gf_cli_rsp for 3.3.x and later and this particular bug is not in 3.3.x and later releases. BUG: 837684 Change-Id: I82f61ad2b9827c5b96af14b180a82c3ab350f559 Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com> Reviewed-on: http://review.gluster.com/3707 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Jeff Darcy <jdarcy@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com>
* nfs: memory leak fixesRajesh Amaravathi2012-07-124-7/+31
| | | | | | | | | | | | | | | | | | | | | | | * Fixes a leak of strdups in the parent_inode_loc_fill code path. * Introduces nfs_forget() to cleanup the ctx(addr of inode_q), a major leak involving inode_q hereby fixed. * Using the correct size to allocate fd_entry structures. * Free hashmatch in call_state during call state wipe and reassignment of the same. * Free cs->resolventry in the rename fop code path. Change-Id: I4ff556eba27f0b56ef824c0a86ab3d393d0d38c1 BUG: 835059 Signed-off-by: Rajesh Amaravathi <rajesh@redhat.com> Reviewed-on: http://review.gluster.com/3641 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com>
* cluster/afr: Avoid return value truncationv3.2.7Pranith Kumar K2012-06-101-1/+1
| | | | | | | | | Change-Id: I947316a0e1112bf3c7b7e21aefce96685a9d06db BUG: 804606 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/3543 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/afr: EIO should overwrite ENOENT in lookupPranith Kumar K2012-06-101-2/+3
| | | | | | | | | | | | | | | In case if lookup decides there is a gfid-mismatch, some enoents and self-heal cant remove the stale entry, it tells lookup to unwind with EIO but since ENOENT has more priority it is not over-written, this patch fixes that case. BUG: 765528 Change-Id: Ida03685629be8126365d660a16d8f9252f7ca030 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/2675 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/afr: Handle self-heal of files with holesv3.2.7qa2Pranith Kumar K2012-06-094-44/+65
| | | | | | | | | Change-Id: I7a560b3633151c9c1e5a787d1c808d8a396ee754 BUG: 765424 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/3540 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/afr: Don't reset split-brain when data-self-heal is offPranith Kumar K2012-06-063-1/+3
| | | | | | | | | Change-Id: Id73bdb84c3a1c0af6f0e2954514c3c07c65317bc BUG: 804606 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/3522 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/afr: Fix truncation of offset in 32-bit machinesPranith Kumar K2012-06-061-1/+2
| | | | | | | | | | | Thanks to the fix submitted by Ian Latter and Michael Change-Id: I84a62a9183c70be6da055d23e4d60cc97cfcf531 BUG: 809982 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/3521 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* Fix spelling errorJoe Julian2012-05-052-2/+2
| | | | | | | | | | | | | | "temporary" was misspelled s/tempaory/temporary/ Change-Id: Ia717bb8ef9f3440f4eb9dee42958b6815a96797c BUG: 818884 Signed-off-by: Joe Julian <me@joejulian.name> Reviewed-on: http://review.gluster.com/3273 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com> Reviewed-by: Jeff Darcy <jdarcy@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com>
* gerrit / rfc.sh: further trim of fetch targetCsaba Henk2012-04-231-1/+1
| | | | | | | | | | | | | | We _really_ don't need anything to be fetched from anywhere except for origin. We can do away with all other tracked repos (which still can be a nuisance eg. if they are in a VPN that we are not happen to be joined to). Change-Id: I98b375698f999500153e5a000dc8dac5005dfaec BUG: 764966 Signed-off-by: Csaba Henk <csaba@redhat.com> Reviewed-on: http://review.gluster.com/3163 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
* geo-rep: gsyncd: fix up fallback xtime for orphans on master sidev3.2.7qa1Csaba Henk2012-04-121-7/+8
| | | | | | | | | | | Change-Id: I2fa543b4bd317e06ea621ae968300ffb7223a68a BUG: 771787 Signed-off-by: Csaba Henk <csaba@gluster.com> Reviewed-on: http://review.gluster.com/2580 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaushik BV <kaushikbv@gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com> Reviewed-on: http://review.gluster.com/3103
* nfs: If entry is not found return ENOENT instead of entering an infinite loop.krishna2012-03-191-5/+9
| | | | | | | | | Change-Id: I06d471cb996e18fb656a6470de38d2401d846318 BUG: 772542 Signed-off-by: krishna <ksriniva@redhat.com> Reviewed-on: http://review.gluster.com/2793 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
* mount/fuse: Inherit direct-io-mode values from fds already opened, for a new ↵Raghavendra G2012-03-184-26/+173
| | | | | | | | | | | | | | | | fd being opened. When an fd is being opened, it inherits direct-io-mode characterstics (either enabled or disabled) from the fds already opened on inode. If none are opened and user has not specified the mode through cmdline options, default mode is used. Change-Id: I0c9e959100e9130e46bbd16d63eca278260635b4 BUG: 801 Signed-off-by: Raghavendra G <raghavendra@gluster.com> Reviewed-on: http://review.gluster.com/20 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Csaba Henk <csaba@redhat.com>
* libglusterfs: Destroy stack_lock in STACK_DESTROYPranith Kumar K2012-03-181-0/+1
| | | | | | | | | Change-Id: Icf910a899db136061a439a2e5e0951da55b92c00 BUG: 798907 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/2902 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
* nfs/server: allow deletion of files by defaultRajesh Amaravathi2012-03-181-2/+2
| | | | | | | | | | | | | | | this is needed for Micrsoft Windows-based Nfs clients which do not ignore ACCESS3_DELETE permission not being granted, while linux/unix systems ignore the same. behaviour on linux/unix systems remains unchanged Change-Id: I5d0057548735af542bc3c837b8e2d98c7b0716fa BUG: 790781 Signed-off-by: Rajesh Amaravathi <rajesh@redhat.com> Reviewed-on: http://review.gluster.com/2872 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Krishna Srinivas <krishna@gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
* cluster/afr: Perform conservative merge on dir with xattr split-brainv3.2.6p3Pranith Kumar K2012-03-143-10/+60
| | | | | | | | | Change-Id: I314cf7bbc8a451e42b6addf367f55e431ab0a85c BUG: 802637 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/2945 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/afr: Handle files w.o. xattrs and size mismatch.v3.2.6p2Pranith Kumar K2012-03-132-10/+44
| | | | | | | | | Change-Id: I31458538c738c85a026994584c937c99538fe069 BUG: 802637 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/2932 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/afr: Mark zero size file as sink in absense of xattrs.Pranith Kumar K2012-03-132-2/+53
| | | | | | | | | Change-Id: Ic9381f9caa7021a629afb7401e51a29686880bfe BUG: 802637 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/2931 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* replicate: backport of 0783ca994d9ea95fd9ab3dd95d6407918f19f255Anand Avati2012-03-124-45/+30
| | | | | | | | | | | This is a backport of patch 0783ca994d9ea95fd9ab3dd95d6407918f19f255 which is a fix for bug 798874 Change-Id: I0edac430b2f22dce121ff4889b563cbef14b09cd BUG: 798874 Reviewed-on: http://review.gluster.com/2928 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
* cluster/afr: Handle split-brain/all-fool xattrs for directoryPranith Kumar K2012-03-126-176/+158
| | | | | | | | | | | | In case of split-brain/all-fool xattrs perform conservative merge. Don't treat ignorant subvol as fool. Change-Id: I3044d388d816d79268fec170d202ef23e7d5bf1c BUG: 765528 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/2674 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
* geo-rep / gsyncd: fix cleanup of temporary mountsv3.2.6qa6v3.2.6Csaba Henk2012-03-091-13/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "finally" clause that was meant to cleanup after the temp mount has not covered the case of getting signalled (eg. by monitor, upon worker timing out). So here we "outsource" the cleanup to an ephemeral child process. Child calls setsid(2) so it won't be bothered by internal process management. We use a pipe in between worker and the cleanup child; when child sees the worker end getting closed, it performs the cleanup. Worker end can get closed either because worker closes it (normal case), or because worker has terminated (faulty case) -- thus as bonus, we get a nice uniform handling with no need to differentiate between normal and faulty cases. The faulty case that was seen IRL -- ie., users of maintainance mounts hang in chdir(2) to mount point -- can be simulated for testing purposes by applying the following patch: diff --git a/xlators/mount/fuse/src/fuse-bridge.c b/xlators/mount/fuse/src/fuse-bridge.c index acd3c68..1ce5dc1 100644 --- a/xlators/mount/fuse/src/fuse-bridge.c +++ b/xlators/mount/fuse/src/fuse-bridge.c @@ -2918,7 +2918,7 @@ fuse_init (xlator_t *this, fuse_in_header_t *finh, void *msg) if (fini->minor < 9) *priv->msg0_len_p = sizeof(*finh) + FUSE_COMPAT_WRITE_IN_SIZE; #endif - ret = send_fuse_obj (this, finh, &fino); + ret = priv->client_pid_set ? 0 : send_fuse_obj (this, finh, &fino); if (ret == 0) gf_log ("glusterfs-fuse", GF_LOG_INFO, "FUSE inited with protocol versions:" Change-Id: I1172bf16ac1006bad48958655572155820e5138d BUG: 786291 Signed-off-by: Csaba Henk <csaba@redhat.com> Reviewed-on: http://review.gluster.com/2908 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Venky Shankar <vshankar@redhat.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* gsyncd: geo-rep: Reduce excessive verbosity in loggingv3.2.6qa5Venky Shankar2012-03-061-8/+23
| | | | | | | | | | | | | Backport logging enhancement from master. This patch reduces per-second logging to per-minute logging during crawl Change-Id: Ie51207e6dd8a0257e59f90239cc6960803673223 BUG: 789078 Signed-off-by: Venky Shankar <vshankar@redhat.com> Reviewed-on: http://review.gluster.com/2879 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* xlator/lib: Drain out sent requests to avoid multiple STACK_UNWINDsVijay Bellur2012-03-062-14/+25
| | | | | | | | | Change-Id: Id48c468934ac64ec52e279ded8949c2a7a760598 BUG: 789078 Signed-off-by: Vijay Bellur <vijay@gluster.com> Reviewed-on: http://review.gluster.com/2871 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Venky Shankar <vshankar@redhat.com>
* Refactor xlator/lib for better readability & maintainability.v3.2.6qa4Vijay Bellur2012-03-022-202/+161
| | | | | | | | | Change-Id: I26532f1bf7cab58b5870a6117f2a62288ee1f23d BUG: 789078 Signed-off-by: Vijay Bellur <vijay@gluster.com> Reviewed-on: http://review.gluster.com/2859 Reviewed-by: Pranith Kumar Karampuri <pranithk@gluster.com> Tested-by: Gluster Build System <jenkins@build.gluster.com>
* geo-rep / syncdaemon: make the timeout for establishing the connection to ↵Csaba Henk2012-03-012-1/+2
| | | | | | | | | | | | | | slave configurable It can be set through the connection-timeout tunable but we keep it hidden, intended as a workaround for some special scenarios not for general use. Change-Id: I31f9fa3873afa7babc2106ee34484123a01bdc57 BUG: 789078 Signed-off-by: Csaba Henk <csaba@redhat.com> Reviewed-on: http://review.gluster.com/2840 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* protocol/client: writev(): don't unwind if rpc_submit failedAmar Tumballi2012-02-201-2/+9
| | | | | | | | | | | | because rpc_submit does call 'cbk()' from inside if sending fails. no need to unwind in higher layers Change-Id: I08611ce742fba7032f68e3fe0f52a78a18a5714e Signed-off-by: Amar Tumballi <amar@gluster.com> BUG: 767359 Reviewed-on: http://review.gluster.com/2770 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* nfs/server: Add counters for hard resolutionv3.2.6qa3Vijay Bellur2012-02-143-4/+85
| | | | | | | | | Change-Id: Id7b26731b750b63d645222dde18cd9fe4ed54018 BUG: 785314 Signed-off-by: Vijay Bellur <vijay@gluster.com> Reviewed-on: http://review.gluster.com/2745 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
* support for nano second resolution for mtime,ctime,atime attributes.v3.2.6qa2krishna2012-02-082-22/+23
| | | | | | | | | Change-Id: Id5078f270d0fec280b53d4aa7b16bbaf42a2df05 BUG: 784095 Signed-off-by: krishna <ksriniva@redhat.com> Reviewed-on: http://review.gluster.com/2729 Reviewed-by: Anand Avati <avati@gluster.com> Tested-by: Gluster Build System <jenkins@build.gluster.com>
* Fixes bug-765574, failure to build from hardened sourceLouis Zuckerman2012-01-261-2/+2
| | | | | | | | | | | | | | | | | | | | | | | Glusterfs 3.2.5 failed to build on Debian which prevented the Debian project from updating their package to the latest release version. There were two changes needed, in both cases just adding a printf format string argument to a call to gf_log. The compiler error was that the argument was neither a string literal nor a format string. It was instead a pointer to a string. The obvious solution was to add a format argument of "%s" before the string pointer argument. This has been tested successfully on Debian Sid. Change-Id: I62c8a46938184c8a37224c2481bd1de00369ab19 Signed-off-by: louis zuckerman <glusterbugs@louiszuckerman.com> Reviewed-on: http://review.gluster.com/2598 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com> Reviewed-by: Amar Tumballi <amar@gluster.com>
* mgmt/glusterd: do not assert if op_ctx is NULLRaghavendra Bhat2012-01-241-1/+0
| | | | | | | | | | | | | | In glusterd_op_send_cli_response we are asserting if op_ctx is NULL. But in handler functions of different ops, we are sending op_ctx NULL which is valid, but still glusterd will assert. Change-Id: I37824c694d14e87a234389d639dfb000592f9513 BUG: 783913 Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Reviewed-on: http://review.gluster.com/2678 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Krishnan Parthasarathi <kp@gluster.com> Reviewed-by: Anand Avati <avati@gluster.com>
* cluster/dht: Exit clean when assert_no_child_down is enabledVijay Bellur2012-01-241-1/+2
| | | | | | | | | Change-Id: If90b1080edcf3792f8590492b585a6dd48fac18e BUG: 783249 Signed-off-by: Vijay Bellur <vijay@gluster.com> Reviewed-on: http://review.gluster.com/2682 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@gluster.com>
* features/marker: xtime should not be updated for modifications done by ↵Junaid2012-01-241-1/+7
| | | | | | | | | | | special client. Change-Id: I4b34d449b3ab08e2189fe1ff088299617b640206 BUG: 769494 Signed-off-by: Junaid <junaid@redhat.com> Reviewed-on: http://review.gluster.com/2564 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Csaba Henk <csaba@redhat.com>
* performance/io-cache: if the reconfigure option given is wrong, then unlock ↵Raghavendra Bhat2012-01-191-9/+10
| | | | | | | | | | | | | | | | and return In io-cache reconfigure we take the lock on ioc_table_t, and then get the options from the dict. If some value of some option given is wrong, we are returning without unlocking, leading to the hang. Change-Id: I5b3ebf0a8e9e2470d240b503167ff1731ff1b613 BUG: 782286 Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Reviewed-on: http://review.gluster.com/2649 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Harshavardhana <fharshav@redhat.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* core/stack: perform locked windsv3.2.6qa1Anand Avati2012-01-111-16/+35
| | | | | | | | | | | | | In configurations like pump, where there is a cluster translator on top of io-threads, there are situations where two concurrent stack-winds can be performed on the same call stack in multiple threads. This patch holds locks during the call frame list manipulation Change-Id: I51539210dc8101f7a80cf9bc103b5eff0c86dc9f BUG: 765522 Reviewed-on: http://review.gluster.com/2629 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@gluster.com>
* cluster/stripe: Fix block size reported in pathinfo xattrVenky Shankar2012-01-031-16/+54
| | | | | | | | | | | | | | | | Block size (or stripe size) is reported when request is made for pathinfo xattr (trusted.glusterfs.pathinfo) for a striped volume. When the block size is changed, the new block size is reported for files created with older block size. This can be confusing ( or even erroneous ) for applications relying on pathinfo xattr to find chunks in backend. Change-Id: I79cb6721bbd33f44c3fada4dd52e459d2c128e24 BUG: 766530 Signed-off-by: Venky Shankar <vshankar@redhat.com> Reviewed-on: http://review.gluster.com/2549 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Shishir Gowda <shishirng@gluster.com>
* cluster/stripe: copy the gfid from the dentry structure to locshishir gowda2012-01-031-0/+2
| | | | | | | | | | | | | | | | In stripe_readdirp_cbk we used to send stat call on each of the entry we have got from readdirp, by creating a new inode (but without setting the gfid in loc). Protocol/client asserts if it cannot find gfid in either loc or loc->inode. This patch copies the gfid of the entry from the stat structure to loc. Change-Id: I3f124fb816fc6e3f9239a24fc657b1dbee34d3a2 BUG: 768840 Signed-off-by: shishir gowda <shishirng@gluster.com> Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com> Reviewed-on: http://review.gluster.com/2544 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* Cluster/stripe setattr: Wind only to first_child for special filesshishir gowda2012-01-031-1/+9
| | | | | | | | | Change-Id: Ia60207199ee702f22222b67a1f1d62b8093e871c BUG: 765539 Signed-off-by: shishir gowda <shishirng@gluster.com> Reviewed-on: http://review.gluster.com/723 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* CONTRIBUTING: add a DCOAnand Avati2011-12-051-0/+25
| | | | | | | | | | | | | | The new CONTRIBUTING file has a Developer Certificate of Origin with instructions for contributors to ensure 'Signed-off-by:' line in the commit log (git commit -s). Corresponding changes also done in Gerrit to reject patches which do not have Signed-off-by: line. Change-Id: Ia3e1e8d3cfd4b32e4cfd4d2df91c6dbd57e2f60e BUG: 3234 Signed-off-by: Anand Avati <avati@gluster.com> Reviewed-on: http://review.gluster.com/768 Reviewed-by: Vijay Bellur <vijay@gluster.com> Tested-by: Gluster Build System <jenkins@build.gluster.com>
* storage/posix: posix getxattr log enhancementRajesh Amaravathi2011-11-301-4/+4
| | | | | | | | | | | | | | Now the key is logged with getxattr failure. BUG: 3283 Change-Id: Idc09b26c9196dded314df78ce3c765573160452b Reviewed-on: http://review.gluster.com/373 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@gluster.com> Reviewed-on: http://review.gluster.com/755 Reviewed-by: Harshavardhana <fharshav@redhat.com> Tested-by: Harshavardhana <fharshav@redhat.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* rpm: fix rpmbuild for opensuseHarshavardhana2011-11-231-0/+3
| | | | | | | | | Change-Id: I8adb8d63b0c219d214a6d4cb1d3e12d4d5bc0b2b BUG: 2925 Signed-off-by: Harshavardhana <fharshav@redhat.com> Reviewed-on: http://review.gluster.com/684 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@gluster.com>
* features/marker-quota: Use stack_wind_cookie in place of stack_wind to ↵v3.2.5qa9v3.2.5Junaid2011-11-111-11/+12
| | | | | | | | | | | | | preserve the cookie value. In marker_rename, some of the call back functions make use of cookies sent by the calling functions. In case of stack_wind the frame->cookie is over written with the address of new frame. Change-Id: I8ec98f3305700e2c3295a10dff159ca6a19a380a BUG: 3808 Reviewed-on: http://review.gluster.com/717 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/afr: honor the xattr set by data-self-healPranith Kumar K2011-11-112-1/+4
| | | | | | | | Change-Id: I2b8c3a2d7771463ef94892f2a78fc464aba703ee BUG: 3805 Reviewed-on: http://review.gluster.com/708 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
* cluster/distribute: Trigger selfheal only if rmdir succeeded onceshishir gowda2011-11-112-3/+7
| | | | | | | | | | | A EACCES error also should not trigger a selfheal. Only if rmdir succeeded on any subvol, a selfheal should be triggered Change-Id: I1dd39db830e9396b1dc6d6edbaa6de6ea83e9070 BUG: 3786 Reviewed-on: http://review.gluster.com/715 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Amar Tumballi <amar@gluster.com>
* fusermount: Build problem fixed with new glibcHarshavardhana2011-11-101-1/+1
| | | | | | | | | Change-Id: Id25e688d3dbecb74d820388faec5ee5041f21630 BUG: 3797 Signed-off-by: Harshavardhana <fharshav@redhat.com> Reviewed-on: http://review.gluster.com/692 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Csaba Henk <csaba@gluster.com>
* afr: Fixed backgroundness detection in self-heal algo.Krishnan Parthasarathi2011-11-101-0/+1
| | | | | | | | Change-Id: I9888d8a0b86fdaf6589885766f2de7222d8c8ba2 BUG: 3802 Reviewed-on: http://review.gluster.com/705 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Pranith Kumar Karampuri <pranithk@gluster.com>
* cluster/distribute lookup: Request layout info for dir'sv3.2.5qa8shishir gowda2011-11-101-6/+6
| | | | | | | | Change-Id: I17e40beb2c6e77a802fb987bed6f0e4be8010ef6 BUG: 3803 Reviewed-on: http://review.gluster.com/707 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Amar Tumballi <amar@gluster.com>
* cluster/afr: Handle split-brain/all-fool xattrs for directoryv3.2.5qa7Pranith Kumar K2011-11-095-35/+67
| | | | | | | | Change-Id: I058ed91494e1a9bbcf5e6e6c49e8ee4f7e014e23 BUG: 3796 Reviewed-on: http://review.gluster.com/695 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@gluster.com>
* cluster/distribute stat: Take ref on inode before to prevent crash.shishir gowda2011-11-091-16/+21
| | | | | | | | | | | | | | | | Possible situation is a forget call which unref's the inode, and the layout gets freed before we access it in stat. By taking a ref before we call ctx get, the layout is going to be persistent till we unref it. Also use local call_cnt var instead of layout->cnt when wind is involved Change-Id: Ie04a7afd9e2c89cc14c08dd923d06c5f0193beb8 BUG: 3730 Reviewed-on: http://review.gluster.com/687 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Amar Tumballi <amar@gluster.com>
generated by cgit (git 2.34.1) at 2024-04-18 02:23:34 +0000