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 go-metrics to v0.5.3 #19190

Merged
merged 1 commit into from
Nov 28, 2023
Merged

deps: update go-metrics to v0.5.3 #19190

merged 1 commit into from
Nov 28, 2023

Conversation

lgfa29
Copy link
Contributor

@lgfa29 lgfa29 commented Nov 27, 2023

Update go-metrics to v0.5.3 to pick hashicorp/go-metrics#146.

go-metrics v0.5.0 switch from armon/go-metrics to hashicorp/go-metrics, so update those references as well.

The indirect reference in go.mod comes from hashicorp/memberlist.

$ go mod why -m github.com/armon/go-metrics
# github.com/armon/go-metrics
github.com/hashicorp/nomad/command/agent
github.com/hashicorp/memberlist
github.com/armon/go-metrics

Update `go-metrics` to v0.5.3 to pick
hashicorp/go-metrics#146.
Copy link
Member

@jrasell jrasell left a comment

Choose a reason for hiding this comment

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

LGTM!

$ ag "github.com/armon/go-metrics"
go.mod
165:	github.com/armon/go-metrics v0.4.1 // indirect

CHANGELOG.md
1012:* deps: Updated `github.com/armon/go-metrics` to `v0.4.1` which includes a performance improvement for Prometheus sink [[GH-14493](https://github.com/hashicorp/nomad/issues/14493)]

go.sum
298:github.com/armon/go-metrics v0.0.0-20180917152333-f0300d1749da/go.mod h1:Q73ZrmVTwzkszR9V5SSuryQ31EELlFMUz1kKyl939pY=
299:github.com/armon/go-metrics v0.0.0-20190430140413-ec5e00d3c878/go.mod h1:3AMJUQhVx52RsWOnlkpikZr01T/yAVN2gn0861vByNg=
300:github.com/armon/go-metrics v0.4.1 h1:hR91U9KYmb6bLBYLQjyM+3j+rcd/UhE+G78SFnF8gJA=
301:github.com/armon/go-metrics v0.4.1/go.mod h1:E6amYzXo6aW1tqzoZGT755KkbgrJsSdpwZ+3JqfkOG4=

Did we want to backport this, so that future dep updates can also be backported that include bug fixes?

Copy link
Member

@tgross tgross left a comment

Choose a reason for hiding this comment

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

LGTM! We should probably do the CE->ENT merge (and fixup) right after merging this because we're definitely going to run into compilation issues in the ENT repo because of the import path change.

@lgfa29 lgfa29 added backport/1.4.x backport to 1.4.x release line backport/1.5.x backport to 1.5.x release line backport/1.6.x backport to 1.6.x release line labels Nov 28, 2023
@lgfa29 lgfa29 merged commit ddb060d into main Nov 28, 2023
25 checks passed
@lgfa29 lgfa29 deleted the deps-update-go-metrics branch November 28, 2023 17:37
lgfa29 added a commit that referenced this pull request Nov 28, 2023
lgfa29 added a commit that referenced this pull request Nov 28, 2023
lgfa29 added a commit that referenced this pull request Nov 28, 2023
lgfa29 added a commit that referenced this pull request Nov 28, 2023
lgfa29 added a commit that referenced this pull request Dec 7, 2023
lgfa29 added a commit that referenced this pull request Dec 7, 2023
lgfa29 added a commit that referenced this pull request Dec 7, 2023
tgross pushed a commit that referenced this pull request Dec 8, 2023
* Revert "deps: update go-metrics to v0.5.3 (#19190) (#19209)"

This reverts commit a0d0ff6.

* changelog: add entry for #19376
tgross pushed a commit that referenced this pull request Dec 8, 2023
* Revert "deps: update go-metrics to v0.5.3 (#19190)"

This reverts commit ddb060d.

* changelog: add entry for #19374
tgross pushed a commit that referenced this pull request Dec 8, 2023
* Revert "deps: update go-metrics to v0.5.3 (#19190) (#19208)"

This reverts commit 1112a28.

* changelog add entry for #19375
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.4.x backport to 1.4.x release line backport/1.5.x backport to 1.5.x release line backport/1.6.x backport to 1.6.x release line
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants