-
Notifications
You must be signed in to change notification settings - Fork 748
Test reqs fix #31687
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
base: main
Are you sure you want to change the base?
Test reqs fix #31687
Conversation
Added resource requirements for the test.
Removed executor configurations and added auto config settings.
Added resource requirements for the test.
Removed conditional requirements for sanitizer type.
Added resource requirements for the test scheme shard.
Added resource requirements for the test environment.
Added resource requirements for the test environment.
Add resource requirements for test execution
Added resource requirements for the test.
Add resource requirements for the test
Add requirements for RAM and CPU to ya.make
|
🟢 |
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.
Pull request overview
This PR addresses Out Of Memory (OOM) issues in RelWithDebInfo builds by standardizing resource requirements across test configurations. The changes ensure that tests have adequate RAM (32GB) and CPU (4 cores) allocated, regardless of build type.
Key changes:
- Standardized resource requirements to
ram:32 cpu:4across multiple test suites - Removed SANITIZER_TYPE conditionals where requirements are now unconditional
- Added ELSE clauses to apply requirements to non-sanitizer builds where conditionals remain
Reviewed changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| ydb/tests/stress/viewer/tests/ya.make | Removed SANITIZER_TYPE conditional, made resource requirements unconditional |
| ydb/tests/olap/scenario/ya.make | Added resource requirements for OLAP scenario tests |
| ydb/tests/olap/column_family/compression/ya.make | Added resource requirements for compression tests |
| ydb/tests/functional/sqs/messaging/ya.make | Added resource requirements for non-sanitizer builds |
| ydb/tests/functional/sqs/large/ya.make | Added resource requirements for non-sanitizer builds |
| ydb/tests/functional/sqs/cloud/ya.make | Added resource requirements for non-sanitizer builds |
| ydb/tests/functional/serverless/ya.make | Removed SANITIZER_TYPE conditional, made resource requirements unconditional |
| ydb/tests/functional/scheme_shard/ya.make | Added resource requirements for scheme shard tests |
| ydb/tests/functional/restarts/ya.make | Removed SANITIZER_TYPE conditional, made resource requirements unconditional |
| ydb/tests/functional/rename/ya.make | Added resource requirements for non-sanitizer builds |
| ydb/tests/functional/hive/ya.make | Added resource requirements for non-sanitizer builds (16GB RAM) |
| ydb/tests/functional/encryption/ya.make | Removed SANITIZER_TYPE conditional, made resource requirements unconditional |
| ydb/tests/functional/cms/ya.make | Added resource requirements for non-sanitizer builds (identical to sanitizer builds) |
| ydb/tests/fq/yds/ya.make | Added resource requirements for yds tests |
| ydb/tests/datashard/vector_index/medium/ya.make | Added resource requirements before size conditional |
| ydb/tests/datashard/ttl/ya.make | Added resource requirements for TTL tests |
| ydb/tests/datashard/split_merge/ya.make | Added resource requirements for split/merge tests |
| ydb/tests/datashard/select/ya.make | Added resource requirements for select tests |
| ydb/tests/datashard/s3/ya.make | Added resource requirements for S3 tests |
| ydb/tests/datashard/partitioning/ya.make | Added resource requirements for partitioning tests |
| ydb/tests/datashard/parametrized_queries/ya.make | Added resource requirements for parametrized query tests |
| ydb/tests/datashard/dump_restore/ya.make | Added resource requirements for dump/restore tests |
| ydb/tests/datashard/dml/ya.make | Added resource requirements for DML tests |
| ydb/tests/datashard/copy_table/ya.make | Added resource requirements for copy table tests |
| ydb/tests/datashard/async_replication/ya.make | Added resource requirements for async replication tests |
| ydb/tests/datashard/add_column/ya.make | Added resource requirements for add column tests |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Run-tests started
⚪ ⚪ Ya make output | Test bloat | Test bloat
🟢 |
|
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪ ⚪ DetailsYa make output | Test bloat | Test bloat
⚪ Ya make output | Test bloat | Test bloat | Test bloat
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
Changelog entry
OOM в запуске relwithdebinfo пропал #31669 (comment)

было #31669 (comment)

Changelog category
Description for reviewers
...