Bug 1675076 - [posix]: log the actual path wherever possible
Summary: [posix]: log the actual path wherever possible
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: posix
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-02-11 19:22 UTC by Raghavendra Bhat
Modified: 2019-02-14 08:29 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-02-14 08:29:33 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gluster.org Gerrit 22192 0 None Merged storage/posix: print the actual file path 2019-02-14 08:29:31 UTC

Description Raghavendra Bhat 2019-02-11 19:22:36 UTC
Description of problem:

posix converts incoming operations on files to operations on
corresponding gfid handles. While this in itself is not a problem,
logging of those gfid handles in place of actual file paths can
create confusions during debugging. The best way would be to
print both the actual file (recieved as an argument) for path
based operations and the gfid handle associated with it.


Ex:

[2019-01-29 08:50:24.373828] E [MSGID: 113039] [posix.c:3122:posix_open] 0-vol_f108aba8b492c40e74247293db9d544a-posix: open on /export/brick/.glusterfs/b8/b4/b8b47498-00df-4ab1-814a-b950250ef598, flags: 0 [No such file or directory]

In the above log, if the actual file path was also mentioned (which posix_open gets as an argument), it would have helped in debugging and analysis better.


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Worker Ant 2019-02-14 08:29:33 UTC
REVIEW: https://review.gluster.org/22192 (storage/posix: print the actual file path) merged (#3) on master by Amar Tumballi


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