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

Integrate OpenCensus for tags, stats, and tracing #1078

Closed
guidoiaquinti opened this issue Feb 22, 2018 · 2 comments · Fixed by #1378
Closed

Integrate OpenCensus for tags, stats, and tracing #1078

guidoiaquinti opened this issue Feb 22, 2018 · 2 comments · Fixed by #1378
Assignees

Comments

@guidoiaquinti
Copy link

guidoiaquinti commented Feb 22, 2018

Would you be interested in a PR to integrate OpenCensus in consul-template?

My goal is to increase visibility around application performance and behavior monitoring data.

I have a first (draft) implementation to natively expose some simple stats:

➜  ~ curl -s localhost:9999/metrics | grep build_info
# HELP consul_template_build_info A metric with a constant '1' value labeled by revision and version from which consul-template was built.
# TYPE consul_template_build_info gauge
consul_template_build_info{gitcommit="",version="0.19.5.dev"} 1
@findkim findkim self-assigned this Apr 30, 2020
@findkim findkim added this to the 0.26.0 milestone Apr 30, 2020
@findkim
Copy link
Contributor

findkim commented Apr 30, 2020

Hi @guidoiaquinti, I wanted to let you know that we have targeted monitoring for the 0.26.0 release 🎉. Thank you for everyone who had 👍 this enhancement and for your patience.

OpenCensus would be a good solution for metrics and monitoring and a late thank you for the offer to submit a PR! However with such a gap from when this issue was first opened to now, I can go ahead and start working on this instead of requesting you to dig 1 year in history for your draft.

With the time that has passed, the future of OpenCensus has moved to OpenTelemetry and it is now in beta. There are several other tickets requesting for monitoring, namely statsd/dogstatsd #826 and prometheus #1338, which OpenTelemetry at its current state supports dogstatsd and prometheus. My plan is to first add support for metrics to Consul Template, and then tracing can be added at another time.

We will try to keep the version of OpenTelemetry up to date as GA is released. Please do let flag if there are any particular feature that you need specifically from OpenCensus that is not supported by OpenTelemetry or is targeted for its GA release.

@guidoiaquinti
Copy link
Author

Awesome news, thank you @findkim ! 🎉

@eikenb eikenb removed this from the 0.26.0 milestone Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants