Skip to content

feat: Enable remote pilot logging using HTTP interface to Tornado.#158

Merged
fstagni merged 3 commits into
DIRACGrid:develfrom
martynia:devel_janusz_pilot_logsWrapper.dev
Nov 8, 2022
Merged

feat: Enable remote pilot logging using HTTP interface to Tornado.#158
fstagni merged 3 commits into
DIRACGrid:develfrom
martynia:devel_janusz_pilot_logsWrapper.dev

Conversation

@martynia

@martynia martynia commented Jun 8, 2022

Copy link
Copy Markdown
Contributor

A"wrapper" approach to remote pilot logging. Logging is happening in parallel to stdout and to a string.io buffer which is flushed when its size exceeds a predefined value. The buffer is periodically sent to Tornado server. The logs are handled by a plugin, a FileCachePlugin stores log files in Tornado web area. They can be pulled by an agent and stored on a SE.

@martynia martynia force-pushed the devel_janusz_pilot_logsWrapper.dev branch from 8dd0a87 to f91f303 Compare June 10, 2022 12:10
@martynia martynia force-pushed the devel_janusz_pilot_logsWrapper.dev branch from 46a832b to 9160b6e Compare June 30, 2022 12:27
@martynia martynia force-pushed the devel_janusz_pilot_logsWrapper.dev branch from c1118b6 to 11bef91 Compare August 31, 2022 15:12
Comment thread Pilot/dirac-pilot.py
Comment thread Pilot/dirac-pilot.py
Comment thread Pilot/pilotCommands.py Outdated
@martynia martynia force-pushed the devel_janusz_pilot_logsWrapper.dev branch from c95956e to 3b6bf8a Compare September 6, 2022 14:13
@martynia martynia marked this pull request as ready for review September 6, 2022 14:17
@martynia

martynia commented Sep 6, 2022 via email

Copy link
Copy Markdown
Contributor Author

@martynia martynia force-pushed the devel_janusz_pilot_logsWrapper.dev branch from 7661f05 to 3d788a8 Compare November 2, 2022 20:25
@fstagni

fstagni commented Nov 4, 2022

Copy link
Copy Markdown
Contributor

I am not against merging this PR (see also DIRACGrid/DIRAC#6208 (comment)). But maybe you want to re-check it and add something else?

@martynia

martynia commented Nov 7, 2022

Copy link
Copy Markdown
Contributor Author

I think we could go ahead and merge to in order to be able to test the DIRAC part. If we find something broken I can always create a new PR.

@fstagni fstagni merged commit 6d88991 into DIRACGrid:devel Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants