Skip to content

Feat/validators monitoring - #84

Open
ad2ien wants to merge 20 commits into
mainfrom
feat/validators-monitoring
Open

ad2ien wants to merge 20 commits into
mainfrom
feat/validators-monitoring

Conversation

@ad2ien

@ad2ien ad2ien commented Oct 17, 2022 •

Copy link
Copy Markdown
Contributor

This PR adds a validators monitoring features.

An actor will poll a cosmos node and send messages to a discord channel when its status changed :

  • when status changes (bonded to or from unbonded / bonding)
  • when a validator is jailed or get out of jail

@ad2ien ad2ien self-assigned this Oct 17, 2022
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch from 8a9ab45 to 68c3021 Compare October 18, 2022 06:48
@ad2ien
ad2ien marked this pull request as ready for review October 18, 2022 08:23
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch from 09d867f to 34ebe50 Compare October 25, 2022 15:15
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch from 34ebe50 to 964c633 Compare November 1, 2022 17:36
@ad2ien
ad2ien marked this pull request as draft November 1, 2022 17:38
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch 2 times, most recently from 738ee6a to a30e8c1 Compare November 4, 2022 07:50
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch 2 times, most recently from f49b641 to 7713e27 Compare November 17, 2022 14:05
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #84 (7713e27) into main (5a6cc4b) will increase coverage by 17.90%.
The diff coverage is 63.24%.

@@            Coverage Diff             @@
##            main      #84       +/-   ##
==========================================
+ Coverage   1.54%   19.45%   +17.90%     
==========================================
  Files         32       39        +7     
  Lines        905     1275      +370     
==========================================
+ Hits          14      248      +234     
- Misses       891     1027      +136     
Impacted Files Coverage Δ
src/cli/commands/start.rs 0.90% <0.00%> (-0.07%) ⬇️
src/cli/config.rs 0.00% <0.00%> (ø)
...rc/cosmos/client/handlers/get_validators_status.rs 0.00% <0.00%> (ø)
src/cosmos/client/mod.rs 0.00% <0.00%> (ø)
src/cosmos/validators/actor.rs 0.00% <0.00%> (ø)
src/cosmos/validators/discord_message.rs 0.00% <0.00%> (ø)
src/cosmos/validators/handlers/get_new_state.rs 0.00% <0.00%> (ø)
src/cosmos/validators/handlers/update_state.rs 0.00% <0.00%> (ø)
src/cosmos/validators/discord_msg_util.rs 99.31% <99.31%> (ø)
src/cosmos/validators/mod.rs 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ad2ien
ad2ien marked this pull request as ready for review November 17, 2022 15:03
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch from 7713e27 to 581b85e Compare November 17, 2022 15:44
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch from 581b85e to 09c240a Compare December 19, 2022 11:10
@ad2ien
ad2ien force-pushed the feat/validators-monitoring branch from 09c240a to e1a751f Compare January 12, 2023 09:17
@ccamel
ccamel removed request for bdeneux and ingvaar November 8, 2023 07:17
@ccamel ccamel assigned ccamel and amimart and unassigned ad2ien Nov 8, 2023
@ccamel

ccamel commented Nov 8, 2023

Copy link
Copy Markdown
Member

@amimart up

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.

4 participants