Project

General

Profile

Actions

Feature #17276

closed

stick client PID in client_metadata

Added by Greg Farnum over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Category:
Administration/Usability
Target version:
-
% Done:

0%

Source:
Community (user)
Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

Description

As best I can tell, we no longer provide any way for an admin to backtrack from a misbehaving client session to a specific PID, which makes debugging hard in situations that have multiple clients on a single box.

(In the past, the pid was used for the messenger nonce, but that got killed.)

Actions #1

Updated by Patrick Donnelly over 7 years ago

I will take this as I've been wanting it for my performance work.

I'm also wondering if you think it'd be useful to have a UUID for each running client/mds daemon (probably not a persistent UUID) that can be used to uniquely identify a daemon instance. This would also be useful for my database of performance values. Right now I'm using asok path names with PIDs (/var/run/ceph/ceph-client-$pid.asok) to distinguish clients which sucks.

Actions #2

Updated by Patrick Donnelly over 7 years ago

  • Assignee set to Patrick Donnelly
Actions #3

Updated by Greg Farnum over 7 years ago

There's already a GID (just an increasing integer) that the monitor uniquely associates with each client session — I guess that's not persistent across a whole process lifetime if it changes monitors, but it's probably good enough to get started with if you can dig out where that is.

Actions #4

Updated by Patrick Donnelly over 7 years ago

  • Status changed from New to Fix Under Review
Actions #5

Updated by Patrick Donnelly over 7 years ago

Testing an email filter. Please ignore...

Actions #6

Updated by Patrick Donnelly over 7 years ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF