>From Jeff Layton <jlay...@redhat.com>:
Jeff Layton has uploaded this change for review. (
https://review.gerrithub.io/383279
Change subject: mdcache: don't fail setattr if it succeeded but getattr failed
......................................................................
mdcache: don't fail setattr if it succeeded but getattr failed
We call down into the fsal to do the setattr, and after that succeeds
we do a getattr to update the cache. If that getattr fails, we fail the
setattr, even though the fsal said it succeeded.
Change it to return success in that case, as the setattr supposedly
did succeed, even though the getattr failed.
Change-Id: I52330c2fcb16ca469068f5a66258e21232db1baa
Signed-off-by: Jeff Layton <jlay...@redhat.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/79/383279/1
--
To view, visit https://review.gerrithub.io/383279
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I52330c2fcb16ca469068f5a66258e21232db1baa
Gerrit-Change-Number: 383279
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlay...@redhat.com>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
