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

Add further standard tracing #1056

Closed
evan-forbes opened this issue Aug 7, 2023 · 2 comments
Closed

Add further standard tracing #1056

evan-forbes opened this issue Aug 7, 2023 · 2 comments
Assignees

Comments

@evan-forbes
Copy link
Member

We're adding a few optional tracing points in #1055, but to continue #978, we should add an additional standard table that traces the local state of the mempool, consensus peer state (such as when our peers are getting new valid blocks, votes, peer disconnections, gossiping related to the pex, local address book state, and perhaps more.

Originally posted #1055 (comment)

@evan-forbes evan-forbes self-assigned this Aug 8, 2023
staheri14 added a commit that referenced this issue Aug 17, 2023
## Description

To accurately gauge block time, it's essential to track the timestamps
of committed blocks. The modifications made in this PR incorporate this
feature.

Incremental work toward
#1056
@staheri14
Copy link
Contributor

Created a Notion page exclusively aimed at compiling insights and discoveries pertaining to the performance evaluation of the network under a spamming attack. This encompasses the testing scenario, essential metrics, and performance indicators.

evan-forbes added a commit that referenced this issue Aug 22, 2023
## Description

adds more things to the block table, see
#1062 (comment)
for further context.

part of #1056

---------

Co-authored-by: Sanaz Taheri <35961250+staheri14@users.noreply.github.com>
@evan-forbes
Copy link
Member Author

standard tracing was added, so I'm going to close this for now.

staheri14 added a commit that referenced this issue Nov 21, 2023
## Description

To accurately gauge block time, it's essential to track the timestamps
of committed blocks. The modifications made in this PR incorporate this
feature.

Incremental work toward
#1056
staheri14 added a commit that referenced this issue Nov 21, 2023
## Description

adds more things to the block table, see
#1062 (comment)
for further context.

part of #1056

---------

Co-authored-by: Sanaz Taheri <35961250+staheri14@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants