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

YQ-3644 added validations for resource pool parametres #8958

Conversation

GrigoriyPA
Copy link
Collaborator

@GrigoriyPA GrigoriyPA commented Sep 9, 2024

Changelog entry

Added validations for resource pool parametres

Changelog category

  • Improvement

Additional information

@GrigoriyPA GrigoriyPA requested a review from a team as a code owner September 9, 2024 12:43
Copy link

github-actions bot commented Sep 9, 2024

2024-09-09 12:52:40 UTC Pre-commit check for 1580ce5 has started.
2024-09-09 12:55:54 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-09 13:40:08 UTC Build successful.
2024-09-09 13:40:19 UTC Tests are running...
🔴 2024-09-09 15:27:34 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39846 34598 0 2 5213 33

🟢 2024-09-09 15:28:14 UTC ydbd size 8.2 GiB changed* by +76.2 KiB, which is < 100.0 KiB vs stable-24-3-8-analytics: OK

ydbd size dash stable-24-3-8-analytics: ac22cb5 merge: 1580ce5 diff diff %
ydbd size 8 786 789 232 Bytes 8 786 867 264 Bytes +76.2 KiB +0.001%
ydbd stripped size 479 897 960 Bytes 479 899 176 Bytes +1.2 KiB +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Sep 9, 2024

2024-09-09 12:53:44 UTC Pre-commit check for 1580ce5 has started.
2024-09-09 12:56:28 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-09 13:12:24 UTC Build successful.

Copy link

github-actions bot commented Sep 9, 2024

2024-09-09 13:11:10 UTC Pre-commit check for 1580ce5 has started.
2024-09-09 13:14:27 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-09 13:40:32 UTC Build successful.
2024-09-09 13:40:44 UTC Tests are running...
🔴 2024-09-09 15:40:59 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11311 11235 0 14 28 34

🟢 2024-09-09 15:41:35 UTC ydbd size 5.5 GiB changed* by +7.6 KiB, which is < 100.0 KiB vs stable-24-3-8-analytics: OK

ydbd size dash stable-24-3-8-analytics: ac22cb5 merge: 1580ce5 diff diff %
ydbd size 5 952 089 432 Bytes 5 952 097 224 Bytes +7.6 KiB +0.000%
ydbd stripped size 1 491 168 640 Bytes 1 491 172 992 Bytes +4.2 KiB +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@GrigoriyPA GrigoriyPA force-pushed the merge-to-analytics-stable-YQ-3644-add-validations-on-resource-pool-parametres branch from 11d917e to f9b14d9 Compare September 11, 2024 09:58
Copy link

github-actions bot commented Sep 11, 2024

2024-09-11 10:01:40 UTC Pre-commit check for 57cc5dd has started.
2024-09-11 10:04:13 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-11 10:30:29 UTC Build successful.
2024-09-11 10:30:44 UTC Tests are running...
🔴 2024-09-11 12:23:47 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11349 11270 0 16 24 39

🟡 2024-09-11 12:24:27 UTC ydbd size 5.5 GiB changed* by +104.2 KiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: f837701 merge: 57cc5dd diff diff %
ydbd size 5 953 018 800 Bytes 5 953 125 480 Bytes +104.2 KiB +0.002%
ydbd stripped size 1 491 348 192 Bytes 1 491 373 984 Bytes +25.2 KiB +0.002%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Sep 11, 2024

2024-09-11 10:02:04 UTC Pre-commit check for 57cc5dd has started.
2024-09-11 10:04:38 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-11 10:38:15 UTC Build successful.

Copy link

github-actions bot commented Sep 11, 2024

2024-09-11 10:02:12 UTC Pre-commit check for 57cc5dd has started.
2024-09-11 10:04:52 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-11 10:48:24 UTC Build successful.
2024-09-11 10:48:40 UTC Tests are running...
🔴 2024-09-11 12:36:40 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39888 34628 0 2 5215 43

🟡 2024-09-11 12:37:33 UTC ydbd size 8.2 GiB changed* by +276.2 KiB, which is >= 100.0 KiB vs stable-24-3-8-analytics: Warning

ydbd size dash stable-24-3-8-analytics: f837701 merge: 57cc5dd diff diff %
ydbd size 8 788 124 072 Bytes 8 788 406 864 Bytes +276.2 KiB +0.003%
ydbd stripped size 479 948 840 Bytes 479 960 232 Bytes +11.1 KiB +0.002%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@GrigoriyPA GrigoriyPA merged commit b82247b into ydb-platform:stable-24-3-8-analytics Sep 11, 2024
8 of 12 checks passed
@GrigoriyPA GrigoriyPA deleted the merge-to-analytics-stable-YQ-3644-add-validations-on-resource-pool-parametres branch September 11, 2024 15:41
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.

2 participants