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

Please, Upgrade armon/go-metrics to hashicorp/go-metrics #707

Open
peczenyj opened this issue Nov 8, 2023 · 4 comments · May be fixed by #735
Open

Please, Upgrade armon/go-metrics to hashicorp/go-metrics #707

peczenyj opened this issue Nov 8, 2023 · 4 comments · May be fixed by #735

Comments

@peczenyj
Copy link

peczenyj commented Nov 8, 2023

The package armon/go-metrics was renamed to hashicorp/go-metrics

however, I need to add replace directives in my go.mod files to keep the vendors updated.

can we just update the dependencies and release again?

@peczenyj peczenyj changed the title Upgrade armon/go-metrics to hashicorp/go-metrics Please, Upgrade armon/go-metrics to hashicorp/go-metrics Nov 15, 2023
frebib added a commit to frebib/mgmt that referenced this issue Apr 28, 2024
Fixes the renamed-but-not-actually library that was moved into the
hashicorp org that causes failures when trying to update modules:

    $ go get -u
    go: github.com/armon/[email protected]: parsing go.mod:
            module declares its path as: github.com/hashicorp/go-metrics
                    but was required as: github.com/armon/go-metrics
            trying github.com/armon/[email protected]
    go: github.com/armon/[email protected]: parsing go.mod:
            module declares its path as: github.com/hashicorp/go-metrics
                    but was required as: github.com/armon/go-metrics
            trying github.com/armon/[email protected]
    go: github.com/armon/[email protected]: parsing go.mod:
            module declares its path as: github.com/hashicorp/go-metrics
                    but was required as: github.com/armon/go-metrics
            trying github.com/armon/[email protected]
    go: github.com/armon/[email protected]: parsing go.mod:
            module declares its path as: github.com/hashicorp/go-metrics
                    but was required as: github.com/armon/go-metrics
            trying github.com/armon/[email protected]
    go: github.com/armon/[email protected]: parsing go.mod:
            module declares its path as: github.com/hashicorp/go-metrics
                    but was required as: github.com/armon/go-metrics
            restoring github.com/armon/[email protected]

This is a workaround until hashicorp/serf#707
is fixed.

Signed-off-by: Joe Groocock <[email protected]>
@Dentrax
Copy link

Dentrax commented Jul 28, 2024

Just hit this issue. I was wondering if there are any updates on this?

cc @badrabubker

Dentrax added a commit to Dentrax/serf that referenced this issue Jul 28, 2024
@Dentrax Dentrax linked a pull request Jul 28, 2024 that will close this issue
@rboyer
Copy link
Member

rboyer commented Jul 29, 2024

Quoting @jmurret's reply on a related PR:

Hi all, we will post a more detailed thread on why this is complicated, but the short gist is that this change has to be consistently updated across the dependency chains of libraries used in an application. go-metrics relies on a singleton sink in global state. If any applications have a mix of armon metrics and hashicorp due to underlying dependencies not aligning on only one of them, metrics will be split among different sinks in the global state.
This change has to be coordinated with changing and releasing this dependency across multiple libraries and applications in a thoughtful way. This PR is taking longer than expected due to the prioritization and coordination of those efforts to ensure a smooth transition. Thank you for your patience and understanding.

@godcong
Copy link

godcong commented Oct 15, 2024

hashicorp/go-metrics updated several versions... You can't update this one, or you'll get an error.

go: github.com/armon/[email protected]: parsing go.mod:
        module declares its path as: github.com/hashicorp/go-metrics
                but was required as: github.com/armon/go-metrics
        trying github.com/armon/[email protected]
go: github.com/armon/[email protected]: parsing go.mod:
        module declares its path as: github.com/hashicorp/go-metrics
                but was required as: github.com/armon/go-metrics
        trying github.com/armon/[email protected]
go: github.com/armon/[email protected]: parsing go.mod:
        module declares its path as: github.com/hashicorp/go-metrics
                but was required as: github.com/armon/go-metrics
        trying github.com/armon/[email protected]
go: github.com/armon/[email protected]: parsing go.mod:
        module declares its path as: github.com/hashicorp/go-metrics
                but was required as: github.com/armon/go-metrics
        trying github.com/armon/[email protected]
go: github.com/armon/[email protected]: parsing go.mod:
        module declares its path as: github.com/hashicorp/go-metrics
                but was required as: github.com/armon/go-metrics
        restoring github.com/armon/[email protected]

@godcong
Copy link

godcong commented Nov 12, 2024

Can move the old version back to github.com/armon/go-metrics
When I update the package, the update fails.
I don't really care which package I use, but it always makes me think my code is wrong.

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 a pull request may close this issue.

4 participants