Skip to content
This repository has been archived by the owner on Jun 24, 2024. It is now read-only.

add client interceptor example #71

Closed
wants to merge 2 commits into from
Closed

Conversation

drewwells
Copy link

switches to go mod, I could not get dep ensure to successfully
vendor prometheus.

switches to `go mod`, I could not get dep ensure to successfully
vendor prometheus.
Copy link

@kd7lxl kd7lxl left a comment

Choose a reason for hiding this comment

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

Don't you need to initialize somewhere?

grpc_prometheus.Register(myServer)

https://github.com/grpc-ecosystem/go-grpc-prometheus#server-side

@drewwells
Copy link
Author

That sounds like the job of atlas-toolkit.

@kd7lxl
Copy link

kd7lxl commented Feb 28, 2020

That sounds like the job of atlas-toolkit.

Why would you add a toolkit dependency when all the required features are in grpc-prometheus?

This is already bootstrapped completely for new apps with atlas-cli: infobloxopen/atlas-cli#53

@drewwells
Copy link
Author

project is deprecated, closing

@drewwells drewwells closed this Mar 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants