Bug 1200271

Summary:	Upcall: xlator options for Upcall xlator
Product:	[Community] GlusterFS	Reporter:	Soumya Koduri <skoduri>
Component:	upcall	Assignee:	Soumya Koduri <skoduri>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	bugs, gluster-bugs
Target Milestone:	---	Keywords:	Reopened, Triaged
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	glusterfs-3.8rc2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	1217723 (view as bug list)		Environment:
Last Closed:	2016-06-16 12:43:17 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:
Bug Blocks:	1200262, 1217711, 1217723

Description Soumya Koduri 2015-03-10 07:43:56 UTC

Description of problem:

This bug is to track changes for upcall xlator options required to enable cache_invalidation and lease_locks support.

Also this will track the changes required to handle cache_invalidation for all the fops which are missed out in the bug1200262.

Comment 1 Soumya Koduri 2015-03-16 13:20:52 UTC

In addition - will address below comments received as part of
http://review.gluster.org/#/c/9535 - 

* skip cache_invalidation notifications only in case of internal Gluster fops. Can check using frame->root->pid.

Comment 2 Anand Avati 2015-03-24 05:04:42 UTC

REVIEW: http://review.gluster.org/9975 (Upcall: Added xlator options to enable cache-invalidation) posted (#1) for review on master by soumya k (skoduri)

Comment 3 Anand Avati 2015-03-25 07:37:56 UTC

REVIEW: http://review.gluster.org/9975 (Upcall: Added xlator options to enable cache-invalidation) posted (#2) for review on master by soumya k (skoduri)

Comment 4 Anand Avati 2015-03-25 07:43:51 UTC

REVIEW: http://review.gluster.org/9975 (Upcall: Added xlator options to enable cache-invalidation) posted (#3) for review on master by soumya k (skoduri)

Comment 5 Anand Avati 2015-03-26 07:00:43 UTC

REVIEW: http://review.gluster.org/9975 (Upcall: Added xlator options to enable cache-invalidation) posted (#4) for review on master by soumya k (skoduri)

Comment 6 Anand Avati 2015-03-26 07:09:42 UTC

REVIEW: http://review.gluster.org/9975 (Upcall: Added xlator options to enable cache-invalidation) posted (#5) for review on master by soumya k (skoduri)

Comment 7 Anand Avati 2015-03-30 11:52:13 UTC

REVIEW: http://review.gluster.org/9975 (Upcall: Added xlator options to enable cache-invalidation) posted (#6) for review on master by soumya k (skoduri)

Comment 8 Anand Avati 2015-03-31 06:54:49 UTC

REVIEW: http://review.gluster.org/9975 (Upcall: Added xlator options to enable cache-invalidation) posted (#7) for review on master by soumya k (skoduri)

Comment 9 Anand Avati 2015-03-31 18:16:31 UTC

COMMIT: http://review.gluster.org/9975 committed in master by Kaleb KEITHLEY (kkeithle) 
------
commit 25f97b2788a33efcb0714ad639f8db380b864982
Author: Soumya Koduri <skoduri>
Date:   Tue Mar 24 10:30:31 2015 +0530

    Upcall: Added xlator options to enable cache-invalidation
    
    Added two xlator options to enable cache-invalidation and set
    cache-invalidation-timeout.
    
    In addition, made few minor changes in the upcall processing code
    in gfapi.
    
    Change-Id: Ie0b32ca8348e34e3fe4f1e7df30cc925fa4aad31
    BUG: 1200271
    Signed-off-by: Soumya Koduri <skoduri>
    Reviewed-on: http://review.gluster.org/9975
    Reviewed-by: Meghana M <mmadhusu>
    Reviewed-by: Kaleb KEITHLEY <kkeithle>
    Tested-by: Gluster Build System <jenkins.com>

Comment 10 Anand Avati 2015-04-10 10:35:06 UTC

REVIEW: http://review.gluster.org/10194 (Upcall: Handle missing fops in the upcall xlator) posted (#1) for review on master by soumya k (skoduri)

Comment 11 Soumya Koduri 2015-04-10 10:36:12 UTC

Internal gluster fops cannot be skipped for cache_invalidation as few of them like self-healing will result in data change.

Comment 12 Anand Avati 2015-04-10 18:37:49 UTC

REVIEW: http://review.gluster.org/10194 (Upcall: Handle missing fops in the upcall xlator) posted (#2) for review on master by soumya k (skoduri)

Comment 13 Anand Avati 2015-04-17 10:00:06 UTC

REVIEW: http://review.gluster.org/10194 (Upcall: Handle missing fops in the upcall xlator) posted (#3) for review on master by soumya k (skoduri)

Comment 14 Anand Avati 2015-04-29 18:19:26 UTC

REVIEW: http://review.gluster.org/10194 (eUpcall: Handle missing fops in the upcall xlator) posted (#4) for review on master by soumya k (skoduri)

Comment 15 Anand Avati 2015-04-30 05:05:36 UTC

REVIEW: http://review.gluster.org/10194 (eUpcall: Handle missing fops in the upcall xlator) posted (#5) for review on master by soumya k (skoduri)

Comment 16 Anand Avati 2015-04-30 05:14:03 UTC

REVIEW: http://review.gluster.org/10194 (eUpcall: Handle missing fops in the upcall xlator) posted (#6) for review on master by soumya k (skoduri)

Comment 17 Anand Avati 2015-04-30 08:02:04 UTC

REVIEW: http://review.gluster.org/10194 (Upcall: Handle missing fops in the upcall xlator) posted (#7) for review on master by soumya k (skoduri)

Comment 18 Anand Avati 2015-05-01 09:19:13 UTC

REVIEW: http://review.gluster.org/10194 (Upcall: Handle missing fops in the upcall xlator) posted (#8) for review on master by soumya k (skoduri)

Comment 19 Anand Avati 2015-05-02 17:31:03 UTC

REVIEW: http://review.gluster.org/10194 (Upcall: Handle missing fops in the upcall xlator) posted (#9) for review on master by soumya k (skoduri)

Comment 20 Niels de Vos 2015-05-14 17:26:51 UTC

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report.

glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 21 Niels de Vos 2015-05-14 17:28:28 UTC

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report.

glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 22 Niels de Vos 2015-05-14 17:35:17 UTC

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.0, please open a new bug report.

glusterfs-3.7.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 23 Niels de Vos 2016-06-16 12:43:17 UTC

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user