-
Notifications
You must be signed in to change notification settings - Fork 70
feat(l1): running localnet with client comparisions #3221
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
Merged
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
e494f0d
Make ethrex-only params use spammor and prometheus_grafana
rodrigo-o b0fa623
Added gas used % to the log
rodrigo-o fdb6002
Added more info to logs about gas used and enhanced it changing erctx…
rodrigo-o 6f5f40e
Pump up the throughput a bit more
rodrigo-o 804e004
Enabled metrics exporter and tweaked txs
rodrigo-o 5c2cc5c
Fixed variable in dashboard and added metrics to el extra args
rodrigo-o 0d61c41
Merge remote-tracking branch 'origin/main' into ethrex-only-ggas-test
rodrigo-o f5c2a7e
Added kurtosis name by default to the job variable
rodrigo-o b4eddb1
Add reth to the ethrex-only test
rodrigo-o 54757dc
Add a revm node and fixed an issue with a panel
rodrigo-o dd9b67e
Log the evm on startup
rodrigo-o 471de64
Remove unneded diff from the dashboard
rodrigo-o 6fc9cfe
Merge branch 'main' into ethrex-only-ggas-test
rodrigo-o 59a49dd
Enable metrics and grafana + prometheus for all assertoor runs
rodrigo-o e400ed7
Moved the changes to a new network_params.yaml and autoamtically prov…
rodrigo-o 514949d
removed dashboard
rodrigo-o e945450
remove spamoor count parameter
rodrigo-o 046bee0
removed previously needed image for feature addition
rodrigo-o 4529c7a
fix logs command to choose one ethrex
rodrigo-o d2f26be
Commented grafana and prometheus lines on the assertoor config networ…
rodrigo-o eb70465
Removed missing config in ethrex-only network params
rodrigo-o 85f7d20
Also remove dora from .github network params
rodrigo-o b1b70f0
Merge branch 'main' into ethrex-only-ggas-test
rodrigo-o cfeb334
Merge branch 'main' into ethrex-only-ggas-test
rodrigo-o e0ae484
Merge branch 'main' into ethrex-only-ggas-test
rodrigo-o 6c4f42d
Merge branch 'main' into ethrex-only-ggas-test
rodrigo-o e419bb4
Merge branch 'main' into ethrex-only-ggas-test
rodrigo-o 766ae55
Merge branch 'main' into ethrex-only-ggas-test
rodrigo-o File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -392,9 +392,11 @@ impl Blockchain { | |
METRICS_BLOCKS.set_latest_gigagas(throughput); | ||
); | ||
let base_log = format!( | ||
"[METRIC] BLOCK EXECUTION THROUGHPUT: {:.2} Ggas/s TIME SPENT: {:.0} ms. #Txs: {}.", | ||
"[METRIC] BLOCK EXECUTION THROUGHPUT ({}): {:.2} Ggas/s TIME SPENT: {:.0} ms. Gas Used: {:.0}%, #Txs: {}.", | ||
block.header.number, | ||
throughput, | ||
interval, | ||
(block.header.gas_used as f64 / block.header.gas_limit as f64) * 100.0, | ||
Comment on lines
+395
to
+399
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Small addition to show block number and gas_used% along the already shown data in this log. |
||
block.body.transactions.len() | ||
); | ||
let extra_log = if as_gigas > 0.0 { | ||
|
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is useful to know which evm was configured on startup as with other logs showing config at the init steps.