Skip to content

Fix some logging ptest failures#86

Merged
amstewart merged 6 commits intoni:masterfrom
texasaggie97:dev/texasaggie97/logging-tests-issues
Feb 25, 2026
Merged

Fix some logging ptest failures#86
amstewart merged 6 commits intoni:masterfrom
texasaggie97:dev/texasaggie97/logging-tests-issues

Conversation

@texasaggie97
Copy link
Copy Markdown
Collaborator

@texasaggie97 texasaggie97 commented Feb 13, 2026

Summary of Changes

Multiple logging tests are failing in the ptest tests. These are mainly due to verify requiring logging to specifically be turned on.

There are also a couple of small improvements to error handling and reporting.

Justification

AB#3702383

Testing

  • Ran make install && make installcheck on a NILRT VM - All tests passed

Procedure

  • This PR: changes user-visible behavior, fixes a bug, or impacts the project's security profile; and so it includes a CHANGELOG note.
  • I certify that the contents of this pull request complies with the Developer Certificate of Origin.

…for the tests to pass

Signed-off-by: Mark Silva <mark.silva@emerson.com>
@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/logging-tests-issues branch from 68fb14a to 5d23a5b Compare February 13, 2026 21:57
…led exceptions

Signed-off-by: Mark Silva <mark.silva@emerson.com>
Signed-off-by: Mark Silva <mark.silva@emerson.com>
…meter

Signed-off-by: Mark Silva <mark.silva@emerson.com>
Signed-off-by: Mark Silva <mark.silva@emerson.com>
Signed-off-by: Mark Silva <mark.silva@emerson.com>
@texasaggie97 texasaggie97 force-pushed the dev/texasaggie97/logging-tests-issues branch from 5d23a5b to 00ed59f Compare February 17, 2026 22:24
@texasaggie97 texasaggie97 dismissed amstewart’s stale review February 17, 2026 22:24

Changes implemented

@amstewart amstewart merged commit 9d1ff6c into ni:master Feb 25, 2026
3 checks passed
@texasaggie97 texasaggie97 deleted the dev/texasaggie97/logging-tests-issues branch March 24, 2026 17:50
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