From ace06dc1db9660a24decc1cb8ff2b9827331101c Mon Sep 17 00:00:00 2001 From: Joao Marcal Date: Fri, 23 Dec 2022 22:21:32 +0000 Subject: [PATCH] Sets the default sd-dns-resolver for ruler miekgdns (#5997) Related to issue: https://github.com/thanos-io/thanos/issues/4204 Problem: in the PR https://github.com/thanos-io/thanos/pull/4519 the default sd-dns-resolver for querier was set to miekgdns but this change was never ported to ruler Solution: this PR brings this default to ruler as well to make it consistent Signed-off-by: JoaoBraveCoding Signed-off-by: JoaoBraveCoding --- CHANGELOG.md | 5 ++--- cmd/thanos/config.go | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index a6484f65af0..24c9c56eaeb 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -37,9 +37,8 @@ NOTE: Querier's `query.promql-engine` flag enabling new PromQL engine is now unh - [#5230](https://github.com/thanos-io/thanos/pull/5230) Rule: Stateless ruler support restoring `for` state from query API servers. The query API servers should be able to access the remote write storage. - [#5880](https://github.com/thanos-io/thanos/pull/5880) Query Frontend: Fixes some edge cases of query sharding analysis. - [#5893](https://github.com/thanos-io/thanos/pull/5893) Cache: Fixed redis client not respecting `SetMultiBatchSize` config value. -- [#5966](https://github.com/thanos-io/thanos/pull/5966) Query: Stop relying on non-existent hints for mint and maxt when selecting series for the `api/v1/series` HTTP endpoint. -- [#5948](https://github.com/thanos-io/thanos/pull/5948) Store: `chunks_fetched_duration` wrong calculation. -- [#5910](https://github.com/thanos-io/thanos/pull/5910): Receive: Fixed ketama quorum bug that was could cause success response for failed replication. This also optimize heavily receiver CPU use. +- [#5966](https://github.com/thanos-io/thanos/pull/5966) Query: Fixed mint and maxt when selecting series for the `api/v1/series` HTTP endpoint. +- [#5997](https://github.com/thanos-io/thanos/pull/5997) Rule: switch to miekgdns DNS resolver as the default one. ### Added diff --git a/cmd/thanos/config.go b/cmd/thanos/config.go index 48d459cec95..f6280dd1bdc 100644 --- a/cmd/thanos/config.go +++ b/cmd/thanos/config.go @@ -198,7 +198,7 @@ func (qc *queryConfig) registerFlag(cmd extkingpin.FlagClause) *queryConfig { cmd.Flag("query.http-method", "HTTP method to use when sending queries. Possible options: [GET, POST]"). Default("POST").EnumVar(&qc.httpMethod, "GET", "POST") cmd.Flag("query.sd-dns-resolver", "Resolver to use. Possible options: [golang, miekgdns]"). - Default("golang").Hidden().StringVar(&qc.dnsSDResolver) + Default("miekgdns").Hidden().StringVar(&qc.dnsSDResolver) cmd.Flag("query.default-step", "Default range query step to use. This is only used in stateless Ruler and alert state restoration."). Default("1s").DurationVar(&qc.step) return qc