Skip to content

Commit 46fe40a

Browse files
maximyurchukmaximyurchuk
authored
Remove providers/generic and core/external_sources from asan precommit checks (too long, and not profitable) (#8034)
1 parent 487f050 commit 46fe40a

File tree

9 files changed

+81
-36
lines changed

9 files changed

+81
-36
lines changed

‎ydb/core/external_sources/hive_metastore/ut/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,15 @@ ENDIF()
3232
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3333

3434
IF (OPENSOURCE)
35-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
36-
# which makes it impossible to run them during precommit checks on Github CI.
37-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
38-
SIZE(MEDIUM)
35+
IF (SANITIZER_TYPE)
36+
# Too huge for precommit check with sanitizers
37+
SIZE(LARGE)
38+
ELSE()
39+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
40+
# which makes it impossible to run them during precommit checks on Github CI.
41+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
42+
SIZE(MEDIUM)
43+
ENDIF()
3944
SET(TEST_TAGS_VALUE)
4045
SET(TEST_REQUIREMENTS_VALUE)
4146

‎ydb/core/external_sources/s3/ut/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,15 @@ ENDIF()
3131
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3232

3333
IF (OPENSOURCE)
34-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
35-
# which makes it impossible to run them during precommit checks on Github CI.
36-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
37-
SIZE(MEDIUM)
34+
IF (SANITIZER_TYPE)
35+
# Too huge for precommit check with sanitizers
36+
SIZE(LARGE)
37+
ELSE()
38+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
39+
# which makes it impossible to run them during precommit checks on Github CI.
40+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
41+
SIZE(MEDIUM)
42+
ENDIF()
3843
SET(TEST_TAGS_VALUE)
3944
SET(TEST_REQUIREMENTS_VALUE)
4045

‎ydb/library/yql/providers/generic/connector/tests/datasource/clickhouse/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ ENDIF()
3737
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3838

3939
IF (OPENSOURCE)
40-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
41-
# which makes it impossible to run them during precommit checks on Github CI.
42-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
43-
SIZE(MEDIUM)
40+
IF (SANITIZER_TYPE)
41+
# Too huge for precommit check with sanitizers
42+
SIZE(LARGE)
43+
ELSE()
44+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
45+
# which makes it impossible to run them during precommit checks on Github CI.
46+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
47+
SIZE(MEDIUM)
48+
ENDIF()
4449
SET(TEST_TAGS_VALUE)
4550
SET(TEST_REQUIREMENTS_VALUE)
4651
# This requirement forces tests to be launched consequently,

‎ydb/library/yql/providers/generic/connector/tests/datasource/ms_sql_server/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ ENDIF()
3737
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3838

3939
IF (OPENSOURCE)
40-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
41-
# which makes it impossible to run them during precommit checks on Github CI.
42-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
43-
SIZE(MEDIUM)
40+
IF (SANITIZER_TYPE)
41+
# Too huge for precommit check with sanitizers
42+
SIZE(LARGE)
43+
ELSE()
44+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
45+
# which makes it impossible to run them during precommit checks on Github CI.
46+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
47+
SIZE(MEDIUM)
48+
ENDIF()
4449
SET(TEST_TAGS_VALUE)
4550
SET(TEST_REQUIREMENTS_VALUE)
4651
# This requirement forces tests to be launched consequently,

‎ydb/library/yql/providers/generic/connector/tests/datasource/mysql/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ ENDIF()
3737
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3838

3939
IF (OPENSOURCE)
40-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
41-
# which makes it impossible to run them during precommit checks on Github CI.
42-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
43-
SIZE(MEDIUM)
40+
IF (SANITIZER_TYPE)
41+
# Too huge for precommit check with sanitizers
42+
SIZE(LARGE)
43+
ELSE()
44+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
45+
# which makes it impossible to run them during precommit checks on Github CI.
46+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
47+
SIZE(MEDIUM)
48+
ENDIF()
4449
SET(TEST_TAGS_VALUE)
4550
SET(TEST_REQUIREMENTS_VALUE)
4651
# This requirement forces tests to be launched consequently,

‎ydb/library/yql/providers/generic/connector/tests/datasource/oracle/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ ENDIF()
3737
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3838

3939
IF (OPENSOURCE)
40-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
41-
# which makes it impossible to run them during precommit checks on Github CI.
42-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
43-
SIZE(MEDIUM)
40+
IF (SANITIZER_TYPE)
41+
# Too huge for precommit check with sanitizers
42+
SIZE(LARGE)
43+
ELSE()
44+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
45+
# which makes it impossible to run them during precommit checks on Github CI.
46+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
47+
SIZE(MEDIUM)
48+
ENDIF()
4449
SET(TEST_TAGS_VALUE)
4550
SET(TEST_REQUIREMENTS_VALUE)
4651
# This requirement forces tests to be launched consequently,

‎ydb/library/yql/providers/generic/connector/tests/datasource/postgresql/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ ENDIF()
3737
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3838

3939
IF (OPENSOURCE)
40-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
41-
# which makes it impossible to run them during precommit checks on Github CI.
42-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
43-
SIZE(MEDIUM)
40+
IF (SANITIZER_TYPE)
41+
# Too huge for precommit check with sanitizers
42+
SIZE(LARGE)
43+
ELSE()
44+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
45+
# which makes it impossible to run them during precommit checks on Github CI.
46+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
47+
SIZE(MEDIUM)
48+
ENDIF()
4449
SET(TEST_TAGS_VALUE)
4550
SET(TEST_REQUIREMENTS_VALUE)
4651
# This requirement forces tests to be launched consequently,

‎ydb/library/yql/providers/generic/connector/tests/datasource/ydb/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ ENDIF()
3737
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3838

3939
IF (OPENSOURCE)
40-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
41-
# which makes it impossible to run them during precommit checks on Github CI.
42-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
43-
SIZE(MEDIUM)
40+
IF (SANITIZER_TYPE)
41+
# Too huge for precommit check with sanitizers
42+
SIZE(LARGE)
43+
ELSE()
44+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
45+
# which makes it impossible to run them during precommit checks on Github CI.
46+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
47+
SIZE(MEDIUM)
48+
ENDIF()
4449
SET(TEST_TAGS_VALUE)
4550
SET(TEST_REQUIREMENTS_VALUE)
4651
# This requirement forces tests to be launched consequently,

‎ydb/library/yql/providers/generic/connector/tests/join/ya.make

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,15 @@ ENDIF()
3737
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)
3838

3939
IF (OPENSOURCE)
40-
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
41-
# which makes it impossible to run them during precommit checks on Github CI.
42-
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
43-
SIZE(MEDIUM)
40+
IF (SANITIZER_TYPE)
41+
# Too huge for precommit check with sanitizers
42+
SIZE(LARGE)
43+
ELSE()
44+
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
45+
# which makes it impossible to run them during precommit checks on Github CI.
46+
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
47+
SIZE(MEDIUM)
48+
ENDIF()
4449
SET(TEST_TAGS_VALUE)
4550
SET(TEST_REQUIREMENTS_VALUE)
4651
# This requirement forces tests to be launched consequently,

0 commit comments

Comments
 (0)