Skip to content
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

Tests to run on .github folder update, to verify that all essential scripts are working correctly #285

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

librarian
Copy link
Member

No description provided.

Copy link
Contributor

Treating pr.yaml as a Workflow definition
Treating nightly.yaml as a Workflow definition
Treating build_and_test_on_demand.yaml as a Workflow definition
Treating github_actions_scripts.yaml as a Workflow definition
Treating pr-github-actions.yaml as a Workflow definition
Treating build_and_test_act_cmake.yaml as a Workflow definition
Treating build_and_test_ya.yaml as a Workflow definition
Treating pr-cmake.yaml as a Workflow definition
Treating build_and_test_act.yaml as a Workflow definition
Treating nightly-index-rebuild.yaml as a Workflow definition
Treating build_and_test_cmake.yaml as a Workflow definition
Treating build_and_test_on_demand_cmake.yaml as a Workflow definition

Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🔴 linux-x64-relwithdebinfo: some tests FAILED for commit 25b7bed.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5015 4903 0 13 5 94

Copy link
Collaborator

@BarkovBG BarkovBG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а зачем коммитить все эти файлы? (*testing_out_stuff*)

@librarian
Copy link
Member Author

Это тестовые данные на которых быстро можно прогнать скрипты и сравнить полученный результат с каноничным.
Структура директорий и файлов взята из результата работы ya make и проверяется, что мы положили все нужные файлы и ничего не забыли из того, что у нас есть в отчёте, например всё что упоминается тут https://github-actions-s3.website.nemax.nebius.cloud/ydb-platform/nbs/PR-check/7699802249/1/nebius-x86-64/summary/ya-test.html
Всё это необходимо чтобы можно было быстро править скрипты, а не делать изменение, а потом запускать целиком воркфлоу и обнаруживать, что где-то в синтаксисе ошибся или ещё чём то.

@budevg
Copy link
Collaborator

budevg commented Jan 31, 2024

по моему это плохая идея во первых там куча рандомных данных типа /tmp/toolchain-bf57cad1-ef0a-4e а во вторых регулярно надо будет пере заливать их и величина репоситория будет расти и git clone будет длится долго

@librarian
Copy link
Member Author

Их не надо перезаливать, это статические данные, которые я специально обрезал, чтобы они занимали мало места, они никак не завязаны на сборку которая уже есть. Основное занимаемое место с ними просто в inode-ах.

@librarian
Copy link
Member Author

Я могу это всё в архив положить, если прям кому-то сильно мешает. Но если что, сорцы у нас намного более часто меняются и они занимают существенно больше места.

@budevg
Copy link
Collaborator

budevg commented Jan 31, 2024

Если эти данные никогда не будут менятса можно действительно как к бинарнику относится залить на s3 zip скачивать и прогонять тесты. Можно ещё git lfs заюзать и хранить ссылку в repository. Обычно так делают потому что если заливаеш новую версию бинарника в git истории остаётся старая версия а diff между бинарниками не всегда маленький

@librarian librarian force-pushed the users/librarian/test-the-tests branch from 25b7bed to 021e1db Compare February 1, 2024 11:09
Copy link
Contributor

github-actions bot commented Feb 1, 2024

Treating build_and_test_on_demand_cmake.yaml as a Workflow definition
Treating build_and_test_cmake.yaml as a Workflow definition
Treating nightly-index-rebuild.yaml as a Workflow definition
Treating pr.yaml as a Workflow definition
Treating build_and_test_act.yaml as a Workflow definition
Treating build_and_test_on_demand.yaml as a Workflow definition
Treating pr-github-actions.yaml as a Workflow definition
Treating build_and_test_ya.yaml as a Workflow definition
Treating github_actions_scripts.yaml as a Workflow definition
Treating nightly.yaml as a Workflow definition
Treating pr-cmake.yaml as a Workflow definition
Treating build_and_test_act_cmake.yaml as a Workflow definition

Copy link
Contributor

github-actions bot commented Feb 1, 2024

Note

This is an automated comment that will be appended during run.

🔴 linux-x64-relwithdebinfo: some tests FAILED for commit 021e1db.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5015 4903 0 13 5 94

@librarian
Copy link
Member Author

Унёс структуру директорий на s3

@librarian librarian merged commit 62766ae into main Feb 1, 2024
4 checks passed
@librarian librarian deleted the users/librarian/test-the-tests branch February 1, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants