Skip to content

Upgrade to github.com/oklog/ulid/v2#16168

Merged
aknuds1 merged 1 commit intoprometheus:mainfrom
aknuds1:arve/upgrade-ulid
Mar 5, 2025
Merged

Upgrade to github.com/oklog/ulid/v2#16168
aknuds1 merged 1 commit intoprometheus:mainfrom
aknuds1:arve/upgrade-ulid

Conversation

@aknuds1
Copy link
Contributor

@aknuds1 aknuds1 commented Mar 5, 2025

Upgrade github.com/oklog/ulid to v2.1.0. Having compared v2 to v1, I don't see any significant changes. The motivation is to unblock an automated upgrade of github.com/oklog/ulid to v2 in Grafana Mimir.

Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
@aknuds1 aknuds1 requested a review from jesusvazquez as a code owner March 5, 2025 09:56
@aknuds1 aknuds1 force-pushed the arve/upgrade-ulid branch from 1fa2545 to 4536f4d Compare March 5, 2025 10:10
@aknuds1 aknuds1 requested a review from dgl as a code owner March 5, 2025 10:10
Copy link
Member

@machine424 machine424 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks.

@aknuds1 aknuds1 merged commit 7cbf749 into prometheus:main Mar 5, 2025
27 checks passed
@aknuds1 aknuds1 deleted the arve/upgrade-ulid branch March 5, 2025 15:03
charleskorn added a commit to grafana/mimir that referenced this pull request Mar 21, 2025
charleskorn added a commit to grafana/mimir that referenced this pull request Mar 21, 2025
zenador pushed a commit to grafana/mimir that referenced this pull request Mar 21, 2025
* Upgrade mimir-prometheus

* Adjust tests to match new pretty printing format in prometheus/prometheus#16083

* Fix breaking change from prometheus/prometheus#16156

* Upgrade to github.com/oklog/ulid/v2 (prometheus/prometheus#16168)

* Bring in changes from prometheus/prometheus#16199

* Remove OOO native histograms flag (prometheus/prometheus#16207)

* Add changelog entries

* Ignore deprecation warning for `model.NameValidationScheme`

* Remove outdated native histogram OOO integration test
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.

2 participants