-
Notifications
You must be signed in to change notification settings - Fork 905
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
feat: add local docker telemetry infrastructure #1990
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1990 +/- ##
==========================================
+ Coverage 54.12% 54.32% +0.19%
==========================================
Files 216 216
Lines 14081 14081
==========================================
+ Hits 7622 7649 +27
+ Misses 5645 5619 -26
+ Partials 814 813 -1 see 4 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still need to test this, but have this one comment for now.
## check-docker: Check if docker is installed on the machine | ||
check-docker: | ||
ifeq (,$(shell which docker)) | ||
@echo "docker is not installed, you must install docker first." | ||
exit 1; | ||
endif | ||
.PHONY: check-go | ||
|
||
## check-docker: Check if docker is installed on the machine | ||
check-docker-compose: | ||
ifeq (,$(shell which docker-compose)) | ||
@echo "docker-compose is not installed, you must install it first." | ||
exit 1; | ||
endif | ||
.PHONY: check-go |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unnecessary bloat.
Replaced by #2296 |
Supersedes #1990 Adds local telemetry infra to the Makefile. This includes pyroscope, grafana, jaeger, an otel-collector, prometheus, and loki. --------- Co-authored-by: derrandz <ouaghad.hamza@gmail.com> Co-authored-by: ramin <raminkeene@gmail.com>
Overview
This PR adds local docker telemetry infrastructure, instructions to run the infrastructure are in the
README.md
Checklist