Skip to content

Client certificates options in ydb cli #8406

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

Conversation

UgnineSirdis
Copy link
Collaborator

@UgnineSirdis UgnineSirdis commented Aug 28, 2024

Changelog entry

Command line options for ydb cli that provide client certificate to mTLS connection.
Implement according to RFC: https://github.com/ydb-platform/ydb-rfc/blob/main/cli_client_certificate_options.md
The next PRs will support env vars and improved help.

Changelog category

  • New feature

Additional information

...

Copy link

github-actions bot commented Aug 28, 2024

2024-08-28 15:15:53 UTC Pre-commit check linux-x86_64-release-clang14 for fec8fce has started.
2024-08-28 15:19:40 UTC ya make is running...
🟢 2024-08-28 15:35:51 UTC Build successful.

Copy link

github-actions bot commented Aug 28, 2024

2024-08-28 15:16:33 UTC Pre-commit check linux-x86_64-relwithdebinfo for fec8fce has started.
2024-08-28 15:19:14 UTC ya make is running...
🟡 2024-08-28 16:20:30 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15843 14473 0 4 1351 15

2024-08-28 16:22:16 UTC ya make is running... (failed tests rerun, try 2)
🟡 2024-08-28 16:31:10 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
24 (only retried tests) 10 0 2 0 12

2024-08-28 16:31:18 UTC ya make is running... (failed tests rerun, try 3)
🔴 2024-08-28 16:38:55 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
19 (only retried tests) 6 0 2 0 11

🟢 2024-08-28 16:39:02 UTC Build successful.
🟡 2024-08-28 16:39:40 UTC ydbd size 8.2 GiB changed* by +1.6 MiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: b83f29e merge: fec8fce diff diff %
ydbd size 8 807 257 640 Bytes 8 808 945 392 Bytes +1.6 MiB +0.019%
ydbd stripped size 480 255 656 Bytes 480 369 768 Bytes +111.4 KiB +0.024%

*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 28, 2024

2024-08-28 15:16:33 UTC Pre-commit check linux-x86_64-release-asan for fec8fce has started.
2024-08-28 15:19:17 UTC ya make is running...
🔴 2024-08-28 17:12:35 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11252 11118 0 65 37 32

🟢 2024-08-28 17:13:38 UTC Build successful.
🟡 2024-08-28 17:14:10 UTC ydbd size 5.5 GiB changed* by +916.0 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: c3dbb7f merge: fec8fce diff diff %
ydbd size 5 935 063 608 Bytes 5 936 001 560 Bytes +916.0 KiB +0.016%
ydbd stripped size 1 490 651 376 Bytes 1 490 858 096 Bytes +201.9 KiB +0.014%

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

@UgnineSirdis UgnineSirdis force-pushed the client-certificates-options-in-ydb-cli branch from c6af5d1 to c7fcad4 Compare October 7, 2024 11:24
Copy link

github-actions bot commented Oct 7, 2024

2024-10-07 11:27:44 UTC Pre-commit check linux-x86_64-relwithdebinfo for d772418 has started.
2024-10-07 11:28:15 UTC Artifacts will be uploaded here
2024-10-07 11:31:39 UTC ya make is running...
🟡 2024-10-07 12:30:40 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
41321 35746 0 6 5458 111

2024-10-07 12:34:08 UTC ya make is running... (failed tests rerun, try 2)
🟡 2024-10-07 12:45:31 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
115 (only retried tests) 19 0 1 0 95

2024-10-07 12:45:39 UTC ya make is running... (failed tests rerun, try 3)
🟢 2024-10-07 12:57:04 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
96 (only retried tests) 3 0 0 0 93

🟢 2024-10-07 12:57:11 UTC Build successful.
🟢 2024-10-07 12:57:33 UTC ydbd size 2.8 GiB changed* by +57.6 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 6361cf3 merge: d772418 diff diff %
ydbd size 3 002 359 304 Bytes 3 002 418 288 Bytes +57.6 KiB +0.002%
ydbd stripped size 477 359 032 Bytes 477 364 536 Bytes +5.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 Oct 7, 2024

2024-10-07 11:28:46 UTC Pre-commit check linux-x86_64-release-asan for d772418 has started.
2024-10-07 11:28:57 UTC Artifacts will be uploaded here
2024-10-07 11:31:59 UTC ya make is running...
🟡 2024-10-07 13:06:18 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11649 11224 0 84 293 48

🟢 2024-10-07 13:07:17 UTC Build successful.
🟡 2024-10-07 13:07:52 UTC ydbd size 5.7 GiB changed* by +167.8 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 6361cf3 merge: d772418 diff diff %
ydbd size 6 084 062 752 Bytes 6 084 234 600 Bytes +167.8 KiB +0.003%
ydbd stripped size 1 519 811 472 Bytes 1 519 832 464 Bytes +20.5 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

@UgnineSirdis UgnineSirdis force-pushed the client-certificates-options-in-ydb-cli branch from c7fcad4 to 6f4ceed Compare October 31, 2024 11:07
Copy link

github-actions bot commented Oct 31, 2024

2024-10-31 11:10:04 UTC Pre-commit check linux-x86_64-release-asan for 5287e88 has started.
2024-10-31 11:10:14 UTC Artifacts will be uploaded here
2024-10-31 11:13:33 UTC ya make is running...
🟡 2024-10-31 12:36:00 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11121 11036 0 37 21 27

🟢 2024-10-31 12:36:57 UTC Build successful.
🟡 2024-10-31 12:37:31 UTC ydbd size 5.7 GiB changed* by +167.4 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 55a88ca merge: 5287e88 diff diff %
ydbd size 6 149 801 808 Bytes 6 149 973 256 Bytes +167.4 KiB +0.003%
ydbd stripped size 1 534 429 616 Bytes 1 534 450 352 Bytes +20.2 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 Oct 31, 2024

2024-10-31 11:10:40 UTC Pre-commit check linux-x86_64-relwithdebinfo for 5287e88 has started.
2024-10-31 11:10:51 UTC Artifacts will be uploaded here
2024-10-31 11:14:07 UTC ya make is running...
🟡 2024-10-31 12:32:51 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
75423 61351 0 5 13973 94

2024-10-31 12:39:46 UTC ya make is running... (failed tests rerun, try 2)
🟢 2024-10-31 12:53:35 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
98 (only retried tests) 8 0 0 0 90

🟢 2024-10-31 12:53:42 UTC Build successful.
🟢 2024-10-31 12:54:01 UTC ydbd size 2.8 GiB changed* by +57.5 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 55a88ca merge: 5287e88 diff diff %
ydbd size 3 039 299 072 Bytes 3 039 357 992 Bytes +57.5 KiB +0.002%
ydbd stripped size 481 329 016 Bytes 481 334 456 Bytes +5.3 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

@UgnineSirdis UgnineSirdis force-pushed the client-certificates-options-in-ydb-cli branch from 6f4ceed to db255d7 Compare November 5, 2024 09:45
Copy link

github-actions bot commented Nov 5, 2024

2024-11-05 09:49:26 UTC Pre-commit check linux-x86_64-relwithdebinfo for 466eb55 has started.
2024-11-05 09:49:37 UTC Artifacts will be uploaded here
2024-11-05 09:52:48 UTC ya make is running...
🟡 2024-11-05 11:07:47 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
75486 61360 0 4 14017 105

2024-11-05 11:14:54 UTC ya make is running... (failed tests rerun, try 2)
🟢 2024-11-05 11:26:33 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
134 (only retried tests) 41 0 0 1 92

🟢 2024-11-05 11:26:39 UTC Build successful.
🟢 2024-11-05 11:27:00 UTC ydbd size 2.8 GiB changed* by +57.5 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 7672737 merge: 466eb55 diff diff %
ydbd size 3 039 362 384 Bytes 3 039 421 304 Bytes +57.5 KiB +0.002%
ydbd stripped size 481 619 384 Bytes 481 624 824 Bytes +5.3 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 Nov 5, 2024

2024-11-05 09:49:33 UTC Pre-commit check linux-x86_64-release-asan for 466eb55 has started.
2024-11-05 09:49:43 UTC Artifacts will be uploaded here
2024-11-05 09:52:57 UTC ya make is running...
🟡 2024-11-05 11:13:04 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11129 11044 0 33 19 33

🟢 2024-11-05 11:14:03 UTC Build successful.
🟡 2024-11-05 11:14:33 UTC ydbd size 5.7 GiB changed* by +171.4 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 7672737 merge: 466eb55 diff diff %
ydbd size 6 150 333 736 Bytes 6 150 509 288 Bytes +171.4 KiB +0.003%
ydbd stripped size 1 535 125 648 Bytes 1 535 150 480 Bytes +24.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

@UgnineSirdis UgnineSirdis force-pushed the client-certificates-options-in-ydb-cli branch from db255d7 to 35b603e Compare February 27, 2025 19:17
@UgnineSirdis UgnineSirdis requested review from a team as code owners February 27, 2025 19:17
Copy link

🟢 2025-02-27 19:27:15 UTC The validation of the Pull Request description is successful.

Copy link

github-actions bot commented Feb 27, 2025

2025-02-27 19:28:32 UTC Pre-commit check linux-x86_64-release-asan for 34e5f3b has started.
2025-02-27 19:28:46 UTC Artifacts will be uploaded here
2025-02-27 19:31:42 UTC ya make is running...
🟡 2025-02-27 20:51:56 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13860 13668 0 108 47 37

2025-02-27 20:53:10 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-02-27 21:05:53 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
267 (only retried tests) 233 0 3 1 30

2025-02-27 21:06:03 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-02-27 21:17:39 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
63 (only retried tests) 32 0 1 0 30

🟢 2025-02-27 21:17:50 UTC Build successful.
🟡 2025-02-27 21:18:19 UTC ydbd size 3.7 GiB changed* by +162.9 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 9ac8b04 merge: 34e5f3b diff diff %
ydbd size 3 985 630 368 Bytes 3 985 797 216 Bytes +162.9 KiB +0.004%
ydbd stripped size 1 387 837 256 Bytes 1 387 875 912 Bytes +37.8 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 Feb 27, 2025

2025-02-27 19:28:32 UTC Pre-commit check linux-x86_64-relwithdebinfo for 34e5f3b has started.
2025-02-27 19:28:47 UTC Artifacts will be uploaded here
2025-02-27 19:31:49 UTC ya make is running...
🟡 2025-02-27 20:40:28 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
28240 25638 0 6 2463 133

2025-02-27 20:42:55 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-02-27 20:53:54 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
207 (only retried tests) 80 0 1 0 126

2025-02-27 20:54:03 UTC ya make is running... (failed tests rerun, try 3)
🔴 2025-02-27 21:05:26 UTC Some tests failed, follow the links below.

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
190 (only retried tests) 65 0 1 0 124

🟢 2025-02-27 21:05:36 UTC Build successful.
🟢 2025-02-27 21:05:56 UTC ydbd size 2.1 GiB changed* by +98.9 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 9ac8b04 merge: 34e5f3b diff diff %
ydbd size 2 286 798 992 Bytes 2 286 900 216 Bytes +98.9 KiB +0.004%
ydbd stripped size 479 450 688 Bytes 479 472 640 Bytes +21.4 KiB +0.005%

*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 Feb 28, 2025

2025-02-28 10:49:13 UTC Pre-commit check linux-x86_64-release-asan for ea8e9c7 has started.
2025-02-28 10:49:17 UTC Artifacts will be uploaded here
2025-02-28 10:52:30 UTC ya make is running...
2025-02-28 11:02:39 UTC Check cancelled

Copy link

github-actions bot commented Feb 28, 2025

2025-02-28 10:50:03 UTC Pre-commit check linux-x86_64-relwithdebinfo for ea8e9c7 has started.
2025-02-28 10:50:17 UTC Artifacts will be uploaded here
2025-02-28 10:53:14 UTC ya make is running...
2025-02-28 11:02:39 UTC Check cancelled

Copy link
Collaborator

@pnv1 pnv1 left a comment

Choose a reason for hiding this comment

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

Copy link

github-actions bot commented Feb 28, 2025

2025-02-28 11:03:09 UTC Pre-commit check linux-x86_64-release-asan for 2e77377 has started.
2025-02-28 11:03:24 UTC Artifacts will be uploaded here
2025-02-28 11:06:28 UTC ya make is running...
🟡 2025-02-28 12:30:04 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13868 13695 0 105 34 34

2025-02-28 12:31:37 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-02-28 12:47:17 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet Going to retry failed tests...

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
242 (only retried tests) 202 0 6 4 30

2025-02-28 12:47:26 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-02-28 12:59:35 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
69 (only retried tests) 38 0 0 0 31

🟢 2025-02-28 12:59:46 UTC Build successful.
🟡 2025-02-28 13:00:18 UTC ydbd size 3.7 GiB changed* by +163.0 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 0db71ba merge: 2e77377 diff diff %
ydbd size 3 986 703 248 Bytes 3 986 870 168 Bytes +163.0 KiB +0.004%
ydbd stripped size 1 388 116 936 Bytes 1 388 155 656 Bytes +37.8 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
Collaborator

@pnv1 pnv1 left a comment

Choose a reason for hiding this comment

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

CLI part OK

Copy link

github-actions bot commented Feb 28, 2025

2025-02-28 11:04:37 UTC Pre-commit check linux-x86_64-relwithdebinfo for 2e77377 has started.
2025-02-28 11:04:48 UTC Artifacts will be uploaded here
2025-02-28 11:07:57 UTC ya make is running...
🟡 2025-02-28 12:10:41 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
28251 25649 0 1 2465 136

2025-02-28 12:13:05 UTC ya make is running... (failed tests rerun, try 2)
🟢 2025-02-28 12:30:58 UTC Tests successful.

Test history | Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
201 (only retried tests) 76 0 0 0 125

🟢 2025-02-28 12:31:06 UTC Build successful.
🟢 2025-02-28 12:31:30 UTC ydbd size 2.1 GiB changed* by +98.8 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 0db71ba merge: 2e77377 diff diff %
ydbd size 2 287 412 760 Bytes 2 287 513 944 Bytes +98.8 KiB +0.004%
ydbd stripped size 479 533 664 Bytes 479 555 552 Bytes +21.4 KiB +0.005%

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

@UgnineSirdis UgnineSirdis merged commit c5b531e into ydb-platform:main Feb 28, 2025
12 checks passed
@UgnineSirdis UgnineSirdis deleted the client-certificates-options-in-ydb-cli branch February 28, 2025 13:01
lberserq pushed a commit to lberserq/ydb that referenced this pull request Mar 3, 2025
UgnineSirdis added a commit to UgnineSirdis/ydb that referenced this pull request Mar 10, 2025
UgnineSirdis added a commit to UgnineSirdis/ydb that referenced this pull request Mar 10, 2025
(cherry picked from commit c5b531e)
(cherry picked from commit 50c34e8)
blinkov pushed a commit that referenced this pull request Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants