Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add telemetry #589

Merged
merged 23 commits into from
Feb 20, 2025
Merged

feat: add telemetry #589

merged 23 commits into from
Feb 20, 2025

Conversation

Romsters
Copy link
Contributor

@Romsters Romsters commented Feb 17, 2025

What πŸ’»

  • Added telemetry

Reports:

  • node start event with cli arguments used;
  • anvil and config APIs usage;
  • panics;
  • errors propagated to main;
  • node executor failures;

Why βœ‹

  • Collect anonymous usage data

@Romsters Romsters requested a review from a team as a code owner February 17, 2025 16:43
@Romsters Romsters changed the title Add telemetry feat: add telemetry Feb 17, 2025
Copy link
Collaborator

@dutterbutter dutterbutter left a comment

Choose a reason for hiding this comment

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

How does someone opt-out of telemetry?

@Romsters
Copy link
Contributor Author

How does someone opt-out of telemetry?

for now only by editing/removing config file, we will add a proper way later.

@dutterbutter
Copy link
Collaborator

How does someone opt-out of telemetry?

for now only by editing/removing config file, we will add a proper way later.

Can we update readme / and docs accordingly (foundry book and official) πŸ™

@Romsters
Copy link
Contributor Author

How does someone opt-out of telemetry?

for now only by editing/removing config file, we will add a proper way later.

Can we update readme / and docs accordingly (foundry book and official) πŸ™

Done

@Romsters Romsters merged commit 323687d into main Feb 20, 2025
13 checks passed
@Romsters Romsters deleted the add-telemetry branch February 20, 2025 18:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants