Skip to content
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

LIU-65: Generate individual log files per session. #39

Merged
merged 2 commits into from
Jul 7, 2021
Merged

Conversation

davepallot
Copy link
Contributor

Generate individual log files per session and propagates default/user specified log file location.

@coveralls
Copy link

coveralls commented Jul 7, 2021

Coverage Status

Coverage increased (+0.05%) to 76.914% when pulling 2f6fe91 on LIU-65 into 6a3275a on master.

Copy link
Contributor

@rtobar rtobar left a comment

Choose a reason for hiding this comment

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

This looks great, thanks :) I left two very very minor comments, feel free to disregard them if you want, then go ahead with the merge.

return False

fmt = '%(asctime)-15s [%(levelname)5.5s] [%(threadName)15.15s] '
fmt += '[%(session_id)10.10s] [%(drop_uid)10.10s] '
Copy link
Contributor

Choose a reason for hiding this comment

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

Here I'd remove appending the session_id, as it's already in the name of the file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment on lines 106 to 108
if hasattr(record, 'session_id'):
return record.session_id == self.sessionId
return False
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe simpler to use return getattr(record, 'session_id', None) == self.sessionId

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@davepallot davepallot merged commit 1b2823a into master Jul 7, 2021
@rtobar rtobar deleted the LIU-65 branch July 7, 2021 07:02
awicenec pushed a commit that referenced this pull request May 19, 2022
LIU-65: Generate individual log files per session.
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.

None yet

3 participants