Skip to content

Commit 0f642a3

Browse files
authored
fix: Use default value for num_tokens in ingester configuration (#544)
* fix: reduce num_tokens from 512 to 128 for ingester configuration Fixes: #543 Signed-off-by: Charlie Le <[email protected]> * chore: update CHANGELOG to set default ingester.num-tokens to 128 Signed-off-by: Charlie Le <[email protected]> * default to no override for num_tokens Signed-off-by: Charlie Le <[email protected]> --------- Signed-off-by: Charlie Le <[email protected]>
1 parent f98b14d commit 0f642a3

File tree

3 files changed

+2
-2
lines changed

3 files changed

+2
-2
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
## master / unreleased
44

5+
* [CHANGE] Use default value for num_tokens in ingester configuration #544
6+
57
## 2.6.0 / 2025-05-07
68

79
* [DEPENDENCY] Update Helm release memcached to v6.14.0 #528

README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,6 @@ Kubernetes: `^1.19.0-0`
230230
| config.&ZeroWidthSpace;frontend.&ZeroWidthSpace;log_queries_longer_than | string | `"10s"` | |
231231
| config.&ZeroWidthSpace;ingester.&ZeroWidthSpace;lifecycler.&ZeroWidthSpace;final_sleep | string | `"30s"` | Duration to sleep for before exiting, to ensure metrics are scraped. |
232232
| config.&ZeroWidthSpace;ingester.&ZeroWidthSpace;lifecycler.&ZeroWidthSpace;join_after | string | `"10s"` | We don't want to join immediately, but wait a bit to see other ingesters and their tokens first. It can take a while to have the full picture when using gossip |
233-
| config.&ZeroWidthSpace;ingester.&ZeroWidthSpace;lifecycler.&ZeroWidthSpace;num_tokens | int | `512` | |
234233
| config.&ZeroWidthSpace;ingester.&ZeroWidthSpace;lifecycler.&ZeroWidthSpace;observe_period | string | `"10s"` | To avoid generating same tokens by multiple ingesters, they can "observe" the ring for a while, after putting their own tokens into it. This is only useful when using gossip, since multiple ingesters joining at the same time can have conflicting tokens if they don't see each other yet. |
235234
| config.&ZeroWidthSpace;ingester.&ZeroWidthSpace;lifecycler.&ZeroWidthSpace;ring.&ZeroWidthSpace;kvstore.&ZeroWidthSpace;store | string | `"memberlist"` | |
236235
| config.&ZeroWidthSpace;ingester.&ZeroWidthSpace;lifecycler.&ZeroWidthSpace;ring.&ZeroWidthSpace;replication_factor | int | `3` | Ingester replication factor per default is 3 |

values.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ config:
5454
observe_period: 10s
5555
# -- Duration to sleep for before exiting, to ensure metrics are scraped.
5656
final_sleep: 30s
57-
num_tokens: 512
5857
ring:
5958
# -- Ingester replication factor per default is 3
6059
replication_factor: 3

0 commit comments

Comments
 (0)