Skip to content

Enable auto-block mode by default #7567

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Aug 9, 2024
Merged

Enable auto-block mode by default #7567

merged 3 commits into from
Aug 9, 2024

Conversation

abyss7
Copy link
Collaborator

@abyss7 abyss7 commented Aug 8, 2024

Changelog category

  • New feature

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 8, 2024 •
edited
Loading

⚪ 2024-08-08 11:46:46 UTC Pre-commit check for f864b18 has started.
⚪ 2024-08-08 11:49:42 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-08 12:20:39 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 8, 2024 •
edited
Loading

⚪ 2024-08-08 11:48:38 UTC Pre-commit check for f864b18 has started.
⚪ 2024-08-08 11:51:45 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-08 14:25:34 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39862 33888 0 468 5474 32

⚪ 2024-08-08 14:30:26 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟡 2024-08-08 15:00:02 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
669 (only retried tests) 43 0 475 125 26

⚪ 2024-08-08 15:03:54 UTC Failed tests rerun (try 3) linux-x86_64-relwithdebinfo is running...
🔴 2024-08-08 15:34:02 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
626 (only retried tests) 4 0 474 122 26

🟢 2024-08-08 15:37:36 UTC Build successful.
🟢 2024-08-08 15:38:15 UTC ydbd size 8.1 GiB changed* by +1.1 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: a08fe02 merge: f864b18 diff diff %
ydbd size 8 675 950 208 Bytes 8 675 951 336 Bytes +1.1 KiB +0.000%
ydbd stripped size 472 179 592 Bytes 472 179 784 Bytes +192 Bytes +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

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 8, 2024 •
edited
Loading

⚪ 2024-08-08 11:58:31 UTC Pre-commit check for f864b18 has started.
⚪ 2024-08-08 12:02:50 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-08 15:30:21 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11094 10864 0 162 32 36

🟢 2024-08-08 15:31:38 UTC Build successful.
🟢 2024-08-08 15:32:16 UTC ydbd size 5.4 GiB changed* by +176 Bytes, which is < 100.0 KiB vs main: OK

ydbd size dash main: a08fe02 merge: f864b18 diff diff %
ydbd size 5 834 482 344 Bytes 5 834 482 520 Bytes +176 Bytes +0.000%
ydbd stripped size 1 465 547 824 Bytes 1 465 548 080 Bytes +256 Bytes +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

@abyss7 abyss7 requested a review from pavelvelikhov August 8, 2024 13:08
abyss7
abyss7 commented Aug 8, 2024
View reviewed changes
...sts/canondata/test_postgres.TestPGSQL.test_sql_suite_plan-jointest_join0.test_/query_13.plan
@@ -185,14 +185,7 @@
"PlanNodeType": "Connection",
"Plans": [
{
"Node Type": "ConstantExpr",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suspisious

Copy link
Collaborator

Choose a reason for hiding this comment

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

Вроде норм

@abyss7 abyss7 requested a review from gridnevvvit August 8, 2024 15:24
pavelvelikhov
pavelvelikhov previously approved these changes Aug 8, 2024
View reviewed changes
...sts/canondata/test_postgres.TestPGSQL.test_sql_suite_plan-jointest_join0.test_/query_13.plan
@@ -185,14 +185,7 @@
"PlanNodeType": "Connection",
"Plans": [
{
"Node Type": "ConstantExpr",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Вроде норм

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2024 •
edited
Loading

⚪ 2024-08-09 11:16:26 UTC Pre-commit check for c1a47e3 has started.
⚪ 2024-08-09 11:19:29 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-09 11:55:47 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2024 •
edited
Loading

⚪ 2024-08-09 11:17:56 UTC Pre-commit check for c1a47e3 has started.
⚪ 2024-08-09 11:21:03 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-09 13:44:19 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39882 34556 0 3 5312 11

⚪ 2024-08-09 13:48:58 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟡 2024-08-09 13:56:34 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
18 (only retried tests) 8 0 1 1 8

⚪ 2024-08-09 13:56:42 UTC Failed tests rerun (try 3) linux-x86_64-relwithdebinfo is running...
🔴 2024-08-09 14:03:59 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10 (only retried tests) 1 0 1 0 8

🟢 2024-08-09 14:04:06 UTC Build successful.
🟢 2024-08-09 14:04:45 UTC ydbd size 8.1 GiB changed* by -328 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash main: 403d37f merge: c1a47e3 diff diff %
ydbd size 8 688 414 328 Bytes 8 688 414 000 Bytes -328 Bytes -0.000%
ydbd stripped size 472 744 808 Bytes 472 745 192 Bytes +384 Bytes +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

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2024 •
edited
Loading

⚪ 2024-08-09 11:18:33 UTC Pre-commit check for c1a47e3 has started.
⚪ 2024-08-09 11:21:39 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-09 14:01:58 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11081 11011 0 26 27 17

🟢 2024-08-09 14:03:13 UTC Build successful.
🟢 2024-08-09 14:03:45 UTC ydbd size 5.4 GiB changed* by +1.1 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 403d37f merge: c1a47e3 diff diff %
ydbd size 5 843 569 184 Bytes 5 843 570 320 Bytes +1.1 KiB +0.000%
ydbd stripped size 1 468 062 000 Bytes 1 468 062 192 Bytes +192 Bytes +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

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2024 •
edited
Loading

⚪ 2024-08-09 14:27:24 UTC Pre-commit check for 8fc45c8 has started.
⚪ 2024-08-09 14:30:22 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-09 14:35:55 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2024 •
edited
Loading

⚪ 2024-08-09 14:27:31 UTC Pre-commit check for 8fc45c8 has started.
⚪ 2024-08-09 14:31:35 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-09 16:31:54 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11089 11052 0 2 22 13

🟢 2024-08-09 16:33:37 UTC Build successful.
🟢 2024-08-09 16:34:04 UTC ydbd size 5.4 GiB changed* by +10.2 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 633ff57 merge: 8fc45c8 diff diff %
ydbd size 5 843 780 776 Bytes 5 843 791 232 Bytes +10.2 KiB +0.000%
ydbd stripped size 1 468 104 912 Bytes 1 468 106 384 Bytes +1.4 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

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 9, 2024 •
edited
Loading

⚪ 2024-08-09 14:27:51 UTC Pre-commit check for 8fc45c8 has started.
⚪ 2024-08-09 14:30:57 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-09 17:05:03 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
39885 34557 0 5 5311 12

⚪ 2024-08-09 17:09:49 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟢 2024-08-09 17:18:35 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
19 (only retried tests) 10 0 0 0 9

🟢 2024-08-09 17:18:42 UTC Build successful.
🟢 2024-08-09 17:19:16 UTC ydbd size 8.1 GiB changed* by +6.5 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 633ff57 merge: 8fc45c8 diff diff %
ydbd size 8 688 682 368 Bytes 8 688 689 024 Bytes +6.5 KiB +0.000%
ydbd stripped size 472 752 392 Bytes 472 752 840 Bytes +448 Bytes +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

@abyss7 abyss7 requested a review from pavelvelikhov August 9, 2024 16:16
@abyss7 abyss7 merged commit 898a9ff into ydb-platform:main Aug 9, 2024
10 of 12 checks passed
@GrigoriyPA GrigoriyPA mentioned this pull request Aug 13, 2024
Merged
abyss7 added a commit to abyss7/ydb that referenced this pull request Sep 9, 2024
@abyss7
Enable auto-block mode by default (ydb-platform#7567)
e0d1d96 
(cherry picked from commit 898a9ff)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pavelvelikhov pavelvelikhov pavelvelikhov approved these changes

@spuchin spuchin spuchin approved these changes

@gridnevvvit gridnevvvit Awaiting requested review from gridnevvvit

Assignees
No one assigned
Labels
new-feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@abyss7 @pavelvelikhov @spuchin