Skip to content

Commit

Permalink
Remove providers/generic and core/external_sources from asan precommi…
Browse files Browse the repository at this point in the history 
…t checks (too long, and not profitable) (ydb-platform#8034)
  • Loading branch information
@maximyurchuk
maximyurchuk authored Aug 20, 2024
1 parent 487f050 commit 46fe40a
Show file tree
Hide file tree
Showing 9 changed files with 81 additions and 36 deletions.
13 changes: 9 additions & 4 deletions ydb/core/external_sources/hive_metastore/ut/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

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

Expand Down
13 changes: 9 additions & 4 deletions ydb/core/external_sources/s3/ut/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

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

Expand Down
13 changes: 9 additions & 4 deletions ydb/library/yql/providers/generic/connector/tests/datasource/clickhouse/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

IF (OPENSOURCE)
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
IF (SANITIZER_TYPE)
# Too huge for precommit check with sanitizers
SIZE(LARGE)
ELSE()
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
ENDIF()
SET(TEST_TAGS_VALUE)
SET(TEST_REQUIREMENTS_VALUE)
# This requirement forces tests to be launched consequently,
Expand Down
13 changes: 9 additions & 4 deletions ydb/library/yql/providers/generic/connector/tests/datasource/ms_sql_server/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

IF (OPENSOURCE)
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
IF (SANITIZER_TYPE)
# Too huge for precommit check with sanitizers
SIZE(LARGE)
ELSE()
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
ENDIF()
SET(TEST_TAGS_VALUE)
SET(TEST_REQUIREMENTS_VALUE)
# This requirement forces tests to be launched consequently,
Expand Down
13 changes: 9 additions & 4 deletions ydb/library/yql/providers/generic/connector/tests/datasource/mysql/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

IF (OPENSOURCE)
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
IF (SANITIZER_TYPE)
# Too huge for precommit check with sanitizers
SIZE(LARGE)
ELSE()
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
ENDIF()
SET(TEST_TAGS_VALUE)
SET(TEST_REQUIREMENTS_VALUE)
# This requirement forces tests to be launched consequently,
Expand Down
13 changes: 9 additions & 4 deletions ydb/library/yql/providers/generic/connector/tests/datasource/oracle/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

IF (OPENSOURCE)
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
IF (SANITIZER_TYPE)
# Too huge for precommit check with sanitizers
SIZE(LARGE)
ELSE()
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
ENDIF()
SET(TEST_TAGS_VALUE)
SET(TEST_REQUIREMENTS_VALUE)
# This requirement forces tests to be launched consequently,
Expand Down
13 changes: 9 additions & 4 deletions ydb/library/yql/providers/generic/connector/tests/datasource/postgresql/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

IF (OPENSOURCE)
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
IF (SANITIZER_TYPE)
# Too huge for precommit check with sanitizers
SIZE(LARGE)
ELSE()
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
ENDIF()
SET(TEST_TAGS_VALUE)
SET(TEST_REQUIREMENTS_VALUE)
# This requirement forces tests to be launched consequently,
Expand Down
13 changes: 9 additions & 4 deletions ydb/library/yql/providers/generic/connector/tests/datasource/ydb/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

IF (OPENSOURCE)
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
IF (SANITIZER_TYPE)
# Too huge for precommit check with sanitizers
SIZE(LARGE)
ELSE()
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
ENDIF()
SET(TEST_TAGS_VALUE)
SET(TEST_REQUIREMENTS_VALUE)
# This requirement forces tests to be launched consequently,
Expand Down
13 changes: 9 additions & 4 deletions ydb/library/yql/providers/generic/connector/tests/join/ya.make
View file
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,15 @@ ENDIF()
INCLUDE(${ARCADIA_ROOT}/library/recipes/docker_compose/recipe.inc)

IF (OPENSOURCE)
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
IF (SANITIZER_TYPE)
# Too huge for precommit check with sanitizers
SIZE(LARGE)
ELSE()
# Including of docker_compose/recipe.inc automatically converts these tests into LARGE,
# which makes it impossible to run them during precommit checks on Github CI.
# Next several lines forces these tests to be MEDIUM. To see discussion, visit YDBOPS-8928.
SIZE(MEDIUM)
ENDIF()
SET(TEST_TAGS_VALUE)
SET(TEST_REQUIREMENTS_VALUE)
# This requirement forces tests to be launched consequently,
Expand Down

0 comments on commit 46fe40a

Please sign in to comment.