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

fix scrape failures on ch 18.5.x+ #23

Merged
merged 2 commits into from
Oct 16, 2018
Merged

Conversation

tkroman
Copy link
Contributor

@tkroman tkroman commented Sep 20, 2018

No description provided.

@valo
Copy link

valo commented Sep 20, 2018

I encountered this problem just now and saw your PR. Thanks for fixing it. Now I need to figure out how to get a docker container with the fix :-)

@tkroman
Copy link
Contributor Author

tkroman commented Sep 20, 2018

@valo https://hub.docker.com/r/tkroman/clickhouse_exporter_fresh/

built a newer docker based on my fork. as a bonus you get an actually working dockerfile :)

@valo
Copy link

valo commented Sep 21, 2018

Deployed the image and it works great! Thanks for providing this

@f1yegor
Copy link
Contributor

f1yegor commented Oct 16, 2018

Thanks @tkroman . Do you mind if I borrow you build file for docker-image?

@f1yegor f1yegor merged commit ed5fbec into ClickHouse:master Oct 16, 2018
@tkroman
Copy link
Contributor Author

tkroman commented Oct 16, 2018

Sure, go ahead.

@tkroman tkroman deleted the patch-1 branch October 16, 2018 16:55
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.

None yet

3 participants