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

Workload read without consumer and metrics fixes #1792

Conversation

niksaveliev
Copy link
Collaborator

@niksaveliev niksaveliev commented Feb 12, 2024

Changelog entry

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

...

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 07:17:39 UTC Pre-commit check for e3a492a has started.
2024-02-12 07:17:40 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-12 07:19:51 UTC Build successful.
2024-02-12 07:20:02 UTC Tests are running...
🔴 2024-02-12 08:35:40 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67327 56391 0 5 10892 39

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 07:17:49 UTC Pre-commit check for e3a492a has started.
2024-02-12 07:17:50 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-12 07:19:54 UTC Build successful.
2024-02-12 07:20:03 UTC Tests are running...
🔴 2024-02-12 08:58:54 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14739 14542 0 13 137 47

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 10:31:59 UTC Pre-commit check for ecf65aa has started.
2024-02-12 10:32:02 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-12 10:34:38 UTC Build successful.
2024-02-12 10:34:46 UTC Tests are running...
🔴 2024-02-12 12:16:44 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14740 14551 0 19 137 33

Copy link

github-actions bot commented Feb 12, 2024

2024-02-12 10:32:07 UTC Pre-commit check for ecf65aa has started.
2024-02-12 10:32:10 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-12 10:34:42 UTC Build successful.
2024-02-12 10:34:52 UTC Tests are running...
🔴 2024-02-12 12:20:00 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67425 56494 0 4 10892 35

Copy link

github-actions bot commented Feb 13, 2024

2024-02-13 12:47:51 UTC Pre-commit check for c18370b has started.
2024-02-13 12:47:52 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-13 12:49:56 UTC Build successful.
2024-02-13 12:50:08 UTC Tests are running...
🔴 2024-02-13 14:25:23 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14761 14563 0 15 135 48

Copy link

github-actions bot commented Feb 13, 2024

2024-02-13 12:48:02 UTC Pre-commit check for c18370b has started.
2024-02-13 12:48:04 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-13 12:50:09 UTC Build successful.
2024-02-13 12:50:19 UTC Tests are running...
🔴 2024-02-13 14:03:35 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67470 56450 0 3 10972 45

@niksaveliev niksaveliev marked this pull request as ready for review February 14, 2024 04:58
@@ -536,7 +538,8 @@ void TPersQueueReadBalancer::Handle(TEvPersQueue::TEvUpdateBalancerConfig::TPtr
Consumers[rr];
}
}

Consumers[CLIENTID_WITHOUT_CONSUMER];
Copy link
Collaborator

Choose a reason for hiding this comment

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

Read without consumer doesn't use read balancer. Why do we need this line?

@@ -31,6 +31,20 @@
name=topic.reserve.used_bytes: 0
name=topic.storage_bytes: 0

consumer=$without_consumer:
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is not what we want to see. No label consumer is possible solution. But some sensors are useless in reading without consumer. Sensor topic.partition.committed_end_to_end_lag_milliseconds_max is undefined for example.

Copy link

github-actions bot commented Feb 15, 2024

2024-02-15 06:58:33 UTC Pre-commit check for 177476e has started.
2024-02-15 06:58:34 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-15 07:03:21 UTC Build successful.
2024-02-15 07:03:30 UTC Tests are running...
🔴 2024-02-15 08:34:33 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67530 56498 0 6 10974 52

Copy link

github-actions bot commented Feb 15, 2024

2024-02-15 06:58:57 UTC Pre-commit check for 177476e has started.
2024-02-15 06:58:59 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-15 07:03:57 UTC Build successful.
2024-02-15 07:04:10 UTC Tests are running...
🔴 2024-02-15 08:39:50 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14774 14586 0 14 128 46

@niksaveliev niksaveliev force-pushed the read-without-consumer-workload-and-counters-fixes branch from e81f997 to 8ff4db8 Compare February 20, 2024 14:20
Copy link

github-actions bot commented Feb 20, 2024

2024-02-20 14:23:15 UTC Pre-commit check for 313438d has started.
2024-02-20 14:23:16 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-20 14:40:36 UTC Build successful.
2024-02-20 14:40:47 UTC Tests are running...
🔴 2024-02-20 16:12:43 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67709 56713 0 16 10945 35

Copy link

github-actions bot commented Feb 20, 2024

2024-02-20 14:23:42 UTC Pre-commit check for 313438d has started.
2024-02-20 14:23:44 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-20 14:44:06 UTC Build successful.
2024-02-20 14:44:21 UTC Tests are running...
🔴 2024-02-20 16:28:52 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14692 14471 0 27 153 41

Copy link

github-actions bot commented Feb 21, 2024

2024-02-21 15:28:18 UTC Pre-commit check for f7ef27e has started.
2024-02-21 15:28:20 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-21 15:32:43 UTC Build successful.
2024-02-21 15:32:54 UTC Tests are running...
🔴 2024-02-21 17:06:31 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67906 56912 0 4 10951 39

Copy link

github-actions bot commented Feb 21, 2024

2024-02-21 15:33:49 UTC Pre-commit check for f7ef27e has started.
2024-02-21 15:33:50 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-21 15:39:08 UTC Build successful.
2024-02-21 15:39:18 UTC Tests are running...
🔴 2024-02-21 17:20:42 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14828 14615 0 31 139 43

@niksaveliev niksaveliev force-pushed the read-without-consumer-workload-and-counters-fixes branch from 9958322 to 39df411 Compare February 28, 2024 05:14
Copy link

github-actions bot commented Feb 28, 2024

2024-02-28 05:17:45 UTC Pre-commit check for dade8a1 has started.
2024-02-28 05:17:48 UTC Build linux-x86_64-release-cmake14 is running...
🟢 2024-02-28 05:47:25 UTC Build successful.

Copy link

github-actions bot commented Feb 28, 2024

2024-02-28 05:17:51 UTC Pre-commit check for dade8a1 has started.
2024-02-28 05:17:53 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-28 05:24:10 UTC Build successful.
2024-02-28 05:24:23 UTC Tests are running...
🔴 2024-02-28 07:02:59 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14876 14751 0 26 77 22

Copy link

github-actions bot commented Feb 28, 2024

2024-02-28 05:17:52 UTC Pre-commit check for dade8a1 has started.
2024-02-28 05:17:55 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-28 05:22:48 UTC Build successful.
2024-02-28 05:23:03 UTC Tests are running...
🔴 2024-02-28 06:57:05 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
67982 57068 0 11 10883 20

Copy link

github-actions bot commented Feb 28, 2024

2024-02-28 09:58:52 UTC Pre-commit check for eb7cc11 has started.
2024-02-28 09:58:54 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-28 10:12:54 UTC Build successful.
2024-02-28 10:13:07 UTC Tests are running...
🔴 2024-02-28 12:00:36 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14879 14735 0 27 96 21

Copy link

github-actions bot commented Feb 28, 2024

2024-02-28 12:56:36 UTC Pre-commit check for 1f4857f has started.
2024-02-28 12:56:37 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-28 13:00:49 UTC Build successful.
2024-02-28 13:00:59 UTC Tests are running...
🔴 2024-02-28 14:31:09 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
68013 57092 0 7 10888 26

Copy link

github-actions bot commented Feb 28, 2024

2024-02-28 12:58:14 UTC Pre-commit check for 1f4857f has started.
2024-02-28 12:58:17 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-28 13:03:08 UTC Build successful.
2024-02-28 13:03:21 UTC Tests are running...
🔴 2024-02-28 14:42:14 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14871 14737 0 31 78 25

Copy link

github-actions bot commented Feb 28, 2024

2024-02-28 12:58:38 UTC Pre-commit check for 1f4857f has started.
2024-02-28 12:58:42 UTC Build linux-x86_64-release-cmake14 is running...
🟢 2024-02-28 13:03:02 UTC Build successful.

Copy link

github-actions bot commented Feb 29, 2024

2024-02-29 07:50:15 UTC Pre-commit check for 7042e11 has started.
2024-02-29 07:50:17 UTC Build linux-x86_64-release-asan is running...
🟢 2024-02-29 07:54:55 UTC Build successful.
2024-02-29 07:55:05 UTC Tests are running...
🔴 2024-02-29 09:31:25 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14884 14753 0 19 86 26

Copy link

github-actions bot commented Feb 29, 2024

2024-02-29 07:52:42 UTC Pre-commit check for 7042e11 has started.
2024-02-29 07:52:45 UTC Build linux-x86_64-release-cmake14 is running...
🟢 2024-02-29 07:57:16 UTC Build successful.

Copy link

github-actions bot commented Feb 29, 2024

2024-02-29 07:52:49 UTC Pre-commit check for 7042e11 has started.
2024-02-29 07:52:52 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-02-29 07:57:17 UTC Build successful.
2024-02-29 07:57:31 UTC Tests are running...
🔴 2024-02-29 09:27:19 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
68019 57100 0 6 10890 23

@niksaveliev niksaveliev merged commit dcce157 into ydb-platform:main Feb 29, 2024
3 of 5 checks passed
@shnikd shnikd mentioned this pull request Mar 26, 2024
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.

None yet

2 participants