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

Use the log_path clang sanitizer (ASAN/TSAN/etc.) option to put errors in a separate file #231

Open
mbautin opened this issue Apr 23, 2018 · 1 comment
Assignees
Labels
area/docdb YugabyteDB core features help wanted We welcome your contributions for this issue! kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue

Comments

@mbautin
Copy link
Contributor

mbautin commented Apr 23, 2018

Jira Link: DB-5018
Putting sanitizer errors in a separate file (with a name that can be matched to the process, e.g. ts-1 or m-2) will allow us to parse / display those warnings more easily and detect test failures based on that.

@mbautin mbautin self-assigned this Apr 23, 2018
@rkarthik007 rkarthik007 added the kind/enhancement This is an enhancement of an existing feature label Apr 25, 2018
@rkarthik007 rkarthik007 added the help wanted We welcome your contributions for this issue! label Apr 25, 2018
@mbautin
Copy link
Contributor Author

mbautin commented Oct 31, 2019

This could be done using the log_path option: https://github.com/google/sanitizers/wiki/SanitizerCommonFlags

Flag Description
log_path Write logs to "log_path.pid". The special values are "stdout" and "stderr". The default is "stderr".
log_exe_name Mention name of executable when reporting error and append executable name to logs (as in "log_path.exe_name.pid").

This is becoming important due to large test output files produced without this, e.g.

$ wc -c ./java/yb-cql/target/surefire-reports_org.yb.cql.TestDecimalDataType__testDecimalDataTypeSum/org.yb.cql.TestDecimalDataType-output.txt
567903392 ./java/yb-cql/target/surefire-reports_org.yb.cql.TestDecimalDataType__testDecimalDataTypeSum/org.yb.cql.TestDecimalDataType-output.txt

with the same TSAN error repeated over and over.

@mbautin mbautin assigned WesleyW and unassigned mbautin Jan 15, 2020
@mbautin mbautin assigned mbautin and unassigned WesleyW Oct 22, 2022
@rthallamko3 rthallamko3 added the area/docdb YugabyteDB core features label Jan 11, 2023
@yugabyte-ci yugabyte-ci added the priority/medium Medium priority issue label Jan 11, 2023
devansh-ism pushed a commit to devansh-ism/yugabyte-db that referenced this issue Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docdb YugabyteDB core features help wanted We welcome your contributions for this issue! kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue
Projects
None yet
Development

No branches or pull requests

5 participants