Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Add metric for fullnode/drone panics #664

Merged
merged 3 commits into from
Jul 17, 2018
Merged

Conversation

mvines
Copy link
Contributor

@mvines mvines commented Jul 17, 2018

Fixes #651

Eg:

$ influx -host metrics.solana.com -ssl -database scratch -execute 'SELECT * FROM "scratch"."autogen"."panic" WHERE time > now() - 10m'
name: panic
time                location                  message                                                                                                                                                 thread
----                --------                  -------                                                                                                                                                 ------
1531849375818000000 src/bin/fullnode.rs:24:22 panicked at 'oops, test panic', src/bin/fullnode.rs:24:22                                                                                                           ?
1531849585120000000 libcore/result.rs:945:5   panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 48, kind: AddrInUse, message: "Address already in use" }', libcore/result.rs:945:5 main

@mvines mvines requested review from rob-solana and aeyakovenko July 17, 2018 17:51
@mvines mvines force-pushed the panic branch 2 times, most recently from b93d5a9 to 8e616e9 Compare July 17, 2018 18:01
@mvines
Copy link
Contributor Author

mvines commented Jul 17, 2018

Not quite sure what 1.6 panics are but the graph is nice regardless:
image

@mvines mvines merged commit 47a22c6 into solana-labs:master Jul 17, 2018
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.

1 participant