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

deps: update prometheus/prometheus/tsdb #24752

Closed
wants to merge 3 commits into from

Conversation

willianpaixao
Copy link
Contributor

Similar to #24751, github.com/prometheus/tsdb was moved to github.com/prometheus/prometheus/tsdb in 2019 and no updates on the original path are made since.

This PR simply replaces the path and bumps to the latest stable version.

@holiman
Copy link
Contributor

holiman commented May 6, 2022

The updates in this PR are a lot more than just prometheus. I'll fix it up

@holiman
Copy link
Contributor

holiman commented May 6, 2022

Did a rebase (freezer had changed), but couldn't seem to get rid of the other dependency-changes.

Copy link
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

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

LGTM

@willianpaixao
Copy link
Contributor Author

@holiman could you please assign a reviewer?

@holiman
Copy link
Contributor

holiman commented May 10, 2022

We'll get to this -- it's not a high prio PR, since we only use a very small snippet of code from prometheus/tsdb.

@willianpaixao willianpaixao force-pushed the update-tsdb branch 2 times, most recently from 833a942 to 249ef6d Compare June 21, 2022 10:32
@holiman holiman changed the title Update prometheus/prometheus/tsdb deps: update prometheus/prometheus/tsdb Feb 3, 2023
@holiman
Copy link
Contributor

holiman commented Feb 3, 2023

rebased on master

@holiman
Copy link
Contributor

holiman commented Feb 7, 2023

Triage discussion: it really makes no sense to pull in the gigantic prometheus dependenciy for this one feature. We'll leave it be, and instead try to replace it with something standalone/small at some point in the future.

@holiman holiman closed this Feb 7, 2023
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.

None yet

2 participants