Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Publish fault detection metrics #470

Merged
merged 5 commits into from
Oct 22, 2020
Merged

Publish fault detection metrics #470

merged 5 commits into from
Oct 22, 2020

Conversation

amathur1893
Copy link
Contributor

@amathur1893 amathur1893 commented Oct 14, 2020

*Fixes #:*#471

Description of changes: WIth this change we will start publishing metrics like Latency and failure for follower check and leader check.

Tests:

  1. Tested using Docker
    Tmp file
^fault_detection/follower_check/6544/start
current_time:1602234524352
SourceNodeID:c765a93a762cd59dda8d3379b09a752a
TargetNodeID:b2a5a93a762cd59dda8d3379b09a752a
StartTime:1566413973072$
^fault_detection/follower_check/7627/finish
current_time:1602234524400
SourceNodeID:g52i9a93a762cd59dda8d3379b09a752a
TargetNodeID:b2a5a93a762cd59dda8d3379b09a752a
FinishTime:1566413986732
error:1$

Table created

sqlite> .tables
Latency_FollowerCheck     
Latency_LeaderCheck
Failure_FollowerCheck     
Failure_LeaderCheck     

Contents of the table

sqlite> select * from Latency_FollowerCheck;
c765a93a762cd59dda8d3379b09a752a|b2a5a93a762cd59dda8d3379b09a752a|14914.0|14914.0|14914.0|14914.0
g52i9a93a762cd59dda8d3379b09a752a|b2a5a93a762cd59dda8d3379b09a752a|13660.0|13660.0|13660.0|13660.0
sqlite> select * from Failure_FollowerCheck;
c765a93a762cd59dda8d3379b09a752a|b2a5a93a762cd59dda8d3379b09a752a|0.0|0.0|0.0|0.0
g52i9a93a762cd59dda8d3379b09a752a|b2a5a93a762cd59dda8d3379b09a752a|1.0|1.0|1.0|1.0

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@yojs
Copy link
Contributor

yojs commented Oct 17, 2020

The CI should pass for this PR to be merged. Currently there are a few checkstyle errors. Can we please fix them ?

@yojs yojs merged commit 4fbbbce into opendistro-for-elasticsearch:master Oct 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants