Bug 865926 - Gluster/Swift plugin reads /etc/swift/swift.conf and /etc/swift/fs.conf for every Swift REST API call
Summary: Gluster/Swift plugin reads /etc/swift/swift.conf and /etc/swift/fs.conf for e...
Keywords:
Status: MODIFIED
Alias: None
Product: Gluster-Swift
Classification: Community
Component: object-server
Version: 0.1
Hardware: x86_64
OS: Linux
low
low
Target Milestone: ---
Assignee: Nobody
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 903396 978061
TreeView+ depends on / blocked
 
Reported: 2012-10-12 20:25 UTC by Peter Portante
Modified: 2023-01-31 23:40 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)

Description Peter Portante 2012-10-12 20:25:05 UTC
Description of problem:

Every Swift REST API call ends up opening the /etc/swift/swift.conf file and the /etc/swift/fs.conf file (twice) to check configuration parameters. This is wasted time and effort when servicing requests.

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

RHS 2.0 (3.3.0)

How reproducible:

Very.

Steps to Reproduce:
1. strace all swift processes via:
   strace -tt -f $(pgrep swift | awk '{printf("-p %s ", $0)}')
2. grep output for /etc/swift
3. count references to conf files
  
Actual results:

  1 open of swift.conf for every request
  2 opens of fs.conf for every request

Expected results:

  Both swift.conf and fs.conf should only be opened during initial startup
  of the app servers.

Additional info:

This problem is fixed by commit: I9b25d6e99521b925952d2fc5086ad19fd59d08ee

Comment 1 Peter Portante 2012-10-19 15:29:38 UTC
CHANGE: http://review.gluster.org/4093 (object-storage: Refactor code to reduce Swift diffs carried) merged in master by Anand Avati (avati)

Comment 2 Peter Portante 2013-02-15 05:06:24 UTC
CORRECTION - this bug was fixed by:

https://github.com/gluster/glusterfs/commit/b0cb7aaf04eff033a329e017a8628c84a62e33cd

CHANGE: http://review.gluster.org/4141 (object-storage: remove glusterfs filter requirement) merged in master by Vijay Bellur (vbellur)


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