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.

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

Kafka port assignment through discoveryCache #13197

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Kafka port assignment through discoveryCache #13197

wants to merge 2 commits into from
+486 −34

Conversation

FloatingCrowbar
Copy link
Collaborator

Changelog entry

Use discovery for kafka proxy port (instead of single port assigned in configs).
...

Changelog category

  • New feature

Additional information

...

@FloatingCrowbar FloatingCrowbar requested a review from a team as a code owner January 8, 2025 15:53
@FloatingCrowbar FloatingCrowbar force-pushed the kafka branch 2 times, most recently from d2c14d8 to 15b0d19 Compare January 8, 2025 16:11
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 8, 2025 •
edited
Loading

⚪ 2025-01-08 16:18:20 UTC Pre-commit check linux-x86_64-release-asan for 22c5e8b has started.
⚪ 2025-01-08 16:18:34 UTC Artifacts will be uploaded here
⚪ 2025-01-08 16:21:35 UTC ya make is running...
🟡 2025-01-08 17:27: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?
13167 13097 0 32 8 30

⚪ 2025-01-08 17:29:37 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-08 17:41:39 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?
117 (only retried tests) 83 0 7 1 26

⚪ 2025-01-08 17:41:47 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-01-08 17:53:32 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 4 0 27

🟢 2025-01-08 17:53:40 UTC Build successful.
🟡 2025-01-08 17:54:10 UTC ydbd size 3.6 GiB changed* by +145.8 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 9f31130 merge: 22c5e8b diff diff %
ydbd size 3 873 834 368 Bytes 3 873 983 688 Bytes +145.8 KiB +0.004%
ydbd stripped size 1 350 684 240 Bytes 1 350 725 904 Bytes +40.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

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 8, 2025 •
edited
Loading

⚪ 2025-01-08 16:18:32 UTC Pre-commit check linux-x86_64-relwithdebinfo for 22c5e8b has started.
⚪ 2025-01-08 16:18:45 UTC Artifacts will be uploaded here
⚪ 2025-01-08 16:21:44 UTC ya make is running...
🟡 2025-01-08 17:09:53 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?
20295 18833 0 3 1397 62

⚪ 2025-01-08 17:11:40 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-08 17:23:18 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?
359 (only retried tests) 228 0 1 7 123

⚪ 2025-01-08 17:23:28 UTC ya make is running... (failed tests rerun, try 3)
🔴 2025-01-08 17:35:20 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?
180 (only retried tests) 56 0 1 0 123

🟢 2025-01-08 17:35:27 UTC Build successful.
🟢 2025-01-08 17:35:50 UTC ydbd size 2.1 GiB changed* by +76.9 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 9f31130 merge: 22c5e8b diff diff %
ydbd size 2 244 317 464 Bytes 2 244 396 160 Bytes +76.9 KiB +0.004%
ydbd stripped size 472 336 560 Bytes 472 346 480 Bytes +9.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

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 8, 2025 •
edited
Loading

⚪ 2025-01-08 20:30:12 UTC Pre-commit check linux-x86_64-relwithdebinfo for 458e984 has started.
⚪ 2025-01-08 20:30:24 UTC Artifacts will be uploaded here
⚪ 2025-01-08 20:33:17 UTC ya make is running...
🟡 2025-01-08 21:19:32 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?
20295 18987 0 6 1169 133

⚪ 2025-01-08 21:21:11 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-08 21:32:47 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?
204 (only retried tests) 81 0 3 0 120

⚪ 2025-01-08 21:32:55 UTC ya make is running... (failed tests rerun, try 3)
🔴 2025-01-08 21:44:16 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?
180 (only retried tests) 58 0 2 0 120

🟢 2025-01-08 21:44:23 UTC Build successful.
🟢 2025-01-08 21:44:46 UTC ydbd size 2.1 GiB changed* by +73.6 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 9f31130 merge: 458e984 diff diff %
ydbd size 2 244 317 464 Bytes 2 244 392 832 Bytes +73.6 KiB +0.003%
ydbd stripped size 472 336 560 Bytes 472 345 840 Bytes +9.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

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 8, 2025 •
edited
Loading

⚪ 2025-01-08 20:30:28 UTC Pre-commit check linux-x86_64-release-asan for 458e984 has started.
⚪ 2025-01-08 20:30:40 UTC Artifacts will be uploaded here
⚪ 2025-01-08 20:33:39 UTC ya make is running...
🟡 2025-01-08 21:34:34 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?
13167 13096 0 37 4 30

⚪ 2025-01-08 21:35:40 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-01-08 21:47:52 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?
116 (only retried tests) 82 0 8 0 26

⚪ 2025-01-08 21:48:00 UTC ya make is running... (failed tests rerun, try 3)
🟡 2025-01-08 21:59:46 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) 33 0 5 1 24

🟢 2025-01-08 21:59:52 UTC Build successful.
🟡 2025-01-08 22:00:24 UTC ydbd size 3.6 GiB changed* by +145.8 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 9f31130 merge: 458e984 diff diff %
ydbd size 3 873 834 368 Bytes 3 873 983 712 Bytes +145.8 KiB +0.004%
ydbd stripped size 1 350 684 240 Bytes 1 350 729 552 Bytes +44.2 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@niksaveliev niksaveliev Awaiting requested review from niksaveliev niksaveliev is a code owner automatically assigned from ydb-platform/topics

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
1 participant
@FloatingCrowbar