Skip to content

Commit 4cbca31

Browse files
committed
Store: *breaking ⚠️* set debug.advertise-compatibility-label to false by default (#8509)
In a previous change back in v0.37.0[^1], we stopped explicitely checking for the compatibility label advertised by the store gateway introduced to ensure compatibility with querier <v0.8.0. However, what didn't happen in that PR is that the thanos will still try to add this label by default[^2]. This change modifies the command line argument to default to NOT adding this compatibility label by default anymore, since this is now the assumption made in code. The reason behind this change is that while upgrading from v0.36.x, we quickly realised that querries were fanning out to all our endpoint-groups, and not abiding by our external labels anymore. [^1]: [pull/7645](https://github.com/thanos-io/thanos/pull/7645/files#diff-fb8e209f24ab370a81b6909a08fd58b18b6dd4a7816597879c08cf5590542b60R800) [^2]: [debug.advertise-compatibility-label](https://github.com/thanos-io/thanos/blob/d997eed1749e4d91243efc0137f6a20cb659841b/cmd/thanos/store.go#L182-L183) Signed-off-by: Alexis Lesieur <[email protected]>
1 parent d997eed commit 4cbca31

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ We use *breaking :warning:* to mark changes that are not backward compatible (re
1515
- [#8334](https://github.com/thanos-io/thanos/pull/8334) Query: wait for initial endpoint discovery before becoming ready
1616
- [#8486](https://github.com/thanos-io/thanos/pull/8486) Receive: fix exemplar label corruption from Cap'n Proto memory references
1717
- [#8499](https://github.com/thanos-io/thanos/pull/8499) Query: support UTF-8 label names for the `/api/v1/label/:name/values` API.
18+
- [#8509](https://github.com/thanos-io/thanos/pull/8509) Store: *breaking :warning:* set debug.advertise-compatibility-label to false by default
1819

1920
### Added
2021

cmd/thanos/store.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ func (sc *storeConfig) registerFlag(cmd extkingpin.FlagClause) {
180180
Default("9999-12-31T23:59:59Z").SetValue(&sc.filterConf.MaxTime)
181181

182182
cmd.Flag("debug.advertise-compatibility-label", "If true, Store Gateway in addition to other labels, will advertise special \"@thanos_compatibility_store_type=store\" label set. This makes store Gateway compatible with Querier before 0.8.0").
183-
Hidden().Default("true").BoolVar(&sc.advertiseCompatibilityLabel)
183+
Hidden().Default("false").BoolVar(&sc.advertiseCompatibilityLabel)
184184

185185
sc.selectorRelabelConf = *extkingpin.RegisterSelectorRelabelFlags(cmd)
186186

0 commit comments

Comments
 (0)