-
Notifications
You must be signed in to change notification settings - Fork 149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Netlogger Format to log XRootD Transfers/Accesses #1049
Comments
@juztas - this would be easy to do for third party copy (since that's a separate module from the rest of xrootd). Would that be sufficient? |
It would be a good start, but in the end I would like to see in all (xrootd/dav/http/newsomething). In this case it would also allow to separate exactly the protocol is used for communications |
Hi, @bbockelm do you have news on this? As a beginning for TPC maybe? |
Hi @juztas - I remain fairly overwhelmed when it comes to Xrootd tasks. Any chance I could get you to try your hand at a few of the logging changes? I'm perfectly supportive of the approach, I just have been struggling to find time. |
Try to consistently give a machine-readable format that is friendly to tools like `grep`. Example messages (split over lines for human readability): """ 200315 14:21:06 1875839 TPC_PullRequest: event=PULL_START, \ local=/home/bbockelm/hello_world_copy.txt, \ remote=http://example.com:8000/hello_world_copy.txt, \ user=(anonymous); Starting a push request 200315 14:21:06 1875839 TPC_PullRequest: event=OPEN_FAIL, \ local=/home/bbockelm/hello_world_copy.txt, \ remote=http://example.com:8000/hello_world_copy.txt, \ user=(anonymous), streams=2, status=401; \ Unable to create /hello_world_copy.txt; permission denied """ Fixes xrootd#1049
Awesome!
…On Sun, 15 Mar 2020 at 12:30, Brian P Bockelman ***@***.***> wrote:
@juztas <https://github.com/juztas> - nice quiet weekend here. Please see
#1156 <#1156>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1049 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAE53NVF3M62K57LMXBGCDDRHUUFZANCNFSM4IPJVKXA>
.
--
Justas Balcas
|
Try to consistently give a machine-readable format that is friendly to tools like `grep`. Example messages (split over lines for human readability): """ 200315 14:21:06 1875839 TPC_PullRequest: event=PULL_START, \ local=/home/bbockelm/hello_world_copy.txt, \ remote=http://example.com:8000/hello_world_copy.txt, \ user=(anonymous); Starting a push request 200315 14:21:06 1875839 TPC_PullRequest: event=OPEN_FAIL, \ local=/home/bbockelm/hello_world_copy.txt, \ remote=http://example.com:8000/hello_world_copy.txt, \ user=(anonymous), streams=2, status=401; \ Unable to create /hello_world_copy.txt; permission denied """ Fixes #1049 Conflicts: src/XrdTpc/XrdTpcTPC.cc
Can we re-open the ticket? This fixed for TPC - but not for other accesses (remote reading, davs, https). |
Hi @juztas - can you open a separate ticket? In terms of metadata, I'd prefer to only have a single work item attached to an issue instead of adding scope. |
@bbockelm would it be possible to include also START (and also END) timestamp in each log line? Success/failure to have start timestamp of transfer and finish. Now it is spread between 2 lines:
Also - does it have a thread ID or something? That would also be very beneficial to have (just in case we get multiple transfers transferring same file - rare - but possible). |
@juztas - that seems reasonable. Can you open a separate ticket for this improvement? Any chance you want to take a swing at it? |
I think we also agreed that we would not use an obsolete format but
something more modern like JSON.
…On Tue, 5 Oct 2021, Brian P Bockelman wrote:
@juztas - that seems reasonable. Can you open a separate ticket for this improvement? Any chance you want to take a swing at it?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
#1049 (comment)
|
Oh, I don't think we literally want to use the Netlogger format... I read it as making the logs a bit easier on humans to work through. Anyhow, let's get a new ticket opened and we can bikeshed a bit there. |
@bbockelm ok, I can look into that (Just need to find all the bits to set xrd dev machine). Will create issue later |
It would be beneficial to have xrootd logging improved and maybe use the format like gridftp [1]. In this case sites could filter out transfers from/to or was it transfer or partial read and also what is the return code, start time/end time, etc..
@bbockelm fyi
[1]
http://netlogger.lbl.gov/doc
The text was updated successfully, but these errors were encountered: