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

log: disable telemetry backend by default #1947

Merged

Conversation

edmundnoble
Copy link
Contributor

Just to be clear for any readers: "telemetry" is structured logging for node operators to use to monitor their nodes. It is not sent to Kadena. By default, it is sent to stdout, and it can be configured to be sent to an Elasticsearch instance of the user's choosing.

This commit disables even logging telemetry to the terminal by default, because usually it's very verbose and not useful.

Change-Id: I83ed57c79fa99151909c76d7de70f6edc88106f8

Just to be clear for any readers: "telemetry" is structured logging for node
operators to use to monitor their nodes. It is not sent to Kadena. By default,
it is sent to stdout, and it can be configured to be sent to an Elasticsearch
instance of the user's choosing.

This commit disables even logging telemetry to the terminal by default, because
usually it's very verbose and not useful.

Change-Id: I83ed57c79fa99151909c76d7de70f6edc88106f8
@thoughtpolice thoughtpolice added this pull request to the merge queue May 31, 2024
Merged via the queue into master with commit 2c894c1 May 31, 2024
39 checks passed
@thoughtpolice thoughtpolice deleted the gerrit/I83ed57c79fa99151909c76d7de70f6edc88106f8 branch May 31, 2024 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants