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

PgWire auth with ApiKey #7973

Merged
merged 5 commits into from
Aug 26, 2024
Merged

PgWire auth with ApiKey #7973

merged 5 commits into from
Aug 26, 2024

Conversation

shnikd
Copy link
Collaborator

@shnikd shnikd commented Aug 19, 2024

No description provided.

Copy link

github-actions bot commented Aug 19, 2024

2024-08-19 05:13:38 UTC Pre-commit check for 061aafc has started.
2024-08-19 05:16:16 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-19 07:38:47 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11286 11215 0 13 23 35

🟢 2024-08-19 07:39:45 UTC Build successful.
🟡 2024-08-19 07:40:13 UTC ydbd size 5.5 GiB changed* by +301.2 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 274bfdd merge: 061aafc diff diff %
ydbd size 5 880 316 984 Bytes 5 880 625 416 Bytes +301.2 KiB +0.005%
ydbd stripped size 1 477 405 904 Bytes 1 477 430 480 Bytes +24.0 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 Aug 19, 2024

2024-08-19 05:15:04 UTC Pre-commit check for 061aafc has started.
2024-08-19 05:17:42 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-19 05:24:15 UTC Build successful.

Copy link

github-actions bot commented Aug 19, 2024

2024-08-19 05:15:12 UTC Pre-commit check for 061aafc has started.
2024-08-19 05:17:57 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-19 06:39:16 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15680 14294 0 8 1338 40

2024-08-19 06:40:35 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟢 2024-08-19 06:49:08 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
54 (only retried tests) 26 0 0 0 28

🟢 2024-08-19 06:49:16 UTC Build successful.
🟡 2024-08-19 06:49:51 UTC ydbd size 8.1 GiB changed* by +570.4 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 5522a96 merge: 061aafc diff diff %
ydbd size 8 736 888 376 Bytes 8 737 472 464 Bytes +570.4 KiB +0.007%
ydbd stripped size 475 560 200 Bytes 475 570 504 Bytes +10.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


void Bootstrap() {
if (PgWireAuthData.UserName == "__ydb_apikey") {
SendDescribeRequest();
Copy link
Collaborator

Choose a reason for hiding this comment

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

may be move functions SendDescribeRequest and SendApiKeyRequest to some function "api key handler"?

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 07:08:15 UTC Pre-commit check for 142d44a has started.
2024-08-21 07:11:48 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-21 07:17:52 UTC Build successful.

Copy link

github-actions bot commented Aug 21, 2024

2024-08-21 07:09:42 UTC Pre-commit check for 142d44a has started.
2024-08-21 07:12:44 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-21 08:56:13 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11149 11067 0 27 34 21

🟢 2024-08-21 08:57:10 UTC Build successful.
🟡 2024-08-21 08:57:45 UTC ydbd size 5.5 GiB changed* by +375.7 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 675705a merge: 142d44a diff diff %
ydbd size 5 887 791 976 Bytes 5 888 176 696 Bytes +375.7 KiB +0.007%
ydbd stripped size 1 478 458 288 Bytes 1 478 493 808 Bytes +34.7 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 Aug 21, 2024

2024-08-21 07:10:18 UTC Pre-commit check for 142d44a has started.
2024-08-21 07:13:11 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-21 09:13:20 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15721 14350 0 8 1347 16

2024-08-21 09:14:41 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟡 2024-08-21 09:21:56 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
31 (only retried tests) 21 0 2 0 8

2024-08-21 09:22:04 UTC Failed tests rerun (try 3) linux-x86_64-relwithdebinfo is running...
🟢 2024-08-21 09:29:45 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15 (only retried tests) 7 0 0 0 8

🟢 2024-08-21 09:29:53 UTC Build successful.
🟡 2024-08-21 09:30:32 UTC ydbd size 8.1 GiB changed* by +573.9 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 675705a merge: 142d44a diff diff %
ydbd size 8 748 412 864 Bytes 8 749 000 504 Bytes +573.9 KiB +0.007%
ydbd stripped size 475 824 616 Bytes 475 835 176 Bytes +10.3 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 Aug 22, 2024

2024-08-22 13:02:33 UTC Pre-commit check for b4d886f has started.
2024-08-22 13:05:15 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-22 13:11:51 UTC Build successful.

Copy link

github-actions bot commented Aug 22, 2024

2024-08-22 13:03:53 UTC Pre-commit check for b4d886f has started.
2024-08-22 13:06:37 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-22 14:43:49 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11177 11079 0 38 37 23

🟢 2024-08-22 14:44:47 UTC Build successful.
🟡 2024-08-22 14:45:19 UTC ydbd size 5.5 GiB changed* by +384.6 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: ba138ae merge: b4d886f diff diff %
ydbd size 5 890 308 040 Bytes 5 890 701 864 Bytes +384.6 KiB +0.007%
ydbd stripped size 1 479 110 672 Bytes 1 479 151 312 Bytes +39.7 KiB +0.003%

*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 Aug 22, 2024

2024-08-22 13:05:56 UTC Pre-commit check for b4d886f has started.
2024-08-22 13:10:11 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-22 14:30:20 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15756 14384 0 6 1344 22

2024-08-22 14:31:58 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟢 2024-08-22 14:39:41 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
32 (only retried tests) 24 0 0 0 8

🟢 2024-08-22 14:39:48 UTC Build successful.
🟡 2024-08-22 14:40:27 UTC ydbd size 8.2 GiB changed* by +571.5 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: ba138ae merge: b4d886f diff diff %
ydbd size 8 752 126 272 Bytes 8 752 711 496 Bytes +571.5 KiB +0.007%
ydbd stripped size 476 005 128 Bytes 476 011 720 Bytes +6.4 KiB +0.001%

*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 Aug 23, 2024

2024-08-23 18:15:54 UTC Pre-commit check for a9456dd has started.
2024-08-23 18:18:41 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-23 19:03:14 UTC Build successful.

Copy link

github-actions bot commented Aug 23, 2024

2024-08-23 18:16:00 UTC Pre-commit check for a9456dd has started.
2024-08-23 18:18:55 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-23 19:17:16 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15770 14395 0 4 1348 23

2024-08-23 19:18:35 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟢 2024-08-23 19:26:53 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
24 (only retried tests) 15 0 0 0 9

🟢 2024-08-23 19:27:00 UTC Build successful.
🟡 2024-08-23 19:27:35 UTC ydbd size 8.2 GiB changed* by +574.3 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 92ff6c1 merge: a9456dd diff diff %
ydbd size 8 784 784 440 Bytes 8 785 372 504 Bytes +574.3 KiB +0.007%
ydbd stripped size 479 498 440 Bytes 479 509 064 Bytes +10.4 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 Aug 23, 2024

2024-08-23 18:16:05 UTC Pre-commit check for a9456dd has started.
2024-08-23 18:18:50 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-23 19:49:49 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11182 11096 0 29 42 15

🟢 2024-08-23 19:50:45 UTC Build successful.
🟡 2024-08-23 19:51:14 UTC ydbd size 5.5 GiB changed* by +382.6 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 92ff6c1 merge: a9456dd diff diff %
ydbd size 5 918 942 800 Bytes 5 919 334 576 Bytes +382.6 KiB +0.007%
ydbd stripped size 1 487 376 880 Bytes 1 487 417 200 Bytes +39.4 KiB +0.003%

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

@shnikd shnikd merged commit 276ad71 into ydb-platform:main Aug 26, 2024
10 of 12 checks passed
shnikd added a commit to shnikd/ydb that referenced this pull request Aug 26, 2024
rekby pushed a commit to rekby-forks/ydb that referenced this pull request Aug 28, 2024
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