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

Log output updates #19

Merged
merged 4 commits into from
Oct 28, 2024
Merged

Conversation

cniethammer
Copy link
Contributor

Some changes related to the log output

  • Add new report type 'none'
  • Disable the default per MPI process log output to files for scalability (used only for debugging normally)
  • Minor code improvements

Signed-off-by: Christoph Niethammer <[email protected]>
The report type 'run' is intended to display each test as it is executed.
Correct the check for the output in case another report type is added
between the 'summary' and 'run' type.

Signed-off-by: Christoph Niethammer <[email protected]>
Move the test for an open output stream to the begin of the log output
to skip multi-threaded run related tests.

Signed-off-by: Christoph Niethammer <[email protected]>
The test suite created by default one detailed log file per MPI process.
When running the test suite with many MPI processes this becomes an issue
for file systems. Therefore, this change disables the per MPI process log
output by lowering its level to TST_REPORT_NONE. The output stream is set
to stderr, just in case.

Signed-off-by: Christoph Niethammer <[email protected]>
@cniethammer cniethammer merged commit 12230b3 into open-mpi:main Oct 28, 2024
1 check passed
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.

1 participant