-
Notifications
You must be signed in to change notification settings - Fork 615
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
Enable usage of sanitizers in tests #3377
Conversation
CI MESSAGE: [3065716]: BUILD STARTED |
CI MESSAGE: [3065716]: BUILD FAILED |
b5c3b62
to
75d6703
Compare
CI MESSAGE: [3068059]: BUILD STARTED |
CI MESSAGE: [3068059]: BUILD FAILED |
562a46e
to
6977964
Compare
CI MESSAGE: [3075168]: BUILD STARTED |
CI MESSAGE: [3075168]: BUILD FAILED |
6977964
to
8aa6e80
Compare
CI MESSAGE: [3076590]: BUILD STARTED |
CI MESSAGE: [3076590]: BUILD FAILED |
CI MESSAGE: [3080914]: BUILD STARTED |
CI MESSAGE: [3080914]: BUILD FAILED |
CI MESSAGE: [3081905]: BUILD STARTED |
CI MESSAGE: [3081905]: BUILD FAILED |
b1a7706
to
9210f3b
Compare
CI MESSAGE: [3102019]: BUILD STARTED |
CI MESSAGE: [3102041]: BUILD STARTED |
9210f3b
to
3c95012
Compare
CI MESSAGE: [3102218]: BUILD STARTED |
CI MESSAGE: [3102218]: BUILD FAILED |
3c95012
to
1370bc5
Compare
CI MESSAGE: [3114113]: BUILD STARTED |
CI MESSAGE: [3114113]: BUILD FAILED |
1370bc5
to
25fe382
Compare
CI MESSAGE: [3117393]: BUILD STARTED |
CI MESSAGE: [3117393]: BUILD FAILED |
qa/test_template.sh
Outdated
else | ||
set -e | ||
fi | ||
enable_sanitizer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be a personal preference, but I think it'd read better if the if's were around the calls to enable_sanitizer/disable_sanitizer/process_sanitizers_logs and not inside the functions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@@ -43,8 +43,10 @@ def main(): | |||
"librt.so.1", | |||
"libstdc++.so.6", | |||
"libgcc_s.so.1", | |||
"libz.so.1", | |||
"liblzma.so.5" | |||
"libasan.so", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we always bundling this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No. In contrary. We allow these libs that files inside the wheel depend on to be not bundled inside.
So even if we use sanitizers we won't bundle them.
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
CI MESSAGE: [3134436]: BUILD STARTED |
CI MESSAGE: [3134435]: BUILD STARTED |
CI MESSAGE: [3134436]: BUILD PASSED |
CI MESSAGE: [3134435]: BUILD PASSED |
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
* Enable usage of sanitizers in tests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
Signed-off-by: Janusz Lisiecki jlisiecki@nvidia.com
Description
What happened in this PR
Additional information
Checklist
Tests
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: DALI-2325