Skip to content

[integration] Remote Pilot Logger#5961

Closed
martynia wants to merge 7 commits into
DIRACGrid:integrationfrom
martynia:integration_janusz_pilotlogs_dev
Closed

[integration] Remote Pilot Logger#5961
martynia wants to merge 7 commits into
DIRACGrid:integrationfrom
martynia:integration_janusz_pilotlogs_dev

Conversation

@martynia

@martynia martynia commented Mar 16, 2022

Copy link
Copy Markdown
Contributor

BEGINRELEASENOTES

*WorkloadManagementSystem
CHANGE: Enable remote pilot logging to Tornado

ENDRELEASENOTES

@martynia martynia changed the title Integration janusz pilotlogs dev [integration] Remote Pilot Logger Mar 16, 2022
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py Outdated
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py Outdated
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py Outdated
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py Outdated
Default = authenticated
}
}
TornadoPilotLogging

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an old DISET PilotLogging service that I would just remove (it was never used)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't removed it yet.

Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py Outdated
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py Outdated
Comment thread src/DIRAC/WorkloadManagementSystem/Agent/PilotLoggingAgent.py Outdated
Comment thread src/DIRAC/WorkloadManagementSystem/Service/TornadoPilotLoggingHandler.py Outdated
Comment on lines +39 to +60
def initializeRequest(self):
"""
Called for each request.

:return: None
"""

self.log.info("Request initialised.. ")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not needed.

@martynia martynia Mar 25, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, it will go if I don't need it in the near future.

Comment thread src/DIRAC/WorkloadManagementSystem/Service/TornadoPilotLoggingHandler.py Outdated
@martynia martynia force-pushed the integration_janusz_pilotlogs_dev branch from 86142f8 to f4f2613 Compare March 25, 2022 16:43
@martynia martynia closed this Jun 27, 2022
@martynia

Copy link
Copy Markdown
Contributor Author

Outdated. A new pull request opened (#6208) , based on this one.

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.

2 participants