From 46fe40a3ff5bee0c91076b18cafabaff49f442d7 Mon Sep 17 00:00:00 2001 From: Maxim Yurchuk Date: Tue, 20 Aug 2024 10:28:04 +0300 Subject: [PATCH] Remove providers/generic and core/external_sources from asan precommit checks (too long, and not profitable) (#8034) --- ydb/core/external_sources/hive_metastore/ut/ya.make | 13 +++++++++---- ydb/core/external_sources/s3/ut/ya.make | 13 +++++++++---- .../connector/tests/datasource/clickhouse/ya.make | 13 +++++++++---- .../tests/datasource/ms_sql_server/ya.make | 13 +++++++++---- .../connector/tests/datasource/mysql/ya.make | 13 +++++++++---- .../connector/tests/datasource/oracle/ya.make | 13 +++++++++---- .../connector/tests/datasource/postgresql/ya.make | 13 +++++++++---- .../generic/connector/tests/datasource/ydb/ya.make | 13 +++++++++---- .../providers/generic/connector/tests/join/ya.make | 13 +++++++++---- 9 files changed, 81 insertions(+), 36 deletions(-) diff --git a/ydb/core/external_sources/hive_metastore/ut/ya.make b/ydb/core/external_sources/hive_metastore/ut/ya.make index 9fe9ad0ae516..6fd18fcb47f2 100644 --- a/ydb/core/external_sources/hive_metastore/ut/ya.make +++ b/ydb/core/external_sources/hive_metastore/ut/ya.make @@ -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) diff --git a/ydb/core/external_sources/s3/ut/ya.make b/ydb/core/external_sources/s3/ut/ya.make index da4d33248657..12301fcfe8f5 100644 --- a/ydb/core/external_sources/s3/ut/ya.make +++ b/ydb/core/external_sources/s3/ut/ya.make @@ -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) diff --git a/ydb/library/yql/providers/generic/connector/tests/datasource/clickhouse/ya.make b/ydb/library/yql/providers/generic/connector/tests/datasource/clickhouse/ya.make index 734b9fff908c..c7153bf98c25 100644 --- a/ydb/library/yql/providers/generic/connector/tests/datasource/clickhouse/ya.make +++ b/ydb/library/yql/providers/generic/connector/tests/datasource/clickhouse/ya.make @@ -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, diff --git a/ydb/library/yql/providers/generic/connector/tests/datasource/ms_sql_server/ya.make b/ydb/library/yql/providers/generic/connector/tests/datasource/ms_sql_server/ya.make index 183e6ee75b6b..42fc2aac49af 100644 --- a/ydb/library/yql/providers/generic/connector/tests/datasource/ms_sql_server/ya.make +++ b/ydb/library/yql/providers/generic/connector/tests/datasource/ms_sql_server/ya.make @@ -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, diff --git a/ydb/library/yql/providers/generic/connector/tests/datasource/mysql/ya.make b/ydb/library/yql/providers/generic/connector/tests/datasource/mysql/ya.make index 9be8496d3723..8d5275732ae7 100644 --- a/ydb/library/yql/providers/generic/connector/tests/datasource/mysql/ya.make +++ b/ydb/library/yql/providers/generic/connector/tests/datasource/mysql/ya.make @@ -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, diff --git a/ydb/library/yql/providers/generic/connector/tests/datasource/oracle/ya.make b/ydb/library/yql/providers/generic/connector/tests/datasource/oracle/ya.make index 4cb58b5990c0..184890bb9740 100644 --- a/ydb/library/yql/providers/generic/connector/tests/datasource/oracle/ya.make +++ b/ydb/library/yql/providers/generic/connector/tests/datasource/oracle/ya.make @@ -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, diff --git a/ydb/library/yql/providers/generic/connector/tests/datasource/postgresql/ya.make b/ydb/library/yql/providers/generic/connector/tests/datasource/postgresql/ya.make index 94e241f4f23b..950432aec24d 100644 --- a/ydb/library/yql/providers/generic/connector/tests/datasource/postgresql/ya.make +++ b/ydb/library/yql/providers/generic/connector/tests/datasource/postgresql/ya.make @@ -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, diff --git a/ydb/library/yql/providers/generic/connector/tests/datasource/ydb/ya.make b/ydb/library/yql/providers/generic/connector/tests/datasource/ydb/ya.make index 0f161769a63f..cab225427e95 100644 --- a/ydb/library/yql/providers/generic/connector/tests/datasource/ydb/ya.make +++ b/ydb/library/yql/providers/generic/connector/tests/datasource/ydb/ya.make @@ -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, diff --git a/ydb/library/yql/providers/generic/connector/tests/join/ya.make b/ydb/library/yql/providers/generic/connector/tests/join/ya.make index 265f9762455f..d8aa54bc8173 100644 --- a/ydb/library/yql/providers/generic/connector/tests/join/ya.make +++ b/ydb/library/yql/providers/generic/connector/tests/join/ya.make @@ -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,