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

Handle 2.0 prerelease #22

Closed
wants to merge 2 commits into from
Closed

Conversation

nevans
Copy link
Contributor

@nevans nevans commented May 10, 2018

I'm running a prerelease version of CouchDB, to work around some bugs in the latest stable release. Unfortunately, https://github.com/hashicorp/go-version misidentifies "2.2.0-sha1" as not matching ">= 2.0". And so couchdb-prometheus-exporter misidentifies my server as CouchDB 1.x.

I've simplified the version check to simply match on the major version >= 2, and I've also submitted an issue at hashicorp/go-version#36.

@gesellix
Copy link
Owner

note to self: relates to hashicorp/go-version#35

@gesellix
Copy link
Owner

@nevans thanks!
I would prefer a fix in go-version, but I assume that won't happen soonish (looking at the quite recent change you already referenced in hashicorp/go-version#36).
So, let's go full circle and also remove the go-version dependency from Gopkg and the vendor directory. Would you like to do that in this PR?

@nevans
Copy link
Contributor Author

nevans commented May 11, 2018

Ah, right. I'm relatively new to go, so I haven't wrapped my head around anything but the most basic go get style imports yet. I'll take a look into doing that today.

@gesellix
Copy link
Owner

@nevans did you already have a chance to remove the go-version dependency?

@gesellix
Copy link
Owner

@nevans I'm going to apply your changes via #79, which has been rebased to the current revision. Thanks for your efforts.

@gesellix gesellix closed this in #79 Aug 29, 2020
@nevans
Copy link
Contributor Author

nevans commented Sep 10, 2020

👍 Thanks!

I recently upgraded our deployment, and I gave up on rebasing or merging my patches.

@nevans nevans deleted the handle-v2-prerelease branch September 10, 2020 18:08
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