Bug 1317785
| Summary: | Cache swift xattrs | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Community] GlusterFS | Reporter: | Prashanth Pai <ppai> | ||||||
| Component: | md-cache | Assignee: | Prashanth Pai <ppai> | ||||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |||||||
| Severity: | low | Docs Contact: | |||||||
| Priority: | urgent | ||||||||
| Version: | mainline | CC: | bugs, sasundar | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | --- | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | glusterfs-3.8rc2 | Doc Type: | Bug Fix | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | |||||||||
| : | 1317788 (view as bug list) | Environment: | |||||||
| Last Closed: | 2016-06-16 14:00:28 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: | 1317788, 1317790 | ||||||||
| Attachments: |
|
||||||||
Created attachment 1136420 [details]
tcpdump trace of DELETE object operation from gluster-swift
Attached tcpdump trace of DELETE object operation from gluster-swift
REVIEW: http://review.gluster.org/13735 (md-cache: Cache gluster-swift metadata) posted (#1) for review on master by Prashanth Pai (ppai) Created attachment 1136461 [details]
tcpdump after application of patch
REVIEW: http://review.gluster.org/13735 (md-cache: Cache gluster-swift metadata) posted (#2) for review on master by Prashanth Pai (ppai) REVIEW: http://review.gluster.org/13735 (md-cache: Cache gluster-swift metadata) posted (#3) for review on master by Prashanth Pai (ppai) REVIEW: http://review.gluster.org/13735 (md-cache: Cache gluster-swift metadata) posted (#4) for review on master by Prashanth Pai (ppai) COMMIT: http://review.gluster.org/13735 committed in master by Jeff Darcy (jdarcy) ------ commit 500ad8f3a72053d33120657e8a2e93d844041cf0 Author: Prashanth Pai <ppai> Date: Tue Mar 15 14:21:18 2016 +0530 md-cache: Cache gluster-swift metadata BUG: 1317785 Change-Id: Ie02b8fc294802f8fdf49dee8bf97f1e6177d92bd Signed-off-by: Prashanth Pai <ppai> Reviewed-on: http://review.gluster.org/13735 Smoke: Gluster Build System <jenkins.com> Reviewed-by: Poornima G <pgurusid> NetBSD-regression: NetBSD Build System <jenkins.org> CentOS-regression: Gluster Build System <jenkins.com> Reviewed-by: Niels de Vos <ndevos> Reviewed-by: Gaurav Kumar Garg <ggarg> 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 |
Description of problem: gluster-swift extensively uses "user.swift.metadata" xattr in it's functionality. This is not cached and hence the performance is usually terrible. Every operation (GET,PUT,POST,DELETE,HEAD) in gluster-swift issues getxattr() as follows: getxattr("/mnt/gluster-object/test/c1/o8", "user.swift.metadata", 0x0, 0) = 190 getxattr("/mnt/gluster-object/test/c1/o8", "user.swift.metadata", "{"Content-Length":"11","ETag":"5eb63bbbe01eeed093cb22bb8f5acdc3","X-Timestamp":"1458021187.45380","X-Object-Type":"file","X-Type":"Object","Content-Type":"application/x-www-form-urlencoded"}", 190) = 190 The first getxattr() is to get the size of xattr value and the second getxattr() is to fetch the actual value itself. The entire xattr information is sent thrice by brick to FUSE mount. Once on lookup, again on the first getxattr() to get size and then again on second gexattr() call. These three network calls can be reduced to just one if the xattr information is cached when it's fetched for the first time.