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

[Jenkins/Benchmarks] save the standard output into a file #1528

Merged
merged 1 commit into from
Nov 11, 2016

Conversation

norihiro-w
Copy link
Collaborator

I would like to see log messages for failed benchmarks without manually running them again.

@chleh
Copy link
Collaborator

chleh commented Nov 8, 2016

Doesn't ctest already automatically save stdout/stderr to $BUILDDIR/Testing/Temporary/LastTest.log?

@endJunction
Copy link
Member

@norihiro-w Could you please comment Christoph's question?

@norihiro-w
Copy link
Collaborator Author

Christoph's comment is correct, though my preference is saving the output into a separate file for each benchmark, because it's easier to find outputs from a particular benchmark and do further works, e.g. compare the logs between different versions.

But I don't mind to keep it in my branch and close this PR if it's only me who would like to have the separate output.

@norihiro-w
Copy link
Collaborator Author

It seems one cannot use tail -f with $BUILDDIR/Testing/Temporary/LastTest.log. At least on my environment, the output will be saved into the file after the job is completed. I don't know how to change the buffered behavior in ctest.

Alternative solution is ctest --verbose &> test.log which includes additional messages from ctest

@chleh
Copy link
Collaborator

chleh commented Nov 11, 2016

Thank you for the explanation. If you find the introduced feature handy, I won't block it. Especially comparing different versions will be more convenient after this PR, indeed.
My only concern was that we shouldn't reimplement features that are already provided by the tools we use. But as said, this PR adds some convenience. So 👍

@endJunction endJunction merged commit 7c04044 into ufz:master Nov 11, 2016
@norihiro-w norihiro-w deleted the benchmark-save-log branch November 11, 2016 11:57
@ogsbot
Copy link
Member

ogsbot commented Jun 19, 2020

OpenGeoSys development has been moved to GitLab.

See this pull request on GitLab.

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.

4 participants