diff options
| author | Kotresh HR <khiremat@redhat.com> | 2015-09-21 14:51:13 +0530 |
|---|---|---|
| committer | Venky Shankar <vshankar@redhat.com> | 2015-11-08 23:09:08 -0800 |
| commit | dc8d65381bbead0a00d045b7040e0cd86d80955b (patch) | |
| tree | 00dba743a0e8e20648646450e699b60ef3c7b0a8 /tests/geo-rep.rc | |
| parent | a37eb179111b848925928009903cc4a98c0781e3 (diff) | |
geo-rep: Fix syncing chown in xsync crawl
GEO-REP INTEROP WITH SHARD FEATURE
Problem:
The sequence of entry creation and chown in master
is recorded as creation of entry with resulted
user:group in xsync changelog. During sync, entry
creation is always split into two ops, MKNOD and
SETATTR. Hence the issue is not being hit otherwise
it would have failed with EPERM if parent is owned
by different user. But with shard translator being
enabled on slave, doing entry creation with MKNOD and
SETATTR is not allowed, SETATTR fails as it accesses
inode structure which is not linked.
Solution:
The sequence of entry creation and chown in master
should be recorded as MKNOD and SETATTR separately always
and do entry creation with single op in gfid-access
xlator. The gfid-access patch will be sent separately.
Change-Id: I93e554bf9342397a7660503f5128e9709f8a0cd8
BUG: 1265148
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Reviewed-on: http://review.gluster.org/12205
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Aravinda VK <avishwan@redhat.com>
Diffstat (limited to 'tests/geo-rep.rc')
| -rw-r--r-- | tests/geo-rep.rc | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/tests/geo-rep.rc b/tests/geo-rep.rc index 208069c6918..e811f9a5b76 100644 --- a/tests/geo-rep.rc +++ b/tests/geo-rep.rc @@ -58,6 +58,16 @@ function create_data() # Rename - Dir mv ${master_mnt}/${prefix}_d3 ${master_mnt}/${prefix}_d4 + + # chown + touch ${master_mnt}/${prefix}_chown_f1 + chown 1000:1000 ${master_mnt}/${prefix}_chown_f1 +} + +function chown_file_ok() +{ + local file_owner=$(stat --format "%u:%g" "$1" 2>/dev/null) + if test "X$file_owner" != "X1000:1000"; then return 1;fi } function regular_file_ok() |