Bug 1406743

Summary: BitRot :- bit-rot.signature and bit-rot.version xattr should not be set if bitrot is not enabled on volume
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Riyas Abdulrasak <rnalakka>
Component: bitrotAssignee: Kotresh HR <khiremat>
Status: CLOSED DUPLICATE QA Contact: Sweta Anandpara <sanandpa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rhgs-3.1CC: bugs, hchiramm, khiremat, mzywusko, olim, rabhat, racpatel, rhs-bugs, rmekala, storage-qa-internal
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1359599 Environment:
Last Closed: 2016-12-21 11:25:44 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1224216, 1298258, 1359599    
Bug Blocks: 1186580, 1223636    

Description Riyas Abdulrasak 2016-12-21 11:24:18 UTC 
+++ This bug was initially created as a clone of Bug #1359599 +++

+++ This bug was initially created as a clone of Bug #1224216 +++

+++ This bug was initially created as a clone of Bug #1207152 +++

Description of problem:
=======================
event though bitrot is not enabled on volume it is setting xattr bit-rot.signature and bit-rot.version

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

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

Steps to Reproduce:
===================
1. create and mount volume.
Volume Name: tier_distrep
Type: Distributed-Replicate
Volume ID: ad81ef54-70ec-41f2-800c-17e5025acb26
Status: Started
Number of Bricks: 5 x 2 = 10
Transport-type: tcp
Bricks:
Brick1: rhs-client38:/pavanbrick2/tier_distrep/hb2m
Brick2: rhs-client37:/pavanbrick2/tier_distrep/hb2
Brick3: rhs-client37:/pavanbrick2/tier_distrep/hb1m
Brick4: rhs-client44:/pavanbrick2/tier_distrep/hb1
Brick5: rhs-client44:/pavanbrick1/tier_distrep/b1
Brick6: rhs-client37:/pavanbrick1/tier_distrep/b1m
Brick7: rhs-client37:/pavanbrick1/tier_distrep/b2
Brick8: rhs-client38:/pavanbrick1/tier_distrep/b2m
Brick9: rhs-client44:/pavanbrick1/tier_distrep/b3m
Brick10: rhs-client38:/pavanbrick1/tier_distrep/b3

2. create few files.
3. verify xattr for files in backend

[root@rhs-client44 try]# getfattr -d -m . -e hex /pavanbrick1/tier_distrep/b1/f12
getfattr: Removing leading '/' from absolute path names
# file: pavanbrick1/tier_distrep/b1/f12
trusted.gfid=0x9142b9dfe68c47149d95dfd037177723
trusted.glusterfs.bit-rot.signature=0xff000000000000000000000000000000
trusted.glusterfs.bit-rot.version=0x010000000000000055152f6c000566ef


Actual results:
===============
even though bitrot is not enabled on volume it is setting xattr bit-rot.signature and bit-rot.version

--- Additional comment from Kotresh HR on 2016-07-25 02:22:35 EDT ---

Patch Posted:

http://review.gluster.org/14442

--- Additional comment from Vijay Bellur on 2016-07-25 03:31:37 EDT ---

REVIEW: http://review.gluster.org/14442 (features/bit-rot-stub: bring in optional versioning) posted (#2) for review on master by Kotresh HR (khiremat)

--- Additional comment from Vijay Bellur on 2016-07-28 12:34:30 EDT ---

REVIEW: http://review.gluster.org/14442 (features/bit-rot-stub: bring in optional versioning) posted (#3) for review on master by Kotresh HR (khiremat)

--- Additional comment from Kotresh HR on 2016-08-11 08:14:31 EDT ---
Comment 2 Riyas Abdulrasak 2016-12-21 11:25:44 UTC 

*** This bug has been marked as a duplicate of bug 1224216 ***