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

Sentry integration #58

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dehekkendekrekker
Copy link

This PR integrates sentry into the subnet's code.

Sentry is a toolkit that captures exceptions and sends all relevant information to a central server, where this information can be viewed in a dashboard. Sentry can integrate with github, so problems can be triaged, and allocated to teammembers.

image

All except statements have been hooked, as to create a full overview what is going wrong in the codebase. This means you will get unhandled (really bad) and handled exceptions (maybe bad). The unhandled ones should be resolved first, as this is a script breaking. The handled ones allow you to see if any weird stuff is happening. If the issue is not weird at all, those events can be muted in the dashboard.

In order to get this running, you will need to create an account and create a project. You will be given a trial first, then you'll need to select a paid plan or you'll get busted to the "Developer plan". This project will give you a dsn. Both validator and miner can be started with the --sentry-dsn <dsn> parameter. When this flag is enabled, sentry is enabled and will start sending information to the server.

You can choose to withhold the dns and only run it in a test-net. However, if you make the dsn public, all participants in the network that specify the parameter will report error information. This should allow you to quickly react to fatal problems and put out patches.

I have hooked all my validators in a similar fashion, but I get so many exceptions that I can't really all SN owners of the problems, so it makes more sense to have the SN owners handled this themselves.

Note, this PR has been tested for validator only. As I don't run any miners, this should be verified for miners as well.

If you feel this PR is worthy of a reward, TAO's can go to 5FZ3Cyq4wNX6rgiTD6kSEovZ6hcvcej6N3uwV68ELb6vwQo4

Greets,
Your trusty l0lpusher69

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.

1 participant