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

Switch to go-kit logging #272

Merged
merged 2 commits into from
Jan 10, 2020
Merged

Switch to go-kit logging #272

merged 2 commits into from
Jan 10, 2020

Conversation

mhartenbower
Copy link
Contributor

Fixes #270

Copy link
Contributor

@matthiasr matthiasr left a comment

Choose a reason for hiding this comment

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

Looks good overall, could you please take a look at the build failures?

exporter.go Outdated Show resolved Hide resolved
Fixes prometheus#270

Signed-off-by: mhartenbower <matt.hartenbower@gmail.com>
@mhartenbower mhartenbower force-pushed the master branch 2 times, most recently from 121af1d to 65512ed Compare October 13, 2019 20:01
Signed-off-by: mhartenbower <matt.hartenbower@gmail.com>
@mhartenbower
Copy link
Contributor Author

@matthiasr I'm not actually sure what the problem is at this point.

The build looks good. It's failing on make common-unused, but I'm having trouble following the output and I can't reproduce the issue locally.

Mind taking a look?

@matthiasr
Copy link
Contributor

It looks like it thinks that the vendor directory and go modules are out of sync. Could you

  • use the latest released Go
  • rebase against mainline
  • go mod vendor
  • go mod tidy

and commit the changes that the last two commands make? You can also try running make common-unused locally, and committing the result.

@brian-brazil
Copy link
Contributor

Did you have a chance to look at the vendoring issues?

@mhartenbower
Copy link
Contributor Author

@brian-brazil Will get it resolved this weekend.

@brian-brazil
Copy link
Contributor

Did you get anywhere with this?

@matthiasr matthiasr mentioned this pull request Jan 10, 2020
2 tasks
matthiasr pushed a commit that referenced this pull request Jan 10, 2020
@matthiasr matthiasr merged commit 5f4f780 into prometheus:master Jan 10, 2020
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.

Switch logging to go-kit
3 participants