-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[GHA] Enable thread sanitizer #26634
Open
mryzhov
wants to merge
73
commits into
openvinotoolkit:master
Choose a base branch
from
mryzhov:gha/sanitizers
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+215
−108
Open
Changes from all commits
Commits
Show all changes
73 commits
Select commit
Hold shift + click to select a range
26978c5
enabled all builds
mryzhov 0c09b06
enable pull events
mryzhov 0c4c277
added tsan suppresions.txt
mryzhov 28a3501
fixed path in the tests
mryzhov 0b3f26a
added missed sting
mryzhov 412d661
update suppresions
mryzhov 9f11960
set additional filters
mryzhov 6c7e435
ignore list
mryzhov f2a964d
ignore onnx too
mryzhov 416fef4
exlude tsan from configure
mryzhov 3f1100a
gcc
mryzhov a399af4
removed clang
mryzhov 0eccfff
disable blacklist
mryzhov 0897a1b
Revert "disable blacklist"
mryzhov d97eab0
Revert "removed clang"
mryzhov 0d07a86
Revert "gcc"
mryzhov 036fb3a
disable onnx fe
mryzhov fa02e63
ENABLE_SYSTEM_PROTOBUF=ON
mryzhov 1a52e59
fixed protobuf builds
mryzhov 1dad8fe
Revert "disable onnx fe"
mryzhov a8d8ce2
Revert "ENABLE_SYSTEM_PROTOBUF=ON"
mryzhov 7b39c02
add condition for ENABLE_THREAD_SANITIZER
mryzhov fc82d2a
final variant
mryzhov 44c6547
revert tsan blacklist
mryzhov a217ee7
rerfactoring
mryzhov 809b42d
Apply suggestions from code review
mryzhov 02b466e
fixed envs
mryzhov 0f5fe7e
use docker images
mryzhov 8d10090
fixed image name
mryzhov 27cb932
dedicated sanitizers image
mryzhov 9fc4eba
ccache
mryzhov d909165
set CMAKE_CXX_COMPILER_LAUNCHER
mryzhov f119c03
fixed typo
mryzhov c134d2f
changed test image
mryzhov 2c67e75
improved cache
mryzhov b7a0bc1
revert test envs
mryzhov e3512c8
di not cleanup build
mryzhov c0854a3
increased ccache size
mryzhov bd466ea
ccache size 20g
mryzhov abf58b2
changed steps conditionsa
mryzhov 6119654
fixed syntax
mryzhov 26f13f8
use gcc10
mryzhov 23b0e94
-revert lubsan
mryzhov cf62916
-revert lubsan
mryzhov 7288c03
debug
mryzhov 7dfe47a
tmp disable tests
mryzhov 26e2713
check compilers
mryzhov f9177af
enabled tests
mryzhov 7f0ae9a
final flags
mryzhov c84fb76
debug info
mryzhov 9760f6a
cleanup the ccache
mryzhov cc31b94
reverted tests
mryzhov 90c61ce
sccache instead of ccache
mryzhov cea5621
change sccache prefix
mryzhov 04bdd25
sccache size 50G
mryzhov b0e325a
disabled cleanup cache steps
mryzhov 213d334
Revert "disabled cleanup cache steps"
mryzhov 3cb0d9a
SCCACHE_AZURE_CONNECTION_STRING
mryzhov d518608
diadable cache cleanup
mryzhov 36aa03f
test old tes image
mryzhov 4927057
remove commented code
mryzhov b33f739
split jobs
mryzhov 87b25aa
Revert "test old tes image"
mryzhov 88c08a9
Revert "remove commented code"
mryzhov c1e3895
install python for the tests
mryzhov d266580
fixed descriptions
mryzhov 8481b39
parallel gzip
mryzhov 74d0b67
setup python
mryzhov 0cdfc56
fixed tar command
mryzhov c7307d2
removed fsanitize-undefined-trap-on-error
mryzhov 0ba8bc3
revert test deps
mryzhov 0d63b9a
increase runner ram
mryzhov 7ad87dd
Merge branch 'master' into gha/sanitizers
mryzhov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
pr-26586 | ||
pr-26634 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,70 @@ | ||
ARG REGISTRY="docker.io" | ||
FROM ${REGISTRY}/library/ubuntu:20.04 | ||
|
||
USER root | ||
|
||
# APT configuration | ||
RUN echo 'Acquire::Retries "10";' > /etc/apt/apt.conf && \ | ||
echo 'APT::Get::Assume-Yes "true";' >> /etc/apt/apt.conf && \ | ||
echo 'APT::Get::Fix-Broken "true";' >> /etc/apt/apt.conf && \ | ||
echo 'APT::Get::no-install-recommends "true";' >> /etc/apt/apt.conf | ||
|
||
ENV DEBIAN_FRONTEND="noninteractive" \ | ||
TZ="Europe/London" | ||
|
||
RUN apt-get update && \ | ||
apt-get install software-properties-common && \ | ||
add-apt-repository --yes --no-update ppa:git-core/ppa && \ | ||
add-apt-repository --yes --no-update ppa:deadsnakes/ppa && \ | ||
apt-get update && \ | ||
apt-get install \ | ||
curl \ | ||
git \ | ||
gpg-agent \ | ||
tzdata \ | ||
# parallel gzip | ||
pigz \ | ||
# Pythons | ||
python3.9-dev \ | ||
python3.9-venv \ | ||
python3.9-distutils \ | ||
# Compiler \ | ||
gcc-10 \ | ||
g++-10 \ | ||
&& \ | ||
rm -rf /var/lib/apt/lists/* | ||
|
||
# Install build dependencies | ||
ADD install_build_dependencies.sh /install_build_dependencies.sh | ||
RUN chmod +x /install_build_dependencies.sh && \ | ||
/install_build_dependencies.sh && \ | ||
rm -rf /var/lib/apt/lists/* | ||
|
||
# Set gcc-10 as a default compiler | ||
RUN update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-10 30 && \ | ||
update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-10 30 | ||
|
||
# Install sscache | ||
ARG SCCACHE_VERSION="v0.7.5" | ||
ENV SCCACHE_HOME="/opt/sccache" \ | ||
SCCACHE_PATH="/opt/sccache/sccache" | ||
|
||
RUN mkdir ${SCCACHE_HOME} && cd ${SCCACHE_HOME} && \ | ||
SCCACHE_ARCHIVE="sccache-${SCCACHE_VERSION}-x86_64-unknown-linux-musl.tar.gz" && \ | ||
curl -SLO https://github.com/mozilla/sccache/releases/download/${SCCACHE_VERSION}/${SCCACHE_ARCHIVE} && \ | ||
tar -xzf ${SCCACHE_ARCHIVE} --strip-components=1 && rm ${SCCACHE_ARCHIVE} | ||
|
||
ENV PATH="$SCCACHE_HOME:$PATH" | ||
|
||
# Setup pip | ||
ENV PIP_VERSION="24.0" | ||
RUN curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py && \ | ||
python3 get-pip.py --no-cache-dir pip==${PIP_VERSION} && \ | ||
python3.9 get-pip.py --no-cache-dir pip==${PIP_VERSION} && \ | ||
rm -f get-pip.py | ||
|
||
# Use Python 3.9 as default instead of Python 3.8 | ||
# Using venv here 'cause other methods to switch the default Python on Ubuntu 20 break both system and wheels build | ||
RUN python3.9 -m venv venv | ||
ENV PATH="/venv/bin:$PATH" | ||
ENV PIP_CACHE_DIR=/mount/caches/pip/linux/${PIP_VERSION} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Since a dedicated image is used, I think these changes could be reverted.