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

gRPC metrics not exposed to prometheus #302

Closed
ukclivecox opened this issue Nov 19, 2018 · 12 comments
Closed

gRPC metrics not exposed to prometheus #302

ukclivecox opened this issue Nov 19, 2018 · 12 comments

Comments

@ukclivecox
Copy link
Contributor

Might need to include inspired by linked project in the issue on micrometer

@tilast
Copy link

tilast commented Apr 12, 2019

Does this issue mean that if one uses a GRPC mode for the model then metrics will not be collected by prometheus?

@ukclivecox
Copy link
Contributor Author

Sorry missed this. Yes at present micrometer doesn't support grpc our of the box.

@tilast
Copy link

tilast commented May 1, 2019

thanks! so if you say it does not support grpc out of the box, is there an existing workaround?

@ukclivecox ukclivecox modified the milestones: 0.3.x, 2.0.x, 1.0.x Aug 21, 2019
@pravingadakh
Copy link
Contributor

@cliveseldon Is there any workaround exist for this or will we have to wait until micrometer issue is fixed?

@ukclivecox
Copy link
Contributor Author

ukclivecox commented Oct 3, 2019

Well, yes I suppose a workaround is to create our own but was hoping we could continue to use micrometer. Not sure of the short term fixes that could help.

@ukclivecox
Copy link
Contributor Author

Also, eventually we want to replace the Java engine with GoLang so then hopefully there is a grpc metrics library in GoLang for this.

@ukclivecox
Copy link
Contributor Author

Also, we should allow metrics directly from each model. So I think this also is the best solution going forward. So out python wrappers could expose a /metrics endpoint.

@HunterEl
Copy link

Any movement on this issue?

@RafalSkolasinski
Copy link
Contributor

This should be fixed in recent PR #1507 about exposing metrics directly on Python wrapper.

@RafalSkolasinski
Copy link
Contributor

Please, reopen if issue is not fixed on master.

@ukclivecox
Copy link
Contributor Author

Yes - gRPC and REST metrics will be exposed in 1.1 via the new GoLang executor:

@HunterEl
Copy link

Thanks all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants