-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MON-3967: [bot] Bump openshift/thanos to v0.36.1 #149
MON-3967: [bot] Bump openshift/thanos to v0.36.1 #149
Conversation
Signed-off-by: Kartikay <[email protected]>
This commit adds a resource_attributes field to the OTLP tracing configuration. Signed-off-by: Filip Petkovski <[email protected]>
Signed-off-by: Filip Petkovski <[email protected]>
Allow specifying OTLP resource attributes for traces
For thanos-io#6775, it would be useful to know the exact block IDs to aid debugging. Signed-off-by: Giedrius Statkevičius <[email protected]>
Signed-off-by: Saswata Mukherjee <[email protected]>
…5-to-main Merge release 0.35 to main
Signed-off-by: Michael Hoffmann <[email protected]>
Signed-off-by: Pedro Tanaka <[email protected]>
When using the exemplars proxy to search for exemplars on receivers, if one receiver had tenants that did not match the selector on the external label it would get skipped completely even if it had a tenant that actually matched Signed-off-by: Pedro Tanaka <[email protected]>
Signed-off-by: Pedro Tanaka <[email protected]>
This commit adds a new tracing span for remotely delegated queries with attributes related to the query and remote engine. Signed-off-by: Filip Petkovski <[email protected]>
Signed-off-by: Filip Petkovski <[email protected]>
Signed-off-by: Pedro Tanaka <[email protected]>
Adding a minimal test case for issue thanos-io#6775 - reproduces the panic in the compactor. Signed-off-by: Giedrius Statkevičius <[email protected]>
Emit tracing span for remote queries
* Adding repro case for broken query with distributed engine Signed-off-by: Pedro Tanaka <[email protected]> * Fixing problem with distributed queries and xfunctios Signed-off-by: Pedro Tanaka <[email protected]> * Adding support for extended functions in tenancy enforcement Signed-off-by: Pedro Tanaka <[email protected]> * Moving custom parser to new package Signed-off-by: Pedro Tanaka <[email protected]> * fixing go-lint Signed-off-by: Pedro Tanaka <[email protected]> * Using same opts and reorganize imports Signed-off-by: Pedro Tanaka <[email protected]> * fixing problem with query format Signed-off-by: Pedro Tanaka <[email protected]> * fixing flaky tests Signed-off-by: Pedro Tanaka <[email protected]> * removing extra test Signed-off-by: Pedro Tanaka <[email protected]> * yet another flaky test Signed-off-by: Pedro Tanaka <[email protected]> --------- Signed-off-by: Pedro Tanaka <[email protected]>
Signed-off-by: Vanshikav123 <[email protected]>
* rule Signed-off-by: Vanshikav123 <[email protected]> * rule-changes Signed-off-by: Vanshikav123 <[email protected]> * prettier Signed-off-by: Vanshikav123 <[email protected]> * Rebuild Signed-off-by: Vanshikav123 <[email protected]> * changes after make react-app Signed-off-by: Vanshikav123 <[email protected]> --------- Signed-off-by: Vanshikav123 <[email protected]>
…ore-multitsdb Query: fix exemplar proxying for receivers with multiple tenants
We have seen deadlocks with endpoint discovery caused by the metric collector hanging and not releasing the store labels lock. This causes the endpoint update to hang, which also makes all endpoint readers hang on acquiring a read lock for the resolved endpoints slice. This commit makes sure the Collect method on the metrics collector has a built in timeout to guard against cases where an upstream call never reads from the collection channel. Signed-off-by: Filip Petkovski <[email protected]>
* Update minio-go to v7.0.70 Add support for EKS Pod Identity fix issue: thanos-io#7157 Signed-off-by: farhad <[email protected]> * Changelog - support for EKS Pod Identity Updated changelog Signed-off-by: farhad <[email protected]> --------- Signed-off-by: farhad <[email protected]>
thanos-io#7338) * fixing extended functions support in more places Signed-off-by: Pedro Tanaka <[email protected]> * Adding new failint for the Parse() method Signed-off-by: Pedro Tanaka <[email protected]> * Adding new method for ParseMetricSelector Signed-off-by: Pedro Tanaka <[email protected]> * Fixing missing imports Extending test to check behavior More missing imports Signed-off-by: Pedro Tanaka <[email protected]> * Fixing method name Signed-off-by: Pedro Tanaka <[email protected]> * Solving references to forbidden functions Signed-off-by: Pedro Tanaka <[email protected]> * Treating promql validation from ParseExpr Signed-off-by: Pedro Tanaka <[email protected]> * fixing funcs Signed-off-by: Pedro Tanaka <[email protected]> --------- Signed-off-by: Pedro Tanaka <[email protected]> Signed-off-by: Pedro Tanaka <[email protected]>
Bumps [webpack](https://github.com/webpack/webpack) from 5.70.0 to 5.91.0. - [Release notes](https://github.com/webpack/webpack/releases) - [Commits](webpack/webpack@v5.70.0...v5.91.0) --- updated-dependencies: - dependency-name: webpack dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Filip Petkovski <[email protected]>
…arn/pkg/ui/react-app/webpack-5.91.0 Bump webpack from 5.70.0 to 5.91.0 in /pkg/ui/react-app
* Align tenant pruning according to wall clock. Pruning a tenant currently acquires a lock on the tenant's TSDB, which blocks reads from incoming queries. We have noticed spikes in query latency when tenants get decomissioned since each receiver will prune the tenant at a different time. To reduce the window where queries get degraded, this commit makes sure that pruning happens at predictable intervals by aligning it to the wall clock, similar to how head compaction is aligned. The commit also changes the tenant deletion condition to look at the duration from the min time of the tenant, rather than from the last append time. Signed-off-by: Filip Petkovski <[email protected]> * Improve tests Signed-off-by: Filip Petkovski <[email protected]> --------- Signed-off-by: Filip Petkovski <[email protected]>
Bumps [ip](https://github.com/indutny/node-ip) from 1.1.5 to 1.1.9. - [Commits](indutny/node-ip@v1.1.5...v1.1.9) --- updated-dependencies: - dependency-name: ip dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@openshift-monitoring-bot[bot]: This pull request references MON-3967 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.17.0" version, but no target version was set. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
/retest-required |
/approved |
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: openshift-monitoring-bot[bot], 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 |
/retest-required |
/test e2e-agnostic-cmo |
/hold |
* Proxy: Query goroutine leak when `store.response-timeout` is set (thanos-io#7618) time.AfterFunc() returns a time.Timer object whose C field is nil, accroding to the documentation. A goroutine blocks forever on reading from a `nil` channel, leading to a goroutine leak on random slow queries. Signed-off-by: Mikhail Nozdrachev <[email protected]> * pkg/clientconfig: fix TLS configs with only CA (thanos-io#7634) 065e3dd introduced a regression: TLS configurations for Thanos Ruler query and alerting with only a CA file failed to load. For instance, the following snippet is a valid query configuration: ``` - static_configs: - prometheus.example.com:9090 scheme: https http_config: tls_config: ca_file: /etc/ssl/cert.pem ``` The test fixtures (CA, certificate and key files) are copied from prometheus/common and are valid until 2072. Signed-off-by: Simon Pasquier <[email protected]> * Cut patch release v0.36.1 Signed-off-by: Saswata Mukherjee <[email protected]> * Fix failing e2e test (thanos-io#7620) Signed-off-by: 🌲 Harry 🌊 John 🏔 <[email protected]> Signed-off-by: Saswata Mukherjee <[email protected]> --------- Signed-off-by: Mikhail Nozdrachev <[email protected]> Signed-off-by: Simon Pasquier <[email protected]> Signed-off-by: Saswata Mukherjee <[email protected]> Signed-off-by: 🌲 Harry 🌊 John 🏔 <[email protected]> Co-authored-by: Mikhail Nozdrachev <[email protected]> Co-authored-by: Simon Pasquier <[email protected]> Co-authored-by: Harry John <[email protected]>
FYI we'll have to wait for v0.36.1 which will include thanos-io#7634. |
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
97bfd89
to
712ac2f
Compare
New changes are detected. LGTM label has been removed. |
/retitle MON-3967: [bot] Bump openshift/thanos to v0.36.1 |
/hold cancel |
/lgtm |
@openshift-monitoring-bot[bot]: all tests passed! Full PR test history. Your PR dashboard. 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. |
[ART PR BUILD NOTIFIER] Distgit: thanos |
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 openshift/thanos repo: