internal/cli: add block tracing#397
Merged
Merged
Conversation
POS-122 Bad block debug profiler
https://www.notion.so/polygontechnology/RFC-22-c70aa1e47d4e497fa6e17e0beb93d7d5 We have many situations on the execution of Bor when a "bad block" happens. This is when there is an internal error and Bor is not able to process a valid block. The only option at that point is to rewind the blockchain but even that does not guarantee that the issue is solved. |
Codecov Report
@@ Coverage Diff @@
## v0.3.0-dev #397 +/- ##
==============================================
+ Coverage 55.74% 56.30% +0.55%
==============================================
Files 601 597 -4
Lines 69602 69111 -491
==============================================
+ Hits 38803 38914 +111
+ Misses 27447 26838 -609
- Partials 3352 3359 +7
Continue to review full report at Codecov.
|
manav2401
commented
May 13, 2022
manav2401
commented
May 13, 2022
ferranbt
reviewed
May 30, 2022
ferranbt
approved these changes
May 30, 2022
JekaMas
reviewed
Jun 1, 2022
JekaMas
reviewed
Jun 1, 2022
JekaMas
reviewed
Jun 1, 2022
cffls
reviewed
Jun 1, 2022
JekaMas
approved these changes
Jun 6, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds a new command to print a JSON output of the trace of a block (by number, tag, hash or bad block hash).
A helper utility which can create and destroy a mock server is added. This will abstract the port handling (primarily for the grpc server, http server and any other if added in future) from the tests.