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

feat(watcher): telegram alerting and automatically persist val node #1127

Merged

Conversation

therealdannzor
Copy link
Contributor

Description

  • Add Telegram support for alerting on various events
  • Add automatic restart of tari_validator_node if it crashes and configuration to disable

How Has This Been Tested?

Create TG channel and populate credentials (bot token) and channel_id in the config file. Go through previous steps of starting tari_swarm_daemon and have the tari_watcher register the node.

Look at the watcher logs for the spawned process pid, or use ps aux | grep 'tari_validator_node' and look for the process with the default VN folder, and call kill -9 $PID. This induces a crash of the node and if auto_restart is enabled, the tari_watcher will start a new validator node. It will keep the state of registry since before, as long as the tari_watcher does not exit, and send registration transaction as usual.

@therealdannzor therealdannzor changed the title feat(watcher): TG notifications and automatically persist VN feat(watcher): telegram alerting and automatically persist val node Aug 29, 2024
Copy link

Test Results (CI)

552 tests  ±0   551 ✅  - 1   1h 56m 55s ⏱️ + 28m 49s
 58 suites ±0     0 💤 ±0 
  2 files   ±0     1 ❌ +1 

For more details on these failures, see this check.

Results for commit 5b87876. ± Comparison against base commit 23bbb44.

Copy link
Member

@sdbondi sdbondi left a comment

Choose a reason for hiding this comment

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

utACK

@sdbondi sdbondi merged commit d85bf65 into tari-project:development Aug 30, 2024
11 of 13 checks passed
@therealdannzor therealdannzor deleted the feat-watcher-notifier-part-2 branch August 30, 2024 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants