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

Cut release 0.23.0-rc.0 #4625

Merged
merged 1 commit into from
Sep 8, 2021
Merged

Cut release 0.23.0-rc.0 #4625

merged 1 commit into from
Sep 8, 2021

Conversation

bwplotka
Copy link
Member

@bwplotka bwplotka commented Sep 1, 2021

Signed-off-by: Bartlomiej Plotka bwplotka@gmail.com

@GiedriusS
Copy link
Member

#4586 (comment) I suppose this needs fixing before the release :(

@bwplotka
Copy link
Member Author

bwplotka commented Sep 2, 2021

Yup, that's a small blocker: #4586 (comment)

@bwplotka
Copy link
Member Author

bwplotka commented Sep 6, 2021

rebased release branch on fixed main 💪🏽 Let's go!

@GiedriusS
Copy link
Member

Do we need to mention #4421? With that PR we now pick up all of the APIs that a node exposes automatically, right?

@bwplotka
Copy link
Member Author

bwplotka commented Sep 6, 2021

You are absolutely right I missed this. Just checked the code it does not matter indeed 💪🏽

@bwplotka
Copy link
Member Author

bwplotka commented Sep 6, 2021

Double checking now if this will work will old Rule/Metadata etc APIs?

@bwplotka
Copy link
Member Author

bwplotka commented Sep 6, 2021

cc @hitanshu-mehta

@bwplotka
Copy link
Member Author

bwplotka commented Sep 6, 2021

I think there is regression here - if someone points to old Ruler API (e.g Thanos Ruler) in --rule - we will assume it's store API

@hitanshu-mehta
Copy link
Contributor

@bwplotka Yes, currently we are assuming it exposes both rule and store APIs (see here), because previously we were keeping rule, metadata, target and exemplar APIs only if we have store API at those endpoints. (see here).

@bwplotka
Copy link
Member Author

bwplotka commented Sep 7, 2021

I understand, but I think we can't assume that. One can roll querier without rolling other components. Adding test case and potential fix here: #4645

@bwplotka
Copy link
Member Author

bwplotka commented Sep 7, 2021

I might be wrong, all might be good thanks to fillExpectedAPIs 🤔

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>
@bwplotka
Copy link
Member Author

bwplotka commented Sep 7, 2021

Addressed comment, @GiedriusS @hitanshu-mehta and confirmed all is good, no regression is there (spent ~2h on assuring that = worth it XD!). Thanks for amazing work (:

@bwplotka bwplotka merged commit a581f10 into release-0.23 Sep 8, 2021
@bwplotka bwplotka deleted the cut-0.23-rc.0-v2 branch September 8, 2021 11:10
bwplotka added a commit that referenced this pull request Oct 6, 2021
* Cut release 0.23.0-rc.0 (#4625)

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

* Updated version.

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

* Cut 0.23.0-rc.1 and cherry picked 3 critical commits from main. (#4684)

* Sidecar: Fix process external label on promethues v2.28+ use units.Bytes config type (#4657)

* Sidecar: Fix process external label when promethues v2.28+ use units.Bytes config type (#4656)

Signed-off-by: hanjm <hanjinming@outlook.com>

* E2E: Upgrade prometheus image version

Signed-off-by: hanjm <hanjinming@outlook.com>

* upgrade Prometheus dependency version to v2.30.0 (#4669)

* upgrade Prometheus dependency version to v2.30.0

Signed-off-by: Ben Ye <ben.ye@bytedance.com>

* fix unit test

Signed-off-by: Ben Ye <ben.ye@bytedance.com>
# Conflicts:
#	go.mod
#	go.sum

* Query: Fix (*exemplarsStream).receive/(*metricMetadataStream).receive/(*targetsStreamStream).receive infinite loop when target response Unimplemented error (#4676) (#4681)

Signed-off-by: hanjm <hanjinming@outlook.com>

* Cut 0.23.0-rc.1

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

Co-authored-by: Jimmiehan <hanjinming@outlook.com>
Co-authored-by: Ben Ye <yb532204897@gmail.com>

* Cut 0.23.0 release. (#4697)

* Endpointset: Do not use info client to obtain metadata (for now) (#4714)

* Do not use info client to obtain metadata

Signed-off-by: Matej Gera <matejgera@gmail.com>

* Update CHANGELOG.

Signed-off-by: Matej Gera <matejgera@gmail.com>

* Comment out client.info usage

Signed-off-by: Matej Gera <matejgera@gmail.com>

* Fix lint error

Signed-off-by: Matej Gera <matejgera@gmail.com>

* Cutting 0.23.1 (#4718)

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

* Moved tutorials Thanos versions to 0.23.1

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

* Added volounteer for shepharding, fixed VERSION.

Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>

Co-authored-by: Jimmiehan <hanjinming@outlook.com>
Co-authored-by: Ben Ye <yb532204897@gmail.com>
Co-authored-by: Matej Gera <38492574+matej-g@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants