Skip to content

[ACM Obs bot] Bump stolostron/thanos to v0.41.0#330

Open
acm-observability-bot[bot] wants to merge 422 commits intostolostron:release-2.17from
rhobs:automated-updates-acm-release-2.17
Open

[ACM Obs bot] Bump stolostron/thanos to v0.41.0#330
acm-observability-bot[bot] wants to merge 422 commits intostolostron:release-2.17from
rhobs:automated-updates-acm-release-2.17

Conversation

@acm-observability-bot
Copy link

Description

This is an automated version bump from CI.
The logs for this run can be found in the syncbot repo actions.
If you wish to perform this manually, execute the following commands from stolostron/thanos repo:

git fetch https://github.com/thanos-io/thanos --tags
if ! git merge refs/tags/v0.41.0 --no-edit; then
  git checkout --theirs CHANGELOG.md VERSION docs go.mod go.sum pkg tutorials .busybox-versions .devcontainer
  git checkout --ours OWNERS
  git add CHANGELOG.md VERSION docs go.mod go.sum pkg tutorials .busybox-versions .devcontainer OWNERS
  git merge --continue
fi
go mod tidy
go mod vendor

if [ -f scripts/rh-manifest.sh ]; then
  bash scripts/rh-manifest.sh
  git add rh-manifest.txt
  git diff --cached --exit-code || git commit -s -m "[bot] update rh-manifest.txt"
fi

Saumya40-codes and others added 30 commits July 1, 2025 08:44
Signed-off-by: Saumya Shah <saumyabshah90@gmail.com>
Refactor this check into a separate function so that defer would run at
the end of it and clean up resources properly.

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
Let's avoid using all the Cortex roundtripper machinery by using the
downstream roundtripper directly and then close the body immediately as
to not allocate any memory for the body of the response.

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
Refactor this check into a separate function so that defer would run at
the end of it and clean up resources properly.

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
docs: update changed repositories links in docs/ to correct location
Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
…nos-io#8287)

We add a "service_config" field to endpoint config file that we can use
to override the default service_config for endpoint groups. This enables
us to configure retry policy or loadbalncing on an endpoint level.

Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
Use Prometheus consts instead of using our own.

Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
Signed-off-by: Joel Verezhak <j.verezhak@gmail.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
…nfig

fix: query announced endpoints match relabel-config
Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
…tion-flag

feat: ignore parquet migrated blocks in store gateway
Thanos Query crashes with "concurrent map iteration and map write" panic
in distributed mode when multiple goroutines access the same `annotations.Annotations`
map concurrently.

```
panic: concurrent map iteration and map write
github.com/prometheus/prometheus/util/annotations.(*Annotations).Merge(...)
github.com/thanos-io/promql-engine/engine.(*compatibilityQuery).Exec(...)
```

Here I replaced direct access to `res.Warnings.AsErrors()` with a thread-safe copy:
```go
// Before (unsafe)
warnings = append(warnings, res.Warnings.AsErrors()...)

// After (thread-safe)
safeWarnings := annotations.New().Merge(res.Warnings)
warnings = append(warnings, safeWarnings.AsErrors()...)
```

Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Co-authored-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
Thanos Query crashes with "concurrent map iteration and map write" panic
in distributed mode when multiple goroutines access the same `annotations.Annotations`
map concurrently.

```
panic: concurrent map iteration and map write
github.com/prometheus/prometheus/util/annotations.(*Annotations).Merge(...)
github.com/thanos-io/promql-engine/engine.(*compatibilityQuery).Exec(...)
```

Here I replaced direct access to `res.Warnings.AsErrors()` with a thread-safe copy:
```go
// Before (unsafe)
warnings = append(warnings, res.Warnings.AsErrors()...)

// After (thread-safe)
safeWarnings := annotations.New().Merge(res.Warnings)
warnings = append(warnings, safeWarnings.AsErrors()...)
```

Signed-off-by: Joel Verezhak <jverezhak@open-systems.com>
Co-authored-by: Joel Verezhak <jverezhak@open-systems.com>
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
MichaHoffmann and others added 19 commits January 23, 2026 19:54
receive: make shuffle sharding stable on scale
"Update" spawns multiple concurrent writers all trying to acquire the
write mutex for the different endpoints we are updating. If concurrently
we serve two "GetEndpointStatus" calls we might end up in a situation
where "GetEndpointStatus" A blocks "Update" for some endpoint with a read
lock, now Update is a blocked writer and blocks "GetEndpointStatus" B
from acquiring another read lock, resulting in a deadlock.

Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
…ntset-race

query: fix race in endpointset status vs update
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
…ingestor-only

receive: fast path for ingestor only
* upgrade to Prometheus 3.8

Signed-off-by: Ben Ye <benye@amazon.com>

* update docs

Signed-off-by: Ben Ye <benye@amazon.com>

* fix lint and tests

Signed-off-by: Ben Ye <benye@amazon.com>

* update go sum

Signed-off-by: Ben Ye <benye@amazon.com>

---------

Signed-off-by: Ben Ye <benye@amazon.com>
…oper-labelset-for-remote-engine

query: use proper labelset for fan-out pruning in distributed engine
Signed-off-by: Adam Jasinski <adam.jasinski@vinted.com>
Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>
query: enable Series() batching by default
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
…log-for-0-41-0

*: cut changelog for 0.41.0
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
…lease-41.0-rc.0

VERSION: cut 0.41.0-rc.0
Signed-off-by: James Geisler <geislerjamesd@gmail.com>
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
…lease-41.0-rc.1

VERSION: prepare release-41.0-rc.1
Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com>
@openshift-ci
Copy link

openshift-ci bot commented Mar 4, 2026

Hi @acm-observability-bot[bot]. Thanks for your PR.

I'm waiting for a stolostron member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci
Copy link

openshift-ci bot commented Mar 4, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: acm-observability-bot[bot]
Once this PR has been reviewed and has the lgtm label, please assign coleenquadros for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@coleenquadros
Copy link

/ok-to-test

@coleenquadros
Copy link

/retest-required

Signed-off-by: Coleen Iona Quadros <coleen.quadros27@gmail.com>
@coleenquadros
Copy link

/retest

@openshift-ci
Copy link

openshift-ci bot commented Mar 5, 2026

@acm-observability-bot[bot]: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/test-unit f89bcb8 link true /test test-unit

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.