Bug 1217723 - Upcall: xlator options for Upcall xlator
Summary: Upcall: xlator options for Upcall xlator
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: upcall
Version: 3.7.0
Hardware: All
OS: All
unspecified
high
Target Milestone: ---
Assignee: Soumya Koduri
QA Contact:
URL:
Whiteboard:
Depends On: 1200271
Blocks: glusterfs-3.7.0 1200262 1217711
TreeView+ depends on / blocked
 
Reported: 2015-05-01 11:12 UTC by Soumya Koduri
Modified: 2015-05-15 17:09 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.7.0
Doc Type: Bug Fix
Doc Text:
Clone Of: 1200271
Environment:
Last Closed: 2015-05-15 17:09:35 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Soumya Koduri 2015-05-01 11:12:46 UTC
+++ This bug was initially created as a clone of Bug #1200271 +++

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.

--- Additional comment from Soumya Koduri on 2015-03-16 09:20:52 EDT ---

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.

--- Additional comment from Anand Avati on 2015-03-24 01:04:42 EDT ---

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

--- Additional comment from Anand Avati on 2015-03-25 03:37:56 EDT ---

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

--- Additional comment from Anand Avati on 2015-03-25 03:43:51 EDT ---

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

--- Additional comment from Anand Avati on 2015-03-26 03:00:43 EDT ---

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

--- Additional comment from Anand Avati on 2015-03-26 03:09:42 EDT ---

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

--- Additional comment from Anand Avati on 2015-03-30 07:52:13 EDT ---

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

--- Additional comment from Anand Avati on 2015-03-31 02:54:49 EDT ---

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

--- Additional comment from Anand Avati on 2015-03-31 14:16:31 EDT ---

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>

--- Additional comment from Anand Avati on 2015-04-10 06:35:06 EDT ---

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

--- Additional comment from Soumya Koduri on 2015-04-10 06:36:12 EDT ---

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

--- Additional comment from Anand Avati on 2015-04-10 14:37:49 EDT ---

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

--- Additional comment from Anand Avati on 2015-04-17 06:00:06 EDT ---

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

--- Additional comment from Anand Avati on 2015-04-29 14:19:26 EDT ---

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

--- Additional comment from Anand Avati on 2015-04-30 01:05:36 EDT ---

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

--- Additional comment from Anand Avati on 2015-04-30 01:14:03 EDT ---

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

--- Additional comment from Anand Avati on 2015-04-30 04:02:04 EDT ---

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

--- Additional comment from Anand Avati on 2015-05-01 05:19:13 EDT ---

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 1 Anand Avati 2015-05-05 19:35:47 UTC
REVIEW: http://review.gluster.org/10564 (Upcall: Handle missing fops in the upcall xlator) posted (#2) for review on release-3.7 by Kaleb KEITHLEY (kkeithle)

Comment 2 Anand Avati 2015-05-06 10:00:45 UTC
REVIEW: http://review.gluster.org/10564 (Upcall: Handle missing fops in the upcall xlator) posted (#3) for review on release-3.7 by soumya k (skoduri)

Comment 3 Niels de Vos 2015-05-15 17:09:35 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


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