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

Improvement: Remove telemetry #374

Open
dselans opened this issue Sep 29, 2024 · 0 comments
Open

Improvement: Remove telemetry #374

dselans opened this issue Sep 29, 2024 · 0 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@dselans
Copy link
Contributor

dselans commented Sep 29, 2024

We tried to use Posthog for getting anonymized metrics but it wasn't terribly useful so we might as well get rid of it.

There are lots of parts to plumber that need some TLC + cleanup - Telemetry is one of them.


I don't remember exactly how telemetry is implemented but it most definitely checks the local plumber config to see if telemetry is enabled/disabled OR if it's the first execution ever - that stuff can be removed.

Also assuming that most actions (read/write/relay/etc.) have calls to telemetry that can also be removed.

I would look at the telemetry pkg and see what references it and go from there.

@dselans dselans added good first issue Good for newcomers help wanted Extra attention is needed labels Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant