Bug 811864 - Server xlator uses different identifiers for 'clients' in accept and disconnect log messages
Summary: Server xlator uses different identifiers for 'clients' in accept and disconne...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: protocol
Version: mainline
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ---
Assignee: krishnan parthasarathi
QA Contact: Anush Shetty
URL:
Whiteboard:
Depends On:
Blocks: 817967
TreeView+ depends on / blocked
 
Reported: 2012-04-12 07:22 UTC by krishnan parthasarathi
Modified: 2015-11-03 23:04 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.4.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-24 17:31:58 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description krishnan parthasarathi 2012-04-12 07:22:03 UTC
Description of problem:
Using different identifiers, namely peerinfo->identifier and conn->id for 'clients' when logging the connection accept event and disconnect event respectively, makes it nearly impossible to find the correspondence between the identifiers and the actual machine.

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


How reproducible:
Always

Steps to Reproduce:
1. Create volume of any type.
2. Start the volume and mount it.
3. Kill the mount process or bring the interface down in the client machine
(Assuming client and (at least one) brick reside on different machines).
4. Observe the log messages involving "accept" and "disconnect".

In the steps described above, there is atmost one client and the correspondence between the different identifiers and the actual machine involved is obvious. When there are more than one client (mount process) involved, it becomes difficult.
  
Actual results:


Expected results:


Additional info:

Comment 1 Amar Tumballi 2012-04-13 02:15:46 UTC
kp already sent a patch for this.

Comment 2 Anand Avati 2012-04-13 07:59:21 UTC
CHANGE: http://review.gluster.com/3133 (server: Modified logging information to ease log based analysis) merged in master by Vijay Bellur (vijay)


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