Skip to content
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

Upgrade v0.29.0 #47

Merged

Conversation

ngraham20
Copy link

No description provided.

matej-g and others added 30 commits March 18, 2022 11:31
Fix the compact test's by running this sub-test sequentially. The
further steps depend on this test's results so it's wrong to run it as a
sub-test.

Signed-off-by: Giedrius Statkevičius <[email protected]>
* return store info only when the service is ready

Signed-off-by: Ben Ye <[email protected]>

* fix test

Signed-off-by: Ben Ye <[email protected]>
* Cut 0.25.0-rc.0 (thanos-io#5184)

Signed-off-by: Matej Gera <[email protected]>

* Cut v0.25.0 (thanos-io#5209)

Signed-off-by: Matej Gera <[email protected]>

* Create v0.25.1 built with Go 1.17.8 (thanos-io#5226)

The binaries published with this release are built with Go1.17.8 to
avoid
[CVE-2022-24921](https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-24921).

Signed-off-by: Matthias Loibl <[email protected]>

* *: Cut 0.25.2 rc.0 (thanos-io#5247)

* fix: add null check to exemplar data (thanos-io#5202)

Signed-off-by: Thomas Mota <[email protected]>

* Ruler: Fix WAL directory in stateless mode (thanos-io#5242)

Signed-off-by: Matej Gera <[email protected]>

* Update CHANGELOG, VERSION

Signed-off-by: Matej Gera <[email protected]>

* Updates busybox SHA (thanos-io#5234)

Signed-off-by: GitHub <[email protected]>

Co-authored-by: yeya24 <[email protected]>

Co-authored-by: Tomás Mota <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: yeya24 <[email protected]>

* Cut v0.25.2

Signed-off-by: Matej Gera <[email protected]>

Update tutorials

Signed-off-by: Matej Gera <[email protected]>

Co-authored-by: Matthias Loibl <[email protected]>
Co-authored-by: Tomás Mota <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: yeya24 <[email protected]>
With the current GRPC APIs, layering Thanos Queriers results in
the root querier getting all of the samples and executing the query
in memory. As a result, the intermediary Queriers do not do any
intensive work and merely transport samples from the Stores to the
root Querier.

When data is perfectly sharded, users can implement a pattern where
the root Querier instructs the intermediary ones to execute the queries
from their stores and return back results. The results can then be
concatenated by the root querier and returned to the user.

In order to support this use case, this commit implements a GRPC API
in the Querier which is analogous to the HTTP Query API exposed
by Prometheus.

Signed-off-by: fpetkovski <[email protected]>
…stination dir (thanos-io#5229)

* Change error cleanup in objstore.DownloadDir to delete files not directories

Dst is always a directory. If any file after the first fails to download,
the cleanup will fail because the destination already contains at least one file.
This commit changes the cleanup logic to clean up successfully downloaded files one by one
instead of attempting to clean up the whole dst directory.

Signed-off-by: Dimitar Dimitrov <[email protected]>

* Add cleanup of root dst directory.

Signed-off-by: Dimitar Dimitrov <[email protected]>

* Add unit test for cleanup of DownloadDir

Signed-off-by: Dimitar Dimitrov <[email protected]>

* Fix linter

Signed-off-by: Dimitar Dimitrov <[email protected]>
* Fix miekgdns resolver to work with CNAME records too

Signed-off-by: Marco Pracucci <[email protected]>

* Remove unused context

Signed-off-by: Marco Pracucci <[email protected]>

* Update pkg/discovery/dns/miekgdns/resolver.go

Signed-off-by: Marco Pracucci <[email protected]>
Co-authored-by: Lucas Servén Marín <[email protected]>

Co-authored-by: Lucas Servén Marín <[email protected]>
* remove old ui

Signed-off-by: Augustin Husson <[email protected]>

* add changelog

Signed-off-by: Augustin Husson <[email protected]>

* update assets

Signed-off-by: Augustin Husson <[email protected]>
Signed-off-by: GitHub <[email protected]>

Co-authored-by: yeya24 <[email protected]>
…hanos-io#5274)

Bumps [moment](https://github.com/moment/moment) from 2.29.1 to 2.29.2.
- [Release notes](https://github.com/moment/moment/releases)
- [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md)
- [Commits](moment/moment@2.29.1...2.29.2)

---
updated-dependencies:
- dependency-name: moment
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* docs: fix Ian Billett's GitHub handle

I noticed that CI was failing [0] for PR
thanos-io#5284 because Ian had changed
his GitHub handle from @ianbillett to @bill3tt. This commit fixes this.

[0] https://github.com/thanos-io/thanos/runs/6050355497?check_suite_focus=true#step:5:135

Signed-off-by: Lucas Servén Marín <[email protected]>

* docs: fix broken links to GitHub docs

Currently, documentation generation is failing because mdox can't fetch
some GitHub documentation pages since the URLs for the help content has
changed. This commit updates the links to use the correct URLs.

Signed-off-by: Lucas Servén Marín <[email protected]>

* MAINTAINERS.md: regenerate

Signed-off-by: Lucas Servén Marín <[email protected]>
* refactor global window typings

Use declaration merging for better window types

Signed-off-by: Gabriel Bernal <[email protected]>

* bump vulnerable react-scripts version

Signed-off-by: Gabriel Bernal <[email protected]>
Signed-off-by: claude ebaneck <[email protected]>

Co-authored-by: claude ebaneck <[email protected]>
A recent commit (thanos-io#5250) added a GRPC API to Thanos Query which allows
executing PromQL over GRPC. This API is currently not discoverable
through endpointsets which makes it hard for other Thanos components
to use it.

This commit extends endpointsets with a GetQueryAPIClients method
which returns Query API clients to all components which support
this API.

Signed-off-by: fpetkovski <[email protected]>
* Added support for ppc64le

Signed-off-by: Marvin Giessing <[email protected]>

* Updated Changelog

Signed-off-by: Marvin Giessing <[email protected]>

* Updated promu & protoc

Signed-off-by: Marvin Giessing <[email protected]>

* Updated Makefile comment

Signed-off-by: Marvin Giessing <[email protected]>
Attempted to repro thanos-io#5257, but no good luck.

Signed-off-by: Bartlomiej Plotka <[email protected]>
* query-frontend: Added repro for dedup affecting precision of querying.

Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Douglas Camata <[email protected]>

* QFE does not cache request with dedup=false.

Signed-off-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Douglas Camata <[email protected]>

* Move info about queries that skip cache logic to docs

Signed-off-by: Douglas Camata <[email protected]>

* Update CHANGELOG

Signed-off-by: Douglas Camata <[email protected]>

* Run docs formatter

Signed-off-by: Douglas Camata <[email protected]>

* Fix e2e tests where caching logic is desired

Signed-off-by: Douglas Camata <[email protected]>

Co-authored-by: Bartlomiej Plotka <[email protected]>
…hanos-io#5301)

Use the doWithBatch function to avoid starting goroutines to fetch batched
results from memcached before they are allowed to run via the concurrency
Gate. This avoids starting many goroutines which cannot make any progress
due to a concurrency limit.

Fixes thanos-io#4967

Signed-off-by: Nick Pillitteri <[email protected]>
Signed-off-by: Wiard van Rij <[email protected]>

Co-authored-by: Wiard van Rij <[email protected]>
Signed-off-by: Wiard van Rij <[email protected]>

Co-authored-by: Wiard van Rij <[email protected]>
Copy link

@openshift-ci openshift-ci bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ngraham20: 1 invalid OWNERS file

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 kubernetes/test-infra repository.

@@ -4,6 +4,12 @@ approvers:
- dislbenn
- marcolan018
- subbarao-meduri
<<<<<<< HEAD
- dislbenn
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot parse file: error converting YAML to JSON: yaml: line 8: could not find expected ':'.

Signed-off-by: Nathaniel Graham <[email protected]>
Signed-off-by: Nathaniel Graham <[email protected]>
@ngraham20
Copy link
Author

/retest

1 similar comment
@ngraham20
Copy link
Author

/retest

Signed-off-by: Nathaniel Graham <[email protected]>
Signed-off-by: Nathaniel Graham <[email protected]>
@subbarao-meduri
Copy link

/test sonar-pre-submit

Signed-off-by: Subbarao Meduri <[email protected]>
@ngraham20
Copy link
Author

/retest

1 similar comment
@marcolan018
Copy link

/retest

@ngraham20
Copy link
Author

/override ci/prow/sonar-pre-submit

@openshift-ci
Copy link

openshift-ci bot commented Dec 8, 2022

@ngraham20: ngraham20 unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:.

In response to this:

/override ci/prow/sonar-pre-submit

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/test-infra repository.

@ngraham20
Copy link
Author

Thanos image worked as expected in testing, and the sonar-pre-submit can be debugged at a later point. Overriding for now.

@dislbenn
Copy link

dislbenn commented Dec 8, 2022

/override ci/prow/sonar-pre-submit

Overriding the sonar-pre-submit job to unblock this PR. The unit test and e2e test cases are passing; therefore, we will debug the sonar job within a separate PR.

@openshift-ci
Copy link

openshift-ci bot commented Dec 8, 2022

@dislbenn: Overrode contexts on behalf of dislbenn: ci/prow/sonar-pre-submit

In response to this:

/override ci/prow/sonar-pre-submit

Overriding the sonar-pre-submit job to unblock this PR. The unit test and e2e test cases are passing; therefore, we will debug the sonar job within a separate PR.

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/test-infra repository.

@dislbenn
Copy link

dislbenn commented Dec 8, 2022

/approve

@dislbenn
Copy link

dislbenn commented Dec 8, 2022

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Dec 8, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dislbenn, ngraham20

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit 2434fc9 into stolostron:release-2.7 Dec 8, 2022
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.