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

tlitest tlog_rec fix journal_find_last failures #252

Merged
merged 1 commit into from
Jun 3, 2019

Conversation

spoore1
Copy link
Contributor

@spoore1 spoore1 commented May 29, 2019

journal_find_last was using _COMM only to match journal entries. This
can fail to find some entries if they do not include that field. The
workaround here is to add an additional check for SYSLOG_IDENTIFIER if
_COMM is not found.

@coveralls
Copy link

coveralls commented May 29, 2019

Coverage Status

Coverage remained the same at 29.515% when pulling f3937b1 on spoore1:journal_fix into 51f6600 on Scribery:master.

lib/tlitest/test_tlog_rec.py Show resolved Hide resolved
lib/tlitest/test_tlog_rec.py Show resolved Hide resolved
journal_find_last was using _COMM only to match journal entries.  This
can fail to find some entries if they do not include that field.  The
workaround here is to add an additional check for SYSLOG_IDENTIFIER if
_COMM is not found.
@justin-stephenson justin-stephenson merged commit 7d24895 into Scribery:master Jun 3, 2019
@spoore1 spoore1 deleted the journal_fix branch May 10, 2021 18:42
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