-
Notifications
You must be signed in to change notification settings - Fork 12
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
Can't make bin #9
Comments
@jeff-cook Could you try doing a make common-build ? As I think this should isolate to only build the exporter binary using promu in the current working directory called snowflake-prometheus-exporter/Makefile.common Lines 196 to 198 in a8e2831
golangci-lint only supports linux, darwin and windows platforms on i386/amd64. So perhaps there's some kind of mismatch between Go version, arch/os thing going on. If you could share those details, that would be great! Either way, I'll have someone take a look at the test soonish as I was able to replicate that quickly. |
I actually believe that output is a bit of a red herring and is part of the normal test message output for one of the tests. The I think they key is that the tests are out of date right now. There is already an open PR here to fix this, but it needs to be merged. |
@gaantunes should be able to close this issue now as a PR was merged to fix this |
Is there a way to just download a pre-build bin?
When running
I get the following error
When I comment the
golangci-lint run
command then I get this error.It looks like the build requires a test that connects to a real database before it will build the bin?
The text was updated successfully, but these errors were encountered: