Bug 1299814 - [geo-rep]: Disable recording SETXATTR for glusterfs.volume-mark on slave if changelogs are enabled
[geo-rep]: Disable recording SETXATTR for glusterfs.volume-mark on slave if c...
Status: NEW
Product: Red Hat Gluster Storage
Classification: Red Hat
Component: geo-replication (Show other bugs)
3.1
x86_64 Linux
unspecified Severity medium
: ---
: ---
Assigned To: Bug Updates Notification Mailing List
storage-qa-internal@redhat.com
: ZStream
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2016-01-19 05:28 EST by Rahul Hinduja
Modified: 2017-03-25 12:26 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed:
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Rahul Hinduja 2016-01-19 05:28:44 EST
Description of problem:
=======================

If changelogs are enabled on slave volume in geo-replication setup. All the bricks records changelogs every minute with following:

[root@dhcp47-94 changelogs]# cat CHANGELOG.1453199034
GlusterFS Changelog | version: v1.2 | encoding : 2
M00000000-0000-0000-0000-00000000000117
[root@dhcp47-94 changelogs]# 
[root@dhcp47-94 changelogs]# cat CHANGELOG.1453199094
GlusterFS Changelog | version: v1.2 | encoding : 2
M00000000-0000-0000-0000-00000000000117
[root@dhcp47-94 changelogs]# 

[root@dhcp47-94 changelogs]# ls | grep CHANGELOG | wc -l 
193
[root@dhcp47-94 changelogs]# getfattr -d -e hex -m . /rhs/brick1/b1/
getfattr: Removing leading '/' from absolute path names
# file: rhs/brick1/b1/
security.selinux=0x73797374656d5f753a6f626a6563745f723a676c7573746572645f627269636b5f743a733000
trusted.gfid=0x00000000000000000000000000000001
trusted.glusterfs.59c23385-352d-43db-b11a-0744cee045aa.xtime=0x569de19e00079e23
trusted.glusterfs.dht=0x000000010000000000000000ffffffff
trusted.glusterfs.volume-id=0xfb1b6f0c13ad4dd88c77443b6c0033fe
trusted.glusterfs.volume-mark.59c23385-352d-43db-b11a-0744cee045aa=0x010059c23385352d43dbb11a0744cee045aa00569de18d000b7f2e569e0eed

[root@dhcp47-94 changelogs]# ls | grep CHANGELOG | wc -l 
193
[root@dhcp47-94 changelogs]# getfattr -d -e hex -m . /rhs/brick1/b1/
getfattr: Removing leading '/' from absolute path names
# file: rhs/brick1/b1/
security.selinux=0x73797374656d5f753a6f626a6563745f723a676c7573746572645f627269636b5f743a733000
trusted.gfid=0x00000000000000000000000000000001
trusted.glusterfs.59c23385-352d-43db-b11a-0744cee045aa.xtime=0x569de19e00079e23
trusted.glusterfs.dht=0x000000010000000000000000ffffffff
trusted.glusterfs.volume-id=0xfb1b6f0c13ad4dd88c77443b6c0033fe
trusted.glusterfs.volume-mark.59c23385-352d-43db-b11a-0744cee045aa=0x010059c23385352d43dbb11a0744cee045aa00569de18d000b7f2e569e0f29

[root@dhcp47-94 changelogs]# ls | grep CHANGELOG | wc -l 
194
[root@dhcp47-94 changelogs]# 

The above are unnecessary to be processed which might delay the performance in rsync as the changelogs are picked in bulk. 

Version-Release number of selected component (if applicable):
==============================================================

glusterfs-3.7.5-16.el7rhgs.x86_64

How reproducible:
=================

1/1


Steps to Reproduce:
====================
1. Enable changelog on slave volume
2. Monitor changelogs for the bricks

Actual results:
===============

It records changelogs without any actual IO
Comment 2 Aravinda VK 2016-01-19 05:39:59 EST
No side effects except extra changelogs.

RCA:
Volume-mark xattr is updated by Geo-rep for every heartbeat. Changelog can skip recording SETXATTR if xattr key starts with "trusted.glusterfs.volume-mark" or any other internal xattrs create/update.

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