Skip to content

Commit

Permalink
Merge pull request #414 from grafana/chore/tsdb-add-multi-concurrency
Browse files Browse the repository at this point in the history
tsdb: Add multi concurrency and max idle connections store gateway params
  • Loading branch information
aknuds1 authored Nov 2, 2021
2 parents 591e1f8 + 92d7684 commit f723ef3
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 0 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@

## master / unreleased

* [CHANGE] Store gateway: set `-blocks-storage.bucket-store.index-cache.memcached.max-get-multi-concurrency`,
`-blocks-storage.bucket-store.chunks-cache.memcached.max-get-multi-concurrency`,
`-blocks-storage.bucket-store.metadata-cache.memcached.max-get-multi-concurrency`,
`-blocks-storage.bucket-store.index-cache.memcached.max-idle-connections`,
`-blocks-storage.bucket-store.chunks-cache.memcached.max-idle-connections`,
`-blocks-storage.bucket-store.metadata-cache.memcached.max-idle-connections` to 100 #414
* [CHANGE] Update grafana-builder dependency: use $__rate_interval in qpsPanel and latencyPanel. #372
* [CHANGE] `namespace` template variable in dashboards now only selects namespaces for selected clusters. #311
* [CHANGE] Alertmanager: mounted overrides configmap to alertmanager too. #315
Expand Down
14 changes: 14 additions & 0 deletions cortex/tsdb.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,20 @@
'blocks-storage.bucket-store.index-header-lazy-loading-idle-timeout': '60m',

'blocks-storage.bucket-store.max-chunk-pool-bytes': 12 * 1024 * 1024 * 1024,

// We should keep a number of idle connections equal to the max "get" concurrency,
// in order to avoid re-opening connections continuously (this would be slower
// and fill up the conntrack table too).
//
// The downside of this approach is that we'll end up with an higher number of
// active connections to memcached, so we have to make sure connections limit
// set in memcached is high enough.
'blocks-storage.bucket-store.index-cache.memcached.max-get-multi-concurrency': 100,
'blocks-storage.bucket-store.chunks-cache.memcached.max-get-multi-concurrency': 100,
'blocks-storage.bucket-store.metadata-cache.memcached.max-get-multi-concurrency': 100,
'blocks-storage.bucket-store.index-cache.memcached.max-idle-connections': $.store_gateway_args['blocks-storage.bucket-store.index-cache.memcached.max-get-multi-concurrency'],
'blocks-storage.bucket-store.chunks-cache.memcached.max-idle-connections': $.store_gateway_args['blocks-storage.bucket-store.chunks-cache.memcached.max-get-multi-concurrency'],
'blocks-storage.bucket-store.metadata-cache.memcached.max-idle-connections': $.store_gateway_args['blocks-storage.bucket-store.metadata-cache.memcached.max-get-multi-concurrency'],
} +
$.blocks_chunks_caching_config +
$.blocks_metadata_caching_config +
Expand Down

0 comments on commit f723ef3

Please sign in to comment.