Marco Pracucci comments

Results 427 comments of


                                            Marco Pracucci

Ruler: reduce time between becoming ACTIVE in ring and starting ruler managers

> This has the drawback that instead of loading only the groups that are sharded to this particular ruler, it will download all rule groups I'm concerned about it. We...

Ruler: reduce time between becoming ACTIVE in ring and starting ruler managers

About proposal 2: what we do in other services (e.g. store-gateway), which I think could be applied to this use case too, is to use the JOINING state in the...

Use estimated query cardinality in query sharding

Once #4121 is merged, I would suggest to do few improvements on top of it: - Update documentation - Apply an offset (based on the query hash) to bucketing, so...

Use estimated query cardinality in query sharding

I think we can close this issue, since has been addressed by the PRs referenced above.

Requests with a query time range that's longer than the limit result in 5xx

`/prometheus/api/v1/read` is the remote read API. The issue here could be that the errors remapping we do doesn't apply to the remote read API.

Is there a way to deploy with kubectl(one commad)?

Mimir is a distributed system. The easiest way to deploy it is with the Helm chart. If you run on Kubernetes and want to run Mimir in production, we recommend...

Deprecate thanos_ metrics prefix

We also expose Mimir metrics with `cortex_` prefixed instead of `mimir_` due to the same legacy reasons as with the `thanos_` ones (nowadays Mimir doesn't vendor Thanos anymore). Unfortunately, doing...

helm: GOMAXPROCS automatic value set too high for low-utilisation clusters

> Example latency spikes: Are latency spikes caused by the CPU throttling? I struggle to correlate a 1m 99th latency spike with CPU throttling. > GOMAXPROCS default value for less...

Reduce memory allocations in functions used to propagate contextual information between gRPC calls

> Is there ever a case where we'd want to use `metadata.FromIncomingContext()`? I'm wondering if we should add a linting rule to avoid re-introducing this in the future. Good idea....

Reduce memory allocations in functions used to propagate contextual information between gRPC calls

> where I've spot another place where we can use the metadata.ValueFromIncomingContext() Actually that function is unused and will get removed in #7530, which I will merge after this PR.