Bump thanos to release 0.14.0-rc.1 #31
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Bartlomiej Plotka <[email protected]>
…derstands. Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Bartlomiej Plotka <[email protected]>
…hanos-io#2697) Signed-off-by: Bartlomiej Plotka <[email protected]>
) (thanos-io#2698) * Allow using multiple memcached clients at the same time. Signed-off-by: Peter Štibraný <[email protected]> * Added unit test to verify that we can have multiple clients. Signed-off-by: Peter Štibraný <[email protected]> * Added CHANGELOG.md entry. Signed-off-by: Peter Štibraný <[email protected]> * Remove unused argument. Signed-off-by: Peter Štibraný <[email protected]> # Conflicts: # CHANGELOG.md
…rting for StoreAPI + Optimized iter chunk dedup. (thanos-io#2710) (thanos-io#2711) * Deduplicate chunk dups on proxy StoreAPI level. Recommend chunk sorting for StoreAPI. Also: Merge same series together on proxy level instead select. This allows better dedup efficiency. Partially fixes: thanos-io#2303 Cases like overlapped data from store and sidecar and 1:1 duplicates are optimized as soon as it's possible. This case was highly visible on GitLab repro data and exists in most of Thanos setup. Signed-off-by: Bartlomiej Plotka <[email protected]> * Optimized algorithm to combine series only on start. Signed-off-by: Bartlomiej Plotka <[email protected]> * Optimized chunk comparision for overlaps. Signed-off-by: Bartlomiej Plotka <[email protected]> * Optimized deduplication for deduplicated chunk on query level as well. Never use proto .String() in fast path! Signed-off-by: Bartlomiej Plotka <[email protected]> # Conflicts: # CHANGELOG.md # pkg/store/storepb/custom.go # pkg/store/storepb/custom_test.go
…nts (thanos-io#2705) (thanos-io#2718) * Updated minio dependency to v6.0.56 for af-south-1 region Signed-off-by: Theunis Botha <[email protected]> * Updated CHANGELOG.md Signed-off-by: Theunis Botha <[email protected]>
thanos-io#2719) See related discussion: https://cloud-native.slack.com/archives/CL25937SP/p1590408991325100 Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Bartlomiej Plotka <[email protected]> # Conflicts: # CHANGELOG.md
Signed-off-by: Bartlomiej Plotka <[email protected]>
…v0.13.0-rc.2 (thanos-io#2765) * shipper: Be strict about upload order unless it's specified so. Signed-off-by: Bartlomiej Plotka <[email protected]> * Cut 0.13.0-rc.2 Signed-off-by: Bartlomiej Plotka <[email protected]>
…s-io#2772) * Add more tracing spans for receive Signed-off-by: Kemal Akkoyun <[email protected]> * Add spans for read path Signed-off-by: Kemal Akkoyun <[email protected]> * Fix context propogation issue Signed-off-by: Kemal Akkoyun <[email protected]>
Signed-off-by: yeya24 <[email protected]>
* Implement async select for Querier Signed-off-by: Kemal Akkoyun <[email protected]> * Use prometheus/gate for now Signed-off-by: Kemal Akkoyun <[email protected]> * Add changelog and mention behaviour in documentation Signed-off-by: Kemal Akkoyun <[email protected]> * Use new gate.Gate implementation Signed-off-by: Kemal Akkoyun <[email protected]> * Preserve tracing context Signed-off-by: Kemal Akkoyun <[email protected]>
Various tools in the SRE space use UTC as the default to make it effortless during incidents to not have to calculate between timezones. The Thanos query UI also defaults to this just like Prometheus. Signed-off-by: Frederic Branczyk <[email protected]>
* ui: Add ErrorBoundary to the React UI Signed-off-by: Prem Kumar <[email protected]> * query: Return empty array instead of nil from API Signed-off-by: Prem Kumar <[email protected]> * ui: Fix typo; remove unnecessary inline style Signed-off-by: Prem Kumar <[email protected]> * format bindata.go Signed-off-by: Prem Kumar <[email protected]>
* Add more context for receive logs Signed-off-by: Kemal Akkoyun <[email protected]> * Add request id for receive logs Signed-off-by: Kemal Akkoyun <[email protected]>
* Add memcached server address to Set() error Signed-off-by: Marco Pracucci <[email protected]> * Addressed review comments Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Prem Kumar <[email protected]>
…hanos-io#2787) Signed-off-by: Frederic Branczyk <[email protected]>
Signed-off-by: Bartlomiej Plotka <[email protected]>
Cut release v0.13.0
# Conflicts: # .circleci/config.yml # CHANGELOG.md # go.mod # go.sum # pkg/shipper/shipper.go # pkg/shipper/shipper_e2e_test.go # pkg/shipper/shipper_test.go # pkg/store/storepb/custom.go # pkg/store/storepb/custom_test.go # pkg/ui/bindata.go
Merge 0.13.0 to master
Signed-off-by: soniasingla <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: thisisobate <[email protected]>
Signed-off-by: soniasingla <[email protected]>
Signed-off-by: Frederic Branczyk <[email protected]>
…NoUpload-improvement Improve ThanosReceiveNoUpload to only alert on current instances
* query: Add /status/flags endpoint Signed-off-by: Prem Kumar <[email protected]> * ui: Allow dropdowns in Navbar; enable flags page Signed-off-by: Prem Kumar <[email protected]>
* go.mod: Bump Prometheus to current latest Signed-off-by: Frederic Branczyk <[email protected]> * cmd/thanos/rule: Fix rule manager options field name This change is necessary due to the Prometheus bump, which down-scopes the interface required for the rule manager to replay ALERT_FOR_STATE. Signed-off-by: Frederic Branczyk <[email protected]> * pkg/block/indexheader: Fix reading from index in test Signed-off-by: Frederic Branczyk <[email protected]>
…hanos-io#2816) startTimeMargin and StartTime is used only by Prometheus remote read, Thanos does not use it. Fixed following race: ``` === RUN TestMultiTSDB/run_on_existing_storage ================== WARNING: DATA RACE Read at 0x00c00073ae80 by goroutine 69: github.com/prometheus/prometheus/tsdb.validateOpts() /home/bwplotka/Repos/thanosgopath/pkg/mod/github.com/prometheus/[email protected]/tsdb/db.go:510 +0x55 github.com/prometheus/prometheus/tsdb.Open() /home/bwplotka/Repos/thanosgopath/pkg/mod/github.com/prometheus/[email protected]/tsdb/db.go:502 +0x61 github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).getOrLoadTenant.func1() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:268 +0x56b github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).getOrLoadTenant() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:302 +0x4ef github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).Open.func1() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:142 +0x66 golang.org/x/sync/errgroup.(*Group).Go.func1() /home/bwplotka/Repos/thanosgopath/pkg/mod/golang.org/x/[email protected]/errgroup/errgroup.go:57 +0x85 Previous write at 0x00c00073ae80 by goroutine 57: github.com/prometheus/prometheus/tsdb.validateOpts() /home/bwplotka/Repos/thanosgopath/pkg/mod/github.com/prometheus/[email protected]/tsdb/db.go:511 +0x1f2 github.com/prometheus/prometheus/tsdb.Open() /home/bwplotka/Repos/thanosgopath/pkg/mod/github.com/prometheus/[email protected]/tsdb/db.go:502 +0x61 github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).getOrLoadTenant.func1() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:268 +0x56b github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).getOrLoadTenant() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:302 +0x4ef github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).Open.func1() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:142 +0x66 golang.org/x/sync/errgroup.(*Group).Go.func1() /home/bwplotka/Repos/thanosgopath/pkg/mod/golang.org/x/[email protected]/errgroup/errgroup.go:57 +0x85 Goroutine 69 (running) created at: golang.org/x/sync/errgroup.(*Group).Go() /home/bwplotka/Repos/thanosgopath/pkg/mod/golang.org/x/[email protected]/errgroup/errgroup.go:54 +0x73 github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).Open() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:141 +0x2af github.com/thanos-io/thanos/pkg/receive.TestMultiTSDB.func3() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb_test.go:118 +0x6d3 testing.tRunner() /home/bwplotka/.gvm/gos/go1.14.2/src/testing/testing.go:991 +0x1eb Goroutine 57 (running) created at: golang.org/x/sync/errgroup.(*Group).Go() /home/bwplotka/Repos/thanosgopath/pkg/mod/golang.org/x/[email protected]/errgroup/errgroup.go:54 +0x73 github.com/thanos-io/thanos/pkg/receive.(*MultiTSDB).Open() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb.go:141 +0x2af github.com/thanos-io/thanos/pkg/receive.TestMultiTSDB.func3() /home/bwplotka/Repos/thanos/pkg/receive/multitsdb_test.go:118 +0x6d3 testing.tRunner() /home/bwplotka/.gvm/gos/go1.14.2/src/testing/testing.go:991 +0x1eb ================== ``` Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
* Query: always return a string in the `lastError` field
While testing out the new React UI, I have found out that some errors
return empty dicts when marshalled into the JSON format. The response
looks like this:
```
{..., "lastError": {}}
```
And then, obviously, the UI fails to parse that. Add a
`stringifiedError` type with tests which ensures that we always get a
string.
Signed-off-by: Giedrius Statkevičius <[email protected]>
* CHANGELOG: add PR number + remove whitespace
Signed-off-by: Giedrius Statkevičius <[email protected]>
* query/storeset: fix a subtle bug
Signed-off-by: Giedrius Statkevičius <[email protected]>
* Make changes according to Bartek's suggestions
Signed-off-by: Giedrius Statkevičius <[email protected]>
…leaks ): (thanos-io#2817) Reason: Missing (), probably we need linter for this. Signed-off-by: Bartlomiej Plotka <[email protected]>
Right now the alert actually fires in wrong conditions. We select all sum(instances == 0) which is always 0... Instead we want to sum(instances) == 0 Signed-off-by: Matthias Loibl <[email protected]>
…hanos-io#2824) * Soft cut changelog on master to indicate 0.14 being in progress Signed-off-by: Kemal Akkoyun <[email protected]> * Update version in tutorials Signed-off-by: Kemal Akkoyun <[email protected]>
Signed-off-by: Kemal Akkoyun <[email protected]>
* Manually download promu in crossbuild stage Signed-off-by: Kemal Akkoyun <[email protected]> * Make sure all directories created in the path Signed-off-by: Kemal Akkoyun <[email protected]>
…2848) * correctly marshal errors to JSON and ignore if nil Signed-off-by: Dan Potepa <[email protected]> * LastError should be cleared if the newer update was ok Signed-off-by: Dan Potepa <[email protected]> Signed-off-by: Kemal Akkoyun <[email protected]>
Signed-off-by: Kemal Akkoyun <[email protected]>
* upstream/release-0.14: (46 commits) Cut release v0.14.0-rc.1 (thanos-io#2853) Query: correctly marshal errors to JSON and ignore if nil (thanos-io#2848) ci: Manually download promu in crossbuild stage (thanos-io#2828) Cut release v0.14.0-rc.0 (thanos-io#2826) Soft cut changelog on master to indicate v0.14.0 being in progress (thanos-io#2824) Update ThanosReceiveNoUpload to select sum == 0 (thanos-io#2819) receive: Added more observability, fixed leaktest, to actually check leaks ): (thanos-io#2817) Query: always return a string in the `lastError` field (thanos-io#2809) Added missing CHANGELOG entry for PR 2613 (thanos-io#2820) receive: Fixed small options race; Removed unused StartTime feature. (thanos-io#2816) go.mod: Bump Prometheus to current latest (thanos-io#2814) Implement CLI Flags page in React UI (thanos-io#2796) Improve ThanosReceiveNoUpload to only alert on current instances store: Preallocate output buffer when encoding postings. (thanos-io#2812) compact: introduce flag --block-viewer.global.sync-block-interval (thanos-io#2752) docs: compact: add blurb about how retention policy works (thanos-io#2808) Reduced memory allocations in readIndexRange() (thanos-io#2807) ui: Add Stores page to React UI (thanos-io#2754) Added Kemal to Maintainer Role; Kemal is volounteering to be next release shephard (thanos-io#2804) proposal: Add scalable rule storage proposal (thanos-io#2661) ...
Currently, alert state is rendered as upper case. In Prometheus it is lower case. This fixes it. Signed-off-by: Sergiusz Urbaniak <[email protected]>
openshift-ci-robot
added
the
approved
Closed
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lilic, paulfantom, simonpasquier The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Merged
openshift-cherrypick-robot
pushed a commit
to openshift-cherrypick-robot/thanos
that referenced
this pull request
Jan 10, 2025
…d-2.5 [release-2.5] Triggering image rebuild
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bumping to 0.14.0-rc.1 instead of 0.13.0 due to problems with git history as a direct result of merging #27 and #28 before bumping to 0.13.0. An alternative approach would be to revert all changes from those PRs, merge 0.13.0, and apply them again.
/cc @openshift/openshift-team-monitoring